Project

General

Profile

Bug #13482

FormData.get_substition_variables() export deux fois les informations de geolocation

Added by Benjamin Dauvergne almost 6 years ago. Updated almost 6 years ago.

Status:
Fermé
Priority:
Normal
Target version:
Start date:
05 October 2016
Due date:
% Done:

100%

Estimated time:
Patch proposed:
Yes
Planning:

Description

De deux manières différentes, ce n'est pas gênant mais ce serait mieux que ce soit fait en un seul endroit.

        if self.geolocations:
            for k, v in self.geolocations.items():
                d['form_geoloc_%s' % k] = v

...
        if self.geolocations:
            for k, v in self.geolocations.items():
                d['form_geoloc_%s_lat' % k] = v.get('lat')
                d['form_geoloc_%s_lon' % k] = v.get('lon')   

Files

Associated revisions

Revision 8c2a5004 (diff)
Added by Benjamin Dauvergne almost 6 years ago

formdata: regroup geolocations substitution variables (fixes #13482)

History

#1

Updated by Frédéric Péters (de retour le 10/10) almost 6 years ago

Je ne sais pas trop comment ça s'est trouvé ainsi…

Pour info dans la doc on fait référence à la seconde forme :

Une fois le géocodage réussi, l'information est mise à disposition dans les
variables form_geoloc_base_lat pour la latitude et
form_geoloc_base_lon pour la longitude.

#3

Updated by Benjamin Dauvergne almost 6 years ago

Ça ne me gêne pas d'avoir les deux.

#4

Updated by Benjamin Dauvergne almost 6 years ago

Je peux ajouter la doc sur la première forme.

#5

Updated by Frédéric Péters (de retour le 10/10) almost 6 years ago

Ack ainsi. (sans mise à jour de la doc)

#6

Updated by Benjamin Dauvergne almost 6 years ago

  • Status changed from Nouveau to Résolu (à déployer)
  • % Done changed from 0 to 100
#7

Updated by Frédéric Péters (de retour le 10/10) almost 6 years ago

  • Target version set to v1.61
#8

Updated by Frédéric Péters (de retour le 10/10) almost 6 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF