Projet

Général

Profil

Bug #69458

Filtrage inopérant sur champ liste si la valeur du champ comporte un crochet ouvrant

Ajouté par Emmanuel Cazenave il y a plus d'un an. Mis à jour il y a plus d'un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
22 septembre 2022
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Un champ liste, dedans un texte qui comporte un crochet ouvrant.

Sur une la vue de traitement, tenter de filtrer les demandes sur ce champ, ne sélectionnant la valeur qui comporte ce crochet ouvrant : aucune demande ne remonte.


Fichiers

0001-wip.patch (2,69 ko) 0001-wip.patch Emmanuel Cazenave, 22 septembre 2022 17:00
0001-backoffice-don-t-consider-filters-as-templates-if-th.patch (4 ko) 0001-backoffice-don-t-consider-filters-as-templates-if-th.patch Frédéric Péters, 23 septembre 2022 10:29

Révisions associées

Révision 1433bd32 (diff)
Ajouté par Frédéric Péters il y a plus d'un an

backoffice: don't consider filters as templates if there are brackets (#69458)

Historique

#2

Mis à jour par Emmanuel Cazenave il y a plus d'un an

Un test unitaire qui reproduit.

Dans le ticket lié le champ prend la valeur "Conférence « Toulouse, 1908. Le Congrès pour l’unité socialiste » [Journées Patrimoine 2022 - TLSE]".

J'ai réduit le teste à sa forme la plus simple, un crocher ouvrant tout seul suffit.

#3

Mis à jour par Frédéric Péters il y a plus d'un an

  • Patch proposed changé de Oui à Non

C'est parce que c'est pris comme étant un gabarit (parce qu'EZT).

Évidemment pas le patch parce qu'on ne peut pas perdre de manière générale la détection ezt, mais pour la démo :

@@ -540,7 +540,7 @@ class Template:

     @classmethod
     def is_template_string(cls, string):
-        return isinstance(string, str) and ('{{' in string or '{%' in string or '[' in string)
+        return isinstance(string, str) and ('{{' in string or '{%' in string)

#4

Mis à jour par Frédéric Péters il y a plus d'un an

  • Assigné à mis à Frédéric Péters
#5

Mis à jour par Pierre Cros il y a plus d'un an

  • Sujet changé de Filtrage inopérant sur champ liste si la valur du champ comporte un crochet ouvrant à Filtrage inopérant sur champ liste si la valeur du champ comporte un crochet ouvrant
#6

Mis à jour par Frédéric Péters il y a plus d'un an

#7

Mis à jour par Thomas Noël il y a plus d'un an

  • Statut changé de Solution proposée à Solution validée

C'est effectivement un endroit où l'on peut considérer que EZT n'a pas à exister. (Et en cas de présence de crochets, ça ne sera pas compté comme déprécation ? Je valide sans avoir vérifié, j'avoue...)

#8

Mis à jour par Frédéric Péters il y a plus d'un an

  • Statut changé de Solution validée à Résolu (à déployer)

Et en cas de présence de crochets, ça ne sera pas compté comme dépréciation ?

Là on était même pas nécessairement sur quelque chose de stocké, juste une valeur tapée pour un filtre. Aussi, pour le moment les vues personnalisées ne sont de toute façon pas analysées pour les dépréciations, je n'avais imaginé que quelqu'un pourrait venir y mettre de l'ezt (et compte sur le fait que personne ne l'a fait).

commit 1433bd320da344b8f2f7ff8e90864319532e357d
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Fri Sep 23 09:46:34 2022 +0200

    backoffice: don't consider filters as templates if there are brackets (#69458)
#9

Mis à jour par Transition automatique il y a plus d'un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#10

Mis à jour par Transition automatique il y a plus d'un an

Automatic expiration

Formats disponibles : Atom PDF