Bug #3404
Exporter name, id, status et category_id, category_name, category_url dans /myspace/json/forms
100%
Description
- permettre à l’appelant de formater le nom du formulaire comme il le souhaite via name, id et status par exemple dans le plugin de listing des formulaires du portail citoyen,
- permettre à l'appelant de ranger/sauter par rapport à la catégorie du formulaire sans faire d'autres appels, toujours pour le portail citoyen.
Files
Associated revisions
add informations in myspace/json/forms (fix #3404)
History
Updated by Thomas Noël over 10 years ago
Voici un patch qui fait cela (note : nécessaire à Orléans pour résoudre une confusion dans l'affichage, signalée dans #4306).
Updated by Thomas Noël over 10 years ago
- Status changed from Nouveau to En cours
- Assignee set to Frédéric Péters
- Priority changed from Bas to Normal
(attribué à Fred pour relecture/avis/etc.)
Updated by Frédéric Péters over 10 years ago
- Assignee changed from Frédéric Péters to Thomas Noël
Réassigné à Thomas après discussion, l'idée est de réutiliser les substitutions de variable, en en proposant une version allégée (formdata.get_substitution_variables(self, light=False)).
Updated by Thomas Noël over 10 years ago
- File 0001-add-minimal-flag-in-formdata.getsubstitions-3404.patch 0001-add-minimal-flag-in-formdata.getsubstitions-3404.patch added
- File 0001-add-informations-in-myspace-json-forms.patch 0001-add-informations-in-myspace-json-forms.patch added
Voici un patch pour wcs et un autre pour auquo. A valider par Fred à qui j'assigne donc le ticket.
Updated by Thomas Noël over 10 years ago
- Status changed from En cours to Information nécessaire
- Assignee changed from Thomas Noël to Frédéric Péters
Updated by Frédéric Péters over 10 years ago
- Status changed from Information nécessaire to En cours
- Assignee changed from Frédéric Péters to Thomas Noël
J'aurais tendance à préférer le bloc :
d.update({ 'form_status_url': '%sstatus' % self.get_url(), 'form_details': self.formdef.get_detailed_email_form(self, self.get_url()), })
déplacé sous le "if minimal: ... return", ça dupliquerait certes la condition "if self.id", mais ça me parait plus clair.
Updated by Thomas Noël over 10 years ago
- File 0001-add-minimal-flag-in-formdata.getsubstitions-3404.patch 0001-add-minimal-flag-in-formdata.getsubstitions-3404.patch added
Une autre version, plus lisible (sauf que le patch, pour le coup, est moins lisible, mais bon).
Updated by Frédéric Péters over 10 years ago
Le "if minimal" qui suivant le "d.update(self.formdef.get_substitution_variables())", et qui quittait la fonction plus tôt, il m'allait bien; ça évite de devoir indenter le tout. Mais du coup parce que j'ai l'impression que mon commentaire passé n'a pas du être clair, le flow que j'imaginais, c'était :
[...] # formdef and category variables d.update(self.formdef.get_substitution_variables()) if minimal: # là on a tout ce qu'on veut, on quitte l'affaire dès maintenant d = copy.deepcopy(d) flatten_dict(d) return d if self.id: # on remet un if self.id, mais c'est pas bien grave. d.update({ 'form_status_url': '%sstatus' % self.get_url(), 'form_details': self.formdef.get_detailed_email_form(self, self.get_url()), }) user = self.get_user() if user: d.update(user.get_substitution_variables(prefix='form_')) [...]
Updated by Thomas Noël over 10 years ago
- File 0001-add-minimal-flag-in-formdata.getsubstitions-3404.patch 0001-add-minimal-flag-in-formdata.getsubstitions-3404.patch added
Cette fois-ci, on devrait être ok.
Updated by Thomas Noël over 10 years ago
- Status changed from En cours to Résolu (à déployer)
- % Done changed from 0 to 100
Appliqué par commit 8c9a8c4eeab797f6b1df318d0ea2883b6276b35b.
Updated by Thomas Noël about 10 years ago
- Status changed from Résolu (à déployer) to Solution déployée
add "minimal" flag in formdata.getsubstitions (#3404)
(currently needed in myspace/json/forms in "auquotidien" extra)