Bug #11288
l'api user/forms ne montre pas les demandes en cours sur des formulaires désactivés
0%
Description
cf #11281
pour moi il faut montrer les demandes, car l'usager peut quand même y accéder.
Files
Associated revisions
general: make formdatas visible even if related formdefs are disabled (#11288)
History
Updated by Thomas Noël almost 7 years ago
A priori ça serait juste :
diff --git a/wcs/api.py b/wcs/api.py
index 8948c97..854ddb3 100644
--- a/wcs/api.py
+++ b/wcs/api.py
@@ -417,7 +417,7 @@ class ApiUserDirectory(Directory):
return json.dumps(user_info)
def get_user_forms(self, user):
- formdefs = FormDef.select(lambda x: not x.is_disabled())
+ formdefs = FormDef.select()
user_forms = []
for formdef in formdefs:
user_forms.extend(formdef.data_class().get_with_indexed_value(
Mais en dehors de l'API, il y aurait aussi des choses à faire dans les pages de wcs qui présentent les demandes (page de catégorie, myspace, …).
Updated by Frédéric Péters almost 7 years ago
Oui, et ok pour un patch qui ferait le nécessaire ailleurs, bien sûr. (et tests etc.)
Updated by Thomas Noël almost 7 years ago
- File 0001-formdatas-are-visible-even-if-related-formdefs-are-d.patch 0001-formdatas-are-visible-even-if-related-formdefs-are-d.patch added
- Patch proposed changed from No to Yes
Voici la proposition côté wcs.
Si c'est ok, il va devoir revoir aussi le module auquo qui joue cela dans son coin (et pousser les deux en même temps)
Updated by Frédéric Péters almost 7 years ago
- File 0001-general-make-formdatas-visible-even-if-related-formd.patch 0001-general-make-formdatas-visible-even-if-related-formd.patch added
ok en modifiant le message. Et donc, oui, il faudrait aussi le patch côté auquotidien.
Updated by Thomas Noël almost 7 years ago
- File 0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch 0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch added
Et voici pour auquotidien (je dois regarder pour les tests, comment faire, reprendre ceux de wcs ?)
Updated by Frédéric Péters almost 7 years ago
Ouaip, pour les tests, partir de ce qui a été fait dans wcs ça doit aller.
Updated by Thomas Noël over 6 years ago
- File auquo-0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch auquo-0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch added
- File wcs-0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch wcs-0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch added
Les deux patches renommés, avec un test simple pour auquo.
Updated by Frédéric Péters over 6 years ago
Un peu plus haut je suggérais d'en changer l'intitulé, "general: make formdatas visible even if related formdefs are disabled (#11288)"; à part ça, ok.
Updated by Thomas Noël over 6 years ago
- Status changed from En cours to Résolu (à déployer)
commit f9be1ffdb8cad5c562a83c1797b7e69e94ce2974
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Tue Jun 14 11:20:53 2016 +0200
general: make formdatas visible even if related formdefs are disabled (#11288)
Updated by Thomas Noël over 6 years ago
Pour auquo j'ai oublié de faire un ticket séparé donc :
commit 71dc5cecd9911ad3b572927a65d651987f5dd47f
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Fri Jun 17 17:39:43 2016 +0200
general: make formdatas visible even if related formdefs are disabled (#11288)
general: make formdatas visible even if related formdefs are disabled (#11288)