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/CreateAttributeDefinitionRequest.php → src/Identity/Request/AdminAttributeDefinitionRequest.php

@@ -8,7 +8,7 @@
 use PhpList\RestBundle\Common\Request\RequestInterface;
 use Symfony\Component\Validator\Constraints as Assert;
 
-class CreateAttributeDefinitionRequest implements RequestInterface
+class AdminAttributeDefinitionRequest implements RequestInterface
 {
     #[Assert\NotBlank]
     public string $name;
@@ -17,7 +17,6 @@ class CreateAttributeDefinitionRequest implements RequestInterface
     public ?int $order = null;
     public ?string $defaultValue = null;
     public bool $required = false;
-    public ?string $tableName = null;
 
     public function getDto(): AdminAttributeDefinitionDto
     {
@@ -27,7 +26,6 @@ public function getDto(): AdminAttributeDefinitionDto
             listOrder: $this->order,
             defaultValue: $this->defaultValue,
             required: $this->required,
-            tableName: $this->tableName,
         );
     }
 }

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/OpenApi/SwaggerSchemasResponse.php

@@ -94,9 +94,24 @@
         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'),
+        new OA\Property(
+            property: 'options',
+            type: 'array',
+            items: new OA\Items(ref: '#/components/schemas/DynamicListAttrOption'),
+            nullable: true,
+        ),
     ],
     type: 'object'
 )]
+#[OA\Schema(
+    schema: 'DynamicListAttrOption',
+    properties: [
+        new OA\Property(property: 'id', type: 'integer', example: 1, nullable: false),
+        new OA\Property(property: 'name', type: 'string', example: 'United States'),
+        new OA\Property(property: 'listorder', type: 'integer', example: 1, nullable: false),
+    ],
+    type: 'object',
+)]
 #[OA\Schema(
     schema: 'SubscriberAttributeValue',
     properties: [