Project

General

Profile

Development #28639

sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lon

Added by Thomas Noël 6 months ago. Updated 5 months ago.

Status:
Solution déployée
Priority:
Normal
Start date:
05 Dec 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

Description

... qui seraient des string, pour ne pas se compliquer la vie.

0001-draft-28639.patch View (2.36 KB) Thomas Noël, 05 Dec 2018 05:01 PM

0001-misc-use-a-dedicated-lazy-object-for-map-variables-2.patch View (3.75 KB) Frédéric Péters, 06 Dec 2018 11:29 AM

0001-fields-add-a-structured-form-to-map-fields-28639.patch View (2.61 KB) Frédéric Péters, 06 Dec 2018 12:36 PM


Related issues

Related to w.c.s. - Bug #28638: 'LazyFieldVar' object has no attribute 'split' Rejeté 05 Dec 2018

Associated revisions

Revision b70d6d15 (diff)
Added by Frédéric Péters 6 months ago

fields: add a structured form to map fields (#28639)

History

#1 Updated by Thomas Noël 6 months ago

  • Related to Bug #28638: 'LazyFieldVar' object has no attribute 'split' added

#2 Updated by Frédéric Péters 6 months ago

pour ne pas se compliquer la vie.

Si on veut éviter le localize par défaut de django, on peut avoir un autre ticket.

#3 Updated by Thomas Noël 6 months ago

Frédéric Péters a écrit :

pour ne pas se compliquer la vie.

Si on veut éviter le localize par défaut de django, on peut avoir un autre ticket.

En fait c'était surtout dans l'idée de ne pas inventer quelque chose ici (on n'utilise pas de "decimal" dans les variables natives), et aussi de renvoyer une chaine vide si la carte n'a pas été cliquée...

#4 Updated by Thomas Noël 6 months ago

Bon, marche pas, je dois rater un truc évident, ou alors je suis complètement à côté de la plaque, ou les deux.

    def test_lazy_map_variable(pub, variable_test_data):
        formdef = FormDef.select()[0]
        formdata = formdef.data_class().select()[0]
        pub.substitutions.reset()
        pub.substitutions.feed(formdef)
        pub.substitutions.feed(formdata)
        for mode in (None, 'lazy'):
            assert WorkflowStatusItem.compute('=form_var_map') == '2;4'
            assert WorkflowStatusItem.compute('{{ form_var_map }}') == '2;4'
            assert WorkflowStatusItem.compute('=form_var_map.split(";")[0]') == '2'
>           assert WorkflowStatusItem.compute('=form_var_map_lat') == '2'
E           AssertionError: assert '=form_var_map_lat' == '2'
E             - =form_var_map_lat
E             + 2

#5 Updated by Frédéric Péters 6 months ago

  • Assignee changed from Thomas Noël to Frédéric Péters

#6 Updated by Frédéric Péters 6 months ago

Le truc raté c'était le test, qui ne passait pas sur les deux modes (lazy ou pas).

Voilà le test adapté, pour vérifier les nouveaux attributs uniquement en mode lazy.

Question à voir ici, peut-on se permettre d'avancer avec des nouveautés sur lazy uniquement ?

#7 Updated by Frédéric Péters 6 months ago

Question à voir ici, peut-on se permettre d'avancer avec des nouveautés sur lazy uniquement ?

Sur l'idée que non, pas encore, j'ai repris l'affaire de manière différente, pour que les champs carte exposent une forme structurée.

#8 Updated by Thomas Noël 6 months ago

il manque peut-être un bout du patch, je m'attendais à ce que les tests soient modifiés pour être aussi lancés sur le mode non lazy ...?

#9 Updated by Frédéric Péters 6 months ago

Les tests arrivent tous dans l'autre ticket, c'était trop pénible à diviser.

#10 Updated by Thomas Noël 6 months ago

  • Status changed from Solution proposée to Solution validée

ok donc le dernier patch remplace complétement le précédent avec le LazyFieldVarMap.

Au cas où je ne serais pas contre un petit commentaire autour du "def get_structured_value(self, data):" qui expliquerait que le but est d'avoir une représentation de "lat" et "lon" dans form_var_xxx_lat/lon

#11 Updated by Frédéric Péters 6 months ago

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

Zut poussé en oubliant ton commentaire :/

commit b70d6d150756e65b75a91b234730ff4c52521c72
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Dec 6 12:34:18 2018 +0100

    fields: add a structured form to map fields (#28639)

#12 Updated by Frédéric Péters 5 months ago

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

Also available in: Atom PDF