chore: bump version to v0.1.134

2026-06-18 08:51:01 +00:00
14 changed files with 43 additions and 594 deletions
@@ -1,2 +1,2 @@
 parameters:
-    app.version: '0.1.135'
+    app.version: '0.1.134'
@@ -528,8 +528,7 @@
                "exportError": "L'export du répertoire transporteurs a échoué. Réessayez.",
                "createSuccess": "Transporteur créé avec succès",
                "integrateSuccess": "Transporteur QUALIMAT intégré",
-                "addressSaved": "Adresse enregistrée",
+                "addressSaved": "Adresse enregistrée"
                "contactSaved": "Contact enregistré"
            },
            "containerType": {
                "BENNE": "Benne",
@@ -592,17 +591,6 @@
                    "remove": "Supprimer l'adresse",
                    "degraded": "Service d'adresse indisponible : saisie de la ville et de l'adresse en mode libre."
                },
                "contact": {
                    "lastName": "Nom",
                    "firstName": "Prénom",
                    "jobTitle": "Fonction",
                    "phonePrimary": "Téléphone",
                    "phoneSecondary": "Téléphone (2)",
                    "addPhone": "Ajouter un numéro",
                    "email": "Email",
                    "add": "Nouveau contact",
                    "remove": "Supprimer le contact"
                },
                "confirmDelete": {
                    "title": "Supprimer ce bloc",
                    "message": "Cette suppression est définitive. Confirmer ?",
@@ -1,108 +0,0 @@
 <template>
    <div class="relative grid grid-cols-4 gap-x-[44px] gap-y-4 bg-white py-4 pl-[28px] pr-[60px] shadow-[0_4px_4px_0_rgba(0,0,0,0.25)]">
        <!-- Suppression : ouvre une modal de confirmation côté parent. Masquée si
             non supprimable (1er bloc) ou en lecture seule. -->
        <MalioButtonIcon
            v-if="removable && !readonly"
            icon="mdi:delete-outline"
            variant="ghost"
            button-class="absolute top-3 right-3"
            v-bind="{ ariaLabel: t('transport.carriers.form.contact.remove') }"
            @click="$emit('remove')"
        />
        <MalioInputText
            :model-value="model.lastName"
            :label="t('transport.carriers.form.contact.lastName')"
            :readonly="readonly"
            :error="errors?.lastName"
            @update:model-value="(v: string) => update('lastName', v)"
        />
        <MalioInputText
            :model-value="model.firstName"
            :label="t('transport.carriers.form.contact.firstName')"
            :readonly="readonly"
            :error="errors?.firstName"
            @update:model-value="(v: string) => update('firstName', v)"
        />
        <!-- Fonction sur 2 colonnes : on wrappe car MalioInputText (inheritAttrs:false)
             renvoie `class` sur l'input interne, pas sur la cellule de grille. -->
        <div class="col-span-2">
            <MalioInputText
                :model-value="model.jobTitle"
                :label="t('transport.carriers.form.contact.jobTitle')"
                :readonly="readonly"
                :error="errors?.jobTitle"
                @update:model-value="(v: string) => update('jobTitle', v)"
            />
        </div>
        <MalioInputEmail
            :model-value="model.email"
            :label="t('transport.carriers.form.contact.email')"
            :readonly="readonly"
            :lowercase="true"
            :error="errors?.email"
            @update:model-value="(v: string) => update('email', v)"
        />
        <!-- Téléphone principal + bouton « + » révélant le 2e numéro (max 2). -->
        <MalioInputPhone
            :model-value="model.phonePrimary"
            :label="t('transport.carriers.form.contact.phonePrimary')"
            :mask="PHONE_MASK"
            :readonly="readonly"
            :error="errors?.phonePrimary"
            :addable="!model.hasSecondaryPhone && !readonly"
            :add-button-label="t('transport.carriers.form.contact.addPhone')"
            @update:model-value="(v: string) => update('phonePrimary', v)"
            @add="revealSecondaryPhone"
        />
        <!-- 2e numéro : révélé à la demande (max 2 téléphones — RG-4.08). -->
        <MalioInputPhone
            v-if="model.hasSecondaryPhone"
            :model-value="model.phoneSecondary"
            :label="t('transport.carriers.form.contact.phoneSecondary')"
            :mask="PHONE_MASK"
            :readonly="readonly"
            :error="errors?.phoneSecondary"
            @update:model-value="(v: string) => update('phoneSecondary', v)"
        />
    </div>
 </template>
 <script setup lang="ts">
 import type { CarrierContactFormDraft } from '~/modules/transport/types/carrierForm'
 // Masque téléphone FR : 5 groupes de 2 chiffres (la normalisation finale reste serveur).
 const PHONE_MASK = '## ## ## ## ##'
 const props = defineProps<{
    /** Brouillon du contact (v-model). */
    modelValue: CarrierContactFormDraft
    /** Affiche l'icône de suppression (1er bloc non supprimable). */
    removable?: boolean
    /** Bloc en lecture seule (onglet validé). */
    readonly?: boolean
    /** Erreurs serveur 422 de cette ligne, indexées par champ (ERP-101). */
    errors?: Record<string, string>
 }>()
 const emit = defineEmits<{
    'update:modelValue': [value: CarrierContactFormDraft]
    'remove': []
 }>()
 const { t } = useI18n()
 // Alias local pour la lisibilité du template.
 const model = computed(() => props.modelValue)
 /** Émet un nouveau brouillon avec le champ modifié (immutabilité). */
 function update<K extends keyof CarrierContactFormDraft>(field: K, value: CarrierContactFormDraft[K]): void {
    emit('update:modelValue', { ...props.modelValue, [field]: value })
 }
 /** Révèle le 2e numéro (max 1 secondaire, le « + » disparaît). */
 function revealSecondaryPhone(): void {
    emit('update:modelValue', { ...props.modelValue, hasSecondaryPhone: true })
 }
 </script>
@@ -37,8 +37,6 @@ vi.stubGlobal('useToast', () => ({
 }))
 const { useCarrierForm, CARRIER_TAB_KEYS } = await import('../useCarrierForm')
 const { buildCarrierContactPayload, isCarrierContactBlank, isCarrierContactNamed } = await import('../../utils/forms/carrierContact')
 const { emptyCarrierContact } = await import('../../types/carrierForm')
 describe('useCarrierForm', () => {
    beforeEach(() => {
@@ -557,145 +555,3 @@ describe('useCarrierForm — onglet Adresses (ERP-167)', () => {
        expect(form.addresses.value).toHaveLength(1)
    })
 })
 describe('carrierContact (util) — validité alignée M1/M2/M3 + max 2 téléphones', () => {
    it('isCarrierContactBlank : vrai si vide, faux dès un champ comptant rempli (phoneSecondary exclu)', () => {
        expect(isCarrierContactBlank(emptyCarrierContact())).toBe(true)
        expect(isCarrierContactBlank({ ...emptyCarrierContact(), jobTitle: 'Acheteur' })).toBe(false)
        expect(isCarrierContactBlank({ ...emptyCarrierContact(), phonePrimary: '0102030405' })).toBe(false)
        // phoneSecondary seul ne compte pas (aligné M1/M2/M3).
        expect(isCarrierContactBlank({ ...emptyCarrierContact(), phoneSecondary: '0605040302', hasSecondaryPhone: true })).toBe(true)
    })
    it('isCarrierContactNamed : nommé seulement avec un prénom OU un nom', () => {
        expect(isCarrierContactNamed(emptyCarrierContact())).toBe(false)
        expect(isCarrierContactNamed({ ...emptyCarrierContact(), firstName: 'Jean' })).toBe(true)
        expect(isCarrierContactNamed({ ...emptyCarrierContact(), lastName: 'Doe' })).toBe(true)
        // Fonction / téléphone / email seuls ne « nomment » pas (≠ RG-4.08 large).
        expect(isCarrierContactNamed({ ...emptyCarrierContact(), jobTitle: 'Acheteur' })).toBe(false)
        expect(isCarrierContactNamed({ ...emptyCarrierContact(), email: 'a@b.fr' })).toBe(false)
    })
    it('buildCarrierContactPayload : phones = 1 numéro sans secondaire', () => {
        const body = buildCarrierContactPayload({ ...emptyCarrierContact(), phonePrimary: '0102030405' })
        expect(body.phones).toEqual(['0102030405'])
    })
    it('buildCarrierContactPayload : phones = 2 numéros si secondaire révélé', () => {
        const body = buildCarrierContactPayload({
            ...emptyCarrierContact(),
            phonePrimary: '0102030405',
            phoneSecondary: '0605040302',
            hasSecondaryPhone: true,
        })
        expect(body.phones).toEqual(['0102030405', '0605040302'])
    })
    it('buildCarrierContactPayload : 2e numéro ignoré tant que non révélé', () => {
        const body = buildCarrierContactPayload({
            ...emptyCarrierContact(),
            phonePrimary: '0102030405',
            phoneSecondary: '0605040302',
            hasSecondaryPhone: false,
        })
        expect(body.phones).toEqual(['0102030405'])
    })
 })
 describe('useCarrierForm — onglet Contacts (ERP-168)', () => {
    beforeEach(() => {
        mockPost.mockReset()
        mockPatch.mockReset()
        mockDelete.mockReset()
    })
    /** Transporteur créé, onglet Contacts accessible. */
    function createdForm() {
        const form = useCarrierForm()
        form.carrierId.value = 7
        return form
    }
    it('« + Nouveau contact » désactivé tant que le bloc n\'est pas nommé (prénom OU nom, aligné M1/M2/M3)', () => {
        const form = createdForm()
        expect(form.canAddContact.value).toBe(false)
        // addContact est un no-op tant que le bloc n'est pas nommé.
        form.addContact()
        expect(form.contacts.value).toHaveLength(1)
        // Fonction seule ne suffit PAS à ajouter un nouveau bloc (≠ RG-4.08 large).
        const first = form.contacts.value[0]
        if (first) first.jobTitle = 'Acheteur'
        expect(form.canAddContact.value).toBe(false)
        form.addContact()
        expect(form.contacts.value).toHaveLength(1)
        // Un nom (ou prénom) débloque l'ajout.
        if (first) first.lastName = 'Doe'
        expect(form.canAddContact.value).toBe(true)
        form.addContact()
        expect(form.contacts.value).toHaveLength(2)
    })
    it('submitContacts : POST des nouveaux contacts (phones tableau), capture id, finalise', async () => {
        mockPost.mockResolvedValueOnce({ id: 55 })
        const form = createdForm()
        const c = form.contacts.value[0]
        if (c) { c.firstName = 'Jean'; c.phonePrimary = '0102030405' }
        const ok = await form.submitContacts(vi.fn())
        expect(ok).toBe(true)
        const [url, body, opts] = mockPost.mock.calls[0] ?? []
        expect(url).toBe('/carriers/7/contacts')
        expect(body).toMatchObject({ firstName: 'Jean', phones: ['0102030405'] })
        expect(opts).toMatchObject({ toast: false, headers: { Accept: 'application/ld+json' } })
        expect(form.contacts.value[0]?.id).toBe(55)
        expect(form.isValidated('contacts')).toBe(true)
    })
    it('submitContacts : PATCH des contacts existants sur /carrier_contacts/{id}', async () => {
        mockPatch.mockResolvedValueOnce({})
        const form = createdForm()
        const c = form.contacts.value[0]
        if (c) { c.id = 55; c.lastName = 'Doe' }
        await form.submitContacts(vi.fn())
        expect(mockPost).not.toHaveBeenCalled()
        expect(mockPatch).toHaveBeenCalledWith('/carrier_contacts/55', expect.objectContaining({ lastName: 'Doe' }), { toast: false })
    })
    it('RG-4.08 : onglet vide → soumet l\'amorce pour déclencher la 422 inline', async () => {
        mockPost.mockRejectedValueOnce({
            response: {
                status: 422,
                _data: { violations: [{ propertyPath: 'firstName', message: 'Au moins un champ du contact est obligatoire.' }] },
            },
        })
        const form = createdForm()
        const ok = await form.submitContacts(vi.fn())
        expect(ok).toBe(false)
        expect(mockPost).toHaveBeenCalledTimes(1)
        expect(form.contactErrors.value[0]?.firstName).toBe('Au moins un champ du contact est obligatoire.')
        expect(form.isValidated('contacts')).toBe(false)
    })
    it('removeContact : DELETE /carrier_contacts/{id} puis retrait du bloc', async () => {
        mockDelete.mockResolvedValueOnce({})
        const form = createdForm()
        const c = form.contacts.value[0]
        if (c) { c.id = 90; c.lastName = 'Doe' }
        form.addContact()
        const c2 = form.contacts.value[1]
        if (c2) c2.firstName = 'Jean'
        await form.removeContact(0)
        expect(mockDelete).toHaveBeenCalledWith('/carrier_contacts/90', {}, { toast: false })
        expect(form.contacts.value).toHaveLength(1)
    })
 })
@@ -5,16 +5,13 @@ import { removeCollectionRow } from '~/shared/utils/collectionRow'
 import {
    emptyCarrierAddress,
    emptyCarrierAddressCopy,
    emptyCarrierContact,
    emptyCarrierMain,
    type CarrierAddressCopy,
    type CarrierAddressFormDraft,
    type CarrierContactFormDraft,
    type CarrierMainDraft,
    type CarrierMainResponse,
 } from '~/modules/transport/types/carrierForm'
 import { buildCarrierAddressPayload, isCarrierAddressValid } from '~/modules/transport/utils/forms/carrierAddress'
 import { buildCarrierContactPayload, isCarrierContactBlank, isCarrierContactNamed } from '~/modules/transport/utils/forms/carrierContact'
 import type { QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
 /** Nom du cas spécial « compte-propre » LIOT (comparaison insensible à la casse, RG-4.01). */
@@ -385,86 +382,6 @@ export function useCarrierForm() {
        }
    }
    // ── Onglet Contacts (ERP-168) ─────────────────────────────────────────────
    const contacts = ref<CarrierContactFormDraft[]>([emptyCarrierContact()])
    // Erreurs 422 par ligne (alignées sur l'index du v-for), peuplées par submitRows.
    const contactErrors = ref<Record<string, string>[]>([])
    // « + Nouveau contact » désactivé tant que le DERNIER bloc n'est pas « nommé »
    // (prénom OU nom) — aligné sur M1/M2/M3 (fonction / téléphone / email seuls ne
    // suffisent pas à ajouter un nouveau bloc).
    const canAddContact = computed(() => {
        const last = contacts.value[contacts.value.length - 1]
        return last !== undefined && isCarrierContactNamed(last)
    })
    function addContact(): void {
        if (canAddContact.value) {
            contacts.value.push(emptyCarrierContact())
        }
    }
    /** Suppression immédiate d'un contact existant (DELETE /carrier_contacts/{id}). */
    async function removeContact(index: number): Promise<void> {
        await removeCollectionRow({
            rows: contacts.value,
            errors: contactErrors.value,
            index,
            endpoint: '/carrier_contacts',
            deleteRow: url => api.delete(url, {}, { toast: false }),
            makeEmpty: emptyCarrierContact,
            onError: notifyRemovalError,
        })
    }
    /**
     * Valide l'onglet Contacts : POST des nouveaux contacts sur
     * /carriers/{id}/contacts, PATCH des existants sur /carrier_contacts/{id}
     * (groupe carrier:write:contacts). RG-4.08 (≥ 1 champ rempli, max 2 téléphones)
     * re-validée back → 422 par ligne. Si l'onglet ne contient QUE des amorces
     * vides, on soumet la 1re pour déclencher la 422 RG-4.08 inline plutôt que de
     * finaliser un onglet vide. Retourne true si l'onglet a été validé.
     */
    async function submitContacts(onError: (error: unknown) => void): Promise<boolean> {
        if (carrierId.value === null || tabSubmitting.value) {
            return false
        }
        tabSubmitting.value = true
        try {
            const hasSubmittable = contacts.value.some(c => c.id !== null || !isCarrierContactBlank(c))
            const hasError = await submitRows(
                contacts.value,
                contactErrors,
                async (contact) => {
                    const body = buildCarrierContactPayload(contact)
                    if (contact.id === null) {
                        const created = await api.post<{ id: number }>(
                            `/carriers/${carrierId.value}/contacts`,
                            body,
                            { headers: { Accept: 'application/ld+json' }, toast: false },
                        )
                        contact.id = created.id
                    }
                    else {
                        await api.patch(`/carrier_contacts/${contact.id}`, body, { toast: false })
                    }
                },
                onError,
                // Amorce vide neuve ignorée s'il reste un autre bloc soumettable ;
                // sinon on la soumet pour déclencher la 422 RG-4.08 (sur firstName).
                contact => hasSubmittable && contact.id === null && isCarrierContactBlank(contact),
            )
            if (hasError) {
                return false
            }
            completeTab('contacts')
            return true
        }
        finally {
            tabSubmitting.value = false
        }
    }
    /**
     * Intègre une ligne QUALIMAT sélectionnée dans l'onglet Qualimat (RG-4.01 /
     * § 2.5) : copie le nom, force la certification à « QUALIMAT » (lecture seule),
@@ -564,13 +481,6 @@ export function useCarrierForm() {
        addAddress,
        removeAddress,
        submitAddresses,
        // contacts
        contacts,
        contactErrors,
        canAddContact,
        addContact,
        removeContact,
        submitContacts,
        // actions
        validateMainFront,
        buildMainPayload,
@@ -207,40 +207,7 @@
                </div>
            </template>
-            <!-- Onglet Contacts (ERP-168) : un bloc par contact (RG-4.08 ≥ 1 champ,
+            <!-- Contacts / Prix : contenu aux tickets suivants. -->
                 max 2 téléphones). Erreurs 422 par ligne. -->
            <template #contacts>
                <div class="mt-12 flex flex-col gap-6">
                    <CarrierContactBlock
                        v-for="(contact, index) in contacts"
                        :key="index"
                        :model-value="contact"
                        :removable="isRowRemovable(contacts, index)"
                        :readonly="isValidated('contacts')"
                        :errors="contactErrors[index]"
                        @update:model-value="(v) => contacts[index] = v"
                        @remove="askRemoveContact(index)"
                    />
                    <div v-if="!isValidated('contacts')" class="flex justify-center gap-6">
                        <MalioButton
                            variant="secondary"
                            icon-name="mdi:add-bold"
                            icon-position="left"
                            :label="t('transport.carriers.form.contact.add')"
                            :disabled="!canAddContact"
                            @click="addContact"
                        />
                        <MalioButton
                            variant="primary"
                            :label="t('transport.carriers.form.submit')"
                            :disabled="tabSubmitting || carrierId === null"
                            @click="onSubmitContacts"
                        />
                    </div>
                </div>
            </template>
            <!-- Prix : contenu au ticket suivant. -->
            <template
                v-for="key in placeholderTabs"
                :key="key"
@@ -304,7 +271,6 @@ import { debounce } from '~/shared/utils/debounce'
 import { extractApiErrorMessage } from '~/shared/utils/api'
 import { isRowRemovable } from '~/shared/utils/collectionRow'
 import CarrierAddressBlock from '~/modules/transport/components/CarrierAddressBlock.vue'
 import CarrierContactBlock from '~/modules/transport/components/CarrierContactBlock.vue'
 import { useCarrierForm } from '~/modules/transport/composables/useCarrierForm'
 import { useQualimatSearch, type QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
@@ -349,12 +315,6 @@ const {
    addAddress,
    removeAddress,
    submitAddresses,
    contacts,
    contactErrors,
    canAddContact,
    addContact,
    removeContact,
    submitContacts,
    submitMain,
    applyQualimatSelection,
 } = useCarrierForm()
@@ -448,10 +408,8 @@ const tabs = computed(() => tabKeys.value.map((key, index) => ({
    disabled: index > unlockedIndex.value,
 })))
-// Onglets dont le contenu arrive aux tickets suivants (Prix).
+// Onglets dont le contenu arrive aux tickets suivants (Contacts / Prix).
-const placeholderTabs = computed(() => tabKeys.value.filter(
+const placeholderTabs = computed(() => tabKeys.value.filter(key => key !== 'qualimat' && key !== 'addresses'))
    key => key !== 'qualimat' && key !== 'addresses' && key !== 'contacts',
 ))
 // ── Onglet Adresses (ERP-167) ────────────────────────────────────────────────
 // Pays : France garantie en tete meme si /countries echoue (resilience), pour
@@ -511,7 +469,7 @@ async function onSubmitAddresses(): Promise<void> {
    }
 }
-// Modal de confirmation de suppression (générique : bloc adresse OU contact).
+// Modal de confirmation de suppression (bloc adresse).
 const deleteConfirm = reactive({ open: false, action: null as null | (() => void) })
 function askRemoveAddress(index: number): void {
@@ -519,22 +477,6 @@ function askRemoveAddress(index: number): void {
    deleteConfirm.open = true
 }
 /** Valide l'onglet Contacts (POST/PATCH par ligne ; avance gérée par le composable). */
 async function onSubmitContacts(): Promise<void> {
    const ok = await submitContacts(error => toast.error({
        title: t('transport.carriers.toast.error'),
        message: apiErrorMessage(error),
    }))
    if (ok) {
        toast.success({ title: t('transport.carriers.toast.contactSaved') })
    }
 }
 function askRemoveContact(index: number): void {
    deleteConfirm.action = () => { void removeContact(index) }
    deleteConfirm.open = true
 }
 function runDeleteConfirm(): void {
    deleteConfirm.action?.()
    deleteConfirm.action = null
@@ -94,41 +94,6 @@ export function emptyCarrierAddress(): CarrierAddressFormDraft {
    }
 }
 /**
 * Brouillon d'un bloc Contact (onglet Contacts, ERP-168) — sous-ressource
 * `CarrierContact` (groupe `carrier:write:contacts`). Les téléphones sont saisis
 * en `phonePrimary` / `phoneSecondary` côté UI, puis envoyés au back sous forme du
 * tableau `phones` (max 2 — RG-4.08). `hasSecondaryPhone` pilote l'affichage du 2e
 * numéro (révélé via le bouton « + »). Pas d'`iri` : aucune relation M2M depuis
 * l'adresse au M4 (≠ M3).
 */
 export interface CarrierContactFormDraft {
    /** Id serveur une fois le contact créé (null tant que non persisté). */
    id: number | null
    firstName: string | null
    lastName: string | null
    jobTitle: string | null
    phonePrimary: string | null
    phoneSecondary: string | null
    email: string | null
    /** UI : le 2e numéro a été révélé via le bouton « + » (max 2 téléphones). */
    hasSecondaryPhone: boolean
 }
 /** Brouillon de contact vide (état initial d'un bloc Contact). */
 export function emptyCarrierContact(): CarrierContactFormDraft {
    return {
        id: null,
        firstName: null,
        lastName: null,
        jobTitle: null,
        phonePrimary: null,
        phoneSecondary: null,
        email: null,
        hasSecondaryPhone: false,
    }
 }
 /**
 * Réponse du POST / PATCH principal (groupe `carrier:read`). Le serveur renvoie
 * le nom normalisé (UPPERCASE, RG-4.13) que l'UI réaffiche tel quel.
@@ -1,61 +0,0 @@
 /**
 * Helpers purs de l'onglet Contact transporteur (M4 Transport, ERP-168) — ALIGNÉ
 * sur `providerContact.ts` (M3) / les autres modules : mêmes règles de validité et
 * de gating « + Nouveau contact » (un contact est « nommé » dès qu'il porte un
 * prénom OU un nom). Seule spécificité M4 conservée : les téléphones partent au back
 * dans le tableau virtuel `phones` (max 2), mappés par le CarrierContactProcessor.
 * Testables sans Vue ni API.
 */
 import type { CarrierContactFormDraft } from '~/modules/transport/types/carrierForm'
 /** Vrai si une chaîne porte au moins un caractère non-espace. */
 function isFilled(value: string | null | undefined): boolean {
    return value !== null && value !== undefined && value.trim() !== ''
 }
 /**
 * Un bloc Contact est VIDE tant qu'aucun champ comptant pour la validité n'est
 * rempli — prénom / nom / fonction / téléphone principal / email. `phoneSecondary`
 * est EXCLU (aligné M1/M2/M3 : un bloc ne portant qu'un 2e numéro reste vide). Sert
 * le filtrage des amorces vides à la soumission.
 */
 export function isCarrierContactBlank(contact: CarrierContactFormDraft): boolean {
    return ![
        contact.firstName,
        contact.lastName,
        contact.jobTitle,
        contact.phonePrimary,
        contact.email,
    ].some(isFilled)
 }
 /**
 * Un contact est « nommé » (valide) dès qu'il porte un prénom OU un nom — aligné
 * sur M1/M2/M3. Pilote le gating « + Nouveau contact » : la fonction / le téléphone
 * / l'email seuls ne suffisent pas pour ajouter un nouveau bloc.
 */
 export function isCarrierContactNamed(contact: CarrierContactFormDraft): boolean {
    return isFilled(contact.firstName) || isFilled(contact.lastName)
 }
 /**
 * Payload de la sous-ressource contacts (groupe `carrier:write:contacts`). Les
 * chaînes vides partent à null (le serveur normalise/trim). Les téléphones sont
 * regroupés dans le tableau `phones` (numéros non vides, max 2 — RG-4.08) ; le 2e
 * numéro n'est inclus que s'il a été révélé (`hasSecondaryPhone`).
 */
 export function buildCarrierContactPayload(contact: CarrierContactFormDraft): Record<string, unknown> {
    const phones = [
        contact.phonePrimary,
        contact.hasSecondaryPhone ? contact.phoneSecondary : null,
    ].filter((phone): phone is string => isFilled(phone))
    return {
        firstName: contact.firstName || null,
        lastName: contact.lastName || null,
        jobTitle: contact.jobTitle || null,
        email: contact.email || null,
        phones,
    }
 }
@@ -1,49 +0,0 @@
 <?php
 declare(strict_types=1);
 namespace DoctrineMigrations;
 use Doctrine\DBAL\Schema\Schema;
 use Doctrine\Migrations\AbstractMigration;
 /**
 * RG-4.08 (correctif) — aligne la regle de validite d'un contact transporteur sur
 * le M1/M2/M3 : au moins le PRENOM OU le NOM (et non plus « un champ quelconque
 * parmi prenom/nom/fonction/telephone/email »). Remplace le CHECK
 * chk_carrier_contact_filled par chk_carrier_contact_name et met a jour les
 * commentaires de colonnes. La garde applicative (CarrierContactProcessor::validateName)
 * est alignee dans le meme commit ; le catalogue ColumnCommentsCatalog aussi.
 *
 * Placee au namespace racine DoctrineMigrations (et non en modulaire Transport) :
 * elle ALTERE une table creee par une migration racine (Version20260615150000) ;
 * le tri par version au sein du meme namespace garantit qu'elle joue APRES l'init
 * (cf. CLAUDE.md regle 11 — le tri cross-namespace casserait l'ordre sur base vide).
 */
 final class Version20260617120000 extends AbstractMigration
 {
    public function getDescription(): string
    {
        return 'RG-4.08 : contact transporteur valide si prenom OU nom (alignement M1/M2/M3) — CHECK chk_carrier_contact_name.';
    }
    public function up(Schema $schema): void
    {
        $this->addSql('ALTER TABLE carrier_contact DROP CONSTRAINT chk_carrier_contact_filled');
        $this->addSql('ALTER TABLE carrier_contact ADD CONSTRAINT chk_carrier_contact_name CHECK (first_name IS NOT NULL OR last_name IS NOT NULL)');
        $this->addSql('COMMENT ON TABLE carrier_contact IS $_$Contacts d un transporteur (1:n) — onglet Contact (M4). Au moins le prenom OU le nom rempli (RG-4.08, chk_carrier_contact_name), max 2 telephones.$_$');
        $this->addSql('COMMENT ON COLUMN carrier_contact.first_name IS $_$Prenom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-4.08, chk_carrier_contact_name).$_$');
        $this->addSql('COMMENT ON COLUMN carrier_contact.last_name IS $_$Nom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-4.08, chk_carrier_contact_name).$_$');
    }
    public function down(Schema $schema): void
    {
        $this->addSql('ALTER TABLE carrier_contact DROP CONSTRAINT chk_carrier_contact_name');
        $this->addSql('ALTER TABLE carrier_contact ADD CONSTRAINT chk_carrier_contact_filled CHECK (first_name IS NOT NULL OR last_name IS NOT NULL OR job_title IS NOT NULL OR phone_primary IS NOT NULL OR email IS NOT NULL)');
        $this->addSql('COMMENT ON TABLE carrier_contact IS $_$Contacts d un transporteur (1:n) — onglet Contact (M4). Au moins un champ rempli (RG-4.08, chk_carrier_contact_filled), max 2 telephones.$_$');
        $this->addSql('COMMENT ON COLUMN carrier_contact.first_name IS $_$Prenom du contact (capitalise serveur). Au moins un champ du contact est requis (RG-4.08).$_$');
        $this->addSql('COMMENT ON COLUMN carrier_contact.last_name IS $_$Nom du contact (capitalise serveur). Au moins un champ du contact est requis (RG-4.08).$_$');
    }
 }
@@ -21,8 +21,8 @@ use Symfony\Component\Validator\Constraints as Assert;
 /**
 * Contact d'un transporteur (1:n) — onglet Contact (M4). Jumeau de
- * SupplierContact (M2) : au moins le prenom OU le nom (RG-4.08, garanti par le
+ * SupplierContact (M2) : au moins un champ rempli (RG-4.08, garanti par le
- * CHECK chk_carrier_contact_name + le Processor), max 2 telephones.
+ * CHECK chk_carrier_contact_filled + le Processor), max 2 telephones.
 *
 * Lecture : proprietes en `carrier:item:read` (embarquees au detail du
 * transporteur). Ecriture : groupe `carrier:write:contacts`.
@@ -23,21 +23,21 @@ use function is_string;
 /**
 * Processor d'ecriture de la sous-ressource Contact d'un transporteur (M4,
 * spec-back § 4.5). Jumeau du SupplierContactProcessor (M2), recentre sur le
- * perimetre ERP-160. RG-4.08 (correctif, alignement M1/M2/M3) : un contact exige
+ * perimetre ERP-160, AVEC deux specificites M4 : RG-4.08 (≥ 1 champ rempli, max
- * au moins le PRENOM OU le NOM (la fonction / le telephone / l'email seuls ne
+ * 2 telephones) portee a la fois par le CHECK BDD chk_carrier_contact_filled et
- * suffisent pas), porte a la fois par le CHECK BDD chk_carrier_contact_name et par
+ * par ce Processor.
 * ce Processor ; le « max 2 telephones » reste une specificite M4.
 *
 * Sequence :
 *  - POST / PATCH : rattachement au transporteur parent (linkParent),
 *    normalisation serveur RG-4.13 (prenom/nom Title Case, email lowercase),
 *    mapping du tableau d'ecriture `phones` -> phonePrimary/phoneSecondary
- *    (max 2, chiffres uniquement), puis garde « prenom OU nom » avant persistance.
+ *    (max 2, chiffres uniquement), puis garde RG-4.08 (≥ 1 champ) avant
 *    persistance.
 *  - DELETE : aucune regle metier specifique (suppression physique directe).
 *
- * La garde « prenom OU nom » vit ICI (double du CHECK BDD) pour transformer une
+ * RG-4.08 vit ICI (double du CHECK BDD) pour transformer une violation SQL (500
- * violation SQL (500 generique) en 422 propre rattachee au champ `firstName`
+ * generique) en 422 propre rattachee au champ `firstName` (mapping inline
- * (mapping inline ERP-101). Le « max 2 telephones » est rattache au champ `phones` : seul
+ * ERP-101). Le « max 2 telephones » est rattache au champ `phones` : seul
 * point de saisie des numeros (les colonnes phonePrimary/phoneSecondary sont en
 * lecture seule).
 *
@@ -77,7 +77,7 @@ final class CarrierContactProcessor implements ProcessorInterface
        $this->linkParent($data, $uriVariables);
        $this->normalize($data);
        $this->applyPhones($data);
-        $this->validateName($data);
+        $this->validateAtLeastOneField($data);
        return $this->persistProcessor->process($data, $operation, $uriVariables, $context);
    }
@@ -187,18 +187,25 @@ final class CarrierContactProcessor implements ProcessorInterface
    }
    /**
-     * RG-4.08 (alignement M1/M2/M3) : un bloc Contact exige au moins le PRENOM OU le
+     * RG-4.08 : un bloc Contact est valide des qu'au moins 1 champ est rempli
-     * NOM — un contact se materialise par son nom ; fonction / telephone / email
+     * (firstName, lastName, jobTitle, phonePrimary ou email — meme perimetre que
-     * seuls ne suffisent pas. Double garde avec le CHECK BDD chk_carrier_contact_name
+     * le CHECK BDD chk_carrier_contact_filled, qui exclut phoneSecondary). Double
-     * — leve une 422 propre rattachee a `firstName` plutot qu'une 500 SQL. Joue apres
+     * garde : leve une 422 propre rattachee a `firstName` plutot qu'une 500 SQL.
-     * normalisation + mapping telephones, donc les chaines vides sont deja null.
+     * Joue apres normalisation + mapping telephones, donc les chaines vides sont
     * deja ramenees a null.
     */
-    private function validateName(CarrierContact $contact): void
+    private function validateAtLeastOneField(CarrierContact $contact): void
    {
-        if (null === $contact->getFirstName() && null === $contact->getLastName()) {
+        if (
            null === $contact->getFirstName()
            && null === $contact->getLastName()
            && null === $contact->getJobTitle()
            && null === $contact->getPhonePrimary()
            && null === $contact->getEmail()
        ) {
            $violations = new ConstraintViolationList();
            $violations->add(new ConstraintViolation(
-                'Le prénom ou le nom du contact est obligatoire.',
+                'Renseignez au moins un champ pour le contact.',
                null,
                [],
                $contact,
@@ -212,8 +219,8 @@ final class CarrierContactProcessor implements ProcessorInterface
    /**
     * Trim + chaine vide -> null (la fonction n'est pas normalisee en casse,
-     * contrairement aux noms de personne). Evite de persister une chaine vide
+     * contrairement aux noms de personne). Garantit que RG-4.08 detecte un champ
-     * (« » devient null) cote fonction du contact.
+     * « non rempli » meme si le client envoie une chaine vide.
     */
    private function blankToNull(?string $value): ?string
    {
@@ -194,7 +194,7 @@ class CarrierFixtures extends Fixture implements DependentFixtureInterface
    /**
     * Ajoute un contact normalise au transporteur (cascade persist via
-     * Carrier.contacts). Prenom OU nom toujours fourni (RG-4.08, chk_carrier_contact_name).
+     * Carrier.contacts). Au moins un champ est toujours fourni (RG-4.08).
     */
    private function addContact(
        Carrier $carrier,
@@ -509,11 +509,11 @@ final class ColumnCommentsCatalog
            ] + self::timestampableBlamableComments(),
            'carrier_contact' => [
-                '_table'          => 'Contacts d un transporteur (1:n) — onglet Contact (M4). Au moins le prenom OU le nom rempli (RG-4.08, chk_carrier_contact_name), max 2 telephones.',
+                '_table'          => 'Contacts d un transporteur (1:n) — onglet Contact (M4). Au moins un champ rempli (RG-4.08, chk_carrier_contact_filled), max 2 telephones.',
                'id'              => 'Identifiant interne auto-incremente.',
                'carrier_id'      => 'FK -> carrier.id, ON DELETE CASCADE — transporteur proprietaire du contact.',
-                'first_name'      => 'Prenom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-4.08, chk_carrier_contact_name).',
+                'first_name'      => 'Prenom du contact (capitalise serveur). Au moins un champ du contact est requis (RG-4.08).',
-                'last_name'       => 'Nom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-4.08, chk_carrier_contact_name).',
+                'last_name'       => 'Nom du contact (capitalise serveur). Au moins un champ du contact est requis (RG-4.08).',
                'job_title'       => 'Fonction / intitule de poste du contact (≤ 120 caracteres).',
                'phone_primary'   => 'Telephone principal — chiffres uniquement (normalisation serveur).',
                'phone_secondary' => 'Telephone secondaire — chiffres uniquement (max 2 telephones, RG-4.08).',
@@ -15,8 +15,8 @@ use Symfony\Component\Console\Output\NullOutput;
 * POST /api/carriers/{id}/contacts, PATCH/DELETE /api/carrier_contacts/{id}.
 *
 * Contrat verifie :
- *  - RG-4.08 : contact sans prenom ni nom -> 422 (alignement M1/M2/M3) ;
+ *  - RG-4.08 : contact totalement vide -> 422 (au moins 1 champ requis) ;
- *  - RG-4.08 : un nom (ou prenom) suffit -> 201 ;
+ *  - RG-4.08 : 1 seul champ rempli -> 201 ;
 *  - RG-4.08 : 3 telephones (tableau `phones`) -> 422 (max 2) ;
 *  - mapping `phones[]` -> phonePrimary / phoneSecondary + normalisation (RG-4.13) ;
 *  - PATCH / DELETE OK avec transport.carriers.manage, 403 sans (view seul).
@@ -51,8 +51,7 @@ final class CarrierContactApiTest extends AbstractCarrierApiTestCase
    public function testEmptyContactReturns422(): void
    {
-        // RG-4.08 (alignement M1/M2/M3) : sans prenom ni nom -> 422 (garde Processor,
+        // RG-4.08 : aucun champ rempli -> 422 (garde Processor, double du CHECK BDD).
        // double du CHECK BDD chk_carrier_contact_name).
        $carrier = $this->seedCarrier('Contact Vide');
        $client  = $this->createAdminClient();
@@ -61,13 +60,13 @@ final class CarrierContactApiTest extends AbstractCarrierApiTestCase
            'json'    => [],
        ]);
        self::assertResponseStatusCodeSame(422);
-        // La violation est rattachee a `firstName` (mapping inline ERP-101).
+        // RG-4.08 : la violation est rattachee a `firstName` (mapping inline ERP-101).
        self::assertViolationOnPath($response, 'firstName');
    }
    public function testSingleFieldContactIsCreated(): void
    {
-        // RG-4.08 : un nom (ou prenom) suffit a valider le bloc.
+        // RG-4.08 : un seul champ suffit a valider le bloc.
        $carrier = $this->seedCarrier('Contact Mono');
        $client  = $this->createAdminClient();
`@@ -1,2 +1,2 @@`
	`parameters:`	`parameters:`
	`app.version: '0.1.135'`	`app.version: '0.1.134'`