feat(transport) : consultation + modification transporteur (ERP-170) #129

Merged

tristan merged 17 commits from feat/erp-170-carrier-view-edit into develop 2026-06-18 08:50:09 +00:00

Conversation 1 Commits 17 Files Changed 13

+1553 -36

tristan commented 2026-06-17 09:35:26 +00:00

Owner

Copy Link

Copy Source

Readonly + archive + édition par onglet. Ticket ERP-170. Stack sur #128→#127→#126→#123→#122→#121.

Readonly + archive + édition par onglet. Ticket ERP-170. Stack sur #128→#127→#126→#123→#122→#121.

tristan added 18 commits 2026-06-17 09:35:27 +00:00

feat(transport) : page répertoire transporteurs (ERP-164) 1ef4215ebf

feat(transport) : filtres checkbox, toggle « Voir les archivés », transporteurs dans Administration (ERP-164) 8046de76c6

chore(frontend) : bump @malio/layer-ui ^1.7.12 + commentaire useSuppliersRepository 597c63bb2e

feat(transport) : écran ajout transporteur — layout + formulaire principal (ERP-165) 5734aaef54

fix(transport) : centre verticalement la case « Affréter » sur la ligne de champ (ERP-165) f1b18cfbbe

feat(transport) : saisie assistée QUALIMAT + champs conditionnels (ERP-166) f70e701854

fix(transport) : décharge après volume (nouvelle ligne) + contenant Benne/FM en select (ERP-166) 8cc2cea444

fix(transport) : ordre des champs Nom/Certif/Décharge/Affréter/Indexation/Benne/Volume (ERP-166) be03f4e51a

fix(transport) : colonne 3 réservée à la décharge, « Affréter » toujours en colonne 4 (ERP-166) 9864dbc00f

fix(transport) : certification obligatoire en pré-validation front, sauf cas LIOT (ERP-166) 3804362546

fix(transport) : pré-validation front des champs conditionnels obligatoires (décharge AUTRE, affrètement) (ERP-166) 6a519874ed

feat(transport) : croix de suppression sur le champ Décharge (clearable) (ERP-166) d6d2144cc1

refactor(transport) : onglet Qualimat en MalioDataTable paginé, recherche branchée sur le nom (ERP-166) 388d39a379

feat(transport) : onglet Qualimat accessible dès le départ, recherche réactive au nom, sélection remplit le formulaire (ERP-166) cf645493c1

feat(transport) : datatable Qualimat vide par défaut, n'affiche que les résultats de recherche (ERP-166) 0733a239a8

Merge remote-tracking branch 'origin/develop' into feat/erp-164-carriers-list 45158af920

Merge branch 'feat/erp-164-carriers-list' into feat/erp-165-carrier-new 4202977950

Merge branch 'feat/erp-165-carrier-new' into feat/erp-166-qualimat-search 40fdded7e2

tristan added the frontM4-Transporteurtype/feat labels 2026-06-17 09:35:27 +00:00

tristan commented 2026-06-17 13:57:57 +00:00

Author

Owner

Copy Link

Copy Source

Review front — ERP-170 (consultation + modification)

Verdict : à corriger avant merge.

La plus grosse de la stack ([id]/index.vue + [id]/edit.vue + mappers). Points solides : useCarrierForm() n'est pas un singleton → aucune fuite d'état entre new et edit ; états loading/error/404 gérés ; mappers aller-retour fidèles (iriOf, ?? null/?? '', formatAmount qui évite « NaN € »). Fix réel au passage : name du radio prix rendu unique via useId() (corrige une désélection mutuelle entre blocs).

Findings

• MAJOR — index.vue : le 409 d'homonyme à la restauration est avalé par un catch qui toast un message générique, alors que useCarrier.setArchived propage l'erreur exprès pour que l'appelant l'affiche (documenté dans useCarrier.ts). L'utilisateur ne comprend pas pourquoi la restauration échoue.
  → Extraire extractApiErrorMessage(err?.response?._data) (déjà fait en edit.vue) et le passer en message: du toast, avec fallback i18n.
• MINOR — sanitizeDecimal('30,5') → '305' : la virgule est supprimée au lieu d'être convertie en point (le test fige ce comportement). Un utilisateur FR tapant « 30,5 » obtient 305 m³. clampPercent juste à côté gère pourtant la virgule.
  → value.replace(',', '.') avant le nettoyage.
• MINOR — edit.vue : upload de la décharge non câblé (déféré à ERP-171, légitime) mais sans le TODO ERP-171 que porte new.vue → asymétrie trompeuse. Ajouter le même commentaire.
• MINOR — prefillFrom initialise les prix sans bloc vide de repli, alors qu'adresses/contacts retombent sur un bloc vide → incohérence à assumer ou documenter.
• NIT — Cast as unknown as pour l'export blob (trou TS-strict, identique à l'existant).

Incrément reviewé : erp-169..erp-170 (front).

## Review front — ERP-170 (consultation + modification) **Verdict : à corriger avant merge.** La plus grosse de la stack (`[id]/index.vue` + `[id]/edit.vue` + mappers). Points solides : `useCarrierForm()` n'est **pas un singleton** → aucune fuite d'état entre `new` et `edit` ; états `loading`/`error`/404 gérés ; mappers aller-retour fidèles (`iriOf`, `?? null`/`?? ''`, `formatAmount` qui évite « NaN € »). Fix réel au passage : `name` du radio prix rendu unique via `useId()` (corrige une désélection mutuelle entre blocs). **Findings** - **MAJOR** — `index.vue` : le **409 d'homonyme à la restauration** est avalé par un `catch` qui toast un message **générique**, alors que `useCarrier.setArchived` propage l'erreur *exprès* pour que l'appelant l'affiche (documenté dans `useCarrier.ts`). L'utilisateur ne comprend pas pourquoi la restauration échoue. → Extraire `extractApiErrorMessage(err?.response?._data)` (déjà fait en `edit.vue`) et le passer en `message:` du toast, avec fallback i18n. - **MINOR** — `sanitizeDecimal('30,5')` → `'305'` : la virgule est **supprimée** au lieu d'être convertie en point (le test fige ce comportement). Un utilisateur FR tapant « 30,5 » obtient 305 m³. `clampPercent` juste à côté gère pourtant la virgule. → `value.replace(',', '.')` avant le nettoyage. - **MINOR** — `edit.vue` : upload de la décharge non câblé (déféré à ERP-171, légitime) mais **sans** le `TODO ERP-171` que porte `new.vue` → asymétrie trompeuse. Ajouter le même commentaire. - **MINOR** — `prefillFrom` initialise les prix sans bloc vide de repli, alors qu'adresses/contacts retombent sur un bloc vide → incohérence à assumer ou documenter. - **NIT** — Cast `as unknown as` pour l'export blob (trou TS-strict, identique à l'existant). > Incrément reviewé : `erp-169..erp-170` (front).

tristan added 27 commits 2026-06-17 14:09:20 +00:00

fix(transport) : intégration QUALIMAT — copie locale seulement après PATCH réussi (évite un état non persisté) (ERP-166) c6259a96cd

feat(transport) : onglet adresses transporteur (ERP-167) ef996c3672

feat(transport) : onglet contacts transporteur (ERP-168) 5765ba7178

fix(transport) : règle « + Nouveau contact » alignée sur M1/M2/M3 (prénom OU nom) (ERP-168) f27db02cb6

fix(transport) : contact transporteur valide si prénom OU nom (alignement M1/M2/M3) (ERP-168) f29266e5e8

feat(transport) : onglet prix transporteur (ERP-169) 07e0bcbcce

fix(transport) : bloc prix — radios sans label de groupe, sens en colonne 1, défaut Benne/Forfait (ERP-169) a26bb09ee1

fix(transport) : bloc prix par défaut (CLIENT), sens seul en ligne 1, payload omet scalaires vides (422 inline au lieu de 400) (ERP-169) 6ff5b13ce2

fix(transport) : bloc prix — radios sens/contenant/tarif horizontaux et centrés (h-12) en colonne 1 (ERP-169) e1712465f1

fix(transport) : pré-validation front du bloc prix — erreurs inline sous tous les champs requis (selects branche) (ERP-169) fb9c15c52a

feat(transport) : consultation + modification transporteur (ERP-170) e612eae391

fix(transport) : tableau prix consultation — cellule de groupe fusionnée (Fond Mouvant/Benne) + colonnes maquette (ERP-170) 7adf3a511a

fix(transport) : name de radio unique par bloc prix (useId) — plus de groupe partagé entre blocs (ERP-170) d5a01ac85f

fix(transport) : tableau prix consultation — police/bordures/radius alignés MalioDataTable, colonne groupe en épine (ERP-170) 5109b5f57a

fix(transport) : tableau prix — supprime la double bordure du bas + séparateur épais entre groupes Benne/Fond mouvant (ERP-170) b11968f5e5

fix(transport) : tableau prix — en-tête « Contenant » sur la colonne de groupe (ERP-170) 48ca963a9d

fix(transport) : volume m³ en champ texte décimal + indexation en montant % plafonné à 100 (ERP-170) 0d284fe488

fix(transport) : indexation réellement plafonnée à 100 % — re-synchronise le champ amount contrôlé via :key (ERP-170) 02d2fde653

fix(transport) : consultation — disposition du bloc principal alignée sur l'ajout (LIOT, décharge col 3, affréter col 4) (ERP-170) b5749520bc

feat(transport) : contenant du formulaire principal en radios centrés (Benne par défaut) (ERP-170) 07ed57f283

feat(transport) : consultation — contenant en radios lecture seule (aligné ajout/modif) (ERP-170) d86dc69cf2

style(transport) : tableau prix consultation en table-fixed (colonnes à parts égales, contenant étroit) (ERP-170) fe1d012548

style(transport) : datatable qualimat table-fixed (radio étroit, colonnes égales) + icônes onglets prix/qualimat (ERP-170) 20296ac149

style(transport) : tableau prix — libellés colonnes + élargit Transporteurs/Adresse livraisons, réduit Forfait/Tonne/Indexation/État (ERP-170) 5bbd4ddb47

fix(transport) : tableau prix — corrige l'inversion Adresse sites / Adresse livraisons (ERP-170) 5125883e21

style(transport) : tableau prix — réduit Adresse sites au profit d'Adresse livraisons (ERP-170) c371057c0b

fix(transport) : affiche le message 409 (homonyme) à la restauration + virgule décimale dans sanitizeDecimal (ERP-170) b6b5bb06e8

tristan force-pushed feat/erp-170-carrier-view-edit from ca7aef08be to b6b5bb06e8 2026-06-17 14:09:20 +00:00 Compare

tristan merged commit fd89160c4b into develop 2026-06-18 08:50:09 +00:00

tristan deleted branch feat/erp-170-carrier-view-edit 2026-06-18 08:50:10 +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 front M4-Transporteur 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

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MALIO-DEV/Starseed#129