## Description rapide du problème Si un champ extra obligatoire est dans un fieldset conditionné par afficher_si, que la condition afficher_si n'est pas rempli, cela provoque quand même une erreur sur le champ extra. Ex avec le .yaml ``` spip_articles: - options: { nom: conditionnante, label: conditionnante, type: text, size: '40', autocomplete: defaut, restrictions: { secteurs: '', branches: '', voir: { auteur: '' }, modifier: { auteur: '' } }, sql: 'text DEFAULT '''' NOT NULL' } identifiant: '@608ef28dc2db6' verifier: { } saisie: input - options: { nom: conditionne, label: Conditionné, explication: 'Si conditionannte = ''oui'', alors s''affiche', afficher_si: '@conditionnante@ == ''oui''', restrictions: { secteurs: '', branches: '', voir: { auteur: '' }, modifier: { auteur: '' } } } identifiant: '@608ef28908833' verifier: { } saisie: fieldset saisies: [{ options: { nom: obligatoire, label: Obligatoire, type: text, size: '40', autocomplete: defaut, obligatoire: on, restrictions: { secteurs: '', branches: '', voir: { auteur: '' }, modifier: { auteur: '' } }, sql: 'text DEFAULT '''' NOT NULL' }, identifiant: '@608ef28b19403', verifier: { }, saisie: input }] ```` ### Explication de la cause du problème 1. On peut regrouper les cextras dans des fieldsets 2. On peut même dire que les saisies dans ces fieldsets sont obligatoires 3. On peut mettre des afficher_si sur un fieldset 4. `saisies_verifier` est capable de ne pas vérifier les champs qui sont a. Marqués comme obligatoire b. Dans un fieldset avec afficher_si dont la condition n'est pas vérifier 5. Mais pour que 4 marche, il faut bien sûr que la structure hierarchique "fieldset->cextras" soit passé à verifier 6. Pb, dans cextras, on ne passe à `saisies_verifier()` que les saisies associés à une structure SQL, or saisies_lister_avec_sql` (appelé ici https://git.spip.net/spip-contrib-extensions/champs_extras_core/src/branch/master/cextras_pipelines.php#L278) supprime la hiérarchie des saisies. ### Pistes de solution a. Modifier cextras pour passer TOUTE la structure de saisies, donc conserver l'arborescence (concrètement, ne plus appeler `champs_extras_saisies_lister_avec_sql()` avant `saisies_verifier`) b. Faire que `saisies_lister_avec_sql` "intègre" au niveau des soussaisies de fieldset les `afficher_si` de la saisie globale A mon avis a. est clairement le plus simple, le moins usine à gaz, et je ne vois vraiment pas pourquoi on ne vérifierai que les saisies qui non pas de valeur SQL. A priori, si quelqu'un déclare un cextras sans SQL associé, c'est qu'il sait ce qu'il fait.