5e15c1f69f
Auto Tag Develop / tag (push) Successful in 11s
Lot de retours métier **ERP-193** (« Fix tous les retours starseed »), transverse aux 4 répertoires (clients, fournisseurs, prestataires, transporteurs).
## Contenu
- **Pagination** : défaut à 25 items/page sur les 4 répertoires.
- **Libellé** : colonne « Dernière activité » → « Dernière modification ».
- **Consultation** : masquage des onglets vides (coquilles « à venir » + onglets de données sans donnée).
- **Chiffre d'affaires** : plafonné à 999 999 999 999,99 (clamp front + `Assert\LessThanOrEqual` back).
- **Date de création** : interdiction des dates futures (`:max` MalioDate + `Assert\LessThanOrEqual('today')` back).
- **Caractères spéciaux** : blocage des caractères parasites (`²³§~#|…`) dans les champs texte via une allow-list par profil (nom de personne / texte libre / adresse / code alphanumérique) — filtrage front à la frappe + `Assert\Regex` back autoritaire. Email/IBAN/BIC/TVA conservent leurs validateurs de format.
- **UI** : champs en consultation et onglets validés grisés (`readonly` → `disabled`).
- **UI** : boutons « Archiver » en rouge (variant `danger`).
## Tests
- Back : nouveaux tests RG (plafond CA, dates futures, caractères spéciaux) + garde-fou contraintes — suite complète verte (813 tests).
- Front : nouveaux tests unitaires (sanitizers, helpers date/montant) — 615 tests verts, eslint clean.
---------
Co-authored-by: THOLOT DECHENE Matthieu <matthieu@yuno.malio.fr>
Reviewed-on: #139
Co-authored-by: tristan <tristan@yuno.malio.fr>
Co-committed-by: tristan <tristan@yuno.malio.fr>
92 lines
3.7 KiB
TypeScript
92 lines
3.7 KiB
TypeScript
/** Ligne de collection supprimable (contact / adresse / RIB). */
|
|
export interface DeletableRow {
|
|
id?: number | null
|
|
}
|
|
|
|
/**
|
|
* Indique si le bloc d'index `index` peut afficher sa poubelle (ERP-172).
|
|
*
|
|
* Regle metier : on ne peut supprimer un bloc QUE s'il reste au moins un AUTRE
|
|
* bloc deja enregistre (`id` non null, donc persiste en base). Consequences :
|
|
* - tant que rien n'est enregistre -> aucune poubelle (pas de suppression d'un
|
|
* simple brouillon saisi mais pas valide) ;
|
|
* - on peut jeter un brouillon non enregistre s'il reste un bloc enregistre ;
|
|
* - on ne peut jamais supprimer son dernier bloc enregistre.
|
|
*/
|
|
export function isRowRemovable<T extends DeletableRow>(rows: T[], index: number): boolean {
|
|
return rows.some((row, i) => i !== index && row.id != null)
|
|
}
|
|
|
|
/** Options de {@link removeCollectionRow}. */
|
|
export interface RemoveCollectionRowOptions<T extends DeletableRow> {
|
|
/** Tableau reactif des brouillons (passer le `.value` de la ref). */
|
|
rows: T[]
|
|
/** Tableau reactif des erreurs par ligne, aligne sur l'index (passer le `.value`). */
|
|
errors: Record<string, string>[]
|
|
/** Index de la ligne a retirer. */
|
|
index: number
|
|
/** Endpoint de la sous-ressource SANS id (ex: '/client_contacts'). */
|
|
endpoint: string
|
|
/** Suppression serveur : DOIT rejeter en cas d'echec (ex: url => api.delete(url, {}, { toast: false })). */
|
|
deleteRow: (url: string) => Promise<unknown>
|
|
/** Fabrique d'un bloc vide pour garder au moins un bloc visible apres retrait. */
|
|
makeEmpty: () => T
|
|
/** Remontee d'erreur 409/422 mappee proprement (message back, pas de toast fourre-tout). */
|
|
onError: (error: unknown) => void
|
|
/**
|
|
* Callback de succes (toast) appele UNIQUEMENT apres une suppression serveur
|
|
* confirmee d'un bloc persiste (`id` non null). Pas appele sur le simple retrait
|
|
* d'un brouillon local non enregistre (aucune suppression reelle).
|
|
*/
|
|
onSuccess?: () => void
|
|
}
|
|
|
|
/**
|
|
* Retire une ligne de collection (contact / adresse / RIB) sur les ecrans de
|
|
* MODIFICATION, avec DELETE immediat de la sous-ressource (ERP-172). Comportement
|
|
* aligne sur les 3 modules (Client / Fournisseur / Prestataire) :
|
|
*
|
|
* - Bloc jamais persiste (`id` null) : simple retrait local, aucun appel reseau.
|
|
* - Bloc existant (`id` non null) : DELETE `/endpoint/{id}` AVANT le retrait du
|
|
* tableau. On ne retire le bloc QUE si le serveur a confirme — sinon le bloc
|
|
* reste affiche et l'erreur est remontee via `onError` (ex. dernier RIB d'une
|
|
* LCR -> 409 back, RG-x.08).
|
|
*
|
|
* Etat purement local : `rows`/`errors` sont les `.value` des refs (proxies
|
|
* reactifs), le `splice` declenche donc la reactivite.
|
|
*
|
|
* @returns `true` si la ligne a ete retiree (suppression confirmee ou bloc local),
|
|
* `false` si la suppression serveur a echoue (bloc conserve).
|
|
*/
|
|
export async function removeCollectionRow<T extends DeletableRow>(
|
|
options: RemoveCollectionRowOptions<T>,
|
|
): Promise<boolean> {
|
|
const { rows, errors, index, endpoint, deleteRow, makeEmpty, onError, onSuccess } = options
|
|
const removed = rows[index]
|
|
let serverDeleted = false
|
|
|
|
// Bloc existant : suppression serveur d'abord, retrait local seulement si OK.
|
|
if (removed?.id != null) {
|
|
try {
|
|
await deleteRow(`${endpoint}/${removed.id}`)
|
|
}
|
|
catch (error) {
|
|
onError(error)
|
|
return false
|
|
}
|
|
serverDeleted = true
|
|
}
|
|
|
|
rows.splice(index, 1)
|
|
errors.splice(index, 1)
|
|
// Garde au moins un bloc visible (cf. amorce a l'hydratation).
|
|
if (rows.length === 0) {
|
|
rows.push(makeEmpty())
|
|
}
|
|
// Toast de succes uniquement quand le serveur a confirme une vraie suppression.
|
|
if (serverDeleted) {
|
|
onSuccess?.()
|
|
}
|
|
return true
|
|
}
|