fix(ui): corrections visuelles et textes du back-office avant soutenance (F40) #149

Merged
Corentin merged 1 commit from fix/ui-f40 into dev 2026-09-24 16:07:22 +02:00
Owner
No description provided.
fix(ui): corrections visuelles avant soutenance (F40)
All checks were successful
CI / secret-scan (pull_request) Successful in 23s
CI / php-lint (pull_request) Successful in 28s
CI / static-tests (pull_request) Successful in 1m18s
CI / js-tests (pull_request) Successful in 47s
CI / secret-scan (push) Successful in 24s
CI / php-lint (push) Successful in 26s
CI / static-tests (push) Successful in 1m12s
CI / js-tests (push) Successful in 47s
3504385662
Corrige les defauts releves sur les 52 captures du dossier de soutenance
(defauts-visibles.md) :

- point 2 (formulaire de role, capture 40) : RoleController::renderForm
  passait le catalogue complet des permissions sous la cle 'permissions',
  la meme que AdminController::adminView utilise pour la liste de codes
  de l'utilisateur courant ($data + $context, l'union garde la cle de
  gauche) ; le layout recevait alors le catalogue au lieu de la liste et
  masquait tout le menu sauf Tableau de bord. Renomme en
  'permissionCatalog' dans le controleur et dans admin/roles/form.php ;
  aucune autre vue ne lisait l'ancienne cle. Test corrige pour verifier
  un element que seul le gabarit affiche (sidebar-section-label
  "Administration" + le lien de nav avec sa classe), pas juste l'URL
  /admin/roles (partagee avec le bouton Annuler du formulaire) ; verifie
  manuellement rouge avant la correction, vert apres.
- point 3 (commandes comptoir, capture 23) : statuts de cuisine sans
  libelle sur la liste comptoir/drive.
- point 4 (caisse comptoir et drive, captures 21/22/24) : photos de
  tuiles en 404 (chemin relatif resolu sous /counter/orders), menus
  ranges sous "Autres" faute de category_name en base, ligne de menu
  Maxi affichant la taille Normale.
- point 5 (confirmation borne, captures 15/M10) : logo decale a gauche
  (auto-placement de grille sans grid-column explicite sur un en-tete a
  un seul enfant).
- point 6 (statistiques, capture 36) : le compte de produits disponibles
  ignorait la rupture automatique (RG-T21).
- section "Textes techniques ou en anglais" : roles et descriptions
  traduits (migration idempotente 0012 + seed pour installation neuve,
  avec la capacite reelle du role cuisine, pas la description perimee),
  categories capitalisees (migration 0013 + seed), pages d'accueil de
  role affichees en clair, types de slot de menu traduits, codes
  RG-T21/"Reset PIN" retires de l'interface, prix desormais saisis et
  affiches en euros (App\Core\Money, base inchangee en centimes ; motif
  HTML aligne sur le serveur, qui refuse 0 et 0,00), dates au format
  francais lisible.
- section "Mise en page" : bouton "Police adaptee" qui recouvrait le
  bas du contenu, page RGPD et formulaire ingredient sans padding
  (.card sans .card-body), cases Retirable/Ajoutable collees (verifie
  par un ecart strictement positif, pas juste l'absence de
  chevauchement), cadre d'options du formulaire de menu qui coupait sa
  derniere ligne.
- E14 : description du role cuisine alignee sur sa capacite reelle
  (avance l'etat de preparation via order.read).
- E15 : bouton Annuler et description de la permission order.cancel
  alignes sur les statuts reellement acceptes par le serveur
  (pending_payment/paid/preparing/ready).
- commentaires perimes ("encaissement -> paid") corriges dans
  CounterOrderController, OrderController, OrderRepository::pay() et
  checkout.js : l'encaissement pose 'preparing' depuis le retour oral
  #8.
- commentaires factuellement faux ("les boissons n'ont pas de variante
  Maxi") corriges dans counter-order.js et admin/counter/new.php : les
  boissons FONTAINE (Coca Cola, Fanta...) en ont une (seed 0006), seules
  les boissons en bouteille n'en ont pas.

Relecture independante (24/09) : les migrations 0012/0013/0014 gardent
maintenant chaque UPDATE par la valeur ANGLAISE/brute D'ORIGINE (label,
description ou name), pour qu'un role/une categorie/une permission
personnalise en production ne soit jamais ecrase par un rejeu. Verifie
par des tests qui rejouent reellement le contenu SQL des trois fichiers
de migration contre la base : 0 ligne affectee une fois les donnees en
francais, et une personnalisation posee a la main survit a un rejeu.

Nouveaux textes ecrits dans ce lot correctement accentues (l'absence
d'accents pre-existante ailleurs reste le lot separe F38).

Suites vertes : PHPStan 0 erreur, PHPUnit 793/793, JS 285/285,
E2E 16/17 (1 test hors-scope non execute par ce harnais).
Corentin scheduled this pull request to auto merge when all checks succeed 2026-09-24 16:04:17 +02:00
Sign in to join this conversation.
No reviewers
No labels
auto-merge
No milestone
No project
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!149
No description provided.