fix(logistique) : bon de pesée — cartouche tiers + filtrage des listes contrepartie par site (ERP-208) (#155)

## ERP-208 — Fix ticket de pesée

### Bon de pesée (PDF)
Ajout d'un **cartouche bordé en haut à droite** du bon de pesée, contenant le **type de contrepartie** (Client / Fournisseur / Autre, en gras au-dessus) et le **nom du tiers**.
- `WeighingTicket::getCounterpartyName()` + `getCounterpartyTypeLabel()` (testés).
- En-tête du template passé en table 2 colonnes (contrainte Dompdf CSS 2.1).

### Écran de saisie (Ajouter / Modifier)
Les listes **Client / Fournisseur** sont **filtrées sur le site courant** (un tiers est rattaché à un site via les sites de ses adresses) et **rechargées au changement de site**.
- Réutilise le filtre back existant `?siteId[]=` de /clients et /suppliers (aucun changement back sur le filtre).
- Au switch de site : le tiers sélectionné est réinitialisé **uniquement** s'il sort du périmètre du nouveau site.
- Portée limitée au ticket de pesée : les répertoires M1/M2 ne changent pas.

### Tests
- Back : test unitaire `WeighingTicketCounterpartyNameTest` (nom + libellé) ; test PDF existant inchangé.
- Front : specs référentiels + écrans Ajouter/Modifier (673/673).
- Pas de migration, pas de RBAC, pas d'E2E.

### À vérifier en recette
En **modification**, si le tiers d'un ticket n'a pas d'adresse sur le site courant, le select peut s'afficher vide (valeur conservée mais option filtrée).

Reviewed-on: #155
Co-authored-by: tristan <tristan@yuno.malio.fr>
Co-committed-by: tristan <tristan@yuno.malio.fr>

frontend/modules/logistique/composables/__tests__/useWeighingTicketReferentials.spec.ts

@@ -0,0 +1,44 @@
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { useWeighingTicketReferentials } from '../useWeighingTicketReferentials'
+
+const mockApiGet = vi.hoisted(() => vi.fn())
+vi.stubGlobal('useApi', () => ({ get: mockApiGet }))
+
+/**
+ * Tests des référentiels Client/Fournisseur de l'écran ticket de pesée (M5).
+ * Contrat couvert (ERP-208) : `load(siteId)` filtre les deux endpoints par site
+ * courant via `siteId[]` ; sans site → listes complètes (param absent).
+ */
+describe('useWeighingTicketReferentials', () => {
+    beforeEach(() => {
+        mockApiGet.mockReset()
+        mockApiGet.mockResolvedValue({ member: [] })
+    })
+
+    it('passe siteId[] aux deux endpoints quand un site courant est fourni', async () => {
+        const { load } = useWeighingTicketReferentials()
+        await load(7)
+
+        const clientsCall = mockApiGet.mock.calls.find(c => c[0] === '/clients')
+        const suppliersCall = mockApiGet.mock.calls.find(c => c[0] === '/suppliers')
+        expect(clientsCall?.[1]).toMatchObject({ pagination: 'false', 'siteId[]': [7] })
+        expect(suppliersCall?.[1]).toMatchObject({ pagination: 'false', 'siteId[]': [7] })
+    })
+
+    it('ne passe pas siteId[] quand aucun site (liste complète)', async () => {
+        const { load } = useWeighingTicketReferentials()
+        await load(null)
+
+        const clientsCall = mockApiGet.mock.calls.find(c => c[0] === '/clients')
+        expect(clientsCall?.[1]).not.toHaveProperty('siteId[]')
+        expect(clientsCall?.[1]).toMatchObject({ pagination: 'false' })
+    })
+
+    it('mappe les membres Hydra en options { value: @id, label: companyName }', async () => {
+        mockApiGet.mockResolvedValue({ member: [{ '@id': '/api/clients/3', companyName: 'ACME' }] })
+        const { load, clients } = useWeighingTicketReferentials()
+        await load(7)
+
+        expect(clients.value).toEqual([{ value: '/api/clients/3', label: 'ACME' }])
+    })
+})

frontend/modules/logistique/composables/useWeighingTicketReferentials.ts

@@ -32,11 +32,19 @@ export function useWeighingTicketReferentials() {
     const clients = ref<RefOption[]>([])
     const suppliers = ref<RefOption[]>([])
 
-    /** Récupère une collection complète (pagination désactivée) en Hydra. */
-    async function fetchAll(url: string): Promise<PartyMember[]> {
+    /**
+     * Récupère une collection complète (pagination désactivée) en Hydra. Filtre par
+     * site courant si `siteId` est fourni (ERP-208) : un tiers est rattaché à un site
+     * via les sites de ses adresses — param `siteId[]` déjà géré par les providers M1/M2.
+     */
+    async function fetchAll(url: string, siteId?: number | null): Promise<PartyMember[]> {
+        const query: Record<string, unknown> = { pagination: 'false' }
+        if (siteId !== null && siteId !== undefined) {
+            query['siteId[]'] = [siteId]
+        }
         const res = await api.get<{ member?: PartyMember[] }>(
             url,
-            { pagination: 'false' },
+            query,
             { headers: LD_JSON_HEADERS, toast: false },
         )
         return res.member ?? []
@@ -45,14 +53,15 @@ export function useWeighingTicketReferentials() {
     /**
      * Charge en parallèle clients + fournisseurs (résilient : un référentiel en
      * échec — ex. 403 selon le rôle — laisse simplement son select vide sans
-     * faire échouer l'autre).
+     * faire échouer l'autre). `siteId` (site courant) filtre les listes par site
+     * (ERP-208) ; absent → listes complètes.
      */
-    async function load(): Promise<void> {
+    async function load(siteId?: number | null): Promise<void> {
         await Promise.allSettled([
-            fetchAll('/clients').then((list) => {
+            fetchAll('/clients', siteId).then((list) => {
                 clients.value = list.map(c => ({ value: c['@id'], label: c.companyName }))
             }),
-            fetchAll('/suppliers').then((list) => {
+            fetchAll('/suppliers', siteId).then((list) => {
                 suppliers.value = list.map(s => ({ value: s['@id'], label: s.companyName }))
             }),
         ])

frontend/modules/logistique/pages/__tests__/weighingTicketEdit.spec.ts

@@ -8,12 +8,13 @@ const mockFetchTicket = vi.hoisted(() => vi.fn())
 const mockPatch = vi.hoisted(() => vi.fn())
 const mockPush = vi.hoisted(() => vi.fn())
 const mockOpen = vi.hoisted(() => vi.fn())
+const mockRefLoad = vi.hoisted(() => vi.fn())
 
 vi.mock('~/modules/logistique/composables/useWeighingTicket', () => ({
     useWeighingTicket: () => ({ fetchTicket: mockFetchTicket }),
 }))
 vi.mock('~/modules/logistique/composables/useWeighingTicketReferentials', () => ({
-    useWeighingTicketReferentials: () => ({ clients: ref([]), suppliers: ref([]), load: vi.fn().mockResolvedValue(undefined) }),
+    useWeighingTicketReferentials: () => ({ clients: ref([]), suppliers: ref([]), load: mockRefLoad }),
 }))
 vi.mock('~/modules/logistique/composables/useWeighbridge', () => ({
     useWeighbridge: () => ({ triggerAuto: vi.fn(), triggerManual: vi.fn(), extractWeighbridgeError: () => 'err' }),
@@ -100,6 +101,7 @@ describe('Écran Modification ticket de pesée (page /weighing-tickets/{id}/edit
         mockPatch.mockReset().mockResolvedValue({})
         mockPush.mockReset()
         mockOpen.mockReset()
+        mockRefLoad.mockReset().mockResolvedValue(undefined)
     })
 
     it('charge le ticket au montage (pré-remplissage via hydrate)', async () => {
@@ -107,6 +109,12 @@ describe('Écran Modification ticket de pesée (page /weighing-tickets/{id}/edit
         expect(mockFetchTicket).toHaveBeenCalledWith('9')
     })
 
+    it('filtre les référentiels sur le SITE DU TICKET, pas le site courant (ERP-208)', async () => {
+        await mountPage()
+        // DETAIL.site.id = 1 → les listes sont chargées pour le site du ticket (immuable).
+        expect(mockRefLoad).toHaveBeenCalledWith(1)
+    })
+
     it('ticket validé : action principale « Enregistrer » + « Imprimer » (pas « Valider »)', async () => {
         const wrapper = await mountPage()
         // DETAIL.status = VALIDATED → l'action principale s'intitule « Enregistrer ».

frontend/modules/logistique/pages/__tests__/weighingTicketNew.spec.ts

@@ -7,9 +7,10 @@ const mockPost = vi.hoisted(() => vi.fn())
 const mockPatch = vi.hoisted(() => vi.fn())
 const mockPush = vi.hoisted(() => vi.fn())
 const mockOpen = vi.hoisted(() => vi.fn())
+const mockRefLoad = vi.hoisted(() => vi.fn())
 
 vi.mock('~/modules/logistique/composables/useWeighingTicketReferentials', () => ({
-    useWeighingTicketReferentials: () => ({ clients: ref([]), suppliers: ref([]), load: vi.fn().mockResolvedValue(undefined) }),
+    useWeighingTicketReferentials: () => ({ clients: ref([]), suppliers: ref([]), load: mockRefLoad }),
 }))
 vi.mock('~/modules/logistique/composables/useWeighbridge', () => ({
     useWeighbridge: () => ({ triggerAuto: vi.fn(), triggerManual: vi.fn(), extractWeighbridgeError: () => 'err' }),
@@ -23,6 +24,8 @@ vi.stubGlobal('useRouter', () => ({ push: mockPush }))
 vi.stubGlobal('usePermissions', () => ({ can: () => true }))
 vi.stubGlobal('navigateTo', vi.fn())
 vi.stubGlobal('useFormErrors', () => ({ errors: reactive({}), setError: vi.fn(), clearErrors: vi.fn(), handleApiError: vi.fn() }))
+// Site courant (ERP-208) : id 7 → les référentiels doivent être chargés filtrés sur ce site.
+vi.stubGlobal('useCurrentSite', () => ({ currentSite: ref({ id: 7, name: 'Site 7', color: '#000000' }) }))
 globalThis.open = mockOpen
 
 const NewPage = (await import('../weighing-tickets/new.vue')).default
@@ -70,6 +73,12 @@ describe('Écran Ajouter ticket de pesée (page /weighing-tickets/new)', () => {
         mockPatch.mockReset().mockResolvedValue({})
         mockPush.mockReset()
         mockOpen.mockReset()
+        mockRefLoad.mockReset().mockResolvedValue(undefined)
+    })
+
+    it('charge les référentiels filtrés sur le site courant au montage (ERP-208)', async () => {
+        await mountPage()
+        expect(mockRefLoad).toHaveBeenCalledWith(7)
     })
 
     it('un seul bouton « Valider » (pas de « Enregistrer » séparé)', async () => {

frontend/modules/logistique/pages/weighing-tickets/[id]/edit.vue

@@ -189,7 +189,7 @@
 import { computed, onMounted, reactive, ref } from 'vue'
 import { useWeighingTicketForm, type WeighingBlockState } from '~/modules/logistique/composables/useWeighingTicketForm'
 import { useWeighbridge } from '~/modules/logistique/composables/useWeighbridge'
-import { useWeighingTicket } from '~/modules/logistique/composables/useWeighingTicket'
+import { useWeighingTicket, type WeighingTicketDetail } from '~/modules/logistique/composables/useWeighingTicket'
 import { useWeighingTicketReferentials, type RefOption } from '~/modules/logistique/composables/useWeighingTicketReferentials'
 import { NUMERIC_MASK, PLATE_MASK, FREE_PLATE_MASK } from '~/modules/logistique/utils/weighingMasks'
 import { mapViolationsToRecord } from '~/shared/utils/api'
@@ -404,12 +404,34 @@ function printTicket(): void {
     window.open(`/api/weighing_tickets/${ticketId}/print.pdf`, '_blank')
 }
 
+/**
+ * Garantit que la contrepartie DÉJÀ ENREGISTRÉE (hydratée depuis le ticket) reste
+ * affichée même si la liste filtrée par site ne la contient pas (ticket antérieur
+ * à ERP-208, droits restreints sur /clients, contrepartie hors site…) : on injecte
+ * son option plutôt que de la purger. Évite toute perte silencieuse de la
+ * contrepartie en édition (ERP-208, retour review).
+ */
+function ensureSelectedOptionPresent(detail: WeighingTicketDetail): void {
+    const client = detail.client
+    if (client && !referentials.clients.value.some(o => o.value === client['@id'])) {
+        referentials.clients.value.push({ value: client['@id'], label: client.companyName })
+    }
+    const supplier = detail.supplier
+    if (supplier && !referentials.suppliers.value.some(o => o.value === supplier['@id'])) {
+        referentials.suppliers.value.push({ value: supplier['@id'], label: supplier.companyName })
+    }
+}
+
 onMounted(async () => {
-    referentials.load().catch(() => {})
     try {
         const detail = await fetchTicket(ticketId)
         ticketNumber.value = detail.number ?? ''
         form.hydrate(detail)
+        // Listes filtrées sur le SITE DU TICKET (immuable, RG-5.09) — pas le site
+        // courant — et chargées APRÈS hydrate pour ne jamais purger la sélection
+        // existante (pas de race load/hydrate, ERP-208).
+        await referentials.load(detail.site?.id ?? null)
+        ensureSelectedOptionPresent(detail)
     }
     catch {
         error.value = true

frontend/modules/logistique/pages/weighing-tickets/new.vue

@@ -176,7 +176,7 @@
 </template>
 
 <script setup lang="ts">
-import { computed, onMounted, reactive, ref } from 'vue'
+import { computed, onMounted, reactive, ref, watch } from 'vue'
 import { useWeighingTicketForm, type WeighingBlockState } from '~/modules/logistique/composables/useWeighingTicketForm'
 import { useWeighbridge } from '~/modules/logistique/composables/useWeighbridge'
 import { useWeighingTicketReferentials, type RefOption } from '~/modules/logistique/composables/useWeighingTicketReferentials'
@@ -375,7 +375,28 @@ async function submitValidate(): Promise<void> {
     }
 }
 
+const { currentSite } = useCurrentSite()
+
+/**
+ * Recharge les référentiels Client/Fournisseur pour le site donné, puis purge le
+ * tiers sélectionné s'il n'appartient plus à la liste du nouveau site (ERP-208).
+ */
+async function reloadReferentials(siteId: number | null): Promise<void> {
+    await referentials.load(siteId)
+    if (form.clientIri.value && !referentials.clients.value.some(o => o.value === form.clientIri.value)) {
+        form.clientIri.value = null
+    }
+    if (form.supplierIri.value && !referentials.suppliers.value.some(o => o.value === form.supplierIri.value)) {
+        form.supplierIri.value = null
+    }
+}
+
 onMounted(() => {
-    referentials.load().catch(() => {})
+    reloadReferentials(currentSite.value?.id ?? null).catch(() => {})
+})
+
+// Changement de site pendant la saisie → recharge les listes du nouveau site (ERP-208).
+watch(() => currentSite.value?.id, (siteId) => {
+    reloadReferentials(siteId ?? null).catch(() => {})
 })
 </script>