Project

General

Profile

Development #60860

commentaire riche, considérer un unique paragraphe vide comme vide

Added by Frédéric Péters 4 months ago. Updated 4 months ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
20 Jan 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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

Revision eabf99f2 (diff)
Added by Frédéric Péters 4 months ago

forms: consider rich text with no visible element as empty (#60860)

History

#2

Updated by Frédéric Péters 4 months ago

#3

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 ?

#4

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 ?

#5

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.

#6

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.

#7

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)
#8

Updated by Frédéric Péters 4 months ago

  • Status changed from Résolu (à déployer) to Solution déployée
#9

Updated by Transition automatique 2 months ago

Automatic expiration

Also available in: Atom PDF