Project

General

Profile

Bug #69458

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

Added by Emmanuel Cazenave 7 days ago. Updated 3 days ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
22 September 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

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.


Files

0001-wip.patch (2.69 KB) 0001-wip.patch Emmanuel Cazenave, 22 September 2022 05:00 PM
0001-backoffice-don-t-consider-filters-as-templates-if-th.patch (4 KB) 0001-backoffice-don-t-consider-filters-as-templates-if-th.patch Frédéric Péters, 23 September 2022 10:29 AM

Associated revisions

Revision 1433bd32 (diff)
Added by Frédéric Péters 6 days ago

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

History

#2

Updated by Emmanuel Cazenave 7 days ago

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

Updated by Frédéric Péters 7 days ago

  • Patch proposed changed from Yes to No

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

Updated by Frédéric Péters 6 days ago

  • Assignee set to Frédéric Péters
#5

Updated by Pierre Cros 6 days ago

  • Subject changed from Filtrage inopérant sur champ liste si la valur du champ comporte un crochet ouvrant to Filtrage inopérant sur champ liste si la valeur du champ comporte un crochet ouvrant
#6

Updated by Frédéric Péters 6 days ago

#7

Updated by Thomas Noël 6 days ago

  • Status changed from Solution proposée to 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

Updated by Frédéric Péters 6 days ago

  • Status changed from Solution validée to 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

Updated by Transition automatique 3 days ago

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

Also available in: Atom PDF