test(transport) : rigueur RG sous-ressources (propertyPath, 404 parent, 401, certif)

Repond aux retours de review (rigueur d'assertion transversale) :
- mutualise assertViolationOnPath dans AbstractCarrierApiTestCase (au lieu d'un
  duplicata local a CarrierWriteApiTest) ;
- asserte le propertyPath des 422 des sous-ressources (adresses city/street/postalCode,
  contacts firstName/phones/email, prix clientDeliveryAddress/supplierSupplyAddress/price)
  -> evite les faux-verts du mapping inline (ERP-101) ;
- 404 parent (POST sur /carriers/999999/{addresses,contacts,prices}) ;
- 401 anonyme + filtre ?certificationType= sur la collection (trous releves sur le
  contrat de lecture).

config/version.yaml

@@ -1,2 +1,2 @@
 parameters:
-    app.version: '0.1.128'
+    app.version: '0.1.126'

frontend/i18n/locales/fr.json

@@ -2,7 +2,6 @@
     "common": {
         "loading": "Chargement...",
         "save": "Enregistrer",
-        "validate": "Valider",
         "cancel": "Annuler",
         "delete": "Supprimer",
         "edit": "Modifier",
@@ -75,7 +74,7 @@
                 "categories": "Catégories",
                 "sites": "Sites",
                 "status": "Statut",
-                "archivedOnly": "Voir les archivés",
+                "includeArchived": "Inclure les archivés",
                 "apply": "Voir les résultats",
                 "reset": "Réinitialiser"
             },
@@ -124,7 +123,7 @@
                 "back": "Retour au répertoire",
                 "loading": "Chargement du fournisseur…",
                 "notFound": "Fournisseur introuvable.",
-                "save": "Enregistrer"
+                "save": "Valider"
             },
             "form": {
                 "title": "Ajouter un fournisseur",
@@ -267,7 +266,7 @@
                 "back": "Retour au répertoire",
                 "loading": "Chargement du client…",
                 "notFound": "Client introuvable.",
-                "save": "Enregistrer"
+                "save": "Valider"
             },
             "validation": {
                 "informationRequiredForCommercial": "Les informations de l'entreprise sont obligatoires pour le rôle Commerciale.",
@@ -389,7 +388,7 @@
                 "categories": "Catégories",
                 "sites": "Sites",
                 "status": "Statut",
-                "archivedOnly": "Voir les archivés",
+                "includeArchived": "Inclure les archivés",
                 "apply": "Voir les résultats",
                 "reset": "Réinitialiser"
             },

frontend/modules/catalog/components/CategoryDrawer.vue

@@ -59,7 +59,7 @@
             />
             <MalioButton
                 v-if="canShowSave"
-                :label="isCreateMode ? t('common.validate') : t('common.save')"
+                :label="t('common.save')"
                 variant="primary"
                 button-class="w-m-btn-action"
                 :disabled="form.submitting.value || loadingTypes"

frontend/modules/commercial/composables/__tests__/useSuppliersRepository.spec.ts

@@ -51,7 +51,7 @@ describe('useSuppliersRepository', () => {
                 search: 'acme',
                 'categoryCode[]': ['NEGOCIANT', 'TRANSPORTEUR'],
                 'siteId[]': ['86', '17'],
-                archivedOnly: true,
+                includeArchived: true,
             },
             { replace: true },
         )
@@ -63,7 +63,7 @@ describe('useSuppliersRepository', () => {
                 search: 'acme',
                 'categoryCode[]': ['NEGOCIANT', 'TRANSPORTEUR'],
                 'siteId[]': ['86', '17'],
-                archivedOnly: true,
+                includeArchived: true,
                 page: 1,
                 itemsPerPage: 10,
             },
@@ -73,7 +73,7 @@ describe('useSuppliersRepository', () => {
 
     it('repasse a une query propre apres reinitialisation des filtres', async () => {
         const repo = useSuppliersRepository()
-        await repo.setFilters({ search: 'acme', archivedOnly: true }, { replace: true })
+        await repo.setFilters({ search: 'acme', includeArchived: true }, { replace: true })
         await repo.setFilters({}, { replace: true })
 
         expect(mockGet).toHaveBeenLastCalledWith(

frontend/modules/commercial/pages/__tests__/suppliersIndex.spec.ts

@@ -172,16 +172,16 @@ describe('Répertoire fournisseurs (page /suppliers)', () => {
         )
     })
 
-    it('repercute le filtre « Voir les archivés » dans setFilters sans toucher l\'URL', async () => {
+    it('repercute le filtre « Inclure les archivés » dans setFilters sans toucher l\'URL', async () => {
         const wrapper = mountPage()
         await flushPromises()
 
-        // Coche « Voir les archivés » puis applique les filtres.
-        await wrapper.find('input[data-id="filter-archived-only"]').setValue(true)
+        // Coche « Inclure les archivés » puis applique les filtres.
+        await wrapper.find('input[data-id="filter-include-archived"]').setValue(true)
         await wrapper.find('[data-label="commercial.suppliers.filters.apply"]').trigger('click')
 
         expect(mockSetFilters).toHaveBeenLastCalledWith(
-            { archivedOnly: true },
+            { includeArchived: true },
             { replace: true },
         )
         // Etat 100 % local (regle n°6) : aucune navigation/query string declenchee.
@@ -192,7 +192,7 @@ describe('Répertoire fournisseurs (page /suppliers)', () => {
         const wrapper = mountPage()
         await flushPromises()
 
-        await wrapper.find('input[data-id="filter-archived-only"]').setValue(true)
+        await wrapper.find('input[data-id="filter-include-archived"]').setValue(true)
         await wrapper.find('[data-label="commercial.suppliers.filters.apply"]').trigger('click')
 
         // Le libelle du bouton Filtrer porte le compteur (1 filtre actif).

frontend/modules/commercial/pages/suppliers/index.vue

@@ -128,13 +128,13 @@
                     </div>
                 </MalioAccordionItem>
 
-                <!-- Statut : bool unique. Coche = archives uniquement, sinon actifs. -->
+                <!-- Statut : bool unique. Coche = inclut aussi les archives (sinon actifs seuls). -->
                 <MalioAccordionItem :title="t('commercial.suppliers.filters.status')" value="status">
                     <MalioCheckbox
-                        id="filter-archived-only"
-                        :label="t('commercial.suppliers.filters.archivedOnly')"
-                        :model-value="draftArchivedOnly"
-                        @update:model-value="(val: boolean) => draftArchivedOnly = val"
+                        id="filter-include-archived"
+                        :label="t('commercial.suppliers.filters.includeArchived')"
+                        :model-value="draftIncludeArchived"
+                        @update:model-value="(val: boolean) => draftIncludeArchived = val"
                     />
                 </MalioAccordionItem>
             </MalioAccordion>
@@ -254,12 +254,12 @@ const filterDrawerOpen = ref(false)
 const draftSearch = ref('')
 const draftCategoryCodes = ref<string[]>([])
 const draftSiteIds = ref<string[]>([])
-const draftArchivedOnly = ref(false)
+const draftIncludeArchived = ref(false)
 
 const appliedSearch = ref('')
 const appliedCategoryCodes = ref<string[]>([])
 const appliedSiteIds = ref<string[]>([])
-const appliedArchivedOnly = ref(false)
+const appliedIncludeArchived = ref(false)
 
 // Options des selects multi, chargees une fois (referentiels courts).
 const categoryOptions = ref<FilterOption[]>([])
@@ -270,7 +270,7 @@ const activeFilterCount = computed(() => {
     if (appliedSearch.value.trim() !== '') count++
     if (appliedCategoryCodes.value.length > 0) count++
     if (appliedSiteIds.value.length > 0) count++
-    if (appliedArchivedOnly.value) count++
+    if (appliedIncludeArchived.value) count++
     return count
 })
 
@@ -285,7 +285,7 @@ function openFilters(): void {
     draftSearch.value = appliedSearch.value
     draftCategoryCodes.value = [...appliedCategoryCodes.value]
     draftSiteIds.value = [...appliedSiteIds.value]
-    draftArchivedOnly.value = appliedArchivedOnly.value
+    draftIncludeArchived.value = appliedIncludeArchived.value
     filterDrawerOpen.value = true
 }
 
@@ -311,7 +311,7 @@ function buildFilterPayload(): Record<string, string | string[] | boolean> {
     if (appliedSearch.value.trim() !== '') payload.search = appliedSearch.value.trim()
     if (appliedCategoryCodes.value.length > 0) payload['categoryCode[]'] = [...appliedCategoryCodes.value]
     if (appliedSiteIds.value.length > 0) payload['siteId[]'] = [...appliedSiteIds.value]
-    if (appliedArchivedOnly.value) payload.archivedOnly = true
+    if (appliedIncludeArchived.value) payload.includeArchived = true
     return payload
 }
 
@@ -321,7 +321,7 @@ function applyFilters(): void {
     appliedSearch.value = draftSearch.value.trim()
     appliedCategoryCodes.value = [...draftCategoryCodes.value]
     appliedSiteIds.value = [...draftSiteIds.value]
-    appliedArchivedOnly.value = draftArchivedOnly.value
+    appliedIncludeArchived.value = draftIncludeArchived.value
 
     setFilters(buildFilterPayload(), { replace: true })
     filterDrawerOpen.value = false
@@ -333,12 +333,12 @@ function resetFilters(): void {
     draftSearch.value = ''
     draftCategoryCodes.value = []
     draftSiteIds.value = []
-    draftArchivedOnly.value = false
+    draftIncludeArchived.value = false
 
     appliedSearch.value = ''
     appliedCategoryCodes.value = []
     appliedSiteIds.value = []
-    appliedArchivedOnly.value = false
+    appliedIncludeArchived.value = false
 
     setFilters({}, { replace: true })
 }

frontend/modules/core/components/RoleDrawer.vue

@@ -83,7 +83,7 @@
                 @click="emit('update:modelValue', false)"
             />
             <MalioButton
-                :label="isEditMode ? t('common.save') : t('common.validate')"
+                :label="t('common.save')"
                 variant="primary"
                 button-class="w-m-btn-action"
                 :disabled="saving || permissionsLoadFailed"

frontend/modules/sites/components/SiteDrawer.vue

@@ -103,7 +103,7 @@
                 @click="emit('update:modelValue', false)"
             />
             <MalioButton
-                :label="isEditMode ? t('common.save') : t('common.validate')"
+                :label="t('common.save')"
                 variant="primary"
                 button-class="w-m-btn-action"
                 :disabled="saving || !isValidHex"

frontend/modules/technique/composables/__tests__/useProvidersRepository.test.ts

@@ -14,9 +14,9 @@ vi.stubGlobal('useApi', () => ({ get: mockApiGet }))
  *  - l'enveloppe Hydra (member / totalItems) est consommee
  *  - le header `Accept: application/ld+json` est envoye (sinon API Platform 4
  *    renvoie un tableau plat sans pagination)
- *  - EXCLUSION DES ARCHIVES PAR DEFAUT : aucun `archivedOnly` n'est envoye
+ *  - EXCLUSION DES ARCHIVES PAR DEFAUT : aucun `includeArchived` n'est envoye
  *    tant que l'utilisateur ne coche pas le filtre (le back masque alors les
- *    archives) ; le filtre `archivedOnly` est bien transmis une fois applique.
+ *    archives) ; le filtre `includeArchived` est bien transmis une fois applique.
  */
 describe('useProvidersRepository', () => {
     beforeEach(() => {
@@ -53,26 +53,26 @@ describe('useProvidersRepository', () => {
         expect(repo.totalItems.value).toBe(1)
     })
 
-    it('exclut les archives par defaut : aucun archivedOnly au premier fetch', async () => {
+    it('exclut les archives par defaut : aucun includeArchived au premier fetch', async () => {
         mockApiGet.mockResolvedValueOnce({ member: PAGE, totalItems: 1 })
         const repo = useProvidersRepository()
 
         await repo.fetch()
 
         const query = mockApiGet.mock.calls[0][1] as Record<string, unknown>
-        expect(query.archivedOnly).toBeUndefined()
+        expect(query.includeArchived).toBeUndefined()
     })
 
-    it('transmet archivedOnly une fois le filtre applique (retour page 1)', async () => {
+    it('transmet includeArchived une fois le filtre applique (retour page 1)', async () => {
         mockApiGet.mockResolvedValueOnce({ member: PAGE, totalItems: 1 })
         const repo = useProvidersRepository()
         await repo.fetch()
 
         mockApiGet.mockResolvedValueOnce({ member: PAGE, totalItems: 1 })
-        await repo.setFilters({ archivedOnly: true })
+        await repo.setFilters({ includeArchived: true })
 
         expect(repo.currentPage.value).toBe(1)
         const query = mockApiGet.mock.calls.at(-1)?.[1] as Record<string, unknown>
-        expect(query.archivedOnly).toBe(true)
+        expect(query.includeArchived).toBe(true)
     })
 })

frontend/modules/technique/composables/useProvidersRepository.ts

@@ -45,11 +45,10 @@ export interface Provider {
  * sur la ressource `/providers` (pagination serveur obligatoire ; jamais de
  * chargement integral en memoire). Miroir de `useSuppliersRepository` (M2).
  *
- * Les filtres (recherche, categories, sites, archives) sont pilotes par la page
- * via `setFilters` du composable partage — la remise en page 1 est garantie. Par
- * defaut, aucun `archivedOnly` n'est envoye : le back masque donc les prestataires
- * archives (exclusion par defaut, spec-back § 2.11). Cocher « Voir les archivés »
- * envoie `archivedOnly=true` → seules les archives sont listees (aligne sur Client).
+ * Les filtres (recherche, categories, sites, inclusion des archives) sont pilotes
+ * par la page via `setFilters` du composable partage — la remise en page 1 est
+ * garantie. Par defaut, aucun `includeArchived` n'est envoye : le back masque
+ * donc les prestataires archives (exclusion par defaut, spec-back § 2.11).
  *
  * Le cloisonnement par site est applique AUTOMATIQUEMENT cote back (§ 2.13) en
  * fonction de l'utilisateur — rien a filtrer cote front.

frontend/modules/technique/pages/providers/index.vue

@@ -129,13 +129,13 @@
                     </div>
                 </MalioAccordionItem>
 
-                <!-- Statut : bool unique. Coche = archives uniquement, sinon actifs. -->
+                <!-- Statut : bool unique. Coche = inclut aussi les archives (sinon actifs seuls). -->
                 <MalioAccordionItem :title="t('technique.providers.filters.status')" value="status">
                     <MalioCheckbox
-                        id="filter-archived-only"
-                        :label="t('technique.providers.filters.archivedOnly')"
-                        :model-value="draftArchivedOnly"
-                        @update:model-value="(val: boolean) => draftArchivedOnly = val"
+                        id="filter-include-archived"
+                        :label="t('technique.providers.filters.includeArchived')"
+                        :model-value="draftIncludeArchived"
+                        @update:model-value="(val: boolean) => draftIncludeArchived = val"
                     />
                 </MalioAccordionItem>
             </MalioAccordion>
@@ -258,12 +258,12 @@ const filterDrawerOpen = ref(false)
 const draftSearch = ref('')
 const draftCategoryCodes = ref<string[]>([])
 const draftSiteIds = ref<string[]>([])
-const draftArchivedOnly = ref(false)
+const draftIncludeArchived = ref(false)
 
 const appliedSearch = ref('')
 const appliedCategoryCodes = ref<string[]>([])
 const appliedSiteIds = ref<string[]>([])
-const appliedArchivedOnly = ref(false)
+const appliedIncludeArchived = ref(false)
 
 // Options des selects multi, chargees une fois (referentiels courts).
 const categoryOptions = ref<FilterOption[]>([])
@@ -274,7 +274,7 @@ const activeFilterCount = computed(() => {
     if (appliedSearch.value.trim() !== '') count++
     if (appliedCategoryCodes.value.length > 0) count++
     if (appliedSiteIds.value.length > 0) count++
-    if (appliedArchivedOnly.value) count++
+    if (appliedIncludeArchived.value) count++
     return count
 })
 
@@ -289,7 +289,7 @@ function openFilters(): void {
     draftSearch.value = appliedSearch.value
     draftCategoryCodes.value = [...appliedCategoryCodes.value]
     draftSiteIds.value = [...appliedSiteIds.value]
-    draftArchivedOnly.value = appliedArchivedOnly.value
+    draftIncludeArchived.value = appliedIncludeArchived.value
     filterDrawerOpen.value = true
 }
 
@@ -315,7 +315,7 @@ function buildFilterPayload(): Record<string, string | string[] | boolean> {
     if (appliedSearch.value.trim() !== '') payload.search = appliedSearch.value.trim()
     if (appliedCategoryCodes.value.length > 0) payload['categoryCode[]'] = [...appliedCategoryCodes.value]
     if (appliedSiteIds.value.length > 0) payload['siteId[]'] = [...appliedSiteIds.value]
-    if (appliedArchivedOnly.value) payload.archivedOnly = true
+    if (appliedIncludeArchived.value) payload.includeArchived = true
     return payload
 }
 
@@ -325,7 +325,7 @@ function applyFilters(): void {
     appliedSearch.value = draftSearch.value.trim()
     appliedCategoryCodes.value = [...draftCategoryCodes.value]
     appliedSiteIds.value = [...draftSiteIds.value]
-    appliedArchivedOnly.value = draftArchivedOnly.value
+    appliedIncludeArchived.value = draftIncludeArchived.value
 
     setFilters(buildFilterPayload(), { replace: true })
     filterDrawerOpen.value = false
@@ -337,12 +337,12 @@ function resetFilters(): void {
     draftSearch.value = ''
     draftCategoryCodes.value = []
     draftSiteIds.value = []
-    draftArchivedOnly.value = false
+    draftIncludeArchived.value = false
 
     appliedSearch.value = ''
     appliedCategoryCodes.value = []
     appliedSiteIds.value = []
-    appliedArchivedOnly.value = false
+    appliedIncludeArchived.value = false
 
     setFilters({}, { replace: true })
 }