Compare commits

...

5 Commits

18 changed files with 1797 additions and 5 deletions
+10
View File
@@ -172,6 +172,16 @@ return [
'module' => 'catalog',
'permission' => 'catalog.categories.view',
],
// Stockage (M7, ERP-210). Admin-only : gate par `catalog.storages.view`
// et son module owner `catalog`. Reutilise le referentiel StorageType
// du M6. Place pres des autres items Catalog (produits, categories).
[
'label' => 'sidebar.catalog.storages',
'to' => '/admin/storages',
'icon' => 'mdi:warehouse',
'module' => 'catalog',
'permission' => 'catalog.storages.view',
],
[
'label' => 'sidebar.core.audit_log',
'to' => '/admin/audit-log',
+3 -1
View File
@@ -53,7 +53,8 @@
},
"catalog": {
"categories": "Gestion des catégories",
"products": "Catalogue produits"
"products": "Catalogue produits",
"storages": "Gestion des stockages"
}
},
"dashboard": {
@@ -819,6 +820,7 @@
"sites_site": "Site",
"catalog_category": "Catégorie",
"catalog_product": "Produit",
"catalog_storage": "Stockage",
"commercial_client": "Client",
"commercial_clientaddress": "Adresse client",
"commercial_clientcontact": "Contact client",
+10 -4
View File
@@ -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' | 'products'>
expectedAdminLinks: Array<'users' | 'roles' | 'sites' | 'audit-log' | 'categories' | 'products' | 'storages'>
}
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', 'products', 'audit-log'],
expectedAdminLinks: ['users', 'roles', 'sites', 'categories', 'products', 'storages', 'audit-log'],
},
'user-full': {
key: 'user-full',
@@ -71,6 +71,12 @@ export const personas: Record<PersonaKey, Persona> = {
// `/admin/products` -> ajoute le lien `products` a expectedAdminLinks.
'catalog.products.view',
'catalog.products.manage',
// Stockage (M7, ERP-210). Admin-only : mappe sur le persona "tout",
// pas de nouveau persona (regle ABSOLUE n°7). L'item vit dans la
// section Administration sur la route `/admin/storages` -> ajoute le
// lien `storages` a expectedAdminLinks.
'catalog.storages.view',
'catalog.storages.manage',
// Commercial — Repertoire clients (M1). Mappe ici sur le persona
// "tout" en attendant les vrais roles metier (bureau/compta/
// commerciale/usine) seedes par ERP-74. Pas de nouveau persona
@@ -121,7 +127,7 @@ export const personas: Record<PersonaKey, Persona> = {
'logistique.weighing_tickets.view',
'logistique.weighing_tickets.manage',
],
expectedAdminLinks: ['users', 'roles', 'sites', 'categories', 'products', 'audit-log'],
expectedAdminLinks: ['users', 'roles', 'sites', 'categories', 'products', 'storages', 'audit-log'],
},
'user-readonly': {
key: 'user-readonly',
@@ -166,4 +172,4 @@ export function getPersona(key: PersonaKey): Persona {
return personas[key]
}
export const ALL_ADMIN_LINKS = ['users', 'roles', 'sites', 'categories', 'products', 'audit-log'] as const
export const ALL_ADMIN_LINKS = ['users', 'roles', 'sites', 'categories', 'products', 'storages', 'audit-log'] as const
+1
View File
@@ -234,6 +234,7 @@ test-db-setup:
$(SYMFONY_CONSOLE) --env=test dbal:run-sql "CREATE UNIQUE INDEX IF NOT EXISTS uq_provider_company_name_active ON provider (LOWER(company_name)) WHERE is_archived = FALSE AND deleted_at IS NULL"
$(SYMFONY_CONSOLE) --env=test dbal:run-sql "CREATE UNIQUE INDEX IF NOT EXISTS uq_carrier_name_active ON carrier (LOWER(name)) WHERE is_archived = FALSE AND deleted_at IS NULL"
$(SYMFONY_CONSOLE) --env=test dbal:run-sql "CREATE UNIQUE INDEX IF NOT EXISTS uq_product_code_active ON product (code) WHERE deleted_at IS NULL"
$(SYMFONY_CONSOLE) --env=test dbal:run-sql "CREATE UNIQUE INDEX IF NOT EXISTS uq_storage_site_type_numero_active ON storage (site_id, storage_type_id, numero) WHERE deleted_at IS NULL"
fixtures:
$(SYMFONY_CONSOLE) --no-interaction doctrine:fixtures:load
+136
View File
@@ -0,0 +1,136 @@
<?php
declare(strict_types=1);
namespace DoctrineMigrations;
use App\Shared\Infrastructure\Database\ColumnCommentsCatalog;
use Doctrine\DBAL\Schema\Schema;
use Doctrine\Migrations\AbstractMigration;
/**
* M7 — Stockage (ERP-211) : creation du schema BDD de la table `storage`.
*
* Un stockage = 1 site + 1 type de stockage (referentiel storage_type du M6) + 1
* numero, avec des etats multi-valeur (JSONB), soft-delete prepare et colonnes
* Timestampable/Blamable (spec-back § 3.2).
*
* Contraintes metier portees ici :
* - RG-7.01 : unicite du couple (site, type, numero) parmi les stockages ACTIFS,
* via l'index UNIQUE partiel uq_storage_site_type_numero_active (WHERE deleted_at
* IS NULL) — un numero peut etre reutilise apres soft-delete.
* - RG-7.04 : au moins un etat, via chk_storage_states_not_empty
* (jsonb_array_length(states) >= 1). Comme pour product.states (M6), PAS de
* DEFAULT '[]'::jsonb : un tableau vide violerait ce CHECK ; la colonne est
* toujours renseignee par l'app (Processor/ORM).
*
* Namespace racine `DoctrineMigrations` (regle ABSOLUE n°11) et NON modulaire : la
* table storage porte des FK cross-module (site, storage_type, user). Le tri par
* timestamp au sein du namespace racine garantit l'ordre apres la creation de ces
* tables sur base vide ; un namespace modulaire trierait par FQCN alphabetique et
* casserait `make db-reset` (cf. Version20260625110000 pour le M6).
*
* Convention IDs (spec § 2.2) : `INT GENERATED BY DEFAULT AS IDENTITY`,
* horodatages `TIMESTAMP(0) WITHOUT TIME ZONE` (le TimestampableBlamableTrait mappe
* `datetime_immutable`). Chaque colonne porte son `COMMENT ON COLUMN` (regle n°12).
*
* NB schema:update (test-db-setup) : `storage` sera mappee en ORM au ticket suivant
* (entite Storage). D'ici la, `schema:update --force` la drope sur la base de TEST
* uniquement (sans impact : aucun test ne la reference encore, et dev/prod ne lancent
* jamais schema:update). Sa description sera ajoutee a ColumnCommentsCatalog au ticket
* entite (comme product / weighing_ticket).
*/
final class Version20260629120000 extends AbstractMigration
{
public function getDescription(): string
{
return 'ERP-211 (M7) : creation de la table storage (FK site + storage_type, unicite metier partielle RG-7.01, etats JSONB RG-7.04, soft-delete + Timestampable/Blamable).';
}
public function up(Schema $schema): void
{
$this->addSql(<<<'SQL'
CREATE TABLE storage (
id INT GENERATED BY DEFAULT AS IDENTITY NOT NULL,
site_id INT NOT NULL,
storage_type_id INT NOT NULL,
numero VARCHAR(50) NOT NULL,
-- Pas de DEFAULT : un tableau vide violerait chk_storage_states_not_empty
-- (RG-7.04). La colonne est toujours renseignee par l'app (Processor/ORM).
states JSONB NOT NULL,
deleted_at TIMESTAMP(0) WITHOUT TIME ZONE DEFAULT NULL,
created_at TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL,
updated_at TIMESTAMP(0) WITHOUT TIME ZONE NOT NULL,
created_by INT DEFAULT NULL,
updated_by INT DEFAULT NULL,
PRIMARY KEY (id),
CONSTRAINT chk_storage_states_not_empty
CHECK (jsonb_array_length(states) >= 1),
CONSTRAINT fk_storage_site
FOREIGN KEY (site_id) REFERENCES site (id) ON DELETE RESTRICT,
CONSTRAINT fk_storage_storage_type
FOREIGN KEY (storage_type_id) REFERENCES storage_type (id) ON DELETE RESTRICT,
CONSTRAINT fk_storage_created_by
FOREIGN KEY (created_by) REFERENCES "user" (id) ON DELETE SET NULL,
CONSTRAINT fk_storage_updated_by
FOREIGN KEY (updated_by) REFERENCES "user" (id) ON DELETE SET NULL
)
SQL);
// RG-7.01 : unicite (site, type, numero) parmi les stockages actifs uniquement
// (index partiel) — un numero redevient disponible apres soft-delete.
$this->addSql('CREATE UNIQUE INDEX uq_storage_site_type_numero_active ON storage (site_id, storage_type_id, numero) WHERE deleted_at IS NULL');
$this->addSql('CREATE INDEX idx_storage_site ON storage (site_id)');
$this->addSql('CREATE INDEX idx_storage_storage_type ON storage (storage_type_id)');
$this->addSql('CREATE INDEX idx_storage_deleted_at ON storage (deleted_at)');
$this->addSql('CREATE INDEX idx_storage_created_by ON storage (created_by)');
$this->addSql('CREATE INDEX idx_storage_updated_by ON storage (updated_by)');
$this->comment('storage', '_table', 'Emplacements de stockage (M7 Catalog) un stockage = 1 site + 1 type (storage_type) + 1 numero, etats multi-valeur JSONB, soft-delete + Timestampable/Blamable.');
$this->comment('storage', 'id', 'Identifiant interne auto-incremente.');
$this->comment('storage', 'site_id', 'Site du stockage. FK -> site.id, ON DELETE RESTRICT. Composante de l unicite metier (RG-7.01).');
$this->comment('storage', 'storage_type_id', 'Type de stockage (referentiel M6). FK -> storage_type.id, ON DELETE RESTRICT. Composante de l unicite metier (RG-7.01).');
$this->comment('storage', 'numero', 'Numero du stockage ( 50), saisi. Unique par (site, type) parmi les actifs (RG-7.01, uq_storage_site_type_numero_active). Normalise serveur.');
$this->comment('storage', 'states', 'Etats du stockage (JSON) : tableau non vide (>= 1 element, RG-7.04, chk_storage_states_not_empty). Multi-valeur.');
$this->comment('storage', 'deleted_at', 'Horodatage du soft-delete technique null = ligne active. Une ligne supprimee sort de l unicite metier (index partiel uq_storage_site_type_numero_active).');
$this->addTimestampableBlamableComments('storage');
}
public function down(Schema $schema): void
{
$this->addSql('DROP TABLE storage');
}
/**
* Pose les 4 commentaires standardises Timestampable/Blamable sur une table,
* en reutilisant le catalogue partage (source unique, ERP-67).
*/
private function addTimestampableBlamableComments(string $table): void
{
foreach (ColumnCommentsCatalog::timestampableBlamableComments() as $column => $description) {
$this->comment($table, $column, $description);
}
}
/**
* Emet un `COMMENT ON TABLE` (colonne speciale `_table`) ou `COMMENT ON COLUMN`
* en dollar-quoting Postgres ($_$...$_$) pour eviter tout echappement d apostrophe.
*/
private function comment(string $table, string $column, string $description): void
{
$quotedTable = '"'.str_replace('"', '""', $table).'"';
if ('_table' === $column) {
$this->addSql(sprintf('COMMENT ON TABLE %s IS $_$%s$_$', $quotedTable, $description));
return;
}
$this->addSql(sprintf(
'COMMENT ON COLUMN %s.%s IS $_$%s$_$',
$quotedTable,
'"'.str_replace('"', '""', $column).'"',
$description,
));
}
}
@@ -0,0 +1,35 @@
<?php
declare(strict_types=1);
namespace App\Module\Catalog\Application\Service;
/**
* Normalisation serveur des champs texte d'un Storage, appliquee par le
* StorageProcessor AVANT l'unicite metier et la persistance (RG-7.06). Jumeau du
* ProductFieldNormalizer (M6), recentre sur l'unique champ texte du stockage.
*
* - numero : trim simple, SANS changement de casse (HP-M7-05 : pas d'UPPER par
* defaut, contrairement au code produit). Le numero est saisi tel quel et sert
* l'unicite metier (site, type, numero) parmi les actifs (RG-7.01).
*
* La methode est null-safe et trim l'entree ; une chaine vide apres trim devient
* null (c'est l'Assert\NotBlank de l'entite qui rejette le vide, pas le normalizer).
*/
final class StorageFieldNormalizer
{
/**
* Numero de stockage trimme (RG-7.06), sans changement de casse (HP-M7-05).
* Conserve null tel quel ; une chaine vide apres trim devient null.
*/
public function normalizeNumero(?string $value): ?string
{
if (null === $value) {
return null;
}
$value = trim($value);
return '' === $value ? null : $value;
}
}
+5
View File
@@ -47,6 +47,11 @@ final class CatalogModule
// Item sidebar dans la section Administration, sous « Repertoire transporteurs ».
['code' => 'catalog.products.view', 'label' => 'Voir les produits'],
['code' => 'catalog.products.manage', 'label' => 'Gérer les produits (créer, éditer)'],
// Stockage (M7, ERP-210) : admin-only. Reutilise le referentiel
// StorageType du M6. Item sidebar dans la section Administration,
// pres des items Catalog (produits, categories).
['code' => 'catalog.storages.view', 'label' => 'Voir les stockages'],
['code' => 'catalog.storages.manage', 'label' => 'Gérer les stockages (créer, éditer)'],
];
}
}
@@ -0,0 +1,264 @@
<?php
declare(strict_types=1);
namespace App\Module\Catalog\Domain\Entity;
use ApiPlatform\Metadata\ApiResource;
use ApiPlatform\Metadata\Get;
use ApiPlatform\Metadata\GetCollection;
use ApiPlatform\Metadata\Patch;
use ApiPlatform\Metadata\Post;
use App\Module\Catalog\Infrastructure\ApiPlatform\State\Processor\StorageProcessor;
use App\Module\Catalog\Infrastructure\ApiPlatform\State\Provider\StorageProvider;
use App\Module\Catalog\Infrastructure\Doctrine\DoctrineStorageRepository;
use App\Module\Sites\Domain\Entity\Site;
use App\Shared\Domain\Attribute\Auditable;
use App\Shared\Domain\Contract\BlamableInterface;
use App\Shared\Domain\Contract\TimestampableInterface;
use App\Shared\Domain\Trait\TimestampableBlamableTrait;
use DateTimeImmutable;
use Doctrine\ORM\Mapping as ORM;
use Symfony\Component\Serializer\Attribute\Groups;
use Symfony\Component\Validator\Constraints as Assert;
/**
* Emplacement de stockage (M7 Catalog) — entite racine du module Stockage, jumelle
* de Product (M6) cote pattern (#[Auditable], TimestampableBlamable, soft-delete,
* etats multi-valeur JSONB) et cote contrat de serialisation (RETEX M1 — spec § 4.0).
*
* Un stockage = 1 site + 1 type de stockage (referentiel storage_type du M6) + 1
* numero. Le couple (site, type, numero) est unique parmi les stockages ACTIFS
* (RG-7.01, index partiel uq_storage_site_type_numero_active possede par la
* migration). Les etats (RECEPTION / PRODUCTION / TRIAGE) sont multi-valeur, au
* moins un (RG-7.04, CHECK chk_storage_states_not_empty).
*
* Contrat de serialisation :
* - LISTE / DETAIL (storage:read + site:read + storage_type:read + default:read) :
* numero, states, displayName (RG-7.05), site et storageType embarques (ensembles
* bornes -> embed autorise, ne viole pas la regle n°13), createdAt/updatedAt
* (via default:read). L'ecriture passe par storage:write (site, storageType,
* numero, states).
*
* Soft-delete prepare via `deletedAt` (non expose, § 2.8) : pas de Delete dans les
* operations ; la liste exclut les stockages supprimes (Provider, ERP-213). Un
* numero redevient disponible apres soft-delete (index partiel sur les actifs).
*
* NB : `Site` appartient au module Sites, consomme en relation ORM partagee (§ 2.1)
* — on reutilise son read-group `site:read`, sans logique inter-module. `StorageType`
* est dans le meme module Catalog.
*
* @see StorageProvider Lecture (liste paginee filtree soft-delete + item) — ERP-213.
* @see StorageProcessor Ecriture (normalisation, unicite metier RG-7.01) — ERP-213.
*/
#[ApiResource(
operations: [
new GetCollection(
security: "is_granted('catalog.storages.view')",
normalizationContext: ['groups' => ['storage:read', 'site:read', 'storage_type:read', 'default:read']],
provider: StorageProvider::class,
),
new Get(
security: "is_granted('catalog.storages.view')",
normalizationContext: ['groups' => ['storage:read', 'site:read', 'storage_type:read', 'default:read']],
provider: StorageProvider::class,
),
new Post(
security: "is_granted('catalog.storages.manage')",
normalizationContext: ['groups' => ['storage:read', 'site:read', 'storage_type:read', 'default:read']],
denormalizationContext: ['groups' => ['storage:write']],
// Convertit les erreurs de denormalisation (type invalide / null sur une
// relation : site, storageType) en violations 422 portant un propertyPath,
// au lieu d'un 400 qui court-circuite la validation (cf. Product — mapping
// inline useFormErrors, ERP-101).
collectDenormalizationErrors: true,
processor: StorageProcessor::class,
),
new Patch(
security: "is_granted('catalog.storages.manage')",
normalizationContext: ['groups' => ['storage:read', 'site:read', 'storage_type:read', 'default:read']],
denormalizationContext: ['groups' => ['storage:write']],
collectDenormalizationErrors: true,
provider: StorageProvider::class,
processor: StorageProcessor::class,
),
// Pas de Delete au M7 (§ 2.8) ; soft-delete prepare non expose.
],
)]
#[ORM\Entity(repositoryClass: DoctrineStorageRepository::class)]
#[ORM\Table(name: 'storage')]
// Index nommes pour matcher la migration (cf. Product). L'index unique partiel
// `uq_storage_site_type_numero_active` ((site, type, numero) WHERE deleted_at IS
// NULL — unicite metier parmi les actifs, RG-7.01) reste possede par la seule
// migration : Doctrine ORM ne sait pas exprimer un index partiel via attribut.
#[ORM\Index(name: 'idx_storage_site', columns: ['site_id'])]
#[ORM\Index(name: 'idx_storage_storage_type', columns: ['storage_type_id'])]
#[ORM\Index(name: 'idx_storage_deleted_at', columns: ['deleted_at'])]
#[ORM\Index(name: 'idx_storage_created_by', columns: ['created_by'])]
#[ORM\Index(name: 'idx_storage_updated_by', columns: ['updated_by'])]
#[Auditable]
class Storage implements TimestampableInterface, BlamableInterface
{
// === Timestampable + Blamable ===
// Les 4 colonnes (created_at, updated_at, created_by, updated_by) + leurs
// getters/setters viennent du Trait Shared, remplies automatiquement par le
// TimestampableBlamableSubscriber au prePersist / preUpdate.
use TimestampableBlamableTrait;
/** Etats du stockage (RG-7.04) — valeurs autorisees de la colonne JSONB `states`. */
public const string STATE_RECEPTION = 'RECEPTION';
public const string STATE_PRODUCTION = 'PRODUCTION';
public const string STATE_TRIAGE = 'TRIAGE';
#[ORM\Id]
#[ORM\GeneratedValue]
#[ORM\Column]
#[Groups(['storage:read'])]
private ?int $id = null;
// Site du stockage (obligatoire). FK ON DELETE RESTRICT : un site reference par
// un stockage ne peut etre supprime. Composante de l'unicite metier (RG-7.01).
#[ORM\ManyToOne(targetEntity: Site::class)]
#[ORM\JoinColumn(name: 'site_id', referencedColumnName: 'id', nullable: false, onDelete: 'RESTRICT')]
#[Assert\NotNull(message: 'Le site est obligatoire.')]
#[Groups(['storage:read', 'storage:write'])]
private ?Site $site = null;
// Type de stockage (obligatoire, referentiel plat M6). FK ON DELETE RESTRICT :
// un type reference par un stockage ne peut etre supprime. Composante de
// l'unicite metier (RG-7.01).
#[ORM\ManyToOne(targetEntity: StorageType::class)]
#[ORM\JoinColumn(name: 'storage_type_id', referencedColumnName: 'id', nullable: false, onDelete: 'RESTRICT')]
#[Assert\NotNull(message: 'Le type de stockage est obligatoire.')]
#[Groups(['storage:read', 'storage:write'])]
private ?StorageType $storageType = null;
// Numero du stockage, saisi. Unique par (site, type) parmi les actifs (RG-7.01).
// Normalise serveur (trim) par le StorageProcessor (ERP-213).
#[ORM\Column(length: 50)]
#[Assert\NotBlank(message: 'Le numéro du stockage est obligatoire.', normalizer: 'trim')]
#[Assert\Length(max: 50, maxMessage: 'Le numéro du stockage ne peut pas dépasser {{ limit }} caractères.', normalizer: 'trim')]
#[Groups(['storage:read', 'storage:write'])]
private ?string $numero = null;
/**
* Etats du stockage (multi-select), sous-ensemble non vide de
* {RECEPTION, PRODUCTION, TRIAGE} (RG-7.04). Stocke en JSONB (tableau de
* chaines), non-vacuite garantie aussi par le CHECK chk_storage_states_not_empty.
*
* Validation des valeurs via Assert\Choice(multiple: true) plutot que
* Assert\All([Choice]) : equivalent fonctionnel, et seul Choice est gere par le
* garde-fou EntityConstraintsHaveFrenchMessageTest (cf. Product::states).
*
* @var list<string>
*/
// jsonb (pas json) : aligne le mapping ORM sur la colonne JSONB creee par la
// migration (CHECK chk_storage_states_not_empty via jsonb_array_length). Sans
// `options: ['jsonb' => true]`, schema:update tente un ALTER states TYPE JSON
// qui casse le CHECK et fait echouer make test-db-setup (cf. Product::states).
#[ORM\Column(type: 'json', options: ['jsonb' => true])]
#[Assert\Count(min: 1, minMessage: 'Sélectionnez au moins un état.')]
#[Assert\Choice(
choices: [self::STATE_RECEPTION, self::STATE_PRODUCTION, self::STATE_TRIAGE],
multiple: true,
message: 'État de stockage invalide.',
multipleMessage: 'État de stockage invalide.',
)]
#[Groups(['storage:read', 'storage:write'])]
private array $states = [];
/**
* Soft-delete technique : null = actif, valeur = supprime logiquement le {date}.
* Non expose (§ 2.8, aucun groupe) : prepare pour une future suppression. La
* liste exclut par defaut les stockages supprimes (Provider, ERP-213) et le
* numero redevient disponible (index partiel sur les actifs, RG-7.01).
*/
#[ORM\Column(name: 'deleted_at', type: 'datetime_immutable', nullable: true)]
private ?DateTimeImmutable $deletedAt = null;
public function getId(): ?int
{
return $this->id;
}
public function getSite(): ?Site
{
return $this->site;
}
public function setSite(?Site $site): static
{
$this->site = $site;
return $this;
}
public function getStorageType(): ?StorageType
{
return $this->storageType;
}
public function setStorageType(?StorageType $storageType): static
{
$this->storageType = $storageType;
return $this;
}
public function getNumero(): ?string
{
return $this->numero;
}
public function setNumero(string $numero): static
{
$this->numero = $numero;
return $this;
}
/**
* @return list<string>
*/
public function getStates(): array
{
return $this->states;
}
/**
* @param list<string> $states
*/
public function setStates(array $states): static
{
$this->states = $states;
return $this;
}
public function getDeletedAt(): ?DateTimeImmutable
{
return $this->deletedAt;
}
public function setDeletedAt(?DateTimeImmutable $deletedAt): static
{
$this->deletedAt = $deletedAt;
return $this;
}
/**
* RG-7.05 : libelle d'affichage = libelle du type de stockage suivi du numero
* (ex. « Cellule 12 »). Getter virtuel non persiste, expose en lecture
* (storage:read). Null-safe : `storageType` et `numero` sont garantis non nuls a
* la lecture (NOT NULL en base), le `?? ''` couvre un objet en cours de
* construction sans casser la serialisation.
*/
#[Groups(['storage:read'])]
public function getDisplayName(): string
{
$label = $this->storageType?->getLabel() ?? '';
return trim($label.' '.($this->numero ?? ''));
}
}
@@ -0,0 +1,48 @@
<?php
declare(strict_types=1);
namespace App\Module\Catalog\Domain\Repository;
use App\Module\Catalog\Domain\Entity\Storage;
use Doctrine\ORM\QueryBuilder;
interface StorageRepositoryInterface
{
public function findById(int $id): ?Storage;
public function save(Storage $storage): void;
/**
* Vrai si un stockage actif (deleted_at IS NULL) porte deja le triplet
* (site, storageType, numero). `$excludeId` exclut un stockage precis du test
* (cas PATCH). Garantit l'unicite metier parmi les actifs (RG-7.01, index
* partiel uq_storage_site_type_numero_active). Un numero redevient disponible
* apres soft-delete (le test ignore les supprimes).
*/
public function existsActiveBySiteTypeNumero(
int $siteId,
int $storageTypeId,
string $numero,
?int $excludeId = null,
): bool;
/**
* QueryBuilder de la liste stockages (consomme par le StorageProvider) : exclut
* par defaut les soft-deleted (RG-7.07), trie par site.code ASC, storageType.label
* ASC, numero ASC (defaut spec § 4.1) et applique les filtres optionnels :
* - `$search` : recherche partielle case-insensitive sur `numero`.
* - `$siteIds` : stockage rattache a AU MOINS UN des sites passes.
* - `$storageTypeId` : restreint a un type de stockage precis (par id).
* - `$state` : appartenance a la colonne JSONB `states` (RECEPTION|PRODUCTION|TRIAGE).
*
* @param list<int> $siteIds
*/
public function createListQueryBuilder(
bool $includeDeleted = false,
?string $search = null,
array $siteIds = [],
?int $storageTypeId = null,
?string $state = null,
): QueryBuilder;
}
@@ -0,0 +1,112 @@
<?php
declare(strict_types=1);
namespace App\Module\Catalog\Infrastructure\ApiPlatform\State\Processor;
use ApiPlatform\Metadata\Operation;
use ApiPlatform\State\ProcessorInterface;
use App\Module\Catalog\Application\Service\StorageFieldNormalizer;
use App\Module\Catalog\Domain\Entity\Storage;
use App\Module\Catalog\Domain\Repository\StorageRepositoryInterface;
use Doctrine\DBAL\Exception\UniqueConstraintViolationException;
use Symfony\Component\DependencyInjection\Attribute\Autowire;
use Symfony\Component\HttpKernel\Exception\ConflictHttpException;
use Throwable;
use function sprintf;
/**
* Processor d'ecriture du stockage (M7, POST / PATCH). Cf. spec-back M7 § 4.3 /
* § 4.4 + RG-7.01 / RG-7.06. Jumeau du ProductProcessor (normalisation serveur +
* 409 doublon).
*
* Sequence (POST / PATCH) :
* 1. Normalisation serveur (RG-7.06) via StorageFieldNormalizer : numero trim
* (pas d'UPPER — HP-M7-05). Jouee AVANT l'unicite et la persistance ; la
* validation (NotNull site/type, NotBlank/Length numero, Count/Choice states
* RG-7.04) a deja joue cote API Platform sur la saisie brute.
* 2. RG-7.01 : unicite metier du triplet (site, storageType, numero) parmi les
* actifs. Pre-check deterministe (excluant le stockage courant en PATCH) -> 409 ;
* l'index partiel uq_storage_site_type_numero_active reste le filet anti-race
* au flush.
* 3. Persistance via le persist_processor Doctrine ORM.
*
* RG-7.03 (« le type doit etre disponible sur le site choisi ») n'est PAS portee :
* le concept type<->site a ete retire du modele en M6 (StorageType rendu plat,
* jointure storage_type_site droppee — migration Version20260626100000). C'est
* desormais l'entite Storage (1 site + 1 type) qui materialise cette disponibilite ;
* il n'existe plus de referentiel a interroger. A reclarifier cote spec (signale).
*
* Mode strict PATCH (RETEX M1) : la security d'operation exige `catalog.storages.manage`
* pour TOUS les champs ecrivables (un seul niveau de permission au M7 — admin-only).
* Aucun champ « hors-permission » a re-gater finement ici : le 403 global est porte
* par la security d'operation.
*
* @implements ProcessorInterface<Storage, Storage>
*/
final class StorageProcessor implements ProcessorInterface
{
public function __construct(
#[Autowire(service: 'api_platform.doctrine.orm.state.persist_processor')]
private readonly ProcessorInterface $persistProcessor,
private readonly StorageFieldNormalizer $normalizer,
#[Autowire(service: 'App\Module\Catalog\Infrastructure\Doctrine\DoctrineStorageRepository')]
private readonly StorageRepositoryInterface $repository,
) {}
public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): mixed
{
if (!$data instanceof Storage) {
return $this->persistProcessor->process($data, $operation, $uriVariables, $context);
}
// 1. RG-7.06 : normalisation serveur (numero trim, pas d'UPPER).
$this->normalize($data);
// 2. RG-7.01 : unicite metier (site, storageType, numero) parmi les actifs
// (exclut le stockage courant en PATCH). Pre-check explicite -> 409
// deterministe. Le NotNull site/type + NotBlank numero ont deja joue.
$siteId = $data->getSite()?->getId();
$typeId = $data->getStorageType()?->getId();
$numero = (string) $data->getNumero();
if (null !== $siteId && null !== $typeId && '' !== $numero
&& $this->repository->existsActiveBySiteTypeNumero($siteId, $typeId, $numero, $data->getId())) {
throw $this->duplicateConflict($numero);
}
// 3. Persistance, avec filet anti-race sur l'index partiel.
try {
return $this->persistProcessor->process($data, $operation, $uriVariables, $context);
} catch (UniqueConstraintViolationException $e) {
// Insertion concurrente du meme triplet entre le pre-check et le flush
// (collision sur uq_storage_site_type_numero_active).
throw $this->duplicateConflict($numero, $e);
}
}
/**
* Normalisation serveur du stockage (RG-7.06). Le setter n'est touche que si une
* valeur est presente, pour ne jamais ecraser l'existant lors d'un PATCH partiel.
* Le cast (string) est sur : NotBlank a deja rejete le vide en amont.
*/
private function normalize(Storage $data): void
{
if (null !== $data->getNumero()) {
$data->setNumero((string) $this->normalizer->normalizeNumero($data->getNumero()));
}
}
/**
* RG-7.01 : 409 sur doublon (site, type, numero). Le front mappe ce conflit sur
* le champ `numero` (setError('numero', ...) + toast — convention useFormErrors
* ERP-101) : le propertyPath exploitable est `numero`.
*/
private function duplicateConflict(string $numero, ?Throwable $previous = null): ConflictHttpException
{
return new ConflictHttpException(
sprintf('Un stockage portant le numéro « %s » existe déjà pour ce site et ce type de stockage.', $numero),
$previous,
);
}
}
@@ -0,0 +1,172 @@
<?php
declare(strict_types=1);
namespace App\Module\Catalog\Infrastructure\ApiPlatform\State\Provider;
use ApiPlatform\Doctrine\Orm\Paginator;
use ApiPlatform\Metadata\CollectionOperationInterface;
use ApiPlatform\Metadata\Operation;
use ApiPlatform\State\Pagination\Pagination;
use ApiPlatform\State\ProviderInterface;
use App\Module\Catalog\Domain\Entity\Storage;
use App\Module\Catalog\Domain\Repository\StorageRepositoryInterface;
use Doctrine\ORM\Tools\Pagination\Paginator as DoctrinePaginator;
use Symfony\Component\DependencyInjection\Attribute\Autowire;
use function in_array;
use function is_int;
use function is_string;
/**
* Provider Storage (lecture, ERP-213) :
* - LISTE : exclut par defaut les stockages soft-deleted (RG-7.07), trie par
* site.code ASC, storageType.label ASC, numero ASC (defaut spec § 4.1), applique
* les filtres (?search sur numero, ?siteId[], ?storageTypeId, ?state) et renvoie
* une collection PAGINEE Hydra (regle ABSOLUE n°13 : jamais d'array brut sur une
* operation de collection — on enveloppe le QueryBuilder dans le Paginator ORM).
* Echappatoire ?pagination=false respectee (alimentation d'un select).
* - ITEM : recharge le stockage puis renvoie null (404) s'il est soft-deleted — le
* soft-delete n'est jamais expose (§ 2.8), aucun flag includeDeleted.
*
* @implements ProviderInterface<Storage>
*/
final class StorageProvider implements ProviderInterface
{
/** Etats valides du filtre ?state= (enum borne, RG-7.04). */
private const array VALID_STATES = [
Storage::STATE_RECEPTION,
Storage::STATE_PRODUCTION,
Storage::STATE_TRIAGE,
];
public function __construct(
#[Autowire(service: 'App\Module\Catalog\Infrastructure\Doctrine\DoctrineStorageRepository')]
private readonly StorageRepositoryInterface $repository,
private readonly Pagination $pagination,
) {}
public function provide(Operation $operation, array $uriVariables = [], array $context = []): iterable|Paginator|Storage|null
{
if ($operation instanceof CollectionOperationInterface) {
// includeDeleted toujours false : le soft-delete n'est pas expose (§ 2.8).
$qb = $this->repository->createListQueryBuilder(
false,
$this->readSearch($context),
$this->readSiteIds($context),
$this->readStorageTypeId($context),
$this->readState($context),
);
// Echappatoire ?pagination=false : collection complete sans Paginator.
if (!$this->pagination->isEnabled($operation, $context)) {
return $qb->getQuery()->getResult();
}
// Branche paginee standard : offset/limit via Pagination, enveloppe dans le
// Paginator ORM. Les jointures site/storageType sont to-ONE (ManyToOne) :
// pas de duplication de lignes, le comptage reste exact.
$limit = $this->pagination->getLimit($operation, $context);
$page = max(1, $this->pagination->getPage($context));
$offset = ($page - 1) * $limit;
$qb->setFirstResult($offset)->setMaxResults($limit);
return new Paginator(new DoctrinePaginator($qb->getQuery()));
}
// 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;
}
$storage = $this->repository->findById((int) $id);
if (null === $storage) {
return null;
}
// § 2.8 : un stockage soft-deleted n'est jamais expose (404).
if (null !== $storage->getDeletedAt()) {
return null;
}
return $storage;
}
/**
* Lit le filtre `?search=` (recherche partielle sur numero). Renvoie la valeur
* trimmee ou null si absente / vide.
*/
private function readSearch(array $context): ?string
{
$raw = $context['filters']['search'] ?? null;
if (!is_string($raw)) {
return null;
}
$raw = trim($raw);
return '' === $raw ? null : $raw;
}
/**
* Lit le filtre `?siteId[]=` : ids des sites coches (OR). Tolere une valeur
* scalaire unique (`?siteId=1`) ou un tableau. Ignore les entrees non numeriques.
*
* @return list<int>
*/
private function readSiteIds(array $context): array
{
$raw = $context['filters']['siteId'] ?? null;
if (null === $raw) {
return [];
}
$values = is_array($raw) ? $raw : [$raw];
$ids = [];
foreach ($values as $value) {
if (is_int($value) || (is_string($value) && ctype_digit($value))) {
$ids[] = (int) $value;
}
}
return array_values(array_unique($ids));
}
/**
* Lit le filtre `?storageTypeId=` (drawer « Filtrer »). Renvoie l'id entier ou
* null si absent / non numerique.
*/
private function readStorageTypeId(array $context): ?int
{
$raw = $context['filters']['storageTypeId'] ?? null;
if (is_int($raw)) {
return $raw;
}
return is_string($raw) && ctype_digit($raw) ? (int) $raw : null;
}
/**
* Lit le filtre `?state=` (RECEPTION / PRODUCTION / TRIAGE). Normalise en
* majuscules et n'accepte qu'une valeur de l'enum borne ; toute autre valeur est
* ignoree (null).
*/
private function readState(array $context): ?string
{
$raw = $context['filters']['state'] ?? null;
if (!is_string($raw) || '' === trim($raw)) {
return null;
}
$state = mb_strtoupper(trim($raw), 'UTF-8');
return in_array($state, self::VALID_STATES, true) ? $state : null;
}
}
@@ -0,0 +1,234 @@
<?php
declare(strict_types=1);
namespace App\Module\Catalog\Infrastructure\Controller;
use App\Module\Catalog\Domain\Entity\Storage;
use App\Module\Catalog\Domain\Repository\StorageRepositoryInterface;
use App\Module\Sites\Domain\Entity\Site;
use App\Shared\Domain\Contract\SpreadsheetExporterInterface;
use DateTimeImmutable;
use Symfony\Component\DependencyInjection\Attribute\Autowire;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpKernel\Attribute\AsController;
use Symfony\Component\Routing\Attribute\Route;
use Symfony\Component\Security\Http\Attribute\IsGranted;
use function in_array;
use function is_int;
use function is_string;
/**
* Export XLSX de la liste des stockages (M7, spec-back § 4.5). Jumeau du
* ProductExportController (M6) — reference en prose volontairement (pas de {@see}
* inter-module).
*
* Controller Symfony custom (et non operation API Platform) car il produit un
* binaire de fichier, pas une representation Hydra. `priority: 1` est OBLIGATOIRE
* sur la route : sans cela API Platform capterait `/api/storages/export.xlsx`
* comme l'item `GET /api/storages/{id}.{_format}` (id="export", _format="xlsx")
* — cf. CLAUDE.md « controller custom sous /api ». Etant un controller (et non un
* #[ApiResource]), il n'est PAS scanne par CollectionsArePaginatedTest : aucune
* entree EXCLUDED necessaire (comme ProductExportController).
*
* Separation des responsabilites :
* - le COMMENT (generation du fichier) est delegue au service Shared
* {@see SpreadsheetExporterInterface} — generique, reutilisable, sans metier ;
* - le QUOI vit ICI : selection des stockages (MEMES filtres que
* `GET /api/storages` via le StorageProvider, deleguee a
* {@see StorageRepositoryInterface::createListQueryBuilder()} — l'export reflete
* exactement ce que l'utilisateur voit a l'ecran) et mapping metier des colonnes.
* Les stockages soft-deleted (RG-7.07) sont toujours exclus, comme en liste (le
* soft-delete n'est jamais expose, § 2.8).
*/
#[AsController]
final class StorageExportController
{
/**
* Libelles FR des etats (RG-7.04) pour la colonne « États ». L'ordre des cles
* fixe l'ordre d'affichage (Réception, Production, Triage) independamment de
* l'ordre de stockage en base.
*/
private const array STATE_LABELS = [
Storage::STATE_RECEPTION => 'Réception',
Storage::STATE_PRODUCTION => 'Production',
Storage::STATE_TRIAGE => 'Triage',
];
public function __construct(
#[Autowire(service: 'App\Module\Catalog\Infrastructure\Doctrine\DoctrineStorageRepository')]
private readonly StorageRepositoryInterface $repository,
private readonly SpreadsheetExporterInterface $exporter,
) {}
#[Route('/api/storages/export.xlsx', name: 'catalog_storages_export_xlsx', methods: ['GET'], priority: 1)]
#[IsGranted('catalog.storages.view')]
public function __invoke(Request $request): Response
{
// Memes filtres que la vue liste (StorageProvider) pour que l'export reflete
// exactement ce que l'utilisateur voit a l'ecran : recherche (?search sur
// numero), sites (?siteId[]), type (?storageTypeId), etat (?state).
// includeDeleted reste false : le soft-delete n'est jamais expose (§ 2.8).
$search = $request->query->getString('search') ?: null;
$siteIds = $this->readIntList($request->query->all()['siteId'] ?? []);
$storageTypeId = $this->readIntOrNull($request->query->get('storageTypeId'));
$state = $this->readState($request->query->get('state'));
/** @var list<Storage> $storages */
$storages = $this->repository
->createListQueryBuilder(false, $search, $siteIds, $storageTypeId, $state)
->getQuery()
->getResult()
;
$binary = $this->exporter->export(
'Stockages',
$this->buildHeaders(),
$this->buildRows($storages),
);
return $this->buildResponse($binary);
}
/**
* Colonnes de l'export (spec § 4.5).
*
* @return list<string>
*/
private function buildHeaders(): array
{
return [
'Nom',
'Site',
'Type de stockage',
'Numéro',
'États',
'Créé le',
'Modifié le',
];
}
/**
* @param list<Storage> $storages
*
* @return iterable<list<null|scalar>>
*/
private function buildRows(array $storages): iterable
{
foreach ($storages as $storage) {
yield [
$storage->getDisplayName(),
$this->formatSite($storage->getSite()),
$storage->getStorageType()?->getLabel(),
$storage->getNumero(),
$this->formatStates($storage),
$this->formatDate($storage->getCreatedAt()),
$this->formatDate($storage->getUpdatedAt()),
];
}
}
/**
* Libelle du site « Nom (Code) » (ex. « Chatellerault (86) »). Le code peut
* etre absent : on retombe alors sur le seul nom.
*/
private function formatSite(?Site $site): string
{
if (null === $site) {
return '';
}
$name = (string) $site->getName();
$code = $site->getCode();
return null !== $code && '' !== $code ? sprintf('%s (%s)', $name, $code) : $name;
}
/**
* Libelles FR des etats du stockage, dans l'ordre canonique (Réception,
* Production, Triage), joints par virgule. Une valeur inattendue est ignoree.
*/
private function formatStates(Storage $storage): string
{
$states = $storage->getStates();
$labels = [];
foreach (self::STATE_LABELS as $code => $label) {
if (in_array($code, $states, true)) {
$labels[] = $label;
}
}
return implode(', ', $labels);
}
/**
* Formate un horodatage en « jj/mm/aaaa hh:mm » (vide si null).
*/
private function formatDate(?DateTimeImmutable $date): string
{
return $date?->format('d/m/Y H:i') ?? '';
}
private function buildResponse(string $binary): Response
{
$filename = sprintf('stockages-%s.xlsx', new DateTimeImmutable()->format('Ymd'));
$response = new Response($binary);
$response->headers->set('Content-Type', 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet');
$response->headers->set('Content-Disposition', sprintf('attachment; filename="%s"', $filename));
return $response;
}
/**
* Lit le filtre `?state=` comme le StorageProvider : normalise en majuscules et
* n'accepte qu'une valeur de l'enum borne {RECEPTION, PRODUCTION, TRIAGE} ;
* toute autre valeur est ignoree (null).
*/
private function readState(mixed $raw): ?string
{
if (!is_string($raw) || '' === trim($raw)) {
return null;
}
$state = mb_strtoupper(trim($raw), 'UTF-8');
return in_array($state, array_keys(self::STATE_LABELS), true) ? $state : null;
}
/**
* Lit un identifiant entier positif unique (`?storageTypeId=`). Aligne sur
* StorageProvider (tolere int ou chaine numerique).
*/
private function readIntOrNull(mixed $raw): ?int
{
if (is_int($raw)) {
return $raw > 0 ? $raw : null;
}
return is_string($raw) && ctype_digit($raw) && (int) $raw > 0 ? (int) $raw : null;
}
/**
* Normalise un filtre en liste d'identifiants entiers positifs (valeur unique
* ou liste, `?siteId[]=`). Aligne sur StorageProvider.
*
* @return list<int>
*/
private function readIntList(mixed $raw): array
{
$values = is_array($raw) ? $raw : [$raw];
$out = [];
foreach ($values as $value) {
if ((is_int($value) || (is_string($value) && ctype_digit($value))) && (int) $value > 0) {
$out[] = (int) $value;
}
}
return $out;
}
}
@@ -0,0 +1,139 @@
<?php
declare(strict_types=1);
namespace App\Module\Catalog\Infrastructure\Doctrine;
use App\Module\Catalog\Domain\Entity\Storage;
use App\Module\Catalog\Domain\Repository\StorageRepositoryInterface;
use Doctrine\Bundle\DoctrineBundle\Repository\ServiceEntityRepository;
use Doctrine\ORM\QueryBuilder;
use Doctrine\Persistence\ManagerRegistry;
/**
* @extends ServiceEntityRepository<Storage>
*/
class DoctrineStorageRepository extends ServiceEntityRepository implements StorageRepositoryInterface
{
public function __construct(ManagerRegistry $registry)
{
parent::__construct($registry, Storage::class);
}
public function findById(int $id): ?Storage
{
return $this->find($id);
}
public function save(Storage $storage): void
{
$this->getEntityManager()->persist($storage);
$this->getEntityManager()->flush();
}
public function existsActiveBySiteTypeNumero(
int $siteId,
int $storageTypeId,
string $numero,
?int $excludeId = null,
): bool {
$qb = $this->createQueryBuilder('s')
->select('1')
->andWhere('s.site = :siteId')
->andWhere('s.storageType = :storageTypeId')
->andWhere('s.numero = :numero')
->andWhere('s.deletedAt IS NULL')
->setParameter('siteId', $siteId)
->setParameter('storageTypeId', $storageTypeId)
->setParameter('numero', $numero)
->setMaxResults(1)
;
if (null !== $excludeId) {
$qb->andWhere('s.id != :excludeId')->setParameter('excludeId', $excludeId);
}
return [] !== $qb->getQuery()->getResult();
}
public function createListQueryBuilder(
bool $includeDeleted = false,
?string $search = null,
array $siteIds = [],
?int $storageTypeId = null,
?string $state = null,
): QueryBuilder {
// Eager-load des relations embarquees en liste (storage:read) pour eviter un
// N+1 par stockage : site et storageType sont des ManyToOne (to-ONE, sures —
// pas de duplication de lignes, contrairement aux ManyToMany du Product). Les
// jointures servent aussi le tri (site.code, storageType.label).
$qb = $this->createQueryBuilder('s')
->leftJoin('s.site', 'site')->addSelect('site')
->leftJoin('s.storageType', 'st')->addSelect('st')
->orderBy('site.code', 'ASC')
->addOrderBy('st.label', 'ASC')
->addOrderBy('s.numero', 'ASC')
;
// RG-7.07 : la liste exclut par defaut les stockages soft-deleted.
if (!$includeDeleted) {
$qb->andWhere('s.deletedAt IS NULL');
}
// ?search= : recherche partielle case-insensitive sur numero. Les
// metacaracteres LIKE (%, _, \) sont echappes pour rester litteraux.
if (null !== $search && '' !== trim($search)) {
$escaped = str_replace(['\\', '%', '_'], ['\\\\', '\%', '\_'], trim($search));
$pattern = '%'.mb_strtolower($escaped, 'UTF-8').'%';
$qb->andWhere('LOWER(s.numero) LIKE :search')->setParameter('search', $pattern);
}
// ?siteId[]= : stockage rattache a AU MOINS UN des sites passes (OR). site est
// un ManyToOne (to-one) -> filtre direct sur la jointure, sans sous-requete
// EXISTS ni risque de masquer une collection (≠ Product.sites M2M).
if ([] !== $siteIds) {
$qb->andWhere('site.id IN (:siteIds)')->setParameter('siteIds', $siteIds);
}
// ?storageTypeId= : filtre par type de stockage precis (id).
if (null !== $storageTypeId) {
$qb->andWhere('st.id = :storageTypeId')->setParameter('storageTypeId', $storageTypeId);
}
// ?state= : appartenance a la colonne JSONB `states`. DQL ne sait pas exprimer
// la containment jsonb -> on resout les ids matchant en SQL natif (operateur
// @>), puis on contraint le QueryBuilder. Ids vides -> condition toujours
// fausse (aucun stockage), sans casser le reste de la requete.
if (null !== $state) {
$stateIds = $this->matchingStateIds($state);
if ([] === $stateIds) {
$qb->andWhere('1 = 0');
} else {
$qb->andWhere('s.id IN (:stateIds)')->setParameter('stateIds', $stateIds);
}
}
return $qb;
}
/**
* Ids des stockages dont la colonne JSONB `states` contient l'etat donne, via
* l'operateur de containment Postgres `@>`. L'etat est borne a l'enum
* {RECEPTION, PRODUCTION, TRIAGE} en amont (StorageProvider) — pas de saisie
* libre ici.
*
* @return list<int>
*/
private function matchingStateIds(string $state): array
{
$rows = $this->getEntityManager()->getConnection()
->executeQuery(
'SELECT id FROM storage WHERE states @> CAST(:state AS JSONB)',
['state' => (string) json_encode([$state])],
)
->fetchFirstColumn()
;
return array_map(static fn (mixed $id): int => (int) $id, $rows);
}
}
@@ -190,6 +190,10 @@ final class SeedE2ECommand extends Command
// p.3) : mappe sur le persona "tout". Miroir de personas.ts.
'catalog.products.view',
'catalog.products.manage',
// Stockage (M7, ERP-210). Admin-only : mappe sur le persona
// "tout". Miroir de personas.ts.
'catalog.storages.view',
'catalog.storages.manage',
// Commercial — Repertoire clients (M1). Mappe ici sur le
// persona "tout" en attendant les vrais roles metier
// (bureau/compta/commerciale/usine) seedes par ERP-74.
@@ -610,6 +610,20 @@ final class ColumnCommentsCatalog
'product_id' => 'FK -> product.id, ON DELETE CASCADE — produit concerne.',
'storage_type_id' => 'FK -> storage_type.id, ON DELETE RESTRICT — type de stockage rattache au produit.',
],
// M7 Catalog (ERP-212) — table desormais mappee par l'entite Storage :
// schema:update (test) la recree sans COMMENT -> app:apply-column-comments
// les rejoue depuis ce catalogue. Strings identiques aux COMMENT de la
// migration Version20260629120000 (ERP-211).
'storage' => [
'_table' => 'Emplacements de stockage (M7 Catalog) — un stockage = 1 site + 1 type (storage_type) + 1 numero, etats multi-valeur JSONB, soft-delete + Timestampable/Blamable.',
'id' => 'Identifiant interne auto-incremente.',
'site_id' => 'Site du stockage. FK -> site.id, ON DELETE RESTRICT. Composante de l unicite metier (RG-7.01).',
'storage_type_id' => 'Type de stockage (referentiel M6). FK -> storage_type.id, ON DELETE RESTRICT. Composante de l unicite metier (RG-7.01).',
'numero' => 'Numero du stockage (≤ 50), saisi. Unique par (site, type) parmi les actifs (RG-7.01, uq_storage_site_type_numero_active). Normalise serveur.',
'states' => 'Etats du stockage (JSON) : tableau non vide (>= 1 element, RG-7.04, chk_storage_states_not_empty). Multi-valeur.',
'deleted_at' => 'Horodatage du soft-delete technique — null = ligne active. Une ligne supprimee sort de l unicite metier (index partiel uq_storage_site_type_numero_active).',
] + self::timestampableBlamableComments(),
];
}
@@ -0,0 +1,202 @@
<?php
declare(strict_types=1);
namespace App\Tests\Module\Catalog\Api;
use ApiPlatform\Symfony\Bundle\Test\Client;
use App\Module\Catalog\Domain\Entity\Storage;
use App\Module\Catalog\Domain\Entity\StorageType;
use App\Module\Sites\Domain\Entity\Site;
use DateTimeImmutable;
use Symfony\Contracts\HttpClient\ResponseInterface;
/**
* Classe de base des tests fonctionnels de l'entite Storage (M7, module Catalog).
*
* Etend la base Catalog (helpers d'auth + personas metier) et ajoute ce qu'il faut
* pour exercer l'API stockage de bout en bout :
* - `seedStorageType()` : type de stockage de test (code prefixe pour cleanup).
* - `firstSite()` / `siteByCode()` : sites fixtures (86 / 17 / 82).
* - `authView()` : user non-admin portant la permission `catalog.storages.view`.
* - `validStoragePayload()` : payload POST de reference (IRIs site / storageType),
* surchargeable par cle.
* - `seedStorageEntity()` : seede un stockage via l'EM (id existant, soft-deleted).
* - `iri()` / `memberById()` / `violationPaths()` : utilitaires Hydra.
*
* Cleanup : on purge les stockages (toute la table — aucune fixture stockage en env
* test) AVANT le parent, car storage reference site / storage_type en FK ON DELETE
* RESTRICT. Les types de stockage de test (prefixe code) sont purges dans la foulee.
*
* @internal
*/
abstract class AbstractStorageApiTestCase extends AbstractCatalogApiTestCase
{
protected const string LD = 'application/ld+json';
protected const string MERGE = 'application/merge-patch+json';
/** Prefixe des codes de StorageType seedes par ces tests (purge ciblee). */
protected const string TEST_STORAGE_TYPE_PREFIX = 'TESTSTO';
protected function tearDown(): void
{
$em = $this->getEm();
// Stockages d'abord : ils referencent site / storage_type en FK RESTRICT.
$em->createQuery('DELETE FROM '.Storage::class)->execute();
// Types de stockage de test (prefixe code).
$em->createQuery('DELETE FROM '.StorageType::class.' s WHERE s.code LIKE :prefix')
->setParameter('prefix', self::TEST_STORAGE_TYPE_PREFIX.'%')
->execute()
;
parent::tearDown();
}
/**
* Cree un type de stockage de test (code prefixe TESTSTO pour le cleanup).
*/
protected function seedStorageType(string $label = 'Cellule test'): StorageType
{
$em = $this->getEm();
$storageType = new StorageType();
$storageType->setCode($this->uniqueCode(self::TEST_STORAGE_TYPE_PREFIX));
$storageType->setLabel($label);
$em->persist($storageType);
$em->flush();
return $storageType;
}
protected function siteByCode(string $code): Site
{
$site = $this->getEm()->getRepository(Site::class)->findOneBy(['code' => $code]);
self::assertInstanceOf(Site::class, $site, sprintf('Le site de code "%s" doit etre seede.', $code));
return $site;
}
protected function firstSite(): Site
{
$site = $this->getEm()->getRepository(Site::class)->findOneBy([]);
self::assertInstanceOf(Site::class, $site, 'Un site fixture est requis (SitesFixtures).');
return $site;
}
/**
* Client non-admin portant seulement `catalog.storages.view`.
*/
protected function authView(): Client
{
$creds = $this->createUserWithPermission('catalog.storages.view');
return $this->authenticatedClient($creds['username'], $creds['password']);
}
/**
* Payload POST de reference : un stockage valide (1 site, 1 type, 1 numero,
* 1 etat). Surchargeable par cle via $overrides (ex: ['numero' => 'A1']).
*
* @param array<string, mixed> $overrides
*
* @return array<string, mixed>
*/
protected function validStoragePayload(array $overrides = []): array
{
$site = $this->firstSite();
$storageType = $this->seedStorageType();
$base = [
'site' => $this->iri('sites', (int) $site->getId()),
'storageType' => $this->iri('storage_types', (int) $storageType->getId()),
'numero' => $this->uniqueCode('NUM'),
'states' => [Storage::STATE_RECEPTION],
];
return array_replace($base, $overrides);
}
/**
* Seede un stockage directement via l'EM (bypass Processor/Validator). Utile pour
* disposer d'un id existant (RBAC item, PATCH) ou d'un stockage soft-deleted
* (reutilisation du triplet — RG-7.01). Le site / le type manquants sont crees
* a la volee.
*
* @param list<string> $states
*/
protected function seedStorageEntity(
?string $numero = null,
array $states = [Storage::STATE_RECEPTION],
?DateTimeImmutable $deletedAt = null,
?Site $site = null,
?StorageType $storageType = null,
): Storage {
$em = $this->getEm();
$site ??= $this->firstSite();
$storage = new Storage();
$storage->setSite($em->getReference(Site::class, (int) $site->getId()));
$storage->setStorageType($storageType ?? $this->seedStorageType('Seed'));
$storage->setNumero($numero ?? $this->uniqueCode('NUM'));
$storage->setStates($states);
$storage->setDeletedAt($deletedAt);
$em->persist($storage);
$em->flush();
return $storage;
}
/**
* Construit un IRI API Platform (`/api/{resource}/{id}`).
*/
protected function iri(string $resource, int $id): string
{
return sprintf('/api/%s/%d', $resource, $id);
}
/**
* Identifiant unique de test (prefixe + nonce), deja en MAJUSCULE.
*/
protected function uniqueCode(string $prefix): string
{
return $prefix.'_'.strtoupper(substr(bin2hex(random_bytes(5)), 0, 10));
}
/**
* Extrait les `propertyPath` des violations d'une reponse 422.
*
* @return list<string>
*/
protected function violationPaths(ResponseInterface $response): array
{
$body = $response->toArray(false);
return array_values(array_map(
static fn (array $violation): string => (string) ($violation['propertyPath'] ?? ''),
$body['violations'] ?? [],
));
}
/**
* Retrouve un membre d'une collection Hydra par son id (ou null).
*
* @param array<string, mixed> $list
*
* @return null|array<string, mixed>
*/
protected function memberById(array $list, int $id): ?array
{
foreach ($list['member'] ?? [] as $member) {
if (($member['id'] ?? null) === $id) {
return $member;
}
}
return null;
}
}
+204
View File
@@ -0,0 +1,204 @@
<?php
declare(strict_types=1);
namespace App\Tests\Module\Catalog\Api;
use App\Module\Catalog\Domain\Entity\Storage;
use DateTimeImmutable;
/**
* Tests fonctionnels de l'API stockage (M7, spec-back § 4) — StorageProvider +
* StorageProcessor (ERP-213).
*
* Couvre : collection paginee Hydra + contrat de serialisation (site / storageType
* embarques, displayName), creation, normalisation serveur du numero (trim, RG-7.06),
* unicite metier (site, type, numero) parmi les actifs -> 409 (RG-7.01), reutilisation
* du triplet apres soft-delete, soft-delete jamais expose (§ 2.8), et la matrice RBAC
* admin-only (view lit mais ne gere pas ; personas metier 403 partout).
*
* RG-7.03 (« type disponible sur le site choisi ») n'est PAS testee : le concept
* type<->site a ete retire du modele en M6 (StorageType plat), c'est Storage qui le
* porte desormais — aucun referentiel a interroger (cf. StorageProcessor).
*
* @internal
*/
final class StorageApiTest extends AbstractStorageApiTestCase
{
/** Personas metier sans permission stockage (admin-only — ERP-210). */
private const array PERSONAS = ['Bureau', 'Compta', 'Commerciale', 'Usine'];
public function testCollectionIsPaginatedHydraWithEmbeddedRelations(): void
{
$site = $this->firstSite();
$type = $this->seedStorageType('Cellule');
$seed = $this->seedStorageEntity('C3', site: $site, storageType: $type);
$client = $this->createAdminClient();
$response = $client->request('GET', '/api/storages', ['headers' => ['Accept' => self::LD]]);
self::assertResponseStatusCodeSame(200);
$body = $response->toArray();
// Enveloppe Hydra (regle ABSOLUE n°13 : collection paginee, jamais d'array brut).
self::assertArrayHasKey('totalItems', $body);
self::assertArrayHasKey('member', $body);
$member = $this->memberById($body, (int) $seed->getId());
self::assertIsArray($member, 'Le stockage seede doit figurer dans la collection.');
// Contrat de serialisation (§ 4.0.bis) : site / storageType en OBJETS embarques
// (pas un IRI nu), displayName present (RG-7.05).
self::assertIsArray($member['site'], 'site doit etre un objet embarque.');
self::assertSame($site->getCode(), $member['site']['code'] ?? null);
self::assertIsArray($member['storageType'], 'storageType doit etre un objet embarque.');
self::assertSame('Cellule', $member['storageType']['label'] ?? null);
self::assertSame('Cellule C3', $member['displayName'] ?? null);
}
public function testAdminCanCreateStorage(): void
{
$client = $this->createAdminClient();
$client->request('POST', '/api/storages', [
'headers' => ['Content-Type' => self::LD],
'json' => $this->validStoragePayload(),
]);
self::assertResponseStatusCodeSame(201);
}
public function testNumeroIsTrimmedServerSide(): void
{
$client = $this->createAdminClient();
$response = $client->request('POST', '/api/storages', [
'headers' => ['Content-Type' => self::LD],
'json' => $this->validStoragePayload(['numero' => ' Z9 ']),
]);
self::assertResponseStatusCodeSame(201);
// RG-7.06 : trim serveur, sans changement de casse (HP-M7-05).
self::assertSame('Z9', $response->toArray()['numero'] ?? null);
}
public function testDuplicateTripletReturns409(): void
{
$site = $this->firstSite();
$type = $this->seedStorageType();
$this->seedStorageEntity('A1', site: $site, storageType: $type);
$client = $this->createAdminClient();
$client->request('POST', '/api/storages', [
'headers' => ['Content-Type' => self::LD],
'json' => [
'site' => $this->iri('sites', (int) $site->getId()),
'storageType' => $this->iri('storage_types', (int) $type->getId()),
'numero' => 'A1',
'states' => [Storage::STATE_RECEPTION],
],
]);
// RG-7.01 : meme (site, type, numero) parmi les actifs -> 409.
self::assertResponseStatusCodeSame(409);
}
public function testSameNumeroDifferentTypeIsAllowed(): void
{
$site = $this->firstSite();
$typeA = $this->seedStorageType();
$typeB = $this->seedStorageType();
$this->seedStorageEntity('A1', site: $site, storageType: $typeA);
$client = $this->createAdminClient();
$client->request('POST', '/api/storages', [
'headers' => ['Content-Type' => self::LD],
'json' => [
'site' => $this->iri('sites', (int) $site->getId()),
'storageType' => $this->iri('storage_types', (int) $typeB->getId()),
'numero' => 'A1',
'states' => [Storage::STATE_RECEPTION],
],
]);
// Unicite portee par le TRIPLET : un meme numero sur un autre type passe.
self::assertResponseStatusCodeSame(201);
}
public function testSoftDeletedTripletCanBeReused(): void
{
$site = $this->firstSite();
$type = $this->seedStorageType();
$this->seedStorageEntity('B2', deletedAt: new DateTimeImmutable(), site: $site, storageType: $type);
$client = $this->createAdminClient();
$client->request('POST', '/api/storages', [
'headers' => ['Content-Type' => self::LD],
'json' => [
'site' => $this->iri('sites', (int) $site->getId()),
'storageType' => $this->iri('storage_types', (int) $type->getId()),
'numero' => 'B2',
'states' => [Storage::STATE_RECEPTION],
],
]);
// RG-7.01 : l'unicite ne porte que sur les ACTIFS -> reutilisation OK.
self::assertResponseStatusCodeSame(201);
}
public function testSoftDeletedIsNotExposed(): void
{
$deleted = $this->seedStorageEntity('SD', deletedAt: new DateTimeImmutable());
$client = $this->createAdminClient();
// § 2.8 : item soft-deleted -> 404.
$client->request('GET', '/api/storages/'.$deleted->getId(), ['headers' => ['Accept' => self::LD]]);
self::assertResponseStatusCodeSame(404);
// … et absent de la collection (RG-7.07).
$response = $client->request('GET', '/api/storages', ['headers' => ['Accept' => self::LD]]);
self::assertNull($this->memberById($response->toArray(), (int) $deleted->getId()));
}
public function testViewPermissionReadsButCannotManage(): void
{
$storage = $this->seedStorageEntity();
$client = $this->authView();
$client->request('GET', '/api/storages', ['headers' => ['Accept' => self::LD]]);
self::assertResponseStatusCodeSame(200);
$client->request('GET', '/api/storages/'.$storage->getId(), ['headers' => ['Accept' => self::LD]]);
self::assertResponseStatusCodeSame(200);
// view sans manage : creation refusee au niveau securite (403).
$client->request('POST', '/api/storages', [
'headers' => ['Content-Type' => self::LD],
'json' => $this->validStoragePayload(),
]);
self::assertResponseStatusCodeSame(403);
}
public function testBusinessPersonasAreForbiddenEverywhere(): void
{
$storage = $this->seedStorageEntity();
$id = (int) $storage->getId();
foreach (self::PERSONAS as $persona) {
$client = $this->createPersonaClient($persona);
$client->request('GET', '/api/storages', ['headers' => ['Accept' => self::LD]]);
self::assertResponseStatusCodeSame(403, $persona.' ne doit pas lister les stockages.');
$client->request('GET', '/api/storages/'.$id, ['headers' => ['Accept' => self::LD]]);
self::assertResponseStatusCodeSame(403, $persona.' ne doit pas consulter un stockage.');
$client->request('POST', '/api/storages', [
'headers' => ['Content-Type' => self::LD],
'json' => $this->validStoragePayload(),
]);
self::assertResponseStatusCodeSame(403, $persona.' ne doit pas creer de stockage.');
$client->request('PATCH', '/api/storages/'.$id, [
'headers' => ['Content-Type' => self::MERGE],
'json' => ['numero' => 'X'],
]);
self::assertResponseStatusCodeSame(403, $persona.' ne doit pas modifier un stockage.');
}
}
}
@@ -0,0 +1,204 @@
<?php
declare(strict_types=1);
namespace App\Tests\Module\Catalog\Api;
use App\Module\Catalog\Domain\Entity\Storage;
use DateTimeImmutable;
use PhpOffice\PhpSpreadsheet\IOFactory;
/**
* Tests fonctionnels de l'export XLSX des stockages (M7, § 4.5) — ERP-214.
*
* Couvre : reponse 200 (Content-Type + Content-Disposition + en-tetes de colonnes),
* exclusion des stockages soft-deleted par defaut (RG-7.07), respect des filtres
* ?search (numero) / ?storageTypeId / ?state, peuplement des colonnes metier
* (displayName, site « Nom (Code) », type, numero, etats joints, dates), 403 sans
* catalog.storages.view, 401 anonyme.
*
* @internal
*/
final class StorageExportControllerTest extends AbstractStorageApiTestCase
{
private const string XLSX_MIME = 'application/vnd.openxmlformats-officedocument.spreadsheetml.sheet';
private const string EXPORT_URL = '/api/storages/export.xlsx';
public function testExportReturnsXlsxResponseWithHeaderRow(): void
{
$client = $this->createAdminClient();
$this->seedStorageEntity('NUM-A');
$response = $client->request('GET', self::EXPORT_URL);
self::assertResponseIsSuccessful();
$headers = $response->getHeaders(false);
self::assertStringContainsString(self::XLSX_MIME, $headers['content-type'][0] ?? '');
$disposition = $headers['content-disposition'][0] ?? '';
self::assertStringContainsString('attachment; filename="stockages-', $disposition);
self::assertMatchesRegularExpression('/filename="stockages-\d{8}\.xlsx"/', $disposition);
// Le binaire est un XLSX relisible dont la 1re ligne porte les en-tetes (§ 4.5).
$headerCells = $this->gridFromResponse($response->getContent())[0];
self::assertSame('Nom', $headerCells[0]);
self::assertSame('Site', $headerCells[1]);
self::assertSame('Type de stockage', $headerCells[2]);
self::assertSame('Numéro', $headerCells[3]);
self::assertSame('États', $headerCells[4]);
self::assertSame('Créé le', $headerCells[5]);
self::assertSame('Modifié le', $headerCells[6]);
// Au moins une ligne de donnees (le stockage seede) reperee par son numero.
self::assertContains('NUM-A', $this->numeros($response->getContent()));
}
public function testExportExcludesSoftDeletedByDefault(): void
{
$client = $this->createAdminClient();
$this->seedStorageEntity('NUM-ACTIVE');
$this->seedStorageEntity('NUM-DELETED', deletedAt: new DateTimeImmutable());
$numeros = $this->numeros($client->request('GET', self::EXPORT_URL)->getContent());
self::assertContains('NUM-ACTIVE', $numeros);
self::assertNotContains('NUM-DELETED', $numeros);
}
public function testExportRespectsSearchFilter(): void
{
$client = $this->createAdminClient();
$this->seedStorageEntity('ALPHA-1');
$this->seedStorageEntity('BETA-2');
$numeros = $this->numeros(
$client->request('GET', self::EXPORT_URL.'?search=alpha')->getContent(),
);
self::assertContains('ALPHA-1', $numeros);
self::assertNotContains('BETA-2', $numeros);
}
public function testExportRespectsStorageTypeFilter(): void
{
$client = $this->createAdminClient();
$typeA = $this->seedStorageType('Cellule A');
$typeB = $this->seedStorageType('Cellule B');
$this->seedStorageEntity('TYPE-A', storageType: $typeA);
$this->seedStorageEntity('TYPE-B', storageType: $typeB);
$numeros = $this->numeros(
$client->request('GET', self::EXPORT_URL.'?storageTypeId='.$typeA->getId())->getContent(),
);
self::assertContains('TYPE-A', $numeros);
self::assertNotContains('TYPE-B', $numeros);
}
public function testExportRespectsStateFilter(): void
{
$client = $this->createAdminClient();
$this->seedStorageEntity('STATE-PROD', [Storage::STATE_PRODUCTION]);
$this->seedStorageEntity('STATE-RECEP', [Storage::STATE_RECEPTION]);
$numeros = $this->numeros(
$client->request('GET', self::EXPORT_URL.'?state=PRODUCTION')->getContent(),
);
self::assertContains('STATE-PROD', $numeros);
self::assertNotContains('STATE-RECEP', $numeros);
}
public function testExportPopulatesAllBusinessColumns(): void
{
$client = $this->createAdminClient();
$site = $this->firstSite();
$type = $this->seedStorageType('Cellule');
$this->seedStorageEntity(
'C3',
[Storage::STATE_RECEPTION, Storage::STATE_TRIAGE],
site: $site,
storageType: $type,
);
$row = $this->rowForNumero($client->request('GET', self::EXPORT_URL)->getContent(), 'C3');
self::assertNotNull($row, 'Le stockage seede est absent de l\'export.');
// 0 Nom | 1 Site | 2 Type | 3 Numéro | 4 États | 5 Créé le | 6 Modifié le
self::assertSame('Cellule C3', $row[0]);
self::assertSame(sprintf('%s (%s)', $site->getName(), $site->getCode()), $row[1]);
self::assertSame('Cellule', $row[2]);
self::assertSame('C3', $row[3]);
// Ordre canonique (Réception avant Triage) independamment de l'ordre en base.
self::assertSame('Réception, Triage', $row[4]);
// Dates renseignees (Timestampable) au format jj/mm/aaaa hh:mm.
self::assertMatchesRegularExpression('#^\d{2}/\d{2}/\d{4} \d{2}:\d{2}$#', (string) $row[5]);
self::assertMatchesRegularExpression('#^\d{2}/\d{2}/\d{4} \d{2}:\d{2}$#', (string) $row[6]);
}
public function testForbiddenWithoutStoragesViewPermission(): void
{
$creds = $this->createUserWithPermission('core.users.view');
$client = $this->authenticatedClient($creds['username'], $creds['password']);
$client->request('GET', self::EXPORT_URL);
self::assertResponseStatusCodeSame(403);
}
public function testUnauthorizedWhenAnonymous(): void
{
$client = self::createClient();
$client->request('GET', self::EXPORT_URL);
self::assertResponseStatusCodeSame(401);
}
/**
* Relit le binaire XLSX d'une reponse et renvoie la grille de cellules.
*
* @return array<int, array<int, mixed>>
*/
private function gridFromResponse(string $binary): array
{
$tmp = tempnam(sys_get_temp_dir(), 'xlsx_storage_export_test_');
self::assertIsString($tmp);
file_put_contents($tmp, $binary);
try {
return IOFactory::load($tmp)->getActiveSheet()->toArray();
} finally {
@unlink($tmp);
}
}
/**
* Extrait la colonne « Numéro » (4e colonne, index 3) des lignes de donnees.
*
* @return list<string>
*/
private function numeros(string $binary): array
{
$rows = array_slice($this->gridFromResponse($binary), 1); // saute l'en-tete
return array_values(array_map(static fn (array $row): string => (string) ($row[3] ?? ''), $rows));
}
/**
* Renvoie la ligne de donnees dont la colonne « Numéro » vaut $numero, ou null.
*
* @return null|array<int, mixed>
*/
private function rowForNumero(string $binary, string $numero): ?array
{
foreach (array_slice($this->gridFromResponse($binary), 1) as $row) {
if ((string) ($row[3] ?? '') === $numero) {
return $row;
}
}
return null;
}
}