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.
Fichiers
Révisions associées
general: make formdatas visible even if related formdefs are disabled (#11288)
Historique
Mis à jour par Thomas Noël il y a presque 8 ans
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, …).
Mis à jour par Frédéric Péters il y a presque 8 ans
Oui, et ok pour un patch qui ferait le nécessaire ailleurs, bien sûr. (et tests etc.)
Mis à jour par Thomas Noël il y a presque 8 ans
- Fichier 0001-formdatas-are-visible-even-if-related-formdefs-are-d.patch 0001-formdatas-are-visible-even-if-related-formdefs-are-d.patch ajouté
- Patch proposed changé de Non à Oui
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)
Mis à jour par Frédéric Péters il y a presque 8 ans
- Fichier 0001-general-make-formdatas-visible-even-if-related-formd.patch 0001-general-make-formdatas-visible-even-if-related-formd.patch ajouté
ok en modifiant le message. Et donc, oui, il faudrait aussi le patch côté auquotidien.
Mis à jour par Thomas Noël il y a presque 8 ans
- Fichier 0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch 0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch ajouté
Et voici pour auquotidien
(je dois regarder pour les tests, comment faire, reprendre ceux de wcs ?)
Mis à jour par Frédéric Péters il y a presque 8 ans
Ouaip, pour les tests, partir de ce qui a été fait dans wcs ça doit aller.
Mis à jour par Thomas Noël il y a presque 8 ans
- Fichier auquo-0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch auquo-0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch ajouté
- Fichier wcs-0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch wcs-0001-consider-all-user-forms-even-if-formdef-if-disabled-.patch ajouté
Les deux patches renommés, avec un test simple pour auquo.
Mis à jour par Frédéric Péters il y a presque 8 ans
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.
Mis à jour par Thomas Noël il y a presque 8 ans
- Statut changé de En cours à 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)
Mis à jour par Thomas Noël il y a presque 8 ans
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)
Mis à jour par Frédéric Péters il y a plus de 7 ans
- Statut changé de Résolu (à déployer) à Fermé
general: make formdatas visible even if related formdefs are disabled (#11288)