chore: bump version to v0.1.134

config/version.yaml

@@ -1,2 +1,2 @@
 parameters:
-    app.version: '0.1.136'
+    app.version: '0.1.134'

frontend/i18n/locales/fr.json

@@ -528,9 +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",
-                "contactSaved": "Contact enregistré",
-                "priceSaved": "Prix enregistré"
+                "addressSaved": "Adresse enregistrée"
             },
             "containerType": {
                 "BENNE": "Benne",
@@ -593,57 +591,11 @@
                     "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 ?",
                     "cancel": "Annuler",
                     "confirm": "Supprimer"
-                },
-                "price": {
-                    "direction": "Sens",
-                    "directionClient": "Client",
-                    "directionSupplier": "Fournisseur",
-                    "client": "Client",
-                    "clientDeliveryAddress": "Adresse de livraison",
-                    "departureSite": "Adresse de départ",
-                    "supplier": "Fournisseur",
-                    "supplierSupplyAddress": "Adresse d'approvisionnement",
-                    "deliverySite": "Adresse de livraison",
-                    "containerType": "Benne / Fond mouvant",
-                    "pricingUnit": "Forfait / Tonne",
-                    "pricingForfait": "Forfait",
-                    "pricingTonne": "Tonne",
-                    "price": "Prix",
-                    "priceState": "État du prix",
-                    "stateEnCours": "En cours",
-                    "stateValide": "Validé",
-                    "stateNonValide": "Non validé",
-                    "add": "Nouveau prix",
-                    "remove": "Supprimer le prix",
-                    "errors": {
-                        "direction": "Le sens du prix est obligatoire.",
-                        "client": "Le client est obligatoire pour un prix client.",
-                        "clientDeliveryAddress": "L'adresse de livraison du client est obligatoire pour un prix client.",
-                        "departureSite": "Le site de départ est obligatoire pour un prix client.",
-                        "supplier": "Le fournisseur est obligatoire pour un prix fournisseur.",
-                        "supplierSupplyAddress": "L'adresse d'approvisionnement est obligatoire pour un prix fournisseur.",
-                        "deliverySite": "Le site de livraison est obligatoire pour un prix fournisseur.",
-                        "containerType": "Le type de contenant est obligatoire.",
-                        "pricingUnit": "L'unité de tarification est obligatoire.",
-                        "price": "Le prix est obligatoire.",
-                        "priceState": "L'état du prix est obligatoire."
-                    }
                 }
             }
         }
@@ -690,27 +642,27 @@
             "delete": "Suppression"
         },
         "entity": {
-            "core_user": "Utilisateur",
-            "core_role": "Rôle",
-            "core_permission": "Permission",
-            "sites_site": "Site",
-            "catalog_category": "Catégorie",
-            "commercial_client": "Client",
+            "core_user":              "Utilisateur",
+            "core_role":              "Rôle",
+            "core_permission":        "Permission",
+            "sites_site":             "Site",
+            "catalog_category":       "Catégorie",
+            "commercial_client":      "Client",
             "commercial_clientaddress": "Adresse client",
             "commercial_clientcontact": "Contact client",
-            "commercial_clientrib": "RIB client",
-            "commercial_supplier": "Fournisseur",
+            "commercial_clientrib":   "RIB client",
+            "commercial_supplier":    "Fournisseur",
             "commercial_supplieraddress": "Adresse fournisseur",
             "commercial_suppliercontact": "Contact fournisseur",
             "commercial_supplierrib": "RIB fournisseur",
-            "technique_provider": "Prestataire",
+            "technique_provider":     "Prestataire",
             "technique_provideraddress": "Adresse prestataire",
             "technique_providercontact": "Contact prestataire",
-            "technique_providerrib": "RIB prestataire",
-            "transport_carrier": "Transporteur",
-            "transport_carrieraddress": "Adresse transporteur",
-            "transport_carriercontact": "Contact transporteur",
-            "transport_carrierprice": "Prix transporteur"
+            "technique_providerrib":  "RIB prestataire",
+            "transport_carrier":         "Transporteur",
+            "transport_carrieraddress":  "Adresse transporteur",
+            "transport_carriercontact":  "Contact transporteur",
+            "transport_carrierprice":    "Prix transporteur"
         },
         "empty": "Aucune activité enregistrée",
         "no_results": "Aucun résultat pour ces filtres",

frontend/modules/transport/components/CarrierContactBlock.vue

@@ -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>

frontend/modules/transport/components/CarrierPriceBlock.vue

@@ -1,302 +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 : modal de confirmation côté parent. -->
-        <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.price.remove') }"
-            @click="$emit('remove')"
-        />
-
-        <!-- RG-4.09 : sens du prix (CLIENT / FOURNISSEUR) en colonne 1 / ligne 1, radios
-             EN LIGNE (horizontaux), centrés sur la hauteur de champ (h-12) comme la
-             case « Affréter ». Pas de label de groupe. -->
-        <div>
-            <div class="flex h-12 items-center gap-6">
-                <MalioRadioButton
-                    :model-value="model.direction"
-                    name="price-direction"
-                    value="CLIENT"
-                    :label="t('transport.carriers.form.price.directionClient')"
-                    :disabled="readonly"
-                    group-class="mt-0"
-                    @update:model-value="onDirectionChange"
-                />
-                <MalioRadioButton
-                    :model-value="model.direction"
-                    name="price-direction"
-                    value="FOURNISSEUR"
-                    :label="t('transport.carriers.form.price.directionSupplier')"
-                    :disabled="readonly"
-                    group-class="mt-0"
-                    @update:model-value="onDirectionChange"
-                />
-            </div>
-            <p v-if="errors?.direction" class="ml-[2px] text-xs text-m-danger">{{ errors.direction }}</p>
-        </div>
-
-        <!-- Branche CLIENT (RG-4.10). -->
-        <template v-if="model.direction === 'CLIENT'">
-            <MalioSelect
-                :model-value="model.clientIri"
-                :options="clientOptions"
-                :label="t('transport.carriers.form.price.client')"
-                empty-option-label=""
-                :required="true"
-                :readonly="readonly"
-                :error="errors?.client"
-                @update:model-value="onClientChange"
-            />
-            <MalioSelect
-                :model-value="model.clientDeliveryAddressIri"
-                :options="clientAddressOptions"
-                :label="t('transport.carriers.form.price.clientDeliveryAddress')"
-                empty-option-label=""
-                :required="true"
-                :readonly="readonly"
-                :error="errors?.clientDeliveryAddress"
-                @update:model-value="(v: string | number | null) => update('clientDeliveryAddressIri', v === null ? null : String(v))"
-            />
-            <MalioSelect
-                :model-value="model.departureSiteIri"
-                :options="siteOptions"
-                :label="t('transport.carriers.form.price.departureSite')"
-                empty-option-label=""
-                :required="true"
-                :readonly="readonly"
-                :error="errors?.departureSite"
-                @update:model-value="(v: string | number | null) => update('departureSiteIri', v === null ? null : String(v))"
-            />
-        </template>
-
-        <!-- Branche FOURNISSEUR (RG-4.11). -->
-        <template v-else-if="model.direction === 'FOURNISSEUR'">
-            <MalioSelect
-                :model-value="model.supplierIri"
-                :options="supplierOptions"
-                :label="t('transport.carriers.form.price.supplier')"
-                empty-option-label=""
-                :required="true"
-                :readonly="readonly"
-                :error="errors?.supplier"
-                @update:model-value="onSupplierChange"
-            />
-            <MalioSelect
-                :model-value="model.supplierSupplyAddressIri"
-                :options="supplierAddressOptions"
-                :label="t('transport.carriers.form.price.supplierSupplyAddress')"
-                empty-option-label=""
-                :required="true"
-                :readonly="readonly"
-                :error="errors?.supplierSupplyAddress"
-                @update:model-value="(v: string | number | null) => update('supplierSupplyAddressIri', v === null ? null : String(v))"
-            />
-            <MalioSelect
-                :model-value="model.deliverySiteIri"
-                :options="siteOptions"
-                :label="t('transport.carriers.form.price.deliverySite')"
-                empty-option-label=""
-                :required="true"
-                :readonly="readonly"
-                :error="errors?.deliverySite"
-                @update:model-value="(v: string | number | null) => update('deliverySiteIri', v === null ? null : String(v))"
-            />
-        </template>
-
-        <!-- Communs (visibles dès qu'un sens est choisi). -->
-        <template v-if="model.direction !== null">
-            <!-- Contenant : Benne / Fond mouvant (radios centrés h-12, pas de label). -->
-            <div>
-                <div class="flex h-12 items-center gap-4">
-                    <MalioRadioButton
-                        :model-value="model.containerType"
-                        name="price-container"
-                        value="BENNE"
-                        :label="t('transport.carriers.containerType.BENNE')"
-                        :disabled="readonly"
-                        group-class="mt-0"
-                        @update:model-value="(v: string | number | boolean | null) => update('containerType', v === null ? null : String(v))"
-                    />
-                    <MalioRadioButton
-                        :model-value="model.containerType"
-                        name="price-container"
-                        value="FOND_MOUVANT"
-                        :label="t('transport.carriers.containerType.FOND_MOUVANT')"
-                        :disabled="readonly"
-                        group-class="mt-0"
-                        @update:model-value="(v: string | number | boolean | null) => update('containerType', v === null ? null : String(v))"
-                    />
-                </div>
-                <p v-if="errors?.containerType" class="ml-[2px] text-xs text-m-danger">{{ errors.containerType }}</p>
-            </div>
-
-            <!-- Tarification : Forfait / Tonne (radios centrés h-12, pas de label). -->
-            <div>
-                <div class="flex h-12 items-center gap-4">
-                    <MalioRadioButton
-                        :model-value="model.pricingUnit"
-                        name="price-unit"
-                        value="FORFAIT"
-                        :label="t('transport.carriers.form.price.pricingForfait')"
-                        :disabled="readonly"
-                        group-class="mt-0"
-                        @update:model-value="(v: string | number | boolean | null) => update('pricingUnit', v === null ? null : String(v))"
-                    />
-                    <MalioRadioButton
-                        :model-value="model.pricingUnit"
-                        name="price-unit"
-                        value="TONNE"
-                        :label="t('transport.carriers.form.price.pricingTonne')"
-                        :disabled="readonly"
-                        group-class="mt-0"
-                        @update:model-value="(v: string | number | boolean | null) => update('pricingUnit', v === null ? null : String(v))"
-                    />
-                </div>
-                <p v-if="errors?.pricingUnit" class="ml-[2px] text-xs text-m-danger">{{ errors.pricingUnit }}</p>
-            </div>
-
-            <MalioInputAmount
-                :model-value="model.price"
-                :label="t('transport.carriers.form.price.price')"
-                :required="true"
-                :readonly="readonly"
-                :error="errors?.price"
-                @update:model-value="(v: string) => update('price', v)"
-            />
-
-            <MalioSelect
-                :model-value="model.priceState"
-                :options="priceStateOptions"
-                :label="t('transport.carriers.form.price.priceState')"
-                empty-option-label=""
-                :required="true"
-                :readonly="readonly"
-                :error="errors?.priceState"
-                @update:model-value="(v: string | number | null) => update('priceState', v === null ? null : String(v))"
-            />
-        </template>
-    </div>
-</template>
-
-<script setup lang="ts">
-import { computed, ref, watch } from 'vue'
-import type { CarrierPriceFormDraft } from '~/modules/transport/types/carrierForm'
-
-interface SelectOption {
-    value: string
-    label: string
-}
-
-const props = defineProps<{
-    /** Brouillon du prix (v-model). */
-    modelValue: CarrierPriceFormDraft
-    /** Clients disponibles (IRI en value). */
-    clientOptions: SelectOption[]
-    /** Fournisseurs disponibles (IRI en value). */
-    supplierOptions: SelectOption[]
-    /** Sites Starseed (3 sites — IRI en value). */
-    siteOptions: SelectOption[]
-    removable?: boolean
-    readonly?: boolean
-    /** Erreurs serveur 422 de cette ligne, indexées par champ (ERP-101). */
-    errors?: Record<string, string>
-}>()
-
-const emit = defineEmits<{
-    'update:modelValue': [value: CarrierPriceFormDraft]
-    'remove': []
-}>()
-
-const { t } = useI18n()
-const api = useApi()
-
-const model = computed(() => props.modelValue)
-
-const priceStateOptions = computed<SelectOption[]>(() => [
-    { value: 'EN_COURS', label: t('transport.carriers.form.price.stateEnCours') },
-    { value: 'VALIDE', label: t('transport.carriers.form.price.stateValide') },
-    { value: 'NON_VALIDE', label: t('transport.carriers.form.price.stateNonValide') },
-])
-
-// Adresses chargées à la volée pour le client / fournisseur sélectionné (par bloc).
-const clientAddressOptions = ref<SelectOption[]>([])
-const supplierAddressOptions = ref<SelectOption[]>([])
-
-/** Émet un nouveau brouillon avec le champ modifié (immutabilité). */
-function update<K extends keyof CarrierPriceFormDraft>(field: K, value: CarrierPriceFormDraft[K]): void {
-    emit('update:modelValue', { ...props.modelValue, [field]: value })
-}
-
-/** Changement de sens : réinitialise les DEUX branches (cohérence CHECK BDD). */
-function onDirectionChange(value: string | number | boolean | null): void {
-    const direction = value === 'CLIENT' || value === 'FOURNISSEUR' ? value : null
-    emit('update:modelValue', {
-        ...props.modelValue,
-        direction,
-        clientIri: null,
-        clientDeliveryAddressIri: null,
-        departureSiteIri: null,
-        supplierIri: null,
-        supplierSupplyAddressIri: null,
-        deliverySiteIri: null,
-    })
-}
-
-/** Sélection d'un client → maj IRI + reset de l'adresse de livraison (autre client). */
-function onClientChange(value: string | number | null): void {
-    emit('update:modelValue', {
-        ...props.modelValue,
-        clientIri: value === null ? null : String(value),
-        clientDeliveryAddressIri: null,
-    })
-}
-
-/** Sélection d'un fournisseur → maj IRI + reset de l'adresse d'appro. */
-function onSupplierChange(value: string | number | null): void {
-    emit('update:modelValue', {
-        ...props.modelValue,
-        supplierIri: value === null ? null : String(value),
-        supplierSupplyAddressIri: null,
-    })
-}
-
-/** Réponse détail (client / fournisseur) embarquant ses adresses. */
-interface ParentWithAddresses {
-    addresses?: { '@id': string, street?: string | null, postalCode?: string | null, city?: string | null }[]
-}
-
-/** Mappe les adresses embarquées en options (IRI en value, voie · CP · ville en label). */
-function toAddressOptions(parent: ParentWithAddresses): SelectOption[] {
-    return (parent.addresses ?? []).map(a => ({
-        value: a['@id'],
-        label: [a.street, a.postalCode, a.city].filter(Boolean).join(' · ') || a['@id'],
-    }))
-}
-
-/**
- * Charge les adresses d'un parent (client/fournisseur) à la volée via son détail
- * (GET de l'IRI, qui embarque `addresses`). Échec → liste vide (non bloquant).
- */
-async function loadAddresses(iri: string | null, target: typeof clientAddressOptions): Promise<void> {
-    if (!iri) {
-        target.value = []
-        return
-    }
-    try {
-        // L'IRI est absolue (/api/clients/5) ; useApi préfixe déjà /api → on le retire.
-        const path = iri.replace(/^\/api/, '')
-        const data = await api.get<ParentWithAddresses>(path, {}, { headers: { Accept: 'application/ld+json' }, toast: false })
-        target.value = toAddressOptions(data)
-    }
-    catch {
-        target.value = []
-    }
-}
-
-// Recharge les adresses quand le client / fournisseur change (immediate pour le
-// pré-remplissage en édition).
-watch(() => props.modelValue.clientIri, iri => loadAddresses(iri, clientAddressOptions), { immediate: true })
-watch(() => props.modelValue.supplierIri, iri => loadAddresses(iri, supplierAddressOptions), { immediate: true })
-</script>

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

@@ -37,9 +37,6 @@ vi.stubGlobal('useToast', () => ({
 }))
 
 const { useCarrierForm, CARRIER_TAB_KEYS } = await import('../useCarrierForm')
-const { buildCarrierContactPayload, isCarrierContactBlank, isCarrierContactNamed } = await import('../../utils/forms/carrierContact')
-const { buildCarrierPricePayload, isCarrierPriceValid } = await import('../../utils/forms/carrierPrice')
-const { emptyCarrierContact, emptyCarrierPrice } = await import('../../types/carrierForm')
 
 describe('useCarrierForm', () => {
     beforeEach(() => {
@@ -558,357 +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)
-    })
-})
-
-describe('carrierPrice (util) — bascule CLIENT/FOURNISSEUR + champs requis par branche', () => {
-    const CLIENT = '/api/clients/3'
-    const CLIENT_ADDR = '/api/client_addresses/8'
-    const SUPPLIER = '/api/suppliers/5'
-    const SUPPLIER_ADDR = '/api/supplier_addresses/9'
-    const SITE = '/api/sites/1'
-
-    it('buildCarrierPricePayload CLIENT : branche client envoyée, branche fournisseur à null', () => {
-        const body = buildCarrierPricePayload({
-            ...emptyCarrierPrice(),
-            direction: 'CLIENT',
-            clientIri: CLIENT,
-            clientDeliveryAddressIri: CLIENT_ADDR,
-            departureSiteIri: SITE,
-            containerType: 'BENNE',
-            pricingUnit: 'FORFAIT',
-            price: '120.00',
-            priceState: 'EN_COURS',
-        })
-        expect(body).toMatchObject({
-            direction: 'CLIENT',
-            client: CLIENT,
-            clientDeliveryAddress: CLIENT_ADDR,
-            departureSite: SITE,
-            supplier: null,
-            supplierSupplyAddress: null,
-            deliverySite: null,
-            containerType: 'BENNE',
-            pricingUnit: 'FORFAIT',
-            price: '120.00',
-            priceState: 'EN_COURS',
-        })
-    })
-
-    it('buildCarrierPricePayload FOURNISSEUR : branche fournisseur envoyée, branche client à null', () => {
-        const body = buildCarrierPricePayload({
-            ...emptyCarrierPrice(),
-            direction: 'FOURNISSEUR',
-            supplierIri: SUPPLIER,
-            supplierSupplyAddressIri: SUPPLIER_ADDR,
-            deliverySiteIri: SITE,
-            containerType: 'FOND_MOUVANT',
-            pricingUnit: 'TONNE',
-            price: '45',
-            priceState: 'VALIDE',
-        })
-        expect(body).toMatchObject({
-            direction: 'FOURNISSEUR',
-            supplier: SUPPLIER,
-            supplierSupplyAddress: SUPPLIER_ADDR,
-            deliverySite: SITE,
-            client: null,
-            clientDeliveryAddress: null,
-            departureSite: null,
-        })
-    })
-
-    it('isCarrierPriceValid : faux si branche incomplète, vrai si branche complète + communs', () => {
-        const base = { ...emptyCarrierPrice(), containerType: 'BENNE', pricingUnit: 'FORFAIT', price: '10', priceState: 'EN_COURS' }
-        // Direction non choisie → invalide.
-        expect(isCarrierPriceValid({ ...base, direction: null })).toBe(false)
-        // Sens CLIENT par défaut mais branche incomplète → invalide.
-        expect(isCarrierPriceValid(base)).toBe(false)
-        // CLIENT sans adresse/site → invalide.
-        expect(isCarrierPriceValid({ ...base, direction: 'CLIENT', clientIri: CLIENT })).toBe(false)
-        // CLIENT complet → valide.
-        expect(isCarrierPriceValid({ ...base, direction: 'CLIENT', clientIri: CLIENT, clientDeliveryAddressIri: CLIENT_ADDR, departureSiteIri: SITE })).toBe(true)
-        // FOURNISSEUR complet → valide.
-        expect(isCarrierPriceValid({ ...base, direction: 'FOURNISSEUR', supplierIri: SUPPLIER, supplierSupplyAddressIri: SUPPLIER_ADDR, deliverySiteIri: SITE })).toBe(true)
-        // Prix manquant → invalide même branche complète.
-        expect(isCarrierPriceValid({ ...emptyCarrierPrice(), direction: 'CLIENT', clientIri: CLIENT, clientDeliveryAddressIri: CLIENT_ADDR, departureSiteIri: SITE, containerType: 'BENNE', pricingUnit: 'FORFAIT', priceState: 'EN_COURS' })).toBe(false)
-    })
-})
-
-describe('useCarrierForm — onglet Prix (ERP-169)', () => {
-    beforeEach(() => {
-        mockPost.mockReset()
-        mockPatch.mockReset()
-        mockDelete.mockReset()
-    })
-
-    function createdForm() {
-        const form = useCarrierForm()
-        form.carrierId.value = 7
-        return form
-    }
-
-    it('démarre avec un bloc CLIENT par défaut ; « + Nouveau prix » bloqué tant qu\'il est incomplet', () => {
-        const form = createdForm()
-        // Un bloc présent d'office, sens CLIENT pré-sélectionné.
-        expect(form.prices.value).toHaveLength(1)
-        expect(form.prices.value[0]?.direction).toBe('CLIENT')
-        // Bloc incomplet → on ne peut pas en ajouter un autre.
-        expect(form.canAddPrice.value).toBe(false)
-        form.addPrice()
-        expect(form.prices.value).toHaveLength(1)
-
-        // Une fois le bloc complété, l'ajout est autorisé.
-        const p = form.prices.value[0]
-        if (p) {
-            p.clientIri = '/api/clients/3'
-            p.clientDeliveryAddressIri = '/api/client_addresses/8'
-            p.departureSiteIri = '/api/sites/1'
-            p.price = '120'
-            p.priceState = 'EN_COURS'
-        }
-        expect(form.canAddPrice.value).toBe(true)
-        form.addPrice()
-        expect(form.prices.value).toHaveLength(2)
-    })
-
-    it('submitPrices : POST des nouveaux prix (branche CLIENT), capture l\'id, finalise', async () => {
-        mockPost.mockResolvedValueOnce({ id: 50 })
-        const form = createdForm()
-        form.addPrice()
-        const p = form.prices.value[0]
-        if (p) {
-            p.direction = 'CLIENT'
-            p.clientIri = '/api/clients/3'
-            p.clientDeliveryAddressIri = '/api/client_addresses/8'
-            p.departureSiteIri = '/api/sites/1'
-            p.containerType = 'BENNE'
-            p.pricingUnit = 'FORFAIT'
-            p.price = '120'
-            p.priceState = 'EN_COURS'
-        }
-
-        const ok = await form.submitPrices(vi.fn())
-
-        expect(ok).toBe(true)
-        const [url, body, opts] = mockPost.mock.calls[0] ?? []
-        expect(url).toBe('/carriers/7/prices')
-        expect(body).toMatchObject({ direction: 'CLIENT', client: '/api/clients/3', supplier: null })
-        expect(opts).toMatchObject({ toast: false, headers: { Accept: 'application/ld+json' } })
-        expect(form.prices.value[0]?.id).toBe(50)
-        expect(form.isValidated('prices')).toBe(true)
-    })
-
-    it('submitPrices : PATCH des prix existants sur /carrier_prices/{id}', async () => {
-        mockPatch.mockResolvedValueOnce({})
-        const form = createdForm()
-        const p = form.prices.value[0]
-        if (p) {
-            p.id = 50
-            p.direction = 'FOURNISSEUR'
-            p.supplierIri = '/api/suppliers/5'
-            p.supplierSupplyAddressIri = '/api/supplier_addresses/9'
-            p.deliverySiteIri = '/api/sites/1'
-            p.price = '10'
-            p.priceState = 'VALIDE'
-        }
-
-        await form.submitPrices(vi.fn())
-
-        expect(mockPost).not.toHaveBeenCalled()
-        expect(mockPatch).toHaveBeenCalledWith('/carrier_prices/50', expect.objectContaining({ direction: 'FOURNISSEUR' }), { toast: false })
-    })
-
-    it('front : bloc prix incomplet → erreurs inline sous chaque champ requis, pas d\'appel back', async () => {
-        const form = createdForm()
-        // Bloc CLIENT par défaut, rien d'autre rempli.
-        const ok = await form.submitPrices(vi.fn())
-
-        expect(ok).toBe(false)
-        expect(mockPost).not.toHaveBeenCalled()
-        const errs = form.priceErrors.value[0]
-        expect(errs?.client).toBeTruthy()
-        expect(errs?.clientDeliveryAddress).toBeTruthy()
-        expect(errs?.departureSite).toBeTruthy()
-        expect(errs?.price).toBeTruthy()
-        expect(errs?.priceState).toBeTruthy()
-    })
-
-    it('submitPrices : mappe les 422 back par ligne (appartenance adresse) et ne finalise pas', async () => {
-        mockPost.mockRejectedValueOnce({
-            response: { status: 422, _data: { violations: [{ propertyPath: 'clientDeliveryAddress', message: 'L\'adresse de livraison doit appartenir au client selectionne.' }] } },
-        })
-        const form = createdForm()
-        // Tous les champs requis remplis (le pré-check front passe) ; le back 422 sur
-        // une RG qu'il est seul à connaître (appartenance de l'adresse au client).
-        const p = form.prices.value[0]
-        if (p) {
-            p.direction = 'CLIENT'
-            p.clientIri = '/api/clients/3'
-            p.clientDeliveryAddressIri = '/api/client_addresses/8'
-            p.departureSiteIri = '/api/sites/1'
-            p.price = '10'
-            p.priceState = 'EN_COURS'
-        }
-
-        const ok = await form.submitPrices(vi.fn())
-
-        expect(ok).toBe(false)
-        expect(form.priceErrors.value[0]?.clientDeliveryAddress).toBe('L\'adresse de livraison doit appartenir au client selectionne.')
-        expect(form.isValidated('prices')).toBe(false)
-    })
-
-    it('removePrice : DELETE /carrier_prices/{id} puis retrait du bloc', async () => {
-        mockDelete.mockResolvedValueOnce({})
-        const form = createdForm()
-        form.addPrice()
-        const p = form.prices.value[0]
-        if (p) { p.id = 77; p.direction = 'CLIENT'; p.clientIri = '/api/clients/3'; p.clientDeliveryAddressIri = '/api/client_addresses/8'; p.departureSiteIri = '/api/sites/1'; p.containerType = 'BENNE'; p.pricingUnit = 'FORFAIT'; p.price = '10'; p.priceState = 'EN_COURS' }
-        form.addPrice()
-
-        await form.removePrice(0)
-
-        expect(mockDelete).toHaveBeenCalledWith('/carrier_prices/77', {}, { toast: false })
-        expect(form.prices.value).toHaveLength(1)
-    })
-})

frontend/modules/transport/composables/useCarrierForm.ts

@@ -5,19 +5,13 @@ import { removeCollectionRow } from '~/shared/utils/collectionRow'
 import {
     emptyCarrierAddress,
     emptyCarrierAddressCopy,
-    emptyCarrierContact,
     emptyCarrierMain,
-    emptyCarrierPrice,
     type CarrierAddressCopy,
     type CarrierAddressFormDraft,
-    type CarrierContactFormDraft,
     type CarrierMainDraft,
     type CarrierMainResponse,
-    type CarrierPriceFormDraft,
 } 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 { buildCarrierPricePayload, isCarrierPriceValid } from '~/modules/transport/utils/forms/carrierPrice'
 import type { QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
 
 /** Nom du cas spécial « compte-propre » LIOT (comparaison insensible à la casse, RG-4.01). */
@@ -388,203 +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
-        }
-    }
-
-    // ── Onglet Prix (ERP-169) ─────────────────────────────────────────────────
-    // Un bloc présent par défaut (sens CLIENT pré-sélectionné). L'utilisateur ajoute
-    // les suivants via « + Nouveau prix ».
-    const prices = ref<CarrierPriceFormDraft[]>([emptyCarrierPrice()])
-    // Erreurs 422 par ligne (alignées sur l'index du v-for), peuplées par submitRows.
-    const priceErrors = ref<Record<string, string>[]>([])
-
-    // « + Nouveau prix » : autorisé si la liste est vide, sinon le dernier bloc doit
-    // être valide (branche complète + prix — RG-4.09→4.11, pré-check léger).
-    const canAddPrice = computed(() => {
-        const last = prices.value[prices.value.length - 1]
-        return last === undefined || isCarrierPriceValid(last)
-    })
-
-    function addPrice(): void {
-        if (canAddPrice.value) {
-            prices.value.push(emptyCarrierPrice())
-        }
-    }
-
-    /**
-     * Pré-validation FRONT d'un bloc prix (ERP-101) : renvoie les erreurs inline par
-     * champ obligatoire (sens + branche active + communs). Nécessaire car côté back
-     * l'Assert\NotBlank sur les scalaires (price/priceState...) court-circuite la
-     * validation de branche du CarrierPriceProcessor : le 422 ne porterait jamais
-     * client/supplier/adresses en même temps. Messages alignés sur le back.
-     */
-    function validatePriceRow(price: CarrierPriceFormDraft): Record<string, string> {
-        const errs: Record<string, string> = {}
-        const msg = (key: string): string => t(`transport.carriers.form.price.errors.${key}`)
-
-        if (!price.direction) {
-            errs.direction = msg('direction')
-        }
-        if (price.direction === 'CLIENT') {
-            if (!price.clientIri) errs.client = msg('client')
-            if (!price.clientDeliveryAddressIri) errs.clientDeliveryAddress = msg('clientDeliveryAddress')
-            if (!price.departureSiteIri) errs.departureSite = msg('departureSite')
-        }
-        if (price.direction === 'FOURNISSEUR') {
-            if (!price.supplierIri) errs.supplier = msg('supplier')
-            if (!price.supplierSupplyAddressIri) errs.supplierSupplyAddress = msg('supplierSupplyAddress')
-            if (!price.deliverySiteIri) errs.deliverySite = msg('deliverySite')
-        }
-        if (!price.containerType) errs.containerType = msg('containerType')
-        if (!price.pricingUnit) errs.pricingUnit = msg('pricingUnit')
-        if (!price.price || price.price.trim() === '') errs.price = msg('price')
-        if (!price.priceState) errs.priceState = msg('priceState')
-
-        return errs
-    }
-
-    /** Suppression immédiate d'un prix existant (DELETE /carrier_prices/{id}). */
-    async function removePrice(index: number): Promise<void> {
-        await removeCollectionRow({
-            rows: prices.value,
-            errors: priceErrors.value,
-            index,
-            endpoint: '/carrier_prices',
-            deleteRow: url => api.delete(url, {}, { toast: false }),
-            makeEmpty: emptyCarrierPrice,
-            onError: notifyRemovalError,
-        })
-    }
-
-    /**
-     * Valide l'onglet Prix : POST des nouveaux prix sur /carriers/{id}/prices, PATCH
-     * des existants sur /carrier_prices/{id} (groupe carrier:write:prices). La
-     * cohérence de branche CLIENT/FOURNISSEUR (RG-4.09→4.11) est re-validée back →
-     * 422 par ligne. Onglet Prix optionnel : une liste vide finalise sans appel.
-     * Retourne true si l'onglet a été validé (création terminée).
-     */
-    async function submitPrices(onError: (error: unknown) => void): Promise<boolean> {
-        if (carrierId.value === null || tabSubmitting.value) {
-            return false
-        }
-
-        // Pré-check front : affiche toutes les obligations sous leur champ d'un coup
-        // (le back ne peut pas tout renvoyer en une passe — cf. validatePriceRow).
-        const frontErrors = prices.value.map(validatePriceRow)
-        if (frontErrors.some(errs => Object.keys(errs).length > 0)) {
-            priceErrors.value = frontErrors
-            return false
-        }
-
-        tabSubmitting.value = true
-        try {
-            const hasError = await submitRows(
-                prices.value,
-                priceErrors,
-                async (price) => {
-                    const body = buildCarrierPricePayload(price)
-                    if (price.id === null) {
-                        const created = await api.post<{ id: number }>(
-                            `/carriers/${carrierId.value}/prices`,
-                            body,
-                            { headers: { Accept: 'application/ld+json' }, toast: false },
-                        )
-                        price.id = created.id
-                    }
-                    else {
-                        await api.patch(`/carrier_prices/${price.id}`, body, { toast: false })
-                    }
-                },
-                onError,
-            )
-            if (hasError) {
-                return false
-            }
-            completeTab('prices')
-            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),
@@ -684,20 +481,6 @@ export function useCarrierForm() {
         addAddress,
         removeAddress,
         submitAddresses,
-        // contacts
-        contacts,
-        contactErrors,
-        canAddContact,
-        addContact,
-        removeContact,
-        submitContacts,
-        // prix
-        prices,
-        priceErrors,
-        canAddPrice,
-        addPrice,
-        removePrice,
-        submitPrices,
         // actions
         validateMainFront,
         buildMainPayload,

frontend/modules/transport/pages/carriers/new.vue

@@ -207,76 +207,7 @@
                 </div>
             </template>
 
-            <!-- Onglet Contacts (ERP-168) : un bloc par contact (RG-4.08 ≥ 1 champ,
-                 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>
-
-            <!-- Onglet Prix (ERP-169) : blocs multiples, branche CLIENT/FOURNISSEUR
-                 (RG-4.09→4.11). Démarre vide ; l'utilisateur ajoute via « + Nouveau prix ». -->
-            <template #prices>
-                <div class="mt-12 flex flex-col gap-6">
-                    <CarrierPriceBlock
-                        v-for="(price, index) in prices"
-                        :key="index"
-                        :model-value="price"
-                        :client-options="clientOptions"
-                        :supplier-options="supplierOptions"
-                        :site-options="siteOptions"
-                        :removable="!isValidated('prices')"
-                        :readonly="isValidated('prices')"
-                        :errors="priceErrors[index]"
-                        @update:model-value="(v) => prices[index] = v"
-                        @remove="askRemovePrice(index)"
-                    />
-                    <div v-if="!isValidated('prices')" class="flex justify-center gap-6">
-                        <MalioButton
-                            variant="secondary"
-                            icon-name="mdi:add-bold"
-                            icon-position="left"
-                            :label="t('transport.carriers.form.price.add')"
-                            :disabled="!canAddPrice"
-                            @click="addPrice"
-                        />
-                        <MalioButton
-                            variant="primary"
-                            :label="t('transport.carriers.form.submit')"
-                            :disabled="tabSubmitting || carrierId === null"
-                            @click="onSubmitPrices"
-                        />
-                    </div>
-                </div>
-            </template>
-
-            <!-- Plus d'onglet placeholder : tous les onglets ont leur contenu. -->
+            <!-- Contacts / Prix : contenu aux tickets suivants. -->
             <template
                 v-for="key in placeholderTabs"
                 :key="key"
@@ -340,8 +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 CarrierPriceBlock from '~/modules/transport/components/CarrierPriceBlock.vue'
 import { useCarrierForm } from '~/modules/transport/composables/useCarrierForm'
 import { useQualimatSearch, type QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
 
@@ -386,18 +315,6 @@ const {
     addAddress,
     removeAddress,
     submitAddresses,
-    contacts,
-    contactErrors,
-    canAddContact,
-    addContact,
-    removeContact,
-    submitContacts,
-    prices,
-    priceErrors,
-    canAddPrice,
-    addPrice,
-    removePrice,
-    submitPrices,
     submitMain,
     applyQualimatSelection,
 } = useCarrierForm()
@@ -491,41 +408,8 @@ const tabs = computed(() => tabKeys.value.map((key, index) => ({
     disabled: index > unlockedIndex.value,
 })))
 
-// Tous les onglets ont désormais leur contenu (qualimat / addresses / contacts / prices).
-const placeholderTabs = computed(() => tabKeys.value.filter(
-    key => key !== 'qualimat' && key !== 'addresses' && key !== 'contacts' && key !== 'prices',
-))
-
-// ── Référentiels de l'onglet Prix (clients / fournisseurs / sites) ───────────
-const clientOptions = ref<SelectOption[]>([])
-const supplierOptions = ref<SelectOption[]>([])
-const siteOptions = ref<SelectOption[]>([])
-
-/** Charge un référentiel paginé (?pagination=false) et mappe en options { IRI, libellé }. */
-async function loadOptions(
-    url: string,
-    target: typeof clientOptions,
-    labelOf: (m: Record<string, unknown>) => string,
-): Promise<void> {
-    try {
-        const data = await api.get<{ member?: Record<string, unknown>[] }>(
-            url,
-            { pagination: 'false' },
-            { headers: { Accept: 'application/ld+json' }, toast: false },
-        )
-        target.value = (data.member ?? []).map(m => ({ value: String(m['@id']), label: labelOf(m) }))
-    }
-    catch {
-        target.value = []
-    }
-}
-
-/** Charge les référentiels de l'onglet Prix (non bloquant : selects vides si échec). */
-function loadPriceReferentials(): void {
-    void loadOptions('/clients', clientOptions, m => String(m.companyName ?? m['@id']))
-    void loadOptions('/suppliers', supplierOptions, m => String(m.companyName ?? m['@id']))
-    void loadOptions('/sites', siteOptions, m => String(m.name ?? m['@id']))
-}
+// Onglets dont le contenu arrive aux tickets suivants (Contacts / Prix).
+const placeholderTabs = computed(() => tabKeys.value.filter(key => key !== 'qualimat' && key !== 'addresses'))
 
 // ── Onglet Adresses (ERP-167) ────────────────────────────────────────────────
 // Pays : France garantie en tete meme si /countries echoue (resilience), pour
@@ -552,7 +436,6 @@ async function loadCountries(): Promise<void> {
 
 onMounted(() => {
     loadCountries().catch(() => {})
-    loadPriceReferentials()
 })
 
 // Avertissement unique quand l'autocompletion d'adresse bascule en degrade.
@@ -586,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 {
@@ -594,38 +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
-}
-
-/** Valide l'onglet Prix (POST/PATCH par ligne ; avance gérée par le composable). */
-async function onSubmitPrices(): Promise<void> {
-    const ok = await submitPrices(error => toast.error({
-        title: t('transport.carriers.toast.error'),
-        message: apiErrorMessage(error),
-    }))
-    if (ok) {
-        toast.success({ title: t('transport.carriers.toast.priceSaved') })
-    }
-}
-
-function askRemovePrice(index: number): void {
-    deleteConfirm.action = () => { void removePrice(index) }
-    deleteConfirm.open = true
-}
-
 function runDeleteConfirm(): void {
     deleteConfirm.action?.()
     deleteConfirm.action = null

frontend/modules/transport/types/carrierForm.ts

@@ -94,86 +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,
-    }
-}
-
-/**
- * Brouillon d'un bloc Prix (onglet Prix, ERP-169 — RG-4.09→4.11). `direction`
- * pilote la branche active : CLIENT (client + adresse de livraison + site de
- * départ) ou FOURNISSEUR (fournisseur + adresse d'appro + site de livraison). Les
- * relations partent au back en IRI (string). `price` est une chaîne (MalioInputAmount).
- */
-export interface CarrierPriceFormDraft {
-    id: number | null
-    direction: 'CLIENT' | 'FOURNISSEUR' | null
-    // Branche CLIENT (RG-4.10).
-    clientIri: string | null
-    clientDeliveryAddressIri: string | null
-    departureSiteIri: string | null
-    // Branche FOURNISSEUR (RG-4.11).
-    supplierIri: string | null
-    supplierSupplyAddressIri: string | null
-    deliverySiteIri: string | null
-    // Communs (toujours requis).
-    containerType: string | null
-    pricingUnit: string | null
-    price: string | null
-    priceState: string | null
-}
-
-/** Brouillon de prix vide (état initial d'un bloc Prix : tout masqué tant que direction null). */
-export function emptyCarrierPrice(): CarrierPriceFormDraft {
-    return {
-        id: null,
-        // Défaut métier : sens CLIENT pré-sélectionné (un bloc prix CLIENT est présent
-        // d'office à l'ouverture de l'onglet).
-        direction: 'CLIENT',
-        clientIri: null,
-        clientDeliveryAddressIri: null,
-        departureSiteIri: null,
-        supplierIri: null,
-        supplierSupplyAddressIri: null,
-        deliverySiteIri: null,
-        // Défauts métier : Benne + Forfait pré-sélectionnés à l'ajout d'un bloc prix.
-        containerType: 'BENNE',
-        pricingUnit: 'FORFAIT',
-        price: null,
-        priceState: null,
-    }
-}
-
 /**
  * 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.

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

@@ -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,
-    }
-}

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

@@ -1,77 +0,0 @@
-/**
- * Helpers purs de l'onglet Prix transporteur (M4 Transport, ERP-169 — RG-4.09→4.11).
- * Une ligne porte une branche CLIENT ou FOURNISSEUR selon `direction` ; les champs
- * de la branche INACTIVE doivent toujours partir à null (CHECK BDD
- * chk_carrier_price_client_branch / supplier_branch). Testables sans Vue ni API.
- */
-
-import type { CarrierPriceFormDraft } 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() !== ''
-}
-
-/**
- * Payload de la sous-ressource prix (groupe `carrier:write:prices`). Envoie les
- * communs + UNIQUEMENT la branche active (l'autre branche à null, exigée par les
- * CHECK BDD). Les relations partent en IRI (string|null).
- *
- * IMPORTANT : les scalaires obligatoires (direction / containerType / pricingUnit /
- * price / priceState) sont OMIS s'ils sont vides — on n'envoie JAMAIS `null` sur un
- * champ string. Sinon API Platform lève un 400 « The type of the "price" attribute
- * must be "string", "NULL" given. » AVANT la validation (non mappable inline). Omis,
- * le champ reste null côté entité → l'Assert\NotBlank renvoie un 422 propre rattaché
- * au champ, affiché sous l'input comme les autres blocs (ERP-101). Le back re-valide
- * aussi l'obligation conditionnelle de branche + l'appartenance de l'adresse.
- */
-export function buildCarrierPricePayload(price: CarrierPriceFormDraft): Record<string, unknown> {
-    const payload: Record<string, unknown> = {}
-
-    // Scalaires : présents seulement si remplis (jamais `null` → évite le 400 de type).
-    if (isFilled(price.direction)) payload.direction = price.direction
-    if (isFilled(price.containerType)) payload.containerType = price.containerType
-    if (isFilled(price.pricingUnit)) payload.pricingUnit = price.pricingUnit
-    if (isFilled(price.price)) payload.price = price.price
-    if (isFilled(price.priceState)) payload.priceState = price.priceState
-
-    // Branche active en IRI (null toléré sur une relation, ne déclenche pas le 400 de
-    // type) ; branche inactive forcée à null (CHECK BDD chk_carrier_price_*_branch).
-    if (price.direction === 'CLIENT') {
-        payload.client = price.clientIri || null
-        payload.clientDeliveryAddress = price.clientDeliveryAddressIri || null
-        payload.departureSite = price.departureSiteIri || null
-        payload.supplier = null
-        payload.supplierSupplyAddress = null
-        payload.deliverySite = null
-    }
-    else if (price.direction === 'FOURNISSEUR') {
-        payload.supplier = price.supplierIri || null
-        payload.supplierSupplyAddress = price.supplierSupplyAddressIri || null
-        payload.deliverySite = price.deliverySiteIri || null
-        payload.client = null
-        payload.clientDeliveryAddress = null
-        payload.departureSite = null
-    }
-
-    return payload
-}
-
-/**
- * Pré-check léger du gating « + Nouveau prix » : direction choisie, prix rempli, et
- * branche active complète (client/adresse/site OU fournisseur/adresse/site). Le back
- * reste la couche autoritaire (RG-4.09→4.11) ; ce pré-check évite d'empiler des
- * blocs vides.
- */
-export function isCarrierPriceValid(price: CarrierPriceFormDraft): boolean {
-    if (!isFilled(price.price) || !isFilled(price.containerType) || !isFilled(price.pricingUnit) || !isFilled(price.priceState)) {
-        return false
-    }
-    if (price.direction === 'CLIENT') {
-        return isFilled(price.clientIri) && isFilled(price.clientDeliveryAddressIri) && isFilled(price.departureSiteIri)
-    }
-    if (price.direction === 'FOURNISSEUR') {
-        return isFilled(price.supplierIri) && isFilled(price.supplierSupplyAddressIri) && isFilled(price.deliverySiteIri)
-    }
-    return false
-}

migrations/Version20260617120000.php

@@ -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).$_$');
-    }
-}

src/Module/Transport/Domain/Entity/CarrierContact.php

@@ -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
- * CHECK chk_carrier_contact_name + le Processor), max 2 telephones.
+ * SupplierContact (M2) : au moins un champ rempli (RG-4.08, garanti par le
+ * 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`.

src/Module/Transport/Infrastructure/ApiPlatform/State/Processor/CarrierContactProcessor.php

@@ -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
- * au moins le PRENOM OU le NOM (la fonction / le telephone / l'email seuls ne
- * suffisent pas), porte a la fois par le CHECK BDD chk_carrier_contact_name et par
- * ce Processor ; le « max 2 telephones » reste une specificite M4.
+ * perimetre ERP-160, AVEC deux specificites M4 : RG-4.08 (≥ 1 champ rempli, max
+ * 2 telephones) portee a la fois par le CHECK BDD chk_carrier_contact_filled et
+ * par ce Processor.
  *
  * 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
- * violation SQL (500 generique) en 422 propre rattachee au champ `firstName`
- * (mapping inline ERP-101). Le « max 2 telephones » est rattache au champ `phones` : seul
+ * RG-4.08 vit ICI (double du CHECK BDD) pour transformer une violation SQL (500
+ * generique) en 422 propre rattachee au champ `firstName` (mapping inline
+ * 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
-     * NOM — un contact se materialise par son nom ; fonction / telephone / email
-     * seuls ne suffisent pas. Double garde avec le CHECK BDD chk_carrier_contact_name
-     * — leve une 422 propre rattachee a `firstName` plutot qu'une 500 SQL. Joue apres
-     * normalisation + mapping telephones, donc les chaines vides sont deja null.
+     * RG-4.08 : un bloc Contact est valide des qu'au moins 1 champ est rempli
+     * (firstName, lastName, jobTitle, phonePrimary ou email — meme perimetre que
+     * le CHECK BDD chk_carrier_contact_filled, qui exclut phoneSecondary). Double
+     * garde : leve une 422 propre rattachee a `firstName` plutot qu'une 500 SQL.
+     * 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
-     * (« » devient null) cote fonction du contact.
+     * contrairement aux noms de personne). Garantit que RG-4.08 detecte un champ
+     * « non rempli » meme si le client envoie une chaine vide.
      */
     private function blankToNull(?string $value): ?string
     {

src/Module/Transport/Infrastructure/DataFixtures/CarrierFixtures.php

@@ -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,

src/Shared/Infrastructure/Database/ColumnCommentsCatalog.php

@@ -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).',
-                'last_name'       => 'Nom 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). 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).',

tests/Module/Transport/Api/CarrierContactApiTest.php

@@ -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 : un nom (ou prenom) suffit -> 201 ;
+ *  - RG-4.08 : contact totalement vide -> 422 (au moins 1 champ requis) ;
+ *  - 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,
-        // double du CHECK BDD chk_carrier_contact_name).
+        // RG-4.08 : aucun champ rempli -> 422 (garde Processor, double du CHECK BDD).
         $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();