Projet

Général

Profil

Bug #27664

conditions de champ sur texte unicode

Ajouté par Frédéric Péters il y a plus de 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Frédéric Péters
Version cible:
-
Début:
30 octobre 2018
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Aujourd'hui ça échoue, la condition n'est pas vérifiée, etc.

Fichiers

0001-tests-make-field-conditions-work-with-unicode-string.patch (2,15 ko) 0001-tests-make-field-conditions-work-with-unicode-string.patch Frédéric Péters, 30 octobre 2018 10:07
0001-tests-make-field-conditions-work-with-unicode-string.patch (4,61 ko) 0001-tests-make-field-conditions-work-with-unicode-string.patch Frédéric Péters, 30 octobre 2018 13:32

Révisions associées

Révision 16cd56c1 (diff)
Ajouté par Frédéric Péters il y a plus de 5 ans

tests: make field conditions work with unicode strings (#27664)

Historique

#2

Mis à jour par Frédéric Péters il y a plus de 5 ans

#3

Mis à jour par Thomas Noël il y a plus de 5 ans

Même soucis pour in (contains), à corriger en même temps ?

#4

Mis à jour par Frédéric Péters il y a plus de 5 ans

Même soucis pour in (contains), à corriger en même temps ?

Tu fais quel test pour voir ça ?

En adaptant le test de ce patch en :

def test_field_unicode_condition_contains(pub):
    FormDef.wipe()
    formdef = FormDef()
    formdef.name = 'Foo'
    formdef.fields = [
        fields.PageField(id='0', label='2nd page', type='page'),
        fields.StringField(type='string', id='1', label='Bar', size='40',
            required=True, varname='bar'),
        fields.PageField(id='3', label='1st page', type='page'),
        fields.StringField(type='string', id='4', label='Baz', size='40',
            required=True, varname='baz',
            condition={'type': 'django', 'value': 'form_var_bar in "éléphant"|split'}),  # ici que ça change
    ]
    formdef.store()

    resp = get_app(pub).get('/foo/')
    resp.form['f1'] = 'hello'
    resp = resp.form.submit('submit')
    assert not 'f4' in resp.form.fields

    resp = get_app(pub).get('/foo/')
    resp.form['f1'] = 'éléphant'
    resp = resp.form.submit('submit')
    assert 'f4' in resp.form.fields

Ça passe.

#5

Mis à jour par Thomas Noël il y a plus de 5 ans

Tu fais quel test pour voir ça ?

Ça passait pas juste en remplaçant form_var_bar == "éléphant" par "éléphant" in form_var_bar (je pense pas avoir foiré mon test pourtant, mais)

#6

Mis à jour par Frédéric Péters il y a plus de 5 ans

Ah, ce contains façon sous-chaine, je vais regarder.

#8

Mis à jour par Thomas Noël il y a plus de 5 ans

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

Tiens, plus complexe que je l'avais imaginé, mais parfait, ainsi seront gérés les éléphants.

#9

Mis à jour par Frédéric Péters il y a plus de 5 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 16cd56c158330d9eeba29bb09ee538caadb32026
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Oct 30 09:44:29 2018 +0100

    tests: make field conditions work with unicode strings (#27664)
#10

Mis à jour par Frédéric Péters il y a plus de 5 ans

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

Formats disponibles : Atom PDF