Project

General

Profile

Bug #11288

l'api user/forms ne montre pas les demandes en cours sur des formulaires désactivés

Added by Thomas Noël over 6 years ago. Updated almost 6 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Target version:
Start date:
10 June 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Description

cf #11281

pour moi il faut montrer les demandes, car l'usager peut quand même y accéder.


Files

Associated revisions

Revision f9be1ffd (diff)
Added by Thomas Noël over 6 years ago

general: make formdatas visible even if related formdefs are disabled (#11288)

Revision 71dc5cec (diff)
Added by Thomas Noël over 6 years ago

general: make formdatas visible even if related formdefs are disabled (#11288)

History

#2

Updated by Thomas Noël over 6 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, …).

#3

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

Oui, et ok pour un patch qui ferait le nécessaire ailleurs, bien sûr. (et tests etc.)

#4

Updated by Thomas Noël over 6 years ago

  • Assignee set to Thomas Noël
#5

Updated by Thomas Noël over 6 years ago

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)

#6

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

ok en modifiant le message. Et donc, oui, il faudrait aussi le patch côté auquotidien.

#7

Updated by Thomas Noël over 6 years ago

Et voici pour auquotidien (je dois regarder pour les tests, comment faire, reprendre ceux de wcs ?)

#8

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

Ouaip, pour les tests, partir de ce qui a été fait dans wcs ça doit aller.

#9

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

  • Status changed from Nouveau to En cours
#11

Updated by Frédéric Péters (de retour le 10/10) 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.

#12

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)

#13

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)

#14

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

  • Target version set to v1.47
#15

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