MALIO-DEV/Starseed
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(commercial) : messages de validation FR sur les contraintes back + garde-fou (ERP-107) #59

Merged
malio merged 5 commits from feat/erp-107-validation-messages-fr into develop 2026-06-04 09:27:33 +00:00
Conversation 0 Commits 5 Files Changed 18
+1384 -21
matthieu commented 2026-06-04 08:11:58 +00:00
Owner
Copy Link
Copy Source

Contexte

Résout ERP-107 — pendant back du mapping d'erreur par champ front (ERP-101). Le front (useFormErrors / mapViolationsToRecord) affiche sous chaque champ le message renvoyé par le back. Ce ticket garantit que ces messages existent, sont en FR et rattachés au bon champ.

Changements

  • Messages FR explicites sur toutes les contraintes #[Assert\*] des entités métier : Client, ClientContact, ClientAddress, ClientRib, Category, Role, User (Email, NotBlank, Length, Bic, Iban, PositiveOrZero, Count…).
  • Contraintes Assert\Length manquantes ajoutées, calées sur le length de la colonne ORM (téléphones VARCHAR(20), siren, nTva, accountNumber, username…). Évite une erreur Postgres 500 non rattachée au champ → 422 propre.
  • Locale FR globale (symfony/translation + default_locale: fr) comme filet pour les messages natifs Symfony non surchargés.
  • Garde-fou tests/Architecture/EntityConstraintsHaveFrenchMessageTest : échoue si une contrainte n'a pas de message FR explicite (comparaison au défaut Symfony) ou si Assert\Length.max diverge du length ORM. Whitelist justifiée pour les formats auto-bornés (Bic/Iban/Regex CP/couleur hex).
  • Test fonctionnel du JSON 422 réel : message FR + propertyPath consommable par le front.
  • Convention documentée dans .claude/rules/backend.md.

Décisions

  • Stratégie retenue : message FR explicite sur toutes les contraintes + locale FR en filet (les deux leviers du ticket).
  • Garde-fou Length == ORM length : test bloquant (anti-dérive).
  • RG-1.03 (distributor/broker) : pas de Assert\Callback ajouté — le ClientProcessor gère déjà l'exclusivité (422 + propertyPath). Pas de doublon.

Hors périmètre / à suivre

  • Alignement nullable(DB) / NotBlank(back) / required(front) : les champs obligatoires existants ont été confirmés, mais aucun changement de nullabilité DB n'a été fait sans arbitrage métier. À recroiser avec les astérisques front (ERP-101 / PR #58) si divergence constatée.

Vérifications

  • make test : 469 tests verts (1793 assertions), 0 échec/erreur.
  • php-cs-fixer : 0 fichier à corriger.
## Contexte Résout **ERP-107** — pendant back du mapping d'erreur par champ front (ERP-101). Le front (`useFormErrors` / `mapViolationsToRecord`) affiche sous chaque champ le `message` renvoyé par le back. Ce ticket garantit que ces messages existent, sont en FR et rattachés au bon champ. ## Changements - **Messages FR explicites** sur toutes les contraintes `#[Assert\*]` des entités métier : `Client`, `ClientContact`, `ClientAddress`, `ClientRib`, `Category`, `Role`, `User` (Email, NotBlank, Length, Bic, Iban, PositiveOrZero, Count…). - **Contraintes `Assert\Length` manquantes ajoutées**, calées sur le `length` de la colonne ORM (téléphones `VARCHAR(20)`, `siren`, `nTva`, `accountNumber`, `username`…). Évite une erreur Postgres 500 non rattachée au champ → 422 propre. - **Locale FR globale** (`symfony/translation` + `default_locale: fr`) comme filet pour les messages natifs Symfony non surchargés. - **Garde-fou** `tests/Architecture/EntityConstraintsHaveFrenchMessageTest` : échoue si une contrainte n'a pas de message FR explicite (comparaison au défaut Symfony) ou si `Assert\Length.max` diverge du `length` ORM. Whitelist justifiée pour les formats auto-bornés (Bic/Iban/Regex CP/couleur hex). - **Test fonctionnel** du JSON 422 réel : message FR + `propertyPath` consommable par le front. - **Convention documentée** dans `.claude/rules/backend.md`. ## Décisions - Stratégie retenue : message FR **explicite sur toutes** les contraintes + locale FR en filet (les deux leviers du ticket). - Garde-fou `Length == ORM length` : **test bloquant** (anti-dérive). - RG-1.03 (distributor/broker) : pas de `Assert\Callback` ajouté — le `ClientProcessor` gère **déjà** l'exclusivité (422 + `propertyPath`). Pas de doublon. ## Hors périmètre / à suivre - **Alignement `nullable`(DB) / `NotBlank`(back) / `required`(front)** : les champs obligatoires existants ont été confirmés, mais aucun changement de nullabilité DB n'a été fait sans arbitrage métier. À recroiser avec les astérisques front (ERP-101 / PR #58) si divergence constatée. ## Vérifications - `make test` : **469 tests verts** (1793 assertions), 0 échec/erreur. - `php-cs-fixer` : 0 fichier à corriger.
matthieu added 1 commit 2026-06-04 08:11:59 +00:00
feat(commercial) : messages de validation FR sur les contraintes back + garde-fou (ERP-107)
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 1m45s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m12s
Details
9202d2ee6f 
Audit et complete les messages des contraintes #[Assert\*] des entites metier
(pendant back du mapping d'erreur par champ front ERP-101) :

- Message FR explicite sur toutes les contraintes (Email, NotBlank, Length,
  Bic, Iban, PositiveOrZero, Count...) des entites Client, ClientContact,
  ClientAddress, ClientRib, Category, Role, User.
- Ajout des Assert\Length manquantes calees sur le length de la colonne ORM
  (telephones VARCHAR(20), siren, nTva, accountNumber, username...) : evite
  une erreur Postgres 500 non rattachee au champ au profit d'une 422 propre.
- Locale FR globale (symfony/translation + default_locale: fr) comme filet
  pour les messages natifs non surcharges.
- Garde-fou tests/Architecture/EntityConstraintsHaveFrenchMessageTest : echoue
  si une contrainte n'a pas de message FR explicite ou si Assert\Length.max
  diverge du length ORM (whitelist justifiee pour les formats Bic/Iban/Regex).
- Test fonctionnel du JSON 422 reel (message FR + propertyPath consommable
  par useFormErrors cote front).
- Convention documentee dans .claude/rules/backend.md.

Tests : 469 verts (1793 assertions).
matthieu added 1 commit 2026-06-04 08:18:46 +00:00
merge: integrate origin/develop into ERP-107
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 1m48s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m17s
Details
27489c777b 
Resout les conflits avec la suppression du contact inline du Client (#57,
ERP-104) : les champs firstName/lastName/phones/email ne sont plus portes par
Client. Les messages de validation FR correspondants vivent desormais sur
ClientContact. Le test fonctionnel du message email FR est recible sur la
sous-ressource POST /clients/{id}/contacts.
matthieu added the backM1-Clienttype/feat labels 2026-06-04 08:19:16 +00:00
malio added 1 commit 2026-06-04 08:41:40 +00:00
Merge branch 'develop' into feat/erp-107-validation-messages-fr
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 1m38s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m7s
Details
f6ae49e258
matthieu added 1 commit 2026-06-04 09:21:35 +00:00
docs(commercial) : spec + plan validation tous-blocs collections client (500 NonUniqueResult + collecte erreurs par bloc)
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Has been cancelled
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Has been cancelled
Details
a17723cb1c
malio added 1 commit 2026-06-04 09:23:14 +00:00
Merge branch 'develop' into feat/erp-107-validation-messages-fr
Pull Request — Quality gate / Backend (PHP CS + PHPUnit) (pull_request) Successful in 1m48s
Details
Pull Request — Quality gate / Frontend (lint + Vitest + build) (pull_request) Successful in 1m15s
Details
16e376a0d0
malio merged commit 597101262d into develop 2026-06-04 09:27:33 +00:00
malio deleted branch feat/erp-107-validation-messages-fr 2026-06-04 09:27:33 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
back
Backend PHP / Symfony / API Platform
 blocked
Bloqué par une dépendance externe
 breaking
Breaking change — impact API / contrat
 db
Migrations Doctrine / schéma BDD
 front
Frontend Nuxt / Vue / @malio/layer-ui
 infra
Docker / nginx / config infra
 M0-Categorie
M0 — Catégories
 M1-Client
M1 — Clients
 M2-Fournisseur
M2 — Fournisseurs
 M3-Prestataire
M3 — Prestataires
 M4-Transporteur
M4 — Transporteurs
 M5-Ticket-pesee
M5 — Tickets de pesée
 type/chore
Tâche de maintenance (bump, conf, etc.)
 type/ci
CI / pipeline / workflows
 type/docs
Documentation
 type/feat
Nouvelle fonctionnalité
 type/fix
Correction de bug
 type/perf
Amélioration de performance
 type/refactor
Refacto sans changement fonctionnel
 type/test
Ajout ou modification de tests
 wip
Work in progress — ne pas merger
No Label back M1-Client type/feat
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
malio (admin malio) matthieu (THOLOT DECHENE Matthieu) tristan (Autin )
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MALIO-DEV/Starseed#59
Delete Branch "feat/erp-107-validation-messages-fr"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?