fix(front) : masque les onglets vides en consultation des 4 repertoires (ERP-193)

2026-06-18 16:15:41 +02:00
parent 86507486a4
commit 868141e324
12 changed files with 578 additions and 45 deletions
@@ -2,7 +2,10 @@ import { describe, expect, it } from 'vitest'
 import {
    canEditClient,
    categoryOptionsOf,
+    clientConsultationVisibleTabs,
    contactOptionsOf,
+    hasAccountingData,
+    hasInformationData,
    iriOf,
    mapAccountingDraft,
    mapAddressToDraft,
@@ -248,3 +251,73 @@ describe('paymentTypeCodeOf (ERP-121 : RIB masques hors-LCR en consultation)', (
        expect(paymentTypeCodeOf(undefined)).toBeNull()
    })
 })
+
+describe('hasInformationData', () => {
+    it('faux si tous les champs Information sont vides/absents', () => {
+        expect(hasInformationData({ '@id': '/api/clients/1', id: 1 })).toBe(false)
+        expect(hasInformationData({ '@id': '/api/clients/1', id: 1, description: '   ' })).toBe(false)
+    })
+
+    it('vrai des qu\'un champ Information porte une donnee', () => {
+        expect(hasInformationData({ '@id': '/api/clients/1', id: 1, directorName: 'Dupont' })).toBe(true)
+        expect(hasInformationData({ '@id': '/api/clients/1', id: 1, employeesCount: 0 })).toBe(true)
+    })
+})
+
+describe('hasAccountingData', () => {
+    it('faux sans champ comptable ni RIB', () => {
+        expect(hasAccountingData({ '@id': '/api/clients/1', id: 1 })).toBe(false)
+    })
+
+    it('vrai avec un champ comptable scalaire', () => {
+        expect(hasAccountingData({ '@id': '/api/clients/1', id: 1, siren: '123456789' })).toBe(true)
+    })
+
+    it('vrai avec une relation comptable embarquee (paymentType)', () => {
+        expect(hasAccountingData({
+            '@id': '/api/clients/1', id: 1,
+            paymentType: { '@id': '/api/payment_types/1', code: 'LCR' },
+        })).toBe(true)
+    })
+
+    it('vrai avec au moins un RIB', () => {
+        expect(hasAccountingData({
+            '@id': '/api/clients/1', id: 1,
+            ribs: [{ '@id': '/api/ribs/1', id: 1, iban: 'FR76...' }],
+        })).toBe(true)
+    })
+})
+
+describe('clientConsultationVisibleTabs', () => {
+    it('retourne [] tant que le client n\'est pas charge', () => {
+        expect(clientConsultationVisibleTabs(null, { canAccountingView: true })).toEqual([])
+        expect(clientConsultationVisibleTabs(undefined, { canAccountingView: true })).toEqual([])
+    })
+
+    it('masque les coquilles et les onglets vides (client minimal)', () => {
+        const client: ClientDetail = { '@id': '/api/clients/1', id: 1, companyName: 'ACME' }
+        expect(clientConsultationVisibleTabs(client, { canAccountingView: true })).toEqual([])
+    })
+
+    it('affiche les onglets non vides dans l\'ordre information/contact/address/accounting', () => {
+        const client: ClientDetail = {
+            '@id': '/api/clients/1', id: 1,
+            directorName: 'Dupont',
+            contacts: [{ '@id': '/api/client_contacts/1', id: 1 }],
+            addresses: [{ '@id': '/api/client_addresses/1', id: 1 }],
+            siren: '123456789',
+        }
+        expect(clientConsultationVisibleTabs(client, { canAccountingView: true }))
+            .toEqual(['information', 'contact', 'address', 'accounting'])
+    })
+
+    it('masque Comptabilite sans le droit accounting.view meme si des donnees existent', () => {
+        const client: ClientDetail = {
+            '@id': '/api/clients/1', id: 1,
+            contacts: [{ '@id': '/api/client_contacts/1', id: 1 }],
+            siren: '123456789',
+        }
+        expect(clientConsultationVisibleTabs(client, { canAccountingView: false }))
+            .toEqual(['contact'])
+    })
+})
@@ -3,6 +3,8 @@ import {
    canEditSupplier,
    categoryOptionsOf,
    contactOptionsOf,
+    hasAccountingData,
+    hasInformationData,
    iriOf,
    mapAccountingDraft,
    mapAddressToDraft,
@@ -14,6 +16,7 @@ import {
    showArchiveAction,
    showRestoreAction,
    siteOptionsOf,
+    supplierConsultationVisibleTabs,
    type SupplierDetail,
 } from '../supplierConsultation'

@@ -237,3 +240,60 @@ describe('paymentTypeCodeOf (ERP-121 : RIB masques hors-LCR en consultation)', (
        expect(paymentTypeCodeOf(undefined)).toBeNull()
    })
 })
+
+describe('hasInformationData (fournisseur)', () => {
+    it('faux si tous les champs Information (volume previsionnel inclus) sont vides', () => {
+        expect(hasInformationData({ '@id': '/api/suppliers/1', id: 1 })).toBe(false)
+    })
+
+    it('vrai des qu\'un champ Information porte une donnee', () => {
+        expect(hasInformationData({ '@id': '/api/suppliers/1', id: 1, directorName: 'Martin' })).toBe(true)
+        expect(hasInformationData({ '@id': '/api/suppliers/1', id: 1, volumeForecast: 1200 })).toBe(true)
+    })
+})
+
+describe('hasAccountingData (fournisseur)', () => {
+    it('faux sans champ comptable ni RIB', () => {
+        expect(hasAccountingData({ '@id': '/api/suppliers/1', id: 1 })).toBe(false)
+    })
+
+    it('vrai avec un champ comptable ou un RIB', () => {
+        expect(hasAccountingData({ '@id': '/api/suppliers/1', id: 1, siren: '123456789' })).toBe(true)
+        expect(hasAccountingData({
+            '@id': '/api/suppliers/1', id: 1,
+            ribs: [{ '@id': '/api/supplier_ribs/1', id: 1, iban: 'FR76...' }],
+        })).toBe(true)
+    })
+})
+
+describe('supplierConsultationVisibleTabs', () => {
+    it('retourne [] tant que le fournisseur n\'est pas charge', () => {
+        expect(supplierConsultationVisibleTabs(null, { canAccountingView: true })).toEqual([])
+    })
+
+    it('masque les coquilles et les onglets vides (fournisseur minimal)', () => {
+        expect(supplierConsultationVisibleTabs(
+            { '@id': '/api/suppliers/1', id: 1, companyName: 'ACME' },
+            { canAccountingView: true },
+        )).toEqual([])
+    })
+
+    it('affiche information/contacts/addresses/accounting (cles plurielles) dans l\'ordre', () => {
+        const supplier: SupplierDetail = {
+            '@id': '/api/suppliers/1', id: 1,
+            volumeForecast: 1000,
+            contacts: [{ '@id': '/api/supplier_contacts/1', id: 1 }],
+            addresses: [{ '@id': '/api/supplier_addresses/1', id: 1 }],
+            siren: '123456789',
+        }
+        expect(supplierConsultationVisibleTabs(supplier, { canAccountingView: true }))
+            .toEqual(['information', 'contacts', 'addresses', 'accounting'])
+    })
+
+    it('masque Comptabilite sans le droit accounting.view', () => {
+        expect(supplierConsultationVisibleTabs(
+            { '@id': '/api/suppliers/1', id: 1, siren: '123456789' },
+            { canAccountingView: false },
+        )).toEqual([])
+    })
+})
@@ -317,6 +317,77 @@ export function mapAddressView(address: AddressRead): AddressView {
    }
 }

+/**
+ * Vrai si une valeur scalaire porte une donnee affichable (non null/undefined,
+ * et non chaine vide apres trim). Sert aux predicats « onglet vide » ci-dessous.
+ */
+function hasValue(value: unknown): boolean {
+    if (value === null || value === undefined) {
+        return false
+    }
+    if (typeof value === 'string') {
+        return value.trim() !== ''
+    }
+    return true
+}
+
+/**
+ * Vrai si l'onglet Information porte au moins une donnee. ERP-193 : en
+ * consultation on masque les onglets vides ; Information n'echappe pas a la
+ * regle malgre son statut d'onglet d'atterrissage par defaut.
+ */
+export function hasInformationData(client: ClientDetail): boolean {
+    return [
+        client.description,
+        client.competitors,
+        client.foundedAt,
+        client.employeesCount,
+        client.revenueAmount,
+        client.profitAmount,
+        client.directorName,
+    ].some(hasValue)
+}
+
+/**
+ * Vrai si l'onglet Comptabilite porte au moins un champ comptable OU un RIB.
+ * (Le gating permission `accounting.view` reste applique en amont par l'appelant.)
+ */
+export function hasAccountingData(client: ClientDetail): boolean {
+    const draft = mapAccountingDraft(client)
+    const hasField = Object.values(draft).some(hasValue)
+    const hasRib = (client.ribs ?? []).length > 0
+    return hasField || hasRib
+}
+
+/**
+ * Onglets visibles en consultation (ERP-193, retour metier) : on masque les
+ * coquilles non implementees (Transport / Statistiques / Rapports / Echanges)
+ * ET tout onglet de donnees vide. L'ordre reproduit `buildClientFormTabKeys`.
+ * Retourne `[]` tant que le client n'est pas charge.
+ */
+export function clientConsultationVisibleTabs(
+    client: ClientDetail | null | undefined,
+    options: { canAccountingView: boolean },
+): string[] {
+    if (!client) {
+        return []
+    }
+    const visible: string[] = []
+    if (hasInformationData(client)) {
+        visible.push('information')
+    }
+    if ((client.contacts ?? []).length > 0) {
+        visible.push('contact')
+    }
+    if ((client.addresses ?? []).length > 0) {
+        visible.push('address')
+    }
+    if (options.canAccountingView && hasAccountingData(client)) {
+        visible.push('accounting')
+    }
+    return visible
+}
+
 /**
 * Bouton « Modifier » : visible si l'utilisateur peut editer au moins un onglet
 * — `manage` (formulaire/onglets metier) OU `accounting.manage` (le role Compta
@@ -292,6 +292,78 @@ export function mapAddressView(address: AddressRead): AddressView {
    }
 }

+/**
+ * Vrai si une valeur scalaire porte une donnee affichable (non null/undefined,
+ * et non chaine vide apres trim). Sert aux predicats « onglet vide » ci-dessous.
+ */
+function hasValue(value: unknown): boolean {
+    if (value === null || value === undefined) {
+        return false
+    }
+    if (typeof value === 'string') {
+        return value.trim() !== ''
+    }
+    return true
+}
+
+/**
+ * Vrai si l'onglet Information porte au moins une donnee (volume previsionnel
+ * inclus, specifique fournisseur). ERP-193 : en consultation on masque les
+ * onglets vides, Information comprise.
+ */
+export function hasInformationData(supplier: SupplierDetail): boolean {
+    return [
+        supplier.description,
+        supplier.competitors,
+        supplier.foundedAt,
+        supplier.employeesCount,
+        supplier.revenueAmount,
+        supplier.profitAmount,
+        supplier.directorName,
+        supplier.volumeForecast,
+    ].some(hasValue)
+}
+
+/**
+ * Vrai si l'onglet Comptabilite porte au moins un champ comptable OU un RIB.
+ * (Le gating permission `accounting.view` reste applique en amont par l'appelant.)
+ */
+export function hasAccountingData(supplier: SupplierDetail): boolean {
+    const draft = mapAccountingDraft(supplier)
+    const hasField = Object.values(draft).some(hasValue)
+    const hasRib = (supplier.ribs ?? []).length > 0
+    return hasField || hasRib
+}
+
+/**
+ * Onglets visibles en consultation (ERP-193, retour metier) : on masque les
+ * coquilles non implementees (Transport / Statistiques / Rapports / Echanges)
+ * ET tout onglet de donnees vide. L'ordre reproduit `buildSupplierFormTabKeys`.
+ * Retourne `[]` tant que le fournisseur n'est pas charge.
+ */
+export function supplierConsultationVisibleTabs(
+    supplier: SupplierDetail | null | undefined,
+    options: { canAccountingView: boolean },
+): string[] {
+    if (!supplier) {
+        return []
+    }
+    const visible: string[] = []
+    if (hasInformationData(supplier)) {
+        visible.push('information')
+    }
+    if ((supplier.contacts ?? []).length > 0) {
+        visible.push('contacts')
+    }
+    if ((supplier.addresses ?? []).length > 0) {
+        visible.push('addresses')
+    }
+    if (options.canAccountingView && hasAccountingData(supplier)) {
+        visible.push('accounting')
+    }
+    return visible
+}
+
 /**
 * Bouton « Modifier » : visible si l'utilisateur peut editer au moins un onglet
 * — `manage` (formulaire/onglets metier) OU `accounting.manage` (le role Compta