Refactor attribute definition request handling

README.md

@@ -38,7 +38,7 @@ which also has more detailed installation instructions in the README.
 
 ## API Documentation
 
-Visit `/docs` endpoint to access the full interactive documentation for `phpList/rest-api`.
+Visit `https://phplist.github.io/restapi-docs/` endpoint to access the full interactive documentation for `phpList/rest-api`.
 
 Look at the **"API Documentation with Swagger"** section in the [contribution guide](.github/CONTRIBUTING.md) for more information on API documenation.
 

composer.json

@@ -42,7 +42,7 @@
     },
     "require": {
         "php": "^8.1",
-        "phplist/core": "dev-main",
+        "phplist/core": "dev-dev",
         "friendsofsymfony/rest-bundle": "*",
         "symfony/test-pack": "^1.0",
         "symfony/process": "^6.4",

config/services/validators.yml

@@ -36,3 +36,8 @@ services:
     autowire: true
     autoconfigure: true
     tags: [ 'validator.constraint_validator' ]
+
+  PhpList\Core\Domain\Identity\Validator\AttributeTypeValidator:
+    autowire: true
+    autoconfigure: true
+

src/Identity/Controller/AdminAttributeDefinitionController.php

@@ -12,8 +12,7 @@
 use PhpList\RestBundle\Common\Controller\BaseController;
 use PhpList\RestBundle\Common\Service\Provider\PaginatedDataProvider;
 use PhpList\RestBundle\Common\Validator\RequestValidator;
-use PhpList\RestBundle\Identity\Request\CreateAttributeDefinitionRequest;
-use PhpList\RestBundle\Identity\Request\UpdateAttributeDefinitionRequest;
+use PhpList\RestBundle\Identity\Request\AdminAttributeDefinitionRequest;
 use PhpList\RestBundle\Identity\Serializer\AdminAttributeDefinitionNormalizer;
 use Symfony\Bridge\Doctrine\Attribute\MapEntity;
 use Symfony\Component\HttpFoundation\JsonResponse;
@@ -51,7 +50,7 @@ public function __construct(
         requestBody: new OA\RequestBody(
             description: 'Pass parameters to create admin attribute.',
             required: true,
-            content: new OA\JsonContent(ref: '#/components/schemas/CreateAdminAttributeDefinitionRequest')
+            content: new OA\JsonContent(ref: '#/components/schemas/AdminAttributeDefinitionRequest')
         ),
         tags: ['admin-attributes'],
         parameters: [
@@ -87,8 +86,8 @@ public function create(Request $request): JsonResponse
     {
         $this->requireAuthentication($request);
 
-        /** @var CreateAttributeDefinitionRequest $definitionRequest */
-        $definitionRequest = $this->validator->validate($request, CreateAttributeDefinitionRequest::class);
+        /** @var AdminAttributeDefinitionRequest $definitionRequest */
+        $definitionRequest = $this->validator->validate($request, AdminAttributeDefinitionRequest::class);
 
         $attributeDefinition = $this->definitionManager->create($definitionRequest->getDto());
         $this->entityManager->flush();
@@ -107,7 +106,7 @@ public function create(Request $request): JsonResponse
         requestBody: new OA\RequestBody(
             description: 'Pass parameters to update admin attribute.',
             required: true,
-            content: new OA\JsonContent(ref: '#/components/schemas/CreateAdminAttributeDefinitionRequest')
+            content: new OA\JsonContent(ref: '#/components/schemas/AdminAttributeDefinitionRequest')
         ),
         tags: ['admin-attributes'],
         parameters: [
@@ -153,8 +152,8 @@ public function update(
             throw $this->createNotFoundException('Attribute definition not found.');
         }
 
-        /** @var UpdateAttributeDefinitionRequest $definitionRequest */
-        $definitionRequest = $this->validator->validate($request, UpdateAttributeDefinitionRequest::class);
+        /** @var AdminAttributeDefinitionRequest $definitionRequest */
+        $definitionRequest = $this->validator->validate($request, AdminAttributeDefinitionRequest::class);
 
         $attributeDefinition = $this->definitionManager->update(
             attributeDefinition: $attributeDefinition,

src/Identity/OpenApi/SwaggerSchemasRequest.php

@@ -5,6 +5,7 @@
 namespace PhpList\RestBundle\Identity\OpenApi;
 
 use OpenApi\Attributes as OA;
+use PhpList\Core\Domain\Common\Model\AttributeTypeEnum;
 
 #[OA\Schema(
     schema: 'CreateAdministratorRequest',
@@ -96,15 +97,23 @@
     type: 'object'
 )]
 #[OA\Schema(
-    schema: 'CreateAdminAttributeDefinitionRequest',
+    schema: 'AdminAttributeDefinitionRequest',
     required: ['name'],
     properties: [
         new OA\Property(property: 'name', type: 'string', format: 'string', example: 'Country'),
-        new OA\Property(property: 'type', type: 'string', example: 'checkbox'),
+        new OA\Property(
+            property: 'type',
+            type: 'string',
+            enum: [
+                AttributeTypeEnum::TextLine,
+                AttributeTypeEnum::Hidden,
+            ],
+            example: 'hidden',
+            nullable: true
+        ),
         new OA\Property(property: 'order', type: 'number', example: 12),
         new OA\Property(property: 'default_value', type: 'string', example: 'United States'),
         new OA\Property(property: 'required', type: 'boolean', example: true),
-        new OA\Property(property: 'table_name', type: 'string', example: 'list_attributes'),
     ],
     type: 'object'
 )]

src/Identity/OpenApi/SwaggerSchemasResponse.php

@@ -27,11 +27,10 @@
     properties: [
         new OA\Property(property: 'id', type: 'integer', example: 1),
         new OA\Property(property: 'name', type: 'string', example: 'Country'),
-        new OA\Property(property: 'type', type: 'string', example: 'select'),
+        new OA\Property(property: 'type', type: 'string', example: 'hidden'),
         new OA\Property(property: 'list_order', type: 'integer', example: 12),
         new OA\Property(property: 'default_value', type: 'string', example: 'United States'),
         new OA\Property(property: 'required', type: 'boolean', example: true),
-        new OA\Property(property: 'table_name', type: 'string', example: 'ukcounties'),
     ],
     type: 'object'
 )]

src/Identity/Request/AdminAttributeDefinitionRequest.php

@@ -0,0 +1,57 @@
+<?php
+
+declare(strict_types=1);
+
+namespace PhpList\RestBundle\Identity\Request;
+
+use PhpList\Core\Domain\Identity\Model\Dto\AdminAttributeDefinitionDto;
+use PhpList\Core\Domain\Subscription\Validator\AttributeTypeValidator;
+use PhpList\RestBundle\Common\Request\RequestInterface;
+use Symfony\Component\Translation\IdentityTranslator;
+use Symfony\Component\Validator\Constraints as Assert;
+use Symfony\Component\Validator\Context\ExecutionContextInterface;
+use Symfony\Component\Validator\Exception\ValidatorException;
+
+#[Assert\Callback('validateType')]
+class AdminAttributeDefinitionRequest implements RequestInterface
+{
+    #[Assert\NotBlank]
+    public string $name;
+
+    #[Assert\Choice(choices: ['hidden', 'textline'], message: 'Invalid type. Allowed values: hidden, textline.')]
+    public ?string $type = null;
+
+    public ?int $order = null;
+
+    public ?string $defaultValue = null;
+
+    public bool $required = false;
+
+    public function getDto(): AdminAttributeDefinitionDto
+    {
+        return new AdminAttributeDefinitionDto(
+            name: $this->name,
+            type: $this->type,
+            listOrder: $this->order,
+            defaultValue: $this->defaultValue,
+            required: $this->required,
+        );
+    }
+
+    public function validateType(ExecutionContextInterface $context): void
+    {
+        if ($this->type === null) {
+            return;
+        }
+
+        $validator = new AttributeTypeValidator(new IdentityTranslator());
+
+        try {
+            $validator->validate($this->type);
+        } catch (ValidatorException $e) {
+            $context->buildViolation($e->getMessage())
+                ->atPath('type')
+                ->addViolation();
+        }
+    }
+}

src/Subscription/Controller/SubscriberAttributeDefinitionController.php

@@ -12,8 +12,7 @@
 use PhpList\RestBundle\Common\Controller\BaseController;
 use PhpList\RestBundle\Common\Service\Provider\PaginatedDataProvider;
 use PhpList\RestBundle\Common\Validator\RequestValidator;
-use PhpList\RestBundle\Subscription\Request\CreateAttributeDefinitionRequest;
-use PhpList\RestBundle\Subscription\Request\UpdateAttributeDefinitionRequest;
+use PhpList\RestBundle\Subscription\Request\SubscriberAttributeDefinitionRequest;
 use PhpList\RestBundle\Subscription\Serializer\AttributeDefinitionNormalizer;
 use Symfony\Bridge\Doctrine\Attribute\MapEntity;
 use Symfony\Component\HttpFoundation\JsonResponse;
@@ -51,7 +50,7 @@ public function __construct(
         requestBody: new OA\RequestBody(
             description: 'Pass parameters to create subscriber attribute.',
             required: true,
-            content: new OA\JsonContent(ref: '#/components/schemas/CreateSubscriberAttributeDefinitionRequest')
+            content: new OA\JsonContent(ref: '#/components/schemas/SubscriberAttributeDefinitionRequest')
         ),
         tags: ['subscriber-attributes'],
         parameters: [
@@ -85,8 +84,8 @@ public function create(Request $request): JsonResponse
     {
         $this->requireAuthentication($request);
 
-        /** @var CreateAttributeDefinitionRequest $definitionRequest */
-        $definitionRequest = $this->validator->validate($request, CreateAttributeDefinitionRequest::class);
+        /** @var SubscriberAttributeDefinitionRequest $definitionRequest */
+        $definitionRequest = $this->validator->validate($request, SubscriberAttributeDefinitionRequest::class);
 
         $attributeDefinition = $this->definitionManager->create($definitionRequest->getDto());
         $this->entityManager->flush();
@@ -104,7 +103,7 @@ public function create(Request $request): JsonResponse
         requestBody: new OA\RequestBody(
             description: 'Pass parameters to update subscriber attribute.',
             required: true,
-            content: new OA\JsonContent(ref: '#/components/schemas/CreateSubscriberAttributeDefinitionRequest')
+            content: new OA\JsonContent(ref: '#/components/schemas/SubscriberAttributeDefinitionRequest')
         ),
         tags: ['subscriber-attributes'],
         parameters: [
@@ -150,8 +149,8 @@ public function update(
             throw $this->createNotFoundException('Attribute definition not found.');
         }
 
-        /** @var UpdateAttributeDefinitionRequest $definitionRequest */
-        $definitionRequest = $this->validator->validate($request, UpdateAttributeDefinitionRequest::class);
+        /** @var SubscriberAttributeDefinitionRequest $definitionRequest */
+        $definitionRequest = $this->validator->validate($request, SubscriberAttributeDefinitionRequest::class);
 
         $attributeDefinition = $this->definitionManager->update(
             attributeDefinition: $attributeDefinition,

src/Subscription/Controller/SubscriberAttributeValueController.php

@@ -351,8 +351,6 @@ public function getAttributeDefinition(
             throw $this->createNotFoundException('Subscriber attribute not found.');
         }
         $attribute = $this->attributeManager->getSubscriberAttribute($subscriber->getId(), $definition->getId());
-        $this->attributeManager->delete($attribute);
-        $this->entityManager->flush();
 
         return $this->json(
             $this->normalizer->normalize($attribute),