Bug #62223
L'insertion d'une image seule est refusée comme saisie dans le champ commentaire
Début:
28 février 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Si l'on insère juste une image dans un champ commentaire, on a un message d'erreur :
Il y a eu un problème à la soumission du formulaire. Regardez ci-dessous pour le détail. champ obligatoire
Il considère que le champ est vide.
Le html est :
<p><img alt="chronomètre" src="https://portail-caen-la-mer.test.entrouvert.org/media/uploads/2022/02/28/chrono-5-mn.jpg" style="width: 75px; height: 78px;" /></p>
Mais une fois soumis, ça se vide.
Si l'on ajoute un espace insécable, ça passe.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
Mis à jour par Frédéric Péters il y a environ 2 ans
- Fichier 0001-misc-don-t-consider-wysiwyg-widgets-with-a-single-im.patch 0001-misc-don-t-consider-wysiwyg-widgets-with-a-single-im.patch ajouté
- Statut changé de En cours à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Noël il y a environ 2 ans
Note : en lisant que Django conseille d'utiliser bleach plutôt que strip_tags, je me disais qu'on pourrait faire comme ça aussi :
if not Cleaner(tags=['img'], strip=True).clean(self.value).strip(): self.value = ''
Mis à jour par Frédéric Péters il y a environ 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
En l'espèce on n'est pas sur du vrai nettoyage important, le truc grossier de strip_tags est très bien.
commit 382e06027e0718d13f6fef92394e0f3f7eb4b8d5 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Wed Mar 16 16:21:06 2022 +0100 misc: don't consider wysiwyg widgets with a single image as empty (#62223)
Mis à jour par Transition automatique il y a environ 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: don't consider wysiwyg widgets with a single image as empty (#62223)