chore: bump version to v0.1.120

Empilée sur ERP-144 (#106).

## Périmètre ERP-145
Écrans **Consultation** (lecture seule) et **Modification** (édition par onglet), peuplés depuis la **seule** réponse `GET /api/providers/{id}` (embed contacts/adresses/ribs + refs comptables — pas de N+1).

### Consultation — `pages/providers/[id]/index.vue` (`/providers/{id}`)
- Ouverture par défaut sur **Contacts** ; tous champs readonly ; onglets **Contacts · Adresse · Rapports · Échanges · Comptabilité** (navigation libre). Rapports/Échanges = placeholders « À venir ».
- Flèche retour → répertoire. Bouton **Modifier** (si `manage` OU `accounting.manage`). Bouton **Archiver** (Admin seul, `archive`) → modal → PATCH `{isArchived:true}` ; **Restaurer** si archivé.
- Comptabilité visible seulement si `accounting.view` ; banque/RIB affichés selon le type de règlement (VIREMENT/LCR).

### Modification — `pages/providers/[id]/edit.vue` (`/providers/{id}/edit`)
- Pré-rempli ; **bloc principal éditable** (Nom/Catégories/Sites, PATCH `provider:write:main` via `updateMain`) ; onglets Contact/Adresse/Comptabilité en **navigation libre**, PATCH partiel par onglet (réutilise `useProviderForm` en `editMode`).
- Onglets sans permission `manage` / `accounting.manage` restent **readonly** (pas de bouton Valider / suppression). Accès réservé à `manage` OU `accounting.manage`.

### Composables / helpers
- **`useProvider(id)`** : charge le détail (ld+json) + archive/restore (PATCH isArchived seul, puis rechargement).
- **`useProviderForm`** étendu : `updateMain()` (PATCH principal en édition) + `editMode` (completeTab ne verrouille/avance plus).
- **`providerDetail.ts`** : mapping embed → brouillons + options role-indépendantes (libellés depuis l'embed) + règles d'actions (Modifier/Archiver/Restaurer).

## Conformité
- `useApi()` only ; `Malio*` only ; `usePermissions()` pour boutons/onglets ; aucun texte FR en dur ; pas d'import inter-module (règle ABSOLUE n°1).

## Vérifications
- Vitest : 470/470 (16 nouveaux : mapping détail, actions par permission, updateMain + editMode).
- ESLint : OK · `nuxi typecheck` : 0 erreur sur les fichiers source du ticket.
- Golden path navigateur : **Consultation** (ACME) — bloc principal readonly + libellés catégories/sites résolus depuis l'embed, 5 onglets, Modifier+Archiver visibles (admin), Comptabilité readonly. **Modification** — bloc principal éditable pré-rempli (Site « 86 17 »), 3 onglets navigation libre, onglet Contact pré-rempli.

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

config/version.yaml

@@ -1,2 +1,2 @@
 parameters:
-    app.version: '0.1.114'
+    app.version: '0.1.120'

frontend/i18n/locales/fr.json

@@ -406,8 +406,6 @@
                 "back": "Retour au répertoire",
                 "loading": "Chargement…",
                 "notFound": "Prestataire introuvable.",
-                "emptyContacts": "Aucun contact.",
-                "emptyAddresses": "Aucune adresse.",
                 "confirmArchive": "Archiver ce prestataire ? Il n'apparaîtra plus dans le répertoire actif.",
                 "confirmRestore": "Restaurer ce prestataire ? Il réapparaîtra dans le répertoire actif."
             },
@@ -429,6 +427,7 @@
                     "sites": "Site"
                 },
                 "errors": {
+                    "nameRequired": "Le nom du prestataire est obligatoire.",
                     "siteRequired": "Sélectionnez au moins un site.",
                     "categoryRequired": "Sélectionnez au moins une catégorie."
                 },
@@ -485,6 +484,7 @@
                 "exportError": "L'export du répertoire prestataires a échoué. Réessayez.",
                 "createSuccess": "Prestataire créé avec succès",
                 "updateSuccess": "Prestataire mis à jour avec succès",
+                "addComplete": "Prestataire ajouté",
                 "archiveSuccess": "Prestataire archivé avec succès",
                 "restoreSuccess": "Prestataire restauré avec succès"
             }

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

@@ -69,14 +69,14 @@ describe('useProviderForm', () => {
         permState.accountingManage = false
     })
 
-    it('RG-3.03/RG-3.09 (front) : bloque le POST si aucun site / aucune categorie', async () => {
+    it('front : formulaire principal vide -> erreurs sur nom + site + categorie, pas de POST', async () => {
         const form = useProviderForm()
-        form.main.companyName = 'Maintenance Pro'
 
         const created = await form.submitMain()
 
         expect(created).toBe(false)
         expect(mockPost).not.toHaveBeenCalled()
+        expect(form.mainErrors.errors.companyName).toBe('technique.providers.form.errors.nameRequired')
         expect(form.mainErrors.errors.sites).toBe('technique.providers.form.errors.siteRequired')
         expect(form.mainErrors.errors.categories).toBe('technique.providers.form.errors.categoryRequired')
         expect(form.mainLocked.value).toBe(false)
@@ -122,18 +122,17 @@ describe('useProviderForm', () => {
         expect(form.unlockedIndex.value).toBe(0)
     })
 
-    it('omet companyName vide du payload (laisse la 422 NotBlank back mordre)', async () => {
-        mockPost.mockResolvedValueOnce({ id: 1, companyName: null })
+    it('front : nom vide/espaces -> erreur inline sur companyName, pas de POST', async () => {
         const form = useProviderForm()
         form.main.companyName = '   '
         form.main.categoryIris = [CAT_MAINT]
         form.main.siteIris = [SITE_86]
 
-        await form.submitMain()
+        const created = await form.submitMain()
 
-        const body = (mockPost.mock.calls[0] ?? [])[1] as Record<string, unknown>
-        expect(body).not.toHaveProperty('companyName')
-        expect(body).toEqual({ categories: [CAT_MAINT], sites: [SITE_86] })
+        expect(created).toBe(false)
+        expect(mockPost).not.toHaveBeenCalled()
+        expect(form.mainErrors.errors.companyName).toBe('technique.providers.form.errors.nameRequired')
     })
 
     it('409 doublon (RG-3.10) : erreur inline dediee sur companyName, pas de verrouillage', async () => {

frontend/modules/technique/composables/useProviderForm.ts

@@ -20,6 +20,7 @@ import {
 import {
     buildProviderContactPayload,
     isProviderContactBlank,
+    isProviderContactNamed,
 } from '~/modules/technique/utils/forms/providerContact'
 import {
     buildProviderAddressPayload,
@@ -111,6 +112,10 @@ export function useProviderForm() {
      */
     function validateMainFront(): boolean {
         let valid = true
+        if (!main.companyName?.trim()) {
+            mainErrors.setError('companyName', t('technique.providers.form.errors.nameRequired'))
+            valid = false
+        }
         if (main.siteIris.length === 0) {
             mainErrors.setError('sites', t('technique.providers.form.errors.siteRequired'))
             valid = false
@@ -299,10 +304,11 @@ export function useProviderForm() {
     // Erreurs 422 par ligne (alignees sur l'index du v-for), peuplees par submitRows.
     const contactErrors = ref<Record<string, string>[]>([])
 
-    // « + Nouveau contact » desactive tant que le dernier bloc est vide (RG-3.04).
+    // « + Nouveau contact » desactive tant que le dernier bloc n'a pas de nom OU
+    // prenom (RG-3.04, aligne M1/M2 — fonction/tel/email seuls ne suffisent pas).
     const canAddContact = computed(() => {
         const last = contacts.value[contacts.value.length - 1]
-        return last !== undefined && !isProviderContactBlank(last)
+        return last !== undefined && isProviderContactNamed(last)
     })
 
     function addContact(): void {

frontend/modules/technique/pages/providers/[id]/edit.vue

@@ -333,6 +333,7 @@ const { provider, loading, error, load } = useProvider(providerId)
 
 const {
     main,
+    providerId: formProviderId,
     mainErrors,
     mainSubmitting,
     tabSubmitting,
@@ -389,6 +390,9 @@ function prefill(): void {
     const d = provider.value
     if (!d) return
 
+    // Indispensable : pilote les URLs des PATCH/POST par onglet (sinon les submits no-op).
+    formProviderId.value = d.id
+
     main.companyName = d.companyName ?? null
     main.categoryIris = irisOf(d.categories)
     main.siteIris = irisOf(d.sites)

frontend/modules/technique/pages/providers/[id]/index.vue

@@ -78,9 +78,6 @@
                             :model-value="contact"
                             readonly
                         />
-                        <p v-if="contacts.length === 0" class="text-center text-black/60">
-                            {{ t('technique.providers.consultation.emptyContacts') }}
-                        </p>
                     </div>
                 </template>
 
@@ -97,9 +94,6 @@
                             :country-options="countryOptionsFor(view.draft.country)"
                             readonly
                         />
-                        <p v-if="addressViews.length === 0" class="text-center text-black/60">
-                            {{ t('technique.providers.consultation.emptyAddresses') }}
-                        </p>
                     </div>
                 </template>
 
@@ -182,6 +176,7 @@ import {
     siteOptionsOf,
 } from '~/modules/technique/utils/forms/providerDetail'
 import { isBankRequiredForPaymentType, isRibRequiredForPaymentType } from '~/modules/technique/utils/forms/providerAccounting'
+import { emptyProviderAddress, emptyProviderContact } from '~/modules/technique/types/providerForm'
 
 const { t } = useI18n()
 const route = useRoute()
@@ -222,16 +217,26 @@ const mainSiteIris = computed(() => irisOf(provider.value?.sites))
 const mainCategoryOptions = computed(() => categoryOptionsOf(provider.value?.categories))
 const mainSiteOptions = computed(() => siteOptionsOf(provider.value?.sites))
 
-const contacts = computed(() => (provider.value?.contacts ?? []).map(mapContactToDraft))
+// Au moins un bloc affiche meme sans donnee (bloc vide en lecture seule, comme
+// l'onglet Comptabilite et les autres modules — pas de message « Aucun … »).
+const contacts = computed(() => {
+    const list = (provider.value?.contacts ?? []).map(mapContactToDraft)
+    return list.length > 0 ? list : [emptyProviderContact()]
+})
 // Contacts rattachables (pour resoudre les libelles des contacts lies aux adresses).
 const contactOptions = computed(() => contactOptionsOf(provider.value?.contacts))
 
 // Vue par adresse : brouillon + options propres a l'adresse (sites/categories embarques).
-const addressViews = computed(() => (provider.value?.addresses ?? []).map(address => ({
-    draft: mapAddressToDraft(address),
-    siteOptions: siteOptionsOf(address.sites),
-    categoryOptions: categoryOptionsOf(address.categories),
-})))
+const addressViews = computed(() => {
+    const views = (provider.value?.addresses ?? []).map(address => ({
+        draft: mapAddressToDraft(address),
+        siteOptions: siteOptionsOf(address.sites),
+        categoryOptions: categoryOptionsOf(address.categories),
+    }))
+    return views.length > 0
+        ? views
+        : [{ draft: emptyProviderAddress(), siteOptions: [], categoryOptions: [] }]
+})
 
 /** Pays : une seule option (la valeur courante), suffisant pour l'affichage readonly. */
 function countryOptionsFor(country: string): { value: string, label: string }[] {

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

@@ -85,7 +85,7 @@
                         <MalioButton
                             variant="primary"
                             :label="t('technique.providers.form.submit')"
-                            :disabled="tabSubmitting"
+                            :disabled="tabSubmitting || providerId === null"
                             @click="onSubmitContacts"
                         />
                     </div>
@@ -121,7 +121,7 @@
                         <MalioButton
                             variant="primary"
                             :label="t('technique.providers.form.submit')"
-                            :disabled="tabSubmitting"
+                            :disabled="tabSubmitting || providerId === null"
                             @click="onSubmitAddresses"
                         />
                     </div>
@@ -251,7 +251,7 @@
                         <MalioButton
                             variant="primary"
                             :label="t('technique.providers.form.submit')"
-                            :disabled="tabSubmitting"
+                            :disabled="tabSubmitting || providerId === null"
                             @click="onSubmitAccounting"
                         />
                     </div>
@@ -314,6 +314,7 @@ const referentials = useProviderReferentials()
 
 const {
     main,
+    providerId,
     mainLocked,
     mainSubmitting,
     mainErrors,
@@ -362,15 +363,33 @@ function apiErrorMessage(error: unknown): string {
     return extractApiErrorMessage(data) || t('technique.providers.toast.error')
 }
 
+// Dernier onglet REMPLISSABLE par le role : tabKeys exclut deja la Comptabilite
+// si l'user n'a pas accounting.view. Sa validation cloture l'ajout (redirection).
+const lastFillableTab = computed(() => tabKeys.value[tabKeys.value.length - 1])
+
+/**
+ * Apres validation d'un onglet (creation) : si c'est le dernier onglet du role,
+ * l'ajout est termine -> toast final + retour au repertoire (miroir M1/M2) ; sinon
+ * toast de mise a jour (l'onglet suivant a deja ete deverrouille par completeTab).
+ */
+function onTabSaved(key: string): void {
+    if (key === lastFillableTab.value) {
+        toast.success({ title: t('technique.providers.toast.addComplete') })
+        router.push('/providers')
+        return
+    }
+    toast.success({ title: t('technique.providers.toast.updateSuccess') })
+}
+
 // ── Onglet Contact ──────────────────────────────────────────────────────────
-/** Valide l'onglet Contact ; toast de succes si l'onglet a ete finalise. */
+/** Valide l'onglet Contact ; redirige si c'est le dernier onglet du role. */
 async function onSubmitContacts(): Promise<void> {
     const ok = await submitContacts(error => toast.error({
         title: t('technique.providers.toast.error'),
         message: apiErrorMessage(error),
     }))
     if (ok) {
-        toast.success({ title: t('technique.providers.toast.updateSuccess') })
+        onTabSaved('contact')
     }
 }
 
@@ -413,14 +432,14 @@ function onAddressDegraded(): void {
     })
 }
 
-/** Valide l'onglet Adresse ; toast de succes si l'onglet a ete finalise. */
+/** Valide l'onglet Adresse ; redirige si c'est le dernier onglet du role. */
 async function onSubmitAddresses(): Promise<void> {
     const ok = await submitAddresses(error => toast.error({
         title: t('technique.providers.toast.error'),
         message: apiErrorMessage(error),
     }))
     if (ok) {
-        toast.success({ title: t('technique.providers.toast.updateSuccess') })
+        onTabSaved('address')
     }
 }
 
@@ -450,7 +469,7 @@ function askRemoveRib(index: number): void {
     askConfirm(t('technique.providers.form.confirmDelete.rib'), () => removeRib(index))
 }
 
-/** Valide l'onglet Comptabilite ; toast de succes si l'onglet a ete finalise. */
+/** Valide l'onglet Comptabilite ; redirige si c'est le dernier onglet du role. */
 async function onSubmitAccounting(): Promise<void> {
     const ok = await submitAccounting(
         isBankRequired.value,
@@ -461,7 +480,7 @@ async function onSubmitAccounting(): Promise<void> {
         }),
     )
     if (ok) {
-        toast.success({ title: t('technique.providers.toast.updateSuccess') })
+        onTabSaved('accounting')
     }
 }
 

frontend/modules/technique/utils/forms/__tests__/providerContact.spec.ts

@@ -3,6 +3,7 @@ import {
     buildProviderContactPayload,
     hasAtLeastOneFilledContact,
     isProviderContactBlank,
+    isProviderContactNamed,
 } from '../providerContact'
 import { emptyProviderContact } from '~/modules/technique/types/providerForm'
 
@@ -34,15 +35,28 @@ describe('providerContact helpers', () => {
         })
     })
 
-    describe('hasAtLeastOneFilledContact (RG-3.12)', () => {
-        it('false si tous les blocs sont vides', () => {
-            expect(hasAtLeastOneFilledContact([emptyProviderContact(), emptyProviderContact()])).toBe(false)
+    describe('isProviderContactNamed (RG-3.04 — prenom OU nom)', () => {
+        it('vrai avec un prenom seul ou un nom seul', () => {
+            expect(isProviderContactNamed({ ...emptyProviderContact(), firstName: 'Jean' })).toBe(true)
+            expect(isProviderContactNamed({ ...emptyProviderContact(), lastName: 'Dupont' })).toBe(true)
         })
 
-        it('true des qu\'un bloc porte une donnee', () => {
+        it('faux si seuls fonction / telephone / email sont remplis (ne suffit pas)', () => {
+            expect(isProviderContactNamed({ ...emptyProviderContact(), jobTitle: 'Directeur' })).toBe(false)
+            expect(isProviderContactNamed({ ...emptyProviderContact(), email: 'a@b.fr' })).toBe(false)
+            expect(isProviderContactNamed({ ...emptyProviderContact(), phonePrimary: '0102030405' })).toBe(false)
+        })
+    })
+
+    describe('hasAtLeastOneFilledContact (RG-3.12 — au moins un contact nomme)', () => {
+        it('false si aucun bloc n\'est nomme', () => {
+            expect(hasAtLeastOneFilledContact([emptyProviderContact(), { ...emptyProviderContact(), email: 'a@b.fr' }])).toBe(false)
+        })
+
+        it('true des qu\'un bloc porte un nom ou prenom', () => {
             expect(hasAtLeastOneFilledContact([
                 emptyProviderContact(),
-                { ...emptyProviderContact(), email: 'a@b.fr' },
+                { ...emptyProviderContact(), lastName: 'Dupont' },
             ])).toBe(true)
         })
     })

frontend/modules/technique/utils/forms/providerContact.ts

@@ -32,12 +32,21 @@ export function isProviderContactBlank(contact: ProviderContactFormDraft): boole
     ].some(isFilled)
 }
 
+/**
+ * RG-3.04 : un contact est « nomme » (valide) des qu'il porte un prenom OU un nom
+ * — aligne sur le M1/M2. Sert le gating « + Nouveau contact » et la notion de
+ * contact valide (la fonction / le telephone / l'email seuls ne suffisent pas).
+ */
+export function isProviderContactNamed(contact: ProviderContactFormDraft): boolean {
+    return isFilled(contact.firstName) || isFilled(contact.lastName)
+}
+
 /**
  * RG-3.12 : l'onglet Contact ne peut etre finalise que s'il reste au moins un
- * bloc non vide (au moins un contact valide).
+ * contact nomme (prenom ou nom).
  */
 export function hasAtLeastOneFilledContact(contacts: ProviderContactFormDraft[]): boolean {
-    return contacts.some(contact => !isProviderContactBlank(contact))
+    return contacts.some(isProviderContactNamed)
 }
 
 /**

migrations/Version20260615120000.php

@@ -0,0 +1,50 @@
+<?php
+
+declare(strict_types=1);
+
+namespace DoctrineMigrations;
+
+use Doctrine\DBAL\Schema\Schema;
+use Doctrine\Migrations\AbstractMigration;
+
+/**
+ * RG-3.04 (correctif) — aligne la regle de validite d'un contact prestataire sur
+ * le M1/M2 : au moins le PRENOM OU le NOM (et non plus « un champ quelconque parmi
+ * prenom/nom/fonction/telephone/email »). Remplace le CHECK chk_provider_contact_name
+ * et met a jour les commentaires de colonnes. La garde applicative
+ * (ProviderContactProcessor::validateName) est alignee dans le meme commit.
+ *
+ * Placee au namespace racine DoctrineMigrations (et non en modulaire Technique) :
+ * elle ALTERE une table creee par une migration racine (Version20260612100000) ;
+ * 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 Version20260615120000 extends AbstractMigration
+{
+    public function getDescription(): string
+    {
+        return 'RG-3.04 : contact prestataire valide si prenom OU nom (alignement M1/M2) — CHECK chk_provider_contact_name.';
+    }
+
+    public function up(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE provider_contact DROP CONSTRAINT chk_provider_contact_name');
+        $this->addSql('ALTER TABLE provider_contact ADD CONSTRAINT chk_provider_contact_name CHECK (first_name IS NOT NULL OR last_name IS NOT NULL)');
+
+        $this->addSql('COMMENT ON TABLE provider_contact IS $_$Contacts d un prestataire (1:n) — au moins le prenom OU le nom rempli (RG-3.04, chk_provider_contact_name).$_$');
+        $this->addSql('COMMENT ON COLUMN provider_contact.first_name IS $_$Prenom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-3.04, chk_provider_contact_name).$_$');
+        $this->addSql('COMMENT ON COLUMN provider_contact.last_name IS $_$Nom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-3.04, chk_provider_contact_name).$_$');
+        $this->addSql('COMMENT ON COLUMN provider_contact.job_title IS $_$Fonction / intitule de poste du contact (≤ 120 caracteres). Facultatif — ne suffit plus a valider le contact (RG-3.04).$_$');
+    }
+
+    public function down(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE provider_contact DROP CONSTRAINT chk_provider_contact_name');
+        $this->addSql('ALTER TABLE provider_contact ADD CONSTRAINT chk_provider_contact_name 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 provider_contact IS $_$Contacts d un prestataire (1:n) — au moins un champ rempli parmi prenom/nom/fonction/telephone/email (RG-3.04, chk_provider_contact_name).$_$');
+        $this->addSql('COMMENT ON COLUMN provider_contact.first_name IS $_$Prenom du contact (capitalise serveur). Au moins un champ du contact requis (RG-3.04, chk_provider_contact_name).$_$');
+        $this->addSql('COMMENT ON COLUMN provider_contact.last_name IS $_$Nom du contact (capitalise serveur). Au moins un champ du contact requis (RG-3.04, chk_provider_contact_name).$_$');
+        $this->addSql('COMMENT ON COLUMN provider_contact.job_title IS $_$Fonction / intitule de poste du contact (≤ 120 caracteres). Au moins un champ du contact requis (RG-3.04, chk_provider_contact_name).$_$');
+    }
+}

src/Module/Technique/Application/Validator/ProviderAccountingCompletenessValidator.php

@@ -0,0 +1,79 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Module\Technique\Application\Validator;
+
+use ApiPlatform\Validator\Exception\ValidationException;
+use App\Module\Technique\Domain\Entity\Provider;
+use Symfony\Component\Validator\ConstraintViolation;
+use Symfony\Component\Validator\ConstraintViolationList;
+
+/**
+ * Validator metier (spec-front M3 § Onglet Comptabilite — jumeau de
+ * SupplierAccountingCompletenessValidator M2) : a la soumission complete de
+ * l'onglet Comptabilite, les six champs scalaires obligatoires doivent etre
+ * renseignes (SIREN, Numero de compte, Mode de TVA, N de TVA, Delai de reglement,
+ * Type de reglement). La banque reste conditionnelle (RG-3.07) et les RIB aussi
+ * (RG-3.08) : ils ne sont pas couverts ici (Assert\Callback sur l'entite Provider
+ * — validatePaymentTypeConsistency).
+ *
+ * Parti pris (miroir M1/M2) : colonnes nullable en base + validateur contextuel,
+ * plutot qu'un Assert\NotBlank sur l'entite (qui casserait le POST de l'onglet
+ * principal, lequel n'envoie aucun champ comptable).
+ *
+ * Invoque par le ProviderProcessor uniquement quand le payload porte les six
+ * champs (= une validation d'onglet), jamais sur un PATCH ciblant un seul champ.
+ *
+ * Leve une ValidationException (HTTP 422) listant chaque champ manquant, par
+ * coherence avec les violations Symfony rendues par API Platform (mapping inline
+ * front via useFormErrors, ERP-101).
+ */
+final class ProviderAccountingCompletenessValidator
+{
+    public function validate(Provider $provider): void
+    {
+        // Map champ -> valeur courante des champs obligatoires de l'onglet.
+        $fields = [
+            'siren'         => $provider->getSiren(),
+            'accountNumber' => $provider->getAccountNumber(),
+            'tvaMode'       => $provider->getTvaMode(),
+            'nTva'          => $provider->getNTva(),
+            'paymentDelay'  => $provider->getPaymentDelay(),
+            'paymentType'   => $provider->getPaymentType(),
+        ];
+
+        $violations = new ConstraintViolationList();
+
+        foreach ($fields as $property => $value) {
+            if ($this->isMissing($value)) {
+                $violations->add(new ConstraintViolation(
+                    'Ce champ est obligatoire.',
+                    null,
+                    [],
+                    $provider,
+                    $property,
+                    $value,
+                ));
+            }
+        }
+
+        if (count($violations) > 0) {
+            throw new ValidationException($violations);
+        }
+    }
+
+    /**
+     * Une valeur est manquante si null ou, pour une chaine, vide apres trim. Les
+     * references (TvaMode / PaymentDelay / PaymentType) ne sont manquantes que
+     * lorsqu'elles valent null.
+     */
+    private function isMissing(mixed $value): bool
+    {
+        if (null === $value) {
+            return true;
+        }
+
+        return is_string($value) && '' === trim($value);
+    }
+}

src/Module/Technique/Infrastructure/ApiPlatform/State/Processor/ProviderContactProcessor.php

@@ -119,23 +119,18 @@ final class ProviderContactProcessor implements ProcessorInterface
     }
 
     /**
-     * RG-3.04 : un bloc Contact est valide des qu'au moins un champ parmi prenom /
-     * nom / fonction / telephone principal / email est renseigne (double garde avec
-     * le CHECK BDD chk_provider_contact_name — leve une 422 propre rattachee au
-     * champ `firstName` plutot qu'une 500 SQL). Joue apres normalisation, donc les
-     * chaines vides (y compris une fonction ou un phone_secondary vides) sont deja
-     * ramenees a null et ne suffisent pas a valider le bloc.
+     * RG-3.04 : un bloc Contact exige au moins le prenom OU le nom (aligne sur le
+     * M1/M2 — un contact se materialise par son nom ; fonction / telephone / email
+     * seuls ne suffisent pas). Double garde avec le CHECK BDD chk_provider_contact_name
+     * — leve une 422 propre rattachee au champ `firstName` plutot qu'une 500 SQL.
+     * Joue apres normalisation (les chaines vides sont deja ramenees a null).
      */
     private function validateName(ProviderContact $contact): void
     {
-        if (null === $contact->getFirstName()
-            && null === $contact->getLastName()
-            && null === $contact->getJobTitle()
-            && null === $contact->getPhonePrimary()
-            && null === $contact->getEmail()) {
+        if (null === $contact->getFirstName() && null === $contact->getLastName()) {
             $violations = new ConstraintViolationList();
             $violations->add(new ConstraintViolation(
-                'Au moins un champ du contact est obligatoire (nom, prénom, fonction, téléphone ou email).',
+                'Le prénom ou le nom du contact est obligatoire.',
                 null,
                 [],
                 $contact,

src/Module/Technique/Infrastructure/ApiPlatform/State/Processor/ProviderProcessor.php

@@ -9,6 +9,7 @@ use ApiPlatform\State\ProcessorInterface;
 use ApiPlatform\Validator\Exception\ValidationException;
 use App\Module\Core\Domain\Entity\User;
 use App\Module\Technique\Application\Service\ProviderFieldNormalizer;
+use App\Module\Technique\Application\Validator\ProviderAccountingCompletenessValidator;
 use App\Module\Technique\Domain\Entity\Provider;
 use App\Shared\Domain\Contract\SiteInterface;
 use DateTimeImmutable;
@@ -75,6 +76,15 @@ final class ProviderProcessor implements ProcessorInterface
         'paymentType', 'bank',
     ];
 
+    /**
+     * Champs comptables obligatoires a la validation complete de l'onglet
+     * (spec-front M3 § Onglet Comptabilite — miroir M1/M2). bank est exclu :
+     * conditionnel (RG-3.07).
+     */
+    private const array ACCOUNTING_REQUIRED_FIELDS = [
+        'siren', 'accountNumber', 'tvaMode', 'nTva', 'paymentDelay', 'paymentType',
+    ];
+
     /** Champ d'archivage (groupe provider:write:archive). */
     private const string ARCHIVE_FIELD = 'isArchived';
 
@@ -102,6 +112,7 @@ final class ProviderProcessor implements ProcessorInterface
         private readonly Security $security,
         private readonly RequestStack $requestStack,
         private readonly EntityManagerInterface $em,
+        private readonly ProviderAccountingCompletenessValidator $accountingValidator,
     ) {}
 
     public function process(mixed $data, Operation $operation, array $uriVariables = [], array $context = []): mixed
@@ -128,6 +139,10 @@ final class ProviderProcessor implements ProcessorInterface
         // deux cotes (l'etat persiste l'a deja ete).
         $this->guardManage($data);
 
+        // Completude de l'onglet Comptabilite (apres normalize : les chaines vides
+        // sont deja ramenees a null). Joue uniquement sur une soumission d'onglet.
+        $this->validateAccountingCompleteness($data);
+
         try {
             return $this->persistProcessor->process($data, $operation, $uriVariables, $context);
         } catch (UniqueConstraintViolationException $e) {
@@ -496,6 +511,21 @@ final class ProviderProcessor implements ProcessorInterface
      *
      * @return list<string>
      */
+    /**
+     * Completude de l'onglet Comptabilite (miroir SupplierProcessor) : ne se
+     * declenche que si TOUS les champs requis sont presents dans le payload
+     * (= soumission d'onglet, pas un PATCH partiel cible). Delegue au validateur
+     * qui leve une 422 listant chaque champ manquant (mapping inline ERP-101).
+     */
+    private function validateAccountingCompleteness(Provider $data): void
+    {
+        if ([] !== array_diff(self::ACCOUNTING_REQUIRED_FIELDS, $this->payloadKeys())) {
+            return;
+        }
+
+        $this->accountingValidator->validate($data);
+    }
+
     private function payloadKeys(): array
     {
         $request = $this->requestStack->getCurrentRequest();

src/Shared/Infrastructure/Database/ColumnCommentsCatalog.php

@@ -395,12 +395,12 @@ final class ColumnCommentsCatalog
             ],
 
             'provider_contact' => [
-                '_table'          => 'Contacts d un prestataire (1:n) — au moins un champ rempli parmi prenom/nom/telephone/email (RG-3.04, chk_provider_contact_name).',
+                '_table'          => 'Contacts d un prestataire (1:n) — au moins le prenom OU le nom rempli (RG-3.04, chk_provider_contact_name).',
                 'id'              => 'Identifiant interne auto-incremente.',
                 'provider_id'     => 'FK -> provider.id, ON DELETE CASCADE — prestataire proprietaire du contact.',
-                'first_name'      => 'Prenom du contact (capitalise serveur). Au moins un champ du contact requis (RG-3.04, chk_provider_contact_name).',
-                'last_name'       => 'Nom du contact (capitalise serveur). Au moins un champ du contact requis (RG-3.04, chk_provider_contact_name).',
-                'job_title'       => 'Fonction / intitule de poste du contact (≤ 120 caracteres).',
+                'first_name'      => 'Prenom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-3.04, chk_provider_contact_name).',
+                'last_name'       => 'Nom du contact (capitalise serveur). Prenom OU nom obligatoire (RG-3.04, chk_provider_contact_name).',
+                'job_title'       => 'Fonction / intitule de poste du contact (≤ 120 caracteres). Facultatif — ne suffit plus a valider le contact (RG-3.04).',
                 'phone_primary'   => 'Telephone principal du contact — chiffres uniquement (normalisation serveur).',
                 'phone_secondary' => 'Telephone secondaire du contact — chiffres uniquement (normalisation serveur).',
                 'email'           => 'Email du contact (lowercase serveur).',

tests/Module/Technique/Api/ProviderAccountingValidationTest.php

@@ -9,9 +9,9 @@ namespace App\Tests\Module\Technique\Api;
  * de l'entite Provider (M3, RG-3.07 / RG-3.08), via le PATCH de l'onglet
  * Comptabilite (groupe provider:write:accounting). On asserte le code HTTP et le
  * propertyPath de la violation (consommable par extractApiViolations cote front,
- * ERP-101). Jumeau de SupplierAccountingApiTest (M2), sans le bloc « completude de
- * l'onglet » : le prestataire est minimal et n'impose pas les six scalaires
- * comptables (spec M3 § 3.1).
+ * ERP-101). Jumeau de SupplierAccountingApiTest (M2), completude de l'onglet
+ * INCLUSE : a la validation complete de l'onglet, les six scalaires comptables
+ * sont obligatoires (spec-front M3 § Onglet Comptabilite — aligne M1/M2).
  *
  * @internal
  */
@@ -81,5 +81,58 @@ final class ProviderAccountingValidationTest extends AbstractProviderApiTestCase
         self::assertResponseStatusCodeSame(200);
     }
 
+    // === Completude de l'onglet Comptabilite (six scalaires obligatoires) ===
+
+    /**
+     * spec-front M3 § Onglet Comptabilite : a la validation COMPLETE de l'onglet
+     * (les six champs requis presents dans le payload), chacun vide doit renvoyer
+     * une 422 sur son propre propertyPath (mapping inline front, ERP-101). Miroir
+     * M1/M2 (ProviderAccountingCompletenessValidator).
+     */
+    public function testIncompleteAccountingTabReturns422OnEachField(): void
+    {
+        $client = $this->createAdminClient();
+        $seed   = $this->seedProvider('Accounting Incomplete');
+
+        $response = $client->request('PATCH', '/api/providers/'.$seed->getId(), [
+            'headers' => ['Content-Type' => self::MERGE, 'Accept' => self::LD],
+            'json'    => [
+                'siren'         => null,
+                'accountNumber' => null,
+                'tvaMode'       => null,
+                'nTva'          => null,
+                'paymentDelay'  => null,
+                'paymentType'   => null,
+            ],
+        ]);
+
+        self::assertResponseStatusCodeSame(422);
+        $paths = $this->violationsByPath($response->toArray(false));
+        self::assertArrayHasKey('siren', $paths);
+        self::assertArrayHasKey('accountNumber', $paths);
+        self::assertArrayHasKey('tvaMode', $paths);
+        self::assertArrayHasKey('nTva', $paths);
+        self::assertArrayHasKey('paymentDelay', $paths);
+        self::assertArrayHasKey('paymentType', $paths);
+    }
+
+    /**
+     * Un PATCH ciblant un sous-ensemble de champs comptables n'est PAS une
+     * validation d'onglet : la completude ne se declenche pas (edition ponctuelle
+     * preservee, cf. validateAccountingCompleteness).
+     */
+    public function testPartialAccountingPatchSkipsCompleteness(): void
+    {
+        $client = $this->createAdminClient();
+        $seed   = $this->seedProvider('Accounting Partial');
+
+        $client->request('PATCH', '/api/providers/'.$seed->getId(), [
+            'headers' => ['Content-Type' => self::MERGE],
+            'json'    => ['nTva' => 'FR12345678901'],
+        ]);
+
+        self::assertResponseStatusCodeSame(200);
+    }
+
     // violationsByPath() : helper mutualise dans AbstractProviderApiTestCase.
 }

tests/Module/Technique/Api/ProviderSubResourceApiTest.php

@@ -9,7 +9,7 @@ use App\Module\Technique\Domain\Entity\Provider;
 /**
  * Tests fonctionnels des sous-ressources Contacts / Adresses / RIB du prestataire
  * (M3, spec § 4.5 — ERP-135). Couvrent : normalisation contact (RG-3.11), RG-3.04
- * (au moins un champ parmi prenom/nom/fonction/telephone/email), RG-3.05 (>= 1 site sur
+ * (au moins le prenom OU le nom — aligne M1/M2), RG-3.05 (>= 1 site sur
  * l'adresse), RG-3.06 (code postal), RG-3.09 (categorie PRESTATAIRE sur adresse),
  * le cloisonnement d'ecriture des sites de l'adresse (§ 2.13 -> 422 sur `sites`),
  * RG-3.08 (DELETE dernier RIB sous LCR -> 409), DELETE contact libre au M3 (pas de
@@ -53,43 +53,60 @@ final class ProviderSubResourceApiTest extends AbstractProviderApiTestCase
     }
 
     /**
-     * RG-3.04 : un bloc Contact est valide des qu'AU MOINS UN champ est rempli parmi
-     * prenom / nom / FONCTION / telephone / email (spec § RG-3.04, ligne 926). Ici
-     * seul jobTitle (Fonction) est fourni -> le bloc est valide -> 201.
+     * RG-3.04 (aligne M1/M2) : un bloc Contact exige le prenom OU le nom. Une
+     * Fonction seule (sans nom ni prenom) ne suffit plus -> 422 rattachee a firstName.
      */
-    public function testPostContactWithOnlyJobTitleReturns201(): void
+    public function testPostContactWithOnlyJobTitleReturns422(): void
     {
         $client = $this->createAdminClient();
         $seed   = $this->seedProvider('Contact JobTitle Only');
 
-        $data = $client->request('POST', '/api/providers/'.$seed->getId().'/contacts', [
-            'headers' => ['Content-Type' => self::LD, 'Accept' => self::LD],
-            'json'    => ['jobTitle' => 'Directeur'],
-        ])->toArray();
-
-        self::assertResponseStatusCodeSame(201);
-        self::assertSame('Directeur', $data['jobTitle']);
-    }
-
-    /**
-     * RG-3.04 : un bloc Contact TOTALEMENT vide (aucun champ du CHECK
-     * chk_provider_contact_name) est rejete avant la base -> 422 rattachee a
-     * firstName. Une Fonction vide (apres normalisation) ne suffit pas a valider.
-     */
-    public function testPostContactCompletelyEmptyReturns422OnFirstNamePath(): void
-    {
-        $client = $this->createAdminClient();
-        $seed   = $this->seedProvider('Contact No Field');
-
         $response = $client->request('POST', '/api/providers/'.$seed->getId().'/contacts', [
             'headers' => ['Content-Type' => self::LD, 'Accept' => self::LD],
-            'json'    => ['jobTitle' => '   '],
+            'json'    => ['jobTitle' => 'Directeur'],
         ]);
 
         self::assertResponseStatusCodeSame(422);
         self::assertArrayHasKey('firstName', $this->violationsByPath($response->toArray(false)));
     }
 
+    /**
+     * RG-3.04 : un bloc Contact sans prenom NI nom (meme avec d'autres champs ou
+     * apres normalisation des chaines vides) est rejete avant la base -> 422
+     * rattachee a firstName (double garde CHECK chk_provider_contact_name).
+     */
+    public function testPostContactWithoutNameReturns422OnFirstNamePath(): void
+    {
+        $client = $this->createAdminClient();
+        $seed   = $this->seedProvider('Contact No Name');
+
+        $response = $client->request('POST', '/api/providers/'.$seed->getId().'/contacts', [
+            'headers' => ['Content-Type' => self::LD, 'Accept' => self::LD],
+            // Email + telephone fournis mais ni prenom ni nom -> invalide (RG-3.04).
+            'json'    => ['email' => 'contact@acme.fr', 'phonePrimary' => '0612345678'],
+        ]);
+
+        self::assertResponseStatusCodeSame(422);
+        self::assertArrayHasKey('firstName', $this->violationsByPath($response->toArray(false)));
+    }
+
+    /**
+     * RG-3.04 : le prenom SEUL (sans nom) suffit a valider le contact -> 201.
+     */
+    public function testPostContactWithOnlyFirstNameReturns201(): void
+    {
+        $client = $this->createAdminClient();
+        $seed   = $this->seedProvider('Contact FirstName Only');
+
+        $data = $client->request('POST', '/api/providers/'.$seed->getId().'/contacts', [
+            'headers' => ['Content-Type' => self::LD, 'Accept' => self::LD],
+            'json'    => ['firstName' => 'Jean'],
+        ])->toArray();
+
+        self::assertResponseStatusCodeSame(201);
+        self::assertSame('Jean', $data['firstName']);
+    }
+
     public function testPostContactOnMissingProviderReturns404(): void
     {
         $client = $this->createAdminClient();