https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342018-12-05T14:23:30ZRedmine Entr’ouvertw.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1420922018-12-05T14:23:30ZThomas Noël
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-1 status-6 priority-4 priority-default closed" href="/issues/28638">Bug #28638</a>: 'LazyFieldVar' object has no attribute 'split'</i> ajouté</li></ul> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1420952018-12-05T14:25:34ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<p>pour ne pas se compliquer la vie.</p>
</blockquote>
<p>Si on veut éviter le localize par défaut de django, on peut avoir un autre ticket.</p> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1421132018-12-05T15:06:37ZThomas Noël
<ul></ul><p>Frédéric Péters a écrit :</p>
<blockquote><blockquote>
<p>pour ne pas se compliquer la vie.</p>
</blockquote>
<p>Si on veut éviter le localize par défaut de django, on peut avoir un autre ticket.</p>
</blockquote>
<p>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...</p> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1421292018-12-05T16:01:22ZThomas Noël
<ul><li><strong>Fichier</strong> <a href="/attachments/29830">0001-draft-28639.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/29830/0001-draft-28639.patch">0001-draft-28639.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>En cours</i></li><li><strong>Assigné à</strong> mis à <i>Thomas Noël</i></li></ul><p>Bon, marche pas, je dois rater un truc évident, ou alors je suis complètement à côté de la plaque, ou les deux.</p>
<pre>
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
</pre> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1422952018-12-06T10:21:17ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Assigné à</strong> changé de <i>Thomas Noël</i> à <i>Frédéric Péters</i></li></ul> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1423042018-12-06T10:30:31ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/29856">0001-misc-use-a-dedicated-lazy-object-for-map-variables-2.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/29856/0001-misc-use-a-dedicated-lazy-object-for-map-variables-2.patch">0001-misc-use-a-dedicated-lazy-object-for-map-variables-2.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul><p>Le truc raté c'était le test, qui ne passait pas sur les deux modes (lazy ou pas).</p>
<p>Voilà le test adapté, pour vérifier les nouveaux attributs uniquement en mode lazy.</p>
<p>Question à voir ici, peut-on se permettre d'avancer avec des nouveautés sur lazy uniquement ?</p> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1423962018-12-06T11:36:52ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/29865">0001-fields-add-a-structured-form-to-map-fields-28639.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/29865/0001-fields-add-a-structured-form-to-map-fields-28639.patch">0001-fields-add-a-structured-form-to-map-fields-28639.patch</a> ajouté</li></ul><blockquote>
<p>Question à voir ici, peut-on se permettre d'avancer avec des nouveautés sur lazy uniquement ?</p>
</blockquote>
<p>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.</p> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1424332018-12-06T13:16:43ZThomas Noël
<ul></ul><p>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 ...?</p> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1424362018-12-06T13:31:15ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Les tests arrivent tous dans l'autre ticket, c'était trop pénible à diviser.</p> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1424432018-12-06T13:45:49ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>ok donc le dernier patch remplace complétement le précédent avec le LazyFieldVarMap.</p>
<p>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</p> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1424462018-12-06T13:54:03ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><p>Zut poussé en oubliant ton commentaire :/</p>
<pre>
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)
</pre> w.c.s. - Development #28639: sur un champ carte, disposer de form_var_xxx_lat et form_var_xxx_lonhttps://dev.entrouvert.org/issues/28639?journal_id=1459892018-12-23T13:34:28ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul>