feat(employees): remove siteId parameter in favor of siteIds in request, OpenAPI spec and Bruno

documentation/Bruno/Arcadia Api/Api/V1/Employees/List Employees.yml

@@ -35,11 +35,6 @@ http:
       type: query
       description: Filter by establishment code (e.g. 01, PA, LY)
       disabled: true
-    - name: siteId
-      value: LDCSER
-      type: query
-      description: Filter by site ID (e.g. LDCSER, SLELA)
-      disabled: true
     - name: siteIds
       value: LDCSER
       type: query

documentation/OpenApi/openapi.yaml

@@ -68,7 +68,7 @@ paths:
       description: |
         Retourne la liste paginée des employés du système Arcadia.
         Prend en charge le filtrage par date de modification (`since`), code établissement (`establishment`),
-        code site (`siteId`), code service (`serviceId`), ainsi que le tri et la pagination HATEOAS.
+        codes sites (`siteIds`), code service (`serviceId`), ainsi que le tri et la pagination HATEOAS.
       operationId: listEmployees
       parameters:
         - $ref: "#/components/parameters/Page"
@@ -77,7 +77,6 @@ paths:
         - $ref: "#/components/parameters/Limit"
         - $ref: "#/components/parameters/Since"
         - $ref: "#/components/parameters/Establishment"
-        - $ref: "#/components/parameters/SiteId"
         - $ref: "#/components/parameters/SiteIds"
         - $ref: "#/components/parameters/ServiceId"
         - $ref: "#/components/parameters/SortBy"
@@ -455,16 +454,6 @@ components:
         pattern: '^[\dA-Z]{2,}$'
         example: "01"
 
-    SiteId:
-      name: siteId
-      in: query
-      required: false
-      description: "Filtrer par code site (ex: LDCSER, SLELA)."
-      schema:
-        type: string
-        pattern: "^[A-Z]{2,}$"
-        example: "LDCSER"
-
     SiteIds:
       name: siteIds
       in: query

src/Presentation/Api/V1/Http/Requests/Employees/EmployeesIndexRequest.php

@@ -35,7 +35,6 @@ final class EmployeesIndexRequest extends FormRequest
             'sortDirection' => ['sometimes', Rule::in(['Ascending', 'Descending'])],
             'since'         => ['sometimes', 'date'],
             'establishment' => ['sometimes', 'string'],
-            'siteId'        => ['sometimes', 'string'],
             'siteIds'       => ['sometimes', 'array'],
             'siteIds.*'     => ['sometimes', 'string'],
             'serviceId'     => ['sometimes', 'string'],
@@ -77,10 +76,6 @@ final class EmployeesIndexRequest extends FormRequest
             }
         }
 
-        if ($this->filled('siteId')) {
-            $sites[] = new SiteId($this->string('siteId')->value());
-        }
-
         if ($establishment === null && empty($sites) && $service === null && $since === null) {
             return null;
         }

tests/Feature/Api/V1/Employees/EmployeesIndexTest.php

@@ -110,7 +110,7 @@ describe('Employees Index Feature API', function (): void {
         $test = $this;
 
         // WHEN
-        $response = $test->getJson('/api/v1/employees?establishment=01&siteId=LDCSER&serviceId=097');
+        $response = $test->getJson('/api/v1/employees?establishment=01&siteIds=LDCSER&serviceId=097');
 
         // THEN
         $response->assertStatus(200);