chore: bump version to v0.1.44

## Objectif
Couche Domain DDD du module Catalog (ticket M0 · position 0.2). Crée les entités `Category` et `CategoryType`, leurs repositories, et branche le pattern Timestampable + Blamable Shared.

> **Mode stacked PR** : cible `feature/ERP-43-migrer-tables-category`. Quand la MR ERP-43 sera mergée sur develop, Matthieu repointera la cible de cette MR vers develop.

## Contenu
- **`Category`** : `#[ApiResource]` (GetCollection, Get, Post, Patch, Delete), `#[Auditable]`, `TimestampableBlamableTrait` + interfaces, asserts (`NotBlank`/`Length` sur `name`, `NotNull` sur `categoryType`), soft delete via `deletedAt`. Provider/Processor branchés au ticket 0.3 (ERP-45).
- **`CategoryType`** : référentiel statique en lecture seule (GetCollection + Get), embarqué dans `Category` via le groupe `category:read`. Pas de Trait — whitelisté dans `EntitiesAreTimestampableBlamableTest::EXCLUDED` (RG-1.17).
- **Repositories** : interfaces Domain + implémentations Doctrine.
- **`config/packages/doctrine.yaml`** : mapping ORM `Catalog` inconditionnel (miroir de `Sites`) — nécessaire pour que l'ORM reconnaisse les entités. La déclaration du module (`config/modules.php`) reste pour le ticket 0.5 (ERP-47).
- Groupes : `category:read` / `category:write` + `default:read` (expose les 4 colonnes du Trait).

## Notes techniques
- Index nommés déclarés sur les entités pour matcher la migration (cf. Role/Permission/Site).
- L'index unique partiel `uq_category_name_type_active` (`LOWER(name), category_type_id WHERE deleted_at IS NULL`) reste possédé par la seule migration : Doctrine ORM ne sait pas exprimer un index fonctionnel + partiel. Seul diff résiduel de `doctrine:schema:validate`.

## Tests
- `make php-cs-fixer-allow-risky` ✓
- `make test` ✓ (248 tests, 0 échec)
- `make db-reset` ✓
- `debug:router` ✓ (7 routes exposées)
- `doctrine:schema:validate` : mapping correct

---------

Co-authored-by: Matthieu <mtholot19@gmail.com>
Reviewed-on: #15
Reviewed-by: Autin <tristan@yuno.malio.fr>
Co-authored-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr>
Co-committed-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr>

config/modules.php

@@ -1,7 +1,6 @@
 <?php
 
 declare(strict_types=1);
-use App\Module\Catalog\CatalogModule;
 use App\Module\Commercial\CommercialModule;
 use App\Module\Core\CoreModule;
 use App\Module\Sites\SitesModule;
@@ -10,5 +9,4 @@ return [
     CoreModule::class,
     CommercialModule::class,
     SitesModule::class,
-    CatalogModule::class,
 ];

config/sidebar.php

@@ -83,13 +83,6 @@ return [
                 'module'     => 'sites',
                 'permission' => 'sites.view',
             ],
-            [
-                'label'      => 'sidebar.catalog.categories',
-                'to'         => '/admin/categories',
-                'icon'       => 'mdi:tag-multiple-outline',
-                'module'     => 'catalog',
-                'permission' => 'catalog.categories.view',
-            ],
             [
                 'label'      => 'sidebar.core.audit_log',
                 'to'         => '/admin/audit-log',

config/version.yaml

@@ -1,2 +1,2 @@
 parameters:
-    app.version: '0.1.42'
+    app.version: '0.1.44'

frontend/i18n/locales/fr.json

@@ -32,9 +32,6 @@
         },
         "sites": {
             "admin": "Sites"
-        },
-        "catalog": {
-            "categories": "Gestion des catégories"
         }
     },
     "dashboard": {

frontend/tests/e2e/_fixtures/personas.ts

@@ -35,7 +35,7 @@ export interface Persona {
     // sidebar-visibility pour driver la matrice. Les valeurs correspondent
     // aux slugs de route (`/admin/<slug>`), volontairement stables quand
     // la copie/i18n change.
-    expectedAdminLinks: Array<'users' | 'roles' | 'sites' | 'audit-log' | 'categories'>
+    expectedAdminLinks: Array<'users' | 'roles' | 'sites' | 'audit-log'>
 }
 
 const SHARED_PASSWORD = 'e2e-secret'
@@ -47,7 +47,7 @@ export const personas: Record<PersonaKey, Persona> = {
         password: SHARED_PASSWORD,
         isAdmin: true,
         permissions: [],
-        expectedAdminLinks: ['users', 'roles', 'sites', 'categories', 'audit-log'],
+        expectedAdminLinks: ['users', 'roles', 'sites', 'audit-log'],
     },
     'user-full': {
         key: 'user-full',
@@ -63,10 +63,8 @@ export const personas: Record<PersonaKey, Persona> = {
             'sites.view',
             'sites.manage',
             'sites.bypass_scope',
-            'catalog.categories.view',
-            'catalog.categories.manage',
         ],
-        expectedAdminLinks: ['users', 'roles', 'sites', 'categories', 'audit-log'],
+        expectedAdminLinks: ['users', 'roles', 'sites', 'audit-log'],
     },
     'user-readonly': {
         key: 'user-readonly',
@@ -111,4 +109,4 @@ export function getPersona(key: PersonaKey): Persona {
     return personas[key]
 }
 
-export const ALL_ADMIN_LINKS = ['users', 'roles', 'sites', 'categories', 'audit-log'] as const
+export const ALL_ADMIN_LINKS = ['users', 'roles', 'sites', 'audit-log'] as const

src/Module/Catalog/CatalogModule.php

@@ -1,43 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace App\Module\Catalog;
-
-final class CatalogModule
-{
-    public const string ID    = 'catalog';
-    public const string LABEL = 'Catalogue';
-    // REQUIRED = true : Category sera FK NOT NULL cote futurs modules Tiers
-    // (M-Clients, M-Fournisseurs, M-Prestataires). Desactiver Catalog casserait
-    // tout le metier au boot Doctrine. Cf. review Tristan MR #12 + spec M0 § 2.1.
-    public const bool REQUIRED = true;
-
-    /**
-     * Liste declarative des permissions RBAC exposees par le module Catalog.
-     *
-     * Consommee par la commande `app:sync-permissions` (SyncPermissionsCommand)
-     * qui se charge d'upserter ces entrees dans la table `permission`, de
-     * reactiver les codes precedemment marques orphelins et de marquer comme
-     * orphelins ceux qui ont disparu du code source.
-     *
-     * La cle `module` est auto-injectee par le sync command a partir de
-     * `self::ID`, il est donc inutile de la repeter dans chaque entree.
-     *
-     * Convention de nommage des codes : `module.resource[.sub].action` en
-     * snake_case, le prefixe module devant correspondre exactement a
-     * `self::ID` (verifie par la commande de synchronisation).
-     *
-     * Granularite alignee sur Core (view + manage), pas view/create/edit/delete
-     * (cf. spec M0 § 2.7).
-     *
-     * @return array<int, array{code: string, label: string}>
-     */
-    public static function permissions(): array
-    {
-        return [
-            ['code' => 'catalog.categories.view', 'label' => 'Voir les categories'],
-            ['code' => 'catalog.categories.manage', 'label' => 'Gerer les categories (creer, editer, supprimer)'],
-        ];
-    }
-}

src/Module/Catalog/Domain/Entity/Category.php

@@ -10,8 +10,6 @@ use ApiPlatform\Metadata\Get;
 use ApiPlatform\Metadata\GetCollection;
 use ApiPlatform\Metadata\Patch;
 use ApiPlatform\Metadata\Post;
-use App\Module\Catalog\Infrastructure\ApiPlatform\State\Processor\CategoryProcessor;
-use App\Module\Catalog\Infrastructure\ApiPlatform\State\Provider\CategoryProvider;
 use App\Module\Catalog\Infrastructure\Doctrine\DoctrineCategoryRepository;
 use App\Shared\Domain\Attribute\Auditable;
 use App\Shared\Domain\Contract\BlamableInterface;
@@ -35,39 +33,32 @@ use Symfony\Component\Validator\Constraints as Assert;
  * - `#[Auditable]` : chaque create / update / delete (soft) est trace dans
  *   audit_log par l'AuditListener du module Core.
  *
- * Provider (filtre soft-delete + ?includeDeleted + tri name ASC + 404 sur
+ * Les Provider (filtre soft-delete) et Processor (trim, soft delete, 409)
- * soft-deleted) et Processor (trim, 409 sur doublon, soft delete) branches
+ * seront branches au ticket 0.3 (ERP-45). Au ticket 0.2, les operations
- * au ticket 0.3 (ERP-45).
+ * utilisent les state Doctrine par defaut d'API Platform.
  */
 #[ApiResource(
     operations: [
         new GetCollection(
             security: "is_granted('catalog.categories.view')",
             normalizationContext: ['groups' => ['category:read', 'default:read']],
-            provider: CategoryProvider::class,
         ),
         new Get(
             security: "is_granted('catalog.categories.view')",
             normalizationContext: ['groups' => ['category:read', 'default:read']],
-            provider: CategoryProvider::class,
         ),
         new Post(
             security: "is_granted('catalog.categories.manage')",
             normalizationContext: ['groups' => ['category:read', 'default:read']],
             denormalizationContext: ['groups' => ['category:write']],
-            processor: CategoryProcessor::class,
         ),
         new Patch(
             security: "is_granted('catalog.categories.manage')",
             normalizationContext: ['groups' => ['category:read', 'default:read']],
             denormalizationContext: ['groups' => ['category:write']],
-            provider: CategoryProvider::class,
-            processor: CategoryProcessor::class,
         ),
         new Delete(
             security: "is_granted('catalog.categories.manage')",
-            provider: CategoryProvider::class,
-            processor: CategoryProcessor::class,
         ),
     ],
 )]

src/Module/Catalog/Domain/Entity/CategoryType.php

@@ -29,9 +29,6 @@ use Symfony\Component\Serializer\Attribute\Groups;
         new GetCollection(
             security: "is_granted('catalog.categories.view')",
             normalizationContext: ['groups' => ['category_type:read']],
-            // Tri par defaut requis par la spec M0 § 4.6 : ordre alphabetique
-            // stable pour alimenter le <MalioSelect> du formulaire Category.
-            order: ['label' => 'ASC'],
         ),
         new Get(
             security: "is_granted('catalog.categories.view')",

src/Module/Catalog/Domain/Repository/CategoryRepositoryInterface.php

@@ -5,18 +5,10 @@ declare(strict_types=1);
 namespace App\Module\Catalog\Domain\Repository;
 
 use App\Module\Catalog\Domain\Entity\Category;
-use Doctrine\ORM\QueryBuilder;
 
 interface CategoryRepositoryInterface
 {
     public function findById(int $id): ?Category;
 
     public function save(Category $category): void;
-
-    /**
-     * Construit un QueryBuilder de liste avec filtre soft-delete et tri par defaut.
-     * - $includeDeleted = false : exclut les categories soft-deleted (RG-1.08)
-     * - Tri : name ASC (RG-1.10).
-     */
-    public function createListQueryBuilder(bool $includeDeleted = false): QueryBuilder;
 }

src/Module/Catalog/Infrastructure/ApiPlatform/State/Processor/CategoryProcessor.php

@@ -1,76 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace App\Module\Catalog\Infrastructure\ApiPlatform\State\Processor;
-
-use ApiPlatform\Metadata\DeleteOperationInterface;
-use ApiPlatform\Metadata\Operation;
-use ApiPlatform\State\ProcessorInterface;
-use App\Module\Catalog\Domain\Entity\Category;
-use DateTimeImmutable;
-use Doctrine\DBAL\Exception\UniqueConstraintViolationException;
-use Symfony\Component\DependencyInjection\Attribute\Autowire;
-use Symfony\Component\HttpKernel\Exception\HttpException;
-
-/**
- * Processor Category : applique les regles de gestion en ecriture.
- *
- * - POST / PATCH : trim du nom (RG-1.03) puis delegation au persist_processor
- *   Doctrine ORM. Toute UniqueConstraintViolationException remontee par Postgres
- *   (collision sur l'index partiel uq_category_name_type_active) est traduite
- *   en HTTP 409 avec le message attendu par la spec (RG-1.07).
- * - DELETE : soft delete (RG-1.12). On NE delegue PAS au remove_processor ;
- *   on pose deletedAt = now() puis on delegue au persist_processor pour que
- *   le UPDATE Doctrine parte et que le TimestampableBlamableSubscriber mette
- *   a jour updatedAt / updatedBy (RG-1.16) en plus de l'AuditListener.
- *
- * @implements ProcessorInterface<Category, null|Category>
- */
-final class CategoryProcessor implements ProcessorInterface
-{
-    public function __construct(
-        #[Autowire(service: 'api_platform.doctrine.orm.state.persist_processor')]
-        private readonly ProcessorInterface $persistProcessor,
-    ) {}
-
-    public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): mixed
-    {
-        if (!$data instanceof Category) {
-            return $this->persistProcessor->process($data, $operation, $uriVariables, $context);
-        }
-
-        // RG-1.12 : soft delete au lieu d'un remove physique. On bascule la DELETE
-        // en UPDATE en posant deletedAt, puis on persiste via le persist_processor.
-        if ($operation instanceof DeleteOperationInterface) {
-            $data->setDeletedAt(new DateTimeImmutable());
-
-            try {
-                $this->persistProcessor->process($data, $operation, $uriVariables, $context);
-            } catch (UniqueConstraintViolationException $e) {
-                // Par construction, le soft delete ne peut pas violer l'index
-                // partiel (il LIBERE le couple (name, type) au lieu de le creer).
-                // On laisse remonter en 500 pour signaler une anomalie reelle.
-                throw $e;
-            }
-
-            return null;
-        }
-
-        // POST / PATCH : trim du nom avant validation et persistance (RG-1.03).
-        if (null !== $data->getName()) {
-            $data->setName(trim($data->getName()));
-        }
-
-        try {
-            return $this->persistProcessor->process($data, $operation, $uriVariables, $context);
-        } catch (UniqueConstraintViolationException $e) {
-            // RG-1.07 : doublon (LOWER(name), category_type_id) parmi les non-soft-deleted.
-            throw new HttpException(
-                409,
-                sprintf('Une catégorie nommée "%s" existe déjà pour ce type.', $data->getName() ?? ''),
-                $e,
-            );
-        }
-    }
-}

src/Module/Catalog/Infrastructure/ApiPlatform/State/Provider/CategoryProvider.php

@@ -1,83 +0,0 @@
-<?php
-
-declare(strict_types=1);
-
-namespace App\Module\Catalog\Infrastructure\ApiPlatform\State\Provider;
-
-use ApiPlatform\Metadata\CollectionOperationInterface;
-use ApiPlatform\Metadata\Operation;
-use ApiPlatform\State\ProviderInterface;
-use App\Module\Catalog\Domain\Entity\Category;
-use App\Module\Catalog\Domain\Repository\CategoryRepositoryInterface;
-use Symfony\Component\DependencyInjection\Attribute\Autowire;
-
-/**
- * Provider Category : applique le filtre soft-delete par defaut (RG-1.08),
- * accepte ?includeDeleted=true pour inclure les soft-deleted (RG-1.09),
- * trie par name ASC (RG-1.10), et renvoie 404 sur Get d'une soft-deleted
- * sans le flag (RG-1.11, via retour null).
- *
- * Choix d'implementation : QueryBuilder via le repository custom plutot
- * qu'un filtre Doctrine global (cf. spec § 2.3 et arbitrage ticket 0.3).
- * Avantage : pas de magie globale, lisibilite directe du code, controle
- * fin du flag includeDeleted par requete.
- *
- * @implements ProviderInterface<Category>
- */
-final class CategoryProvider implements ProviderInterface
-{
-    public function __construct(
-        #[Autowire(service: 'App\Module\Catalog\Infrastructure\Doctrine\DoctrineCategoryRepository')]
-        private readonly CategoryRepositoryInterface $repository,
-    ) {}
-
-    public function provide(Operation $operation, array $uriVariables = [], array $context = []): Category|iterable|null
-    {
-        $includeDeleted = $this->readIncludeDeleted($context);
-
-        if ($operation instanceof CollectionOperationInterface) {
-            return $this->repository
-                ->createListQueryBuilder($includeDeleted)
-                ->getQuery()
-                ->getResult()
-            ;
-        }
-
-        // Get unitaire : recharger l'entite, puis appliquer le filtre soft-delete.
-        $id = $uriVariables['id'] ?? null;
-        if (!is_int($id) && !(is_string($id) && ctype_digit($id))) {
-            return null;
-        }
-
-        $category = $this->repository->findById((int) $id);
-        if (null === $category) {
-            return null;
-        }
-
-        // RG-1.11 : 404 si soft-deleted et pas de flag includeDeleted.
-        if (!$includeDeleted && null !== $category->getDeletedAt()) {
-            return null;
-        }
-
-        return $category;
-    }
-
-    /**
-     * Lit le flag includeDeleted depuis les filtres API Platform.
-     * Accepte "true" / "1" / true (booleen).
-     */
-    private function readIncludeDeleted(array $context): bool
-    {
-        $raw = $context['filters']['includeDeleted'] ?? false;
-
-        if (is_bool($raw)) {
-            return $raw;
-        }
-
-        if (is_string($raw)) {
-            return in_array(strtolower($raw), ['true', '1'], true);
-        }
-
-        return false;
-    }
-}

src/Module/Catalog/Infrastructure/Doctrine/DoctrineCategoryRepository.php

@@ -7,7 +7,6 @@ namespace App\Module\Catalog\Infrastructure\Doctrine;
 use App\Module\Catalog\Domain\Entity\Category;
 use App\Module\Catalog\Domain\Repository\CategoryRepositoryInterface;
 use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
-use Doctrine\ORM\QueryBuilder;
 use Doctrine\Persistence\ManagerRegistry;
 
 /**
@@ -30,17 +29,4 @@ class DoctrineCategoryRepository extends ServiceEntityRepository implements Cate
         $this->getEntityManager()->persist($category);
         $this->getEntityManager()->flush();
     }
-
-    public function createListQueryBuilder(bool $includeDeleted = false): QueryBuilder
-    {
-        $qb = $this->createQueryBuilder('c')
-            ->orderBy('c.name', 'ASC')
-        ;
-
-        if (!$includeDeleted) {
-            $qb->andWhere('c.deletedAt IS NULL');
-        }
-
-        return $qb;
-    }
 }

src/Module/Core/Infrastructure/Console/SeedE2ECommand.php

@@ -184,8 +184,6 @@ final class SeedE2ECommand extends Command
                     'sites.view',
                     'sites.manage',
                     'sites.bypass_scope',
-                    'catalog.categories.view',
-                    'catalog.categories.manage',
                 ],
             ],
             [