Development #60860
commentaire riche, considérer un unique paragraphe vide comme vide
0%
Description
Une action commentaire laissée vide transmet désormais <p></p> et comme ce n'est pas formellement vide ça fait que la vérification du caractère obligatoire du champ passe.
Files
Associated revisions
History
Updated by Frédéric Péters 4 months ago
- File 0001-forms-consider-rich-text-with-no-visible-element-as-.patch 0001-forms-consider-rich-text-with-no-visible-element-as-.patch added
- Status changed from Nouveau to Solution proposée
- Patch proposed changed from No to Yes
Updated by Thomas Noël 4 months ago
Je me suis dit "et si quelqu'un tape juste deux espaces ça va faire <p> </p> et passer", mais il y a bien un strip après le retrait des balises (strip_tags(self.value).strip()) donc ça doit être ok.
Et donc, ajouter le cas « <p> </p> » dans les tests ?
Updated by Frédéric Péters 4 months ago
Test ajouté dans la branche mais avant, sans éditeur riche, si quelqu'un tapait juste des espaces, ça passait ou pas ?
Updated by Pierre Cros 4 months ago
Aucune idée mais je dirais que si quelqu'un tape des espaces il est normal que ça passe.
Updated by Benjamin Dauvergne 4 months ago
- Status changed from Solution proposée to Solution validée
Pierre Cros a écrit :
Aucune idée mais je dirais que si quelqu'un tape des espaces il est normal que ça passe.
Mais c'est très bien aussi que le nouveau code ne le permette plus. Je valide avec intégration de la remarque de Thomas.
Updated by Frédéric Péters 4 months ago
- Status changed from Solution validée to Résolu (à déployer)
commit eabf99f2cac31d7546edfd94db0bdfa6a270677b
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Thu Jan 20 09:29:00 2022 +0100
forms: consider rich text with no visible element as empty (#60860)
Updated by Frédéric Péters 4 months ago
- Status changed from Résolu (à déployer) to Solution déployée
forms: consider rich text with no visible element as empty (#60860)