Compare commits
1 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
 gitea-actions
|
47fe4ce1bb |
+1
-1
@@ -1,2 +1,2 @@
|
|||||||
parameters:
|
parameters:
|
||||||
app.version: '0.1.136'
|
app.version: '0.1.134'
|
||||||
|
|||||||
@@ -528,9 +528,7 @@
|
|||||||
"exportError": "L'export du répertoire transporteurs a échoué. Réessayez.",
|
"exportError": "L'export du répertoire transporteurs a échoué. Réessayez.",
|
||||||
"createSuccess": "Transporteur créé avec succès",
|
"createSuccess": "Transporteur créé avec succès",
|
||||||
"integrateSuccess": "Transporteur QUALIMAT intégré",
|
"integrateSuccess": "Transporteur QUALIMAT intégré",
|
||||||
"addressSaved": "Adresse enregistrée",
|
"addressSaved": "Adresse enregistrée"
|
||||||
"contactSaved": "Contact enregistré",
|
|
||||||
"priceSaved": "Prix enregistré"
|
|
||||||
},
|
},
|
||||||
"containerType": {
|
"containerType": {
|
||||||
"BENNE": "Benne",
|
"BENNE": "Benne",
|
||||||
@@ -593,57 +591,11 @@
|
|||||||
"remove": "Supprimer l'adresse",
|
"remove": "Supprimer l'adresse",
|
||||||
"degraded": "Service d'adresse indisponible : saisie de la ville et de l'adresse en mode libre."
|
"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": {
|
"confirmDelete": {
|
||||||
"title": "Supprimer ce bloc",
|
"title": "Supprimer ce bloc",
|
||||||
"message": "Cette suppression est définitive. Confirmer ?",
|
"message": "Cette suppression est définitive. Confirmer ?",
|
||||||
"cancel": "Annuler",
|
"cancel": "Annuler",
|
||||||
"confirm": "Supprimer"
|
"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"
|
"delete": "Suppression"
|
||||||
},
|
},
|
||||||
"entity": {
|
"entity": {
|
||||||
"core_user": "Utilisateur",
|
"core_user": "Utilisateur",
|
||||||
"core_role": "Rôle",
|
"core_role": "Rôle",
|
||||||
"core_permission": "Permission",
|
"core_permission": "Permission",
|
||||||
"sites_site": "Site",
|
"sites_site": "Site",
|
||||||
"catalog_category": "Catégorie",
|
"catalog_category": "Catégorie",
|
||||||
"commercial_client": "Client",
|
"commercial_client": "Client",
|
||||||
"commercial_clientaddress": "Adresse client",
|
"commercial_clientaddress": "Adresse client",
|
||||||
"commercial_clientcontact": "Contact client",
|
"commercial_clientcontact": "Contact client",
|
||||||
"commercial_clientrib": "RIB client",
|
"commercial_clientrib": "RIB client",
|
||||||
"commercial_supplier": "Fournisseur",
|
"commercial_supplier": "Fournisseur",
|
||||||
"commercial_supplieraddress": "Adresse fournisseur",
|
"commercial_supplieraddress": "Adresse fournisseur",
|
||||||
"commercial_suppliercontact": "Contact fournisseur",
|
"commercial_suppliercontact": "Contact fournisseur",
|
||||||
"commercial_supplierrib": "RIB fournisseur",
|
"commercial_supplierrib": "RIB fournisseur",
|
||||||
"technique_provider": "Prestataire",
|
"technique_provider": "Prestataire",
|
||||||
"technique_provideraddress": "Adresse prestataire",
|
"technique_provideraddress": "Adresse prestataire",
|
||||||
"technique_providercontact": "Contact prestataire",
|
"technique_providercontact": "Contact prestataire",
|
||||||
"technique_providerrib": "RIB prestataire",
|
"technique_providerrib": "RIB prestataire",
|
||||||
"transport_carrier": "Transporteur",
|
"transport_carrier": "Transporteur",
|
||||||
"transport_carrieraddress": "Adresse transporteur",
|
"transport_carrieraddress": "Adresse transporteur",
|
||||||
"transport_carriercontact": "Contact transporteur",
|
"transport_carriercontact": "Contact transporteur",
|
||||||
"transport_carrierprice": "Prix transporteur"
|
"transport_carrierprice": "Prix transporteur"
|
||||||
},
|
},
|
||||||
"empty": "Aucune activité enregistrée",
|
"empty": "Aucune activité enregistrée",
|
||||||
"no_results": "Aucun résultat pour ces filtres",
|
"no_results": "Aucun résultat pour ces filtres",
|
||||||
|
|||||||
@@ -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>
|
|
||||||
@@ -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>
|
|
||||||
@@ -37,9 +37,6 @@ vi.stubGlobal('useToast', () => ({
|
|||||||
}))
|
}))
|
||||||
|
|
||||||
const { useCarrierForm, CARRIER_TAB_KEYS } = await import('../useCarrierForm')
|
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', () => {
|
describe('useCarrierForm', () => {
|
||||||
beforeEach(() => {
|
beforeEach(() => {
|
||||||
@@ -558,357 +555,3 @@ describe('useCarrierForm — onglet Adresses (ERP-167)', () => {
|
|||||||
expect(form.addresses.value).toHaveLength(1)
|
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)
|
|
||||||
})
|
|
||||||
})
|
|
||||||
|
|||||||
@@ -5,19 +5,13 @@ import { removeCollectionRow } from '~/shared/utils/collectionRow'
|
|||||||
import {
|
import {
|
||||||
emptyCarrierAddress,
|
emptyCarrierAddress,
|
||||||
emptyCarrierAddressCopy,
|
emptyCarrierAddressCopy,
|
||||||
emptyCarrierContact,
|
|
||||||
emptyCarrierMain,
|
emptyCarrierMain,
|
||||||
emptyCarrierPrice,
|
|
||||||
type CarrierAddressCopy,
|
type CarrierAddressCopy,
|
||||||
type CarrierAddressFormDraft,
|
type CarrierAddressFormDraft,
|
||||||
type CarrierContactFormDraft,
|
|
||||||
type CarrierMainDraft,
|
type CarrierMainDraft,
|
||||||
type CarrierMainResponse,
|
type CarrierMainResponse,
|
||||||
type CarrierPriceFormDraft,
|
|
||||||
} from '~/modules/transport/types/carrierForm'
|
} from '~/modules/transport/types/carrierForm'
|
||||||
import { buildCarrierAddressPayload, isCarrierAddressValid } from '~/modules/transport/utils/forms/carrierAddress'
|
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'
|
import type { QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
|
||||||
|
|
||||||
/** Nom du cas spécial « compte-propre » LIOT (comparaison insensible à la casse, RG-4.01). */
|
/** 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 /
|
* 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),
|
* § 2.5) : copie le nom, force la certification à « QUALIMAT » (lecture seule),
|
||||||
@@ -684,20 +481,6 @@ export function useCarrierForm() {
|
|||||||
addAddress,
|
addAddress,
|
||||||
removeAddress,
|
removeAddress,
|
||||||
submitAddresses,
|
submitAddresses,
|
||||||
// contacts
|
|
||||||
contacts,
|
|
||||||
contactErrors,
|
|
||||||
canAddContact,
|
|
||||||
addContact,
|
|
||||||
removeContact,
|
|
||||||
submitContacts,
|
|
||||||
// prix
|
|
||||||
prices,
|
|
||||||
priceErrors,
|
|
||||||
canAddPrice,
|
|
||||||
addPrice,
|
|
||||||
removePrice,
|
|
||||||
submitPrices,
|
|
||||||
// actions
|
// actions
|
||||||
validateMainFront,
|
validateMainFront,
|
||||||
buildMainPayload,
|
buildMainPayload,
|
||||||
|
|||||||
@@ -207,76 +207,7 @@
|
|||||||
</div>
|
</div>
|
||||||
</template>
|
</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>
|
|
||||||
|
|
||||||
<!-- 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. -->
|
|
||||||
<template
|
<template
|
||||||
v-for="key in placeholderTabs"
|
v-for="key in placeholderTabs"
|
||||||
:key="key"
|
:key="key"
|
||||||
@@ -340,8 +271,6 @@ import { debounce } from '~/shared/utils/debounce'
|
|||||||
import { extractApiErrorMessage } from '~/shared/utils/api'
|
import { extractApiErrorMessage } from '~/shared/utils/api'
|
||||||
import { isRowRemovable } from '~/shared/utils/collectionRow'
|
import { isRowRemovable } from '~/shared/utils/collectionRow'
|
||||||
import CarrierAddressBlock from '~/modules/transport/components/CarrierAddressBlock.vue'
|
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 { useCarrierForm } from '~/modules/transport/composables/useCarrierForm'
|
||||||
import { useQualimatSearch, type QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
|
import { useQualimatSearch, type QualimatCarrierRow } from '~/modules/transport/composables/useQualimatSearch'
|
||||||
|
|
||||||
@@ -386,18 +315,6 @@ const {
|
|||||||
addAddress,
|
addAddress,
|
||||||
removeAddress,
|
removeAddress,
|
||||||
submitAddresses,
|
submitAddresses,
|
||||||
contacts,
|
|
||||||
contactErrors,
|
|
||||||
canAddContact,
|
|
||||||
addContact,
|
|
||||||
removeContact,
|
|
||||||
submitContacts,
|
|
||||||
prices,
|
|
||||||
priceErrors,
|
|
||||||
canAddPrice,
|
|
||||||
addPrice,
|
|
||||||
removePrice,
|
|
||||||
submitPrices,
|
|
||||||
submitMain,
|
submitMain,
|
||||||
applyQualimatSelection,
|
applyQualimatSelection,
|
||||||
} = useCarrierForm()
|
} = useCarrierForm()
|
||||||
@@ -491,41 +408,8 @@ const tabs = computed(() => tabKeys.value.map((key, index) => ({
|
|||||||
disabled: index > unlockedIndex.value,
|
disabled: index > unlockedIndex.value,
|
||||||
})))
|
})))
|
||||||
|
|
||||||
// Tous les onglets ont désormais leur contenu (qualimat / addresses / contacts / prices).
|
// 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' && 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']))
|
|
||||||
}
|
|
||||||
|
|
||||||
// ── Onglet Adresses (ERP-167) ────────────────────────────────────────────────
|
// ── Onglet Adresses (ERP-167) ────────────────────────────────────────────────
|
||||||
// Pays : France garantie en tete meme si /countries echoue (resilience), pour
|
// Pays : France garantie en tete meme si /countries echoue (resilience), pour
|
||||||
@@ -552,7 +436,6 @@ async function loadCountries(): Promise<void> {
|
|||||||
|
|
||||||
onMounted(() => {
|
onMounted(() => {
|
||||||
loadCountries().catch(() => {})
|
loadCountries().catch(() => {})
|
||||||
loadPriceReferentials()
|
|
||||||
})
|
})
|
||||||
|
|
||||||
// Avertissement unique quand l'autocompletion d'adresse bascule en degrade.
|
// 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) })
|
const deleteConfirm = reactive({ open: false, action: null as null | (() => void) })
|
||||||
|
|
||||||
function askRemoveAddress(index: number): void {
|
function askRemoveAddress(index: number): void {
|
||||||
@@ -594,38 +477,6 @@ function askRemoveAddress(index: number): void {
|
|||||||
deleteConfirm.open = true
|
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 {
|
function runDeleteConfirm(): void {
|
||||||
deleteConfirm.action?.()
|
deleteConfirm.action?.()
|
||||||
deleteConfirm.action = null
|
deleteConfirm.action = null
|
||||||
|
|||||||
@@ -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
|
* 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.
|
* 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,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
|
|
||||||
}
|
|
||||||
@@ -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
|
* 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
|
* Lecture : proprietes en `carrier:item:read` (embarquees au detail du
|
||||||
* transporteur). Ecriture : groupe `carrier:write:contacts`.
|
* transporteur). Ecriture : groupe `carrier:write:contacts`.
|
||||||
|
|||||||
+26
-19
@@ -23,21 +23,21 @@ use function is_string;
|
|||||||
/**
|
/**
|
||||||
* Processor d'ecriture de la sous-ressource Contact d'un transporteur (M4,
|
* Processor d'ecriture de la sous-ressource Contact d'un transporteur (M4,
|
||||||
* spec-back § 4.5). Jumeau du SupplierContactProcessor (M2), recentre sur le
|
* 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 :
|
* Sequence :
|
||||||
* - POST / PATCH : rattachement au transporteur parent (linkParent),
|
* - POST / PATCH : rattachement au transporteur parent (linkParent),
|
||||||
* normalisation serveur RG-4.13 (prenom/nom Title Case, email lowercase),
|
* normalisation serveur RG-4.13 (prenom/nom Title Case, email lowercase),
|
||||||
* mapping du tableau d'ecriture `phones` -> phonePrimary/phoneSecondary
|
* 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).
|
* - 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
|
* point de saisie des numeros (les colonnes phonePrimary/phoneSecondary sont en
|
||||||
* lecture seule).
|
* lecture seule).
|
||||||
*
|
*
|
||||||
@@ -77,7 +77,7 @@ final class CarrierContactProcessor implements ProcessorInterface
|
|||||||
$this->linkParent($data, $uriVariables);
|
$this->linkParent($data, $uriVariables);
|
||||||
$this->normalize($data);
|
$this->normalize($data);
|
||||||
$this->applyPhones($data);
|
$this->applyPhones($data);
|
||||||
$this->validateName($data);
|
$this->validateAtLeastOneField($data);
|
||||||
|
|
||||||
return $this->persistProcessor->process($data, $operation, $uriVariables, $context);
|
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 = new ConstraintViolationList();
|
||||||
$violations->add(new ConstraintViolation(
|
$violations->add(new ConstraintViolation(
|
||||||
'Le prénom ou le nom du contact est obligatoire.',
|
'Renseignez au moins un champ pour le contact.',
|
||||||
null,
|
null,
|
||||||
[],
|
[],
|
||||||
$contact,
|
$contact,
|
||||||
@@ -212,8 +219,8 @@ final class CarrierContactProcessor implements ProcessorInterface
|
|||||||
|
|
||||||
/**
|
/**
|
||||||
* Trim + chaine vide -> null (la fonction n'est pas normalisee en casse,
|
* 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
|
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
|
* 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(
|
private function addContact(
|
||||||
Carrier $carrier,
|
Carrier $carrier,
|
||||||
|
|||||||
@@ -509,11 +509,11 @@ final class ColumnCommentsCatalog
|
|||||||
] + self::timestampableBlamableComments(),
|
] + self::timestampableBlamableComments(),
|
||||||
|
|
||||||
'carrier_contact' => [
|
'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.',
|
'id' => 'Identifiant interne auto-incremente.',
|
||||||
'carrier_id' => 'FK -> carrier.id, ON DELETE CASCADE — transporteur proprietaire du contact.',
|
'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).',
|
'job_title' => 'Fonction / intitule de poste du contact (≤ 120 caracteres).',
|
||||||
'phone_primary' => 'Telephone principal — chiffres uniquement (normalisation serveur).',
|
'phone_primary' => 'Telephone principal — chiffres uniquement (normalisation serveur).',
|
||||||
'phone_secondary' => 'Telephone secondaire — chiffres uniquement (max 2 telephones, RG-4.08).',
|
'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}.
|
* POST /api/carriers/{id}/contacts, PATCH/DELETE /api/carrier_contacts/{id}.
|
||||||
*
|
*
|
||||||
* Contrat verifie :
|
* 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) ;
|
* - RG-4.08 : 3 telephones (tableau `phones`) -> 422 (max 2) ;
|
||||||
* - mapping `phones[]` -> phonePrimary / phoneSecondary + normalisation (RG-4.13) ;
|
* - mapping `phones[]` -> phonePrimary / phoneSecondary + normalisation (RG-4.13) ;
|
||||||
* - PATCH / DELETE OK avec transport.carriers.manage, 403 sans (view seul).
|
* - PATCH / DELETE OK avec transport.carriers.manage, 403 sans (view seul).
|
||||||
@@ -51,8 +51,7 @@ final class CarrierContactApiTest extends AbstractCarrierApiTestCase
|
|||||||
|
|
||||||
public function testEmptyContactReturns422(): void
|
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');
|
$carrier = $this->seedCarrier('Contact Vide');
|
||||||
$client = $this->createAdminClient();
|
$client = $this->createAdminClient();
|
||||||
|
|
||||||
@@ -61,13 +60,13 @@ final class CarrierContactApiTest extends AbstractCarrierApiTestCase
|
|||||||
'json' => [],
|
'json' => [],
|
||||||
]);
|
]);
|
||||||
self::assertResponseStatusCodeSame(422);
|
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');
|
self::assertViolationOnPath($response, 'firstName');
|
||||||
}
|
}
|
||||||
|
|
||||||
public function testSingleFieldContactIsCreated(): void
|
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');
|
$carrier = $this->seedCarrier('Contact Mono');
|
||||||
$client = $this->createAdminClient();
|
$client = $this->createAdminClient();
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user