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

frontend/modules/transport/composables/__tests__/useCarrierForm.test.ts

@@ -412,23 +412,6 @@ describe('useCarrierForm — copie QUALIMAT (ERP-166)', () => {
         )
     })
 
-    it('après création : PATCH en échec → pas de copie locale (rollback) et retour false', async () => {
-        mockPost.mockResolvedValueOnce({ id: 9, name: 'X', certificationType: 'GMP_PLUS' })
-        mockPatch.mockRejectedValueOnce({ response: { status: 500, _data: {} } })
-        const form = useCarrierForm()
-        form.main.name = 'X'
-        form.main.certificationType = 'GMP_PLUS'
-        await form.submitMain()
-
-        const ok = await form.applyQualimatSelection(QUALIMAT_ROW)
-
-        // Échec serveur : l'UI ne doit pas refléter une intégration QUALIMAT non persistée.
-        expect(ok).toBe(false)
-        expect(form.main.name).toBe('X')
-        expect(form.main.certificationType).toBe('GMP_PLUS')
-        expect(form.main.qualimatCarrierIri).toBeNull()
-    })
-
     it('buildMainPayload inclut qualimatCarrier + certificationType QUALIMAT après intégration', async () => {
         const form = useCarrierForm()
         form.main.name = 'Acme'

frontend/modules/transport/composables/useCarrierForm.ts

@@ -659,27 +659,10 @@ export function useCarrierForm() {
      * § 2.5) : copie le nom, force la certification à « QUALIMAT » (lecture seule),
      * pose la FK `qualimatCarrier` (IRI) et copie l'adresse (pour l'onglet Adresses).
      * Si le transporteur existe déjà (post-POST, cas nominal de l'onglet), persiste
-     * d'abord la copie via un PATCH partiel `carrier:write:main` : la copie locale
-     * (nom, certification figée « QUALIMAT », FK, adresse) n'est appliquée qu'en cas
-     * de succès, pour ne pas laisser l'UI dans un état QUALIMAT non sauvegardé si le
-     * PATCH échoue. Retourne true si l'intégration a abouti.
+     * la copie via un PATCH partiel `carrier:write:main`. La copie locale a lieu
+     * dans tous les cas. Retourne true si l'intégration a abouti.
      */
     async function applyQualimatSelection(row: QualimatCarrierRow): Promise<boolean> {
-        // Transporteur déjà créé : on persiste avant de refléter localement.
-        if (carrierId.value !== null) {
-            try {
-                await patchCarrier({
-                    qualimatCarrier: row['@id'],
-                    name: row.name,
-                    certificationType: 'QUALIMAT',
-                })
-            }
-            catch (error) {
-                mainErrors.handleApiError(error, { fallbackMessage: t('transport.carriers.toast.error') })
-                return false
-            }
-        }
-
         main.name = row.name ?? ''
         main.certificationType = 'QUALIMAT'
         main.qualimatCarrierIri = row['@id']
@@ -699,7 +682,23 @@ export function useCarrierForm() {
             street: row.address || null,
             streetComplement: null,
         }]
-        return true
+
+        if (carrierId.value === null) {
+            return true
+        }
+
+        try {
+            await patchCarrier({
+                qualimatCarrier: row['@id'],
+                name: row.name,
+                certificationType: 'QUALIMAT',
+            })
+            return true
+        }
+        catch (error) {
+            mainErrors.handleApiError(error, { fallbackMessage: t('transport.carriers.toast.error') })
+            return false
+        }
     }
 
     /**

frontend/modules/transport/pages/carriers/[id]/index.vue

@@ -255,7 +255,6 @@ import {
     showRestoreAction,
     type CarrierPriceRead,
 } from '~/modules/transport/utils/forms/carrierMappers'
-import { extractApiErrorMessage } from '~/shared/utils/api'
 
 interface SelectOption {
     value: string
@@ -482,14 +481,8 @@ async function runToggleArchive(): Promise<void> {
                 : t('transport.carriers.toast.restoreSuccess'),
         })
     }
-    catch (err) {
-        // Surface le message back (ex. 409 « homonyme actif » à la restauration),
-        // propagé exprès par useCarrier ; fallback générique sinon.
-        const data = (err as { response?: { _data?: unknown } })?.response?._data
-        toast.error({
-            title: t('transport.carriers.toast.error'),
-            message: extractApiErrorMessage(data) || undefined,
-        })
+    catch {
+        toast.error({ title: t('transport.carriers.toast.error') })
     }
 }
 

frontend/modules/transport/utils/forms/__tests__/numberInput.test.ts

@@ -5,7 +5,7 @@ describe('numberInput — saisie volume / indexation (ERP-170)', () => {
     it('sanitizeDecimal : ne garde que chiffres + un seul point', () => {
         expect(sanitizeDecimal('30')).toBe('30')
         expect(sanitizeDecimal('30.5')).toBe('30.5')
-        expect(sanitizeDecimal('30,5 kg')).toBe('30.5') // virgule FR → point ; espace + lettres retirés
+        expect(sanitizeDecimal('30,5 kg')).toBe('305') // virgule + espace + lettres retirés
         expect(sanitizeDecimal('1.2.3')).toBe('1.23') // un seul point conservé
         expect(sanitizeDecimal('abc12.3x')).toBe('12.3')
         expect(sanitizeDecimal('')).toBe('')

frontend/modules/transport/utils/forms/numberInput.ts

@@ -5,11 +5,10 @@
 
 /**
  * Restreint une saisie à un nombre décimal : chiffres + UN seul point (RG volume m³,
- * « nombres avec des points » comme les autres modules). La virgule décimale FR est
- * convertie en point (« 30,5 » → « 30.5 ») ; tout autre caractère est supprimé.
+ * « nombres avec des points » comme les autres modules). Supprime tout autre caractère.
  */
 export function sanitizeDecimal(value: string): string {
-    let cleaned = (value ?? '').replace(/,/g, '.').replace(/[^0-9.]/g, '')
+    let cleaned = (value ?? '').replace(/[^0-9.]/g, '')
     const dot = cleaned.indexOf('.')
     if (dot !== -1) {
         // Conserve le 1er point, retire les suivants.