refactor(transport) : supprime les reliquats multi-adresses — colonne position, dead code front, docblocks 1:n (ERP-172)

frontend/i18n/locales/fr.json

@@ -631,8 +631,6 @@
                     "street": "Adresse",
                     "streetComplement": "Adresse complémentaire",
                     "streetNotFound": "Adresse introuvable ? Saisissez-la directement.",
-                    "add": "Nouvelle adresse",
-                    "remove": "Supprimer l'adresse",
                     "degraded": "Service d'adresse indisponible : saisie de la ville et de l'adresse en mode libre."
                 },
                 "contact": {

frontend/modules/transport/components/CarrierAddressBlock.vue

@@ -1,15 +1,6 @@
 <template>
+    <!-- Adresse UNIQUE par transporteur (ERP-172) : un seul bloc, jamais supprimable. -->
     <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 cote 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.address.remove') }"
-            @click="$emit('remove')"
-        />
-
         <!-- Pays : prerempli « France » (RG-4.05). -->
         <MalioSelect
             :model-value="model.country"
@@ -114,7 +105,6 @@ const props = defineProps<{
     modelValue: CarrierAddressFormDraft
     /** Pays disponibles (France par defaut). */
     countryOptions: RefOption[]
-    removable?: boolean
     readonly?: boolean
     /** Erreurs serveur 422 de cette ligne, indexees par champ (ERP-101). */
     errors?: Record<string, string>
@@ -122,7 +112,6 @@ const props = defineProps<{
 
 const emit = defineEmits<{
     'update:modelValue': [value: CarrierAddressFormDraft]
-    'remove': []
     /** Emis une fois quand le service d'autocompletion bascule en indisponible. */
     'degraded': []
 }>()

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

@@ -137,7 +137,6 @@
                         <CarrierAddressBlock
                             :model-value="address"
                             :country-options="countryOptions"
-                            :removable="false"
                             :errors="addressErrors"
                             @update:model-value="(v) => address = v"
                             @degraded="onAddressDegraded"

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

@@ -174,7 +174,6 @@
                     <CarrierAddressBlock
                         :model-value="address"
                         :country-options="countryOptions"
-                        :removable="false"
                         :readonly="isQualimat || isValidated('addresses')"
                         :errors="addressErrors"
                         @update:model-value="(v) => address = v"

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

@@ -8,18 +8,7 @@
 import type { CarrierAddressFormDraft } from '~/modules/transport/types/carrierForm'
 
 /**
- * RG-4.05 : gate du bouton « + Nouvelle adresse ». Une adresse est « complète »
- * (donc on autorise l'ajout d'un nouveau bloc) dès qu'elle porte un code postal,
- * une ville ET une rue. Les RG conditionnelles (obligatoire si affrété) restent
- * validées par le back (422 inline) — ce gate empêche seulement d'empiler des
- * blocs vides.
- */
-export function isCarrierAddressValid(address: CarrierAddressFormDraft): boolean {
-    return Boolean(address.postalCode?.trim() && address.city?.trim() && address.street?.trim())
-}
-
-/**
- * Payload de la sous-ressource addresses (groupe `carrier:write:addresses`). Les
+ * Payload de la sous-ressource address (groupe `carrier:write:addresses`). Les
  * scalaires sont nullable côté entité : on envoie `null` quand le champ est vide
  * (le `CarrierAddressProcessor` re-valide la présence si affrété — RG-4.05 — et
  * renvoie une 422 par champ).

migrations/Version20260617160000.php

@@ -0,0 +1,41 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DoctrineMigrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * ERP-172 — nettoyage des reliquats du multi-adresses sur carrier_address, suite a
+ * la bascule en adresse UNIQUE (Version20260617140000). La colonne `position`
+ * servait a ordonner une LISTE d'adresses ; avec une seule adresse par transporteur
+ * (OneToOne) elle n'a plus de sens -> on la supprime. On reactualise aussi le
+ * COMMENT ON TABLE qui annoncait encore une relation 1:n.
+ *
+ * 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
+ * et apres la bascule OneToOne (cf. CLAUDE.md regle 11).
+ */
+final class Version20260617160000 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'ERP-172 : retire la colonne carrier_address.position (relique multi-adresses) + COMMENT ON TABLE 1:1.';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE carrier_address DROP COLUMN position');
+        $this->addSql("COMMENT ON TABLE carrier_address IS 'Adresse d un transporteur (1:1, OneToOne — ERP-172 : adresse UNIQUE) — onglet Adresse (M4). Pre-remplie depuis QUALIMAT si applicable (RG-4.05).'");
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE carrier_address ADD COLUMN position INT DEFAULT 0 NOT NULL');
+        $this->addSql("COMMENT ON COLUMN carrier_address.position IS 'Ordre d affichage de l adresse dans la liste du transporteur (croissant).'");
+        $this->addSql("COMMENT ON TABLE carrier_address IS 'Adresses d un transporteur (1:n) — onglet Adresse (M4). Pre-remplie depuis QUALIMAT si applicable (RG-4.05).'");
+    }
+}

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

@@ -20,9 +20,10 @@ use Symfony\Component\Serializer\Attribute\Groups;
 use Symfony\Component\Validator\Constraints as Assert;
 
 /**
- * Adresse d'un transporteur (1:n) — onglet Adresse (M4). Jumelle de
- * SupplierAddress (M2), version simplifiee (pas de type d'adresse, pas de M2M
- * sites/categories sur l'adresse : les sites du M4 vivent dans l'onglet Prix).
+ * Adresse d'un transporteur (1:1, OneToOne — decision metier ERP-172) — onglet
+ * Adresse (M4). Jumelle de SupplierAddress (M2), version simplifiee (pas de type
+ * d'adresse, pas de M2M sites/categories sur l'adresse : les sites du M4 vivent
+ * dans l'onglet Prix), et UNIQUE par transporteur (la jumelle M2 est 1:n).
  *
  * Lecture : proprietes en `carrier:item:read` (embarquees au detail du
  * transporteur). Ecriture : groupe `carrier:write:addresses`.
@@ -30,9 +31,10 @@ use Symfony\Component\Validator\Constraints as Assert;
  * Sous-ressource API (ERP-159, spec § 4.5) — jumelle de SupplierAddress (M2) /
  * ProviderAddress (M3), sans address_type ni M2M (les sites du M4 vivent dans
  * l'onglet Prix) :
- *  - POST /api/carriers/{carrierId}/addresses : creation rattachee au
+ *  - POST /api/carriers/{carrierId}/address : creation rattachee au
  *    transporteur parent (Link toProperty 'carrier'), security
- *    transport.carriers.manage.
+ *    transport.carriers.manage. 409 si le transporteur a deja une adresse
+ *    (CarrierAddressProcessor::guardSingleAddress, avant la contrainte d'unicite).
  *  - PATCH / DELETE /api/carrier_addresses/{id} : security
  *    transport.carriers.manage.
  *  - GET /api/carrier_addresses/{id} : lecture unitaire (security view) — la
@@ -134,9 +136,6 @@ class CarrierAddress implements TimestampableInterface, BlamableInterface
     #[Groups(['carrier:item:read', 'carrier:write:addresses'])]
     private ?string $streetComplement = null;
 
-    #[ORM\Column(options: ['default' => 0])]
-    private int $position = 0;
-
     public function getId(): ?int
     {
         return $this->id;
@@ -213,16 +212,4 @@ class CarrierAddress implements TimestampableInterface, BlamableInterface
 
         return $this;
     }
-
-    public function getPosition(): int
-    {
-        return $this->position;
-    }
-
-    public function setPosition(int $position): static
-    {
-        $this->position = $position;
-
-        return $this;
-    }
 }

src/Shared/Infrastructure/Database/ColumnCommentsCatalog.php

@@ -497,15 +497,14 @@ final class ColumnCommentsCatalog
             ] + self::timestampableBlamableComments(),
 
             'carrier_address' => [
-                '_table'            => 'Adresses d un transporteur (1:n) — onglet Adresse (M4). Pre-remplie depuis QUALIMAT si applicable (RG-4.05).',
+                '_table'            => 'Adresse d un transporteur (1:1, OneToOne — ERP-172 : adresse UNIQUE) — onglet Adresse (M4). Pre-remplie depuis QUALIMAT si applicable (RG-4.05).',
                 'id'                => 'Identifiant interne auto-incremente.',
-                'carrier_id'        => 'FK -> carrier.id, ON DELETE CASCADE — transporteur proprietaire de l adresse.',
+                'carrier_id'        => 'FK -> carrier.id, ON DELETE CASCADE, UNIQUE (uniq_carrier_address_carrier) — transporteur proprietaire de l unique adresse.',
                 'country'           => 'Pays de l adresse — defaut France.',
                 'postal_code'       => 'Code postal (saisie assistee BAN cote front, RG-4.06).',
                 'city'              => 'Ville — preremplie depuis le code postal via API BAN cote front.',
                 'street'            => 'Numero et voie de l adresse.',
                 'street_complement' => 'Complement d adresse (etage, batiment...) — optionnel.',
-                'position'          => 'Ordre d affichage de l adresse dans la liste du transporteur (croissant).',
             ] + self::timestampableBlamableComments(),
 
             'carrier_contact' => [