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.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-forms-consider-rich-text-with-no-visible-element-as-.patch 0001-forms-consider-rich-text-with-no-visible-element-as-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 2 ans
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 ?
Mis à jour par Frédéric Péters il y a environ 2 ans
Test ajouté dans la branche mais avant, sans éditeur riche, si quelqu'un tapait juste des espaces, ça passait ou pas ?
Mis à jour par Pierre Cros il y a environ 2 ans
Aucune idée mais je dirais que si quelqu'un tape des espaces il est normal que ça passe.
Mis à jour par Benjamin Dauvergne il y a environ 2 ans
- Statut changé de Solution proposée à 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.
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution validée à 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)
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
forms: consider rich text with no visible element as empty (#60860)