AcadeNice/corentin_wakdo
7
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix(admin): conflits HTTP 409 au lieu de 422 (delete FK-bloque + course unicite) #33

Merged
Corentin merged 1 commit from fix/http-409-conflict-codes into dev 2026-06-17 10:11:29 +02:00
Conversation 0 Commits 1 Files changed 10 +31 -29
Corentin commented 2026-06-17 10:08:47 +02:00
Owner
Copy link
No description provided.
Corentin added 1 commit 2026-06-17 10:08:48 +02:00
fix(admin): conflits HTTP 409 au lieu de 422 (delete FK-bloque + course unicite)
Some checks failed
CI / secret-scan (push) Successful in 16s
Details
CI / php-lint (push) Successful in 23s
Details
CI / static-tests (push) Successful in 53s
Details
CI / secret-scan (pull_request) Successful in 9s
Details
CI / php-lint (pull_request) Successful in 28s
Details
CI / static-tests (pull_request) Successful in 56s
Details
CI / auto-merge (push) Has been skipped
Details
CI / auto-merge (pull_request) Failing after 6s
Details
a5d48cacbd 
Aligne les controleurs admin sur le contrat byan-api.md / conventions.md :
un conflit remonte par la base (SQLSTATE 23000) renvoie 409 Conflict, pas 422.
Couvre la suppression dure bloquee par une FK RESTRICT (Product, Menu) et la
course d'unicite slug/name a l'insertion (Category). La validation simple en
amont (champ/format/bornes, PIN invalide, acteur verrouille) et la
pre-verification nameExists/slugExists restent en 422.

renderDelete gagne un parametre de statut optionnel (defaut inchange).
Tests : assertions de conflit flippees en 409 (TDD). 201 tests, 597 assertions,
PHPStan L6 propre.
Corentin scheduled this pull request to auto merge when all checks succeed 2026-06-17 10:08:49 +02:00
Corentin added the
auto-merge
 label 2026-06-17 10:09:04 +02:00
Corentin merged commit 0666a22562 into dev 2026-06-17 10:11:29 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
auto-merge

Fusion automatique sur CI verte

No labels
auto-merge
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: AcadeNice/corentin_wakdo#33
No description provided.
Delete branch "fix/http-409-conflict-codes"

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?