Produits Entr'ouvert » w.c.s.

Frédéric Péters a écrit :

Ça ne devrait sans doute pas être enregistré (dans formdata.user_id) tant que le formulaire est en brouillon, pour éviter que l'usager en frontoffice ait un accès à la demande alors que l'agent est occupé à la saisir.

(je note ça ici) Il y a, au niveau de l'api, formdata.submission_context['user_id] qui pourrait devenir "officiel". Affichage pendant la saisie sur colonne de droite, et stockage dans formdata.user_id uniquement quand le statut n'est plus "draft".

(ceci n'est pas une review)

Je teste la branche en local pour essayer de comprendre #48354
A priori il manque l'inclusion de select2.js, j'ai des erreurs JS et un menu qui s'affiche mal sans ça

diff --git a/wcs/backoffice/root.py b/wcs/backoffice/root.py
index 71e1d5ef..d5f731ec 100644
--- a/wcs/backoffice/root.py
+++ b/wcs/backoffice/root.py
@@ -133,7 +133,7 @@ class RootDirectory(BackofficeRootDirectory):
     def _q_access(self):
         get_response().breadcrumb = [] # reinit, root the breadcrumb in the backoffice
         get_response().breadcrumb.append( ('backoffice/', _('Back Office')) )
-        get_response().add_javascript(['jquery.js', 'qommon.admin.js'])
+        get_response().add_javascript(['jquery.js', 'select2.js', 'qommon.admin.js'])
         req = get_request()

         if self.check_admin_for_all():

Tu as bien #47867 ? (il est dans la branche mais sait-on jamais). Tu as l'erreur sur quelle page ?

Je ne rencontre pas ça, tu as des infos dans la console javascript, pour voir ce qui appellerait select2 ?

Ok, j'ai ajouté un if ($('div.submit-user-selection').length) { devant; sans tester mais ça doit permettre de passer.

c'est mieux, merci :)

                if self.selected_user_id:
                    r += htmltext('<option value="%s">%s</option>') % (
                        self.selected_user_id,
                        get_publisher().user_class.get(self.selected_user_id)
                    )

Il me semble que get_publisher().user_class.get(self.selected_user_id) crashe si le user id n'existe pas

Ça ne devrait pas arriver vu qu'il vient d'être choisi mais sait-on jamais et branche modifiée pour se protéger du crash (ajout de ignore_errors=True), ça afficherait "None" ce qui est moyen mais comme je ne pense pas que cette situation puisse vraiment arriver, ça me semble suffisant.

J'ai un peu testé, regardé le diff, ça m'a l'air ok, mais je préfèrerais qu'un autre dev valide, genre Thomas qui a l'air de plus maîtriser le sujet :)

Ca sera quand même à revoir, parce que à terme welco est dédié uniquement au canal papier, les autres canaux (telephone, guichet) sont gérés via le portail agent combo directement -- mais c'est plutôt l'objet d'un ticket/projet à part sur ce sujet précis, parcours de saisie backoffice etc.

En pratique la variable welco_url n'est plus posée automatiquement, mais oui il y aura à un moment, genre si jamais welco se retrouve vraiment utilisé pour le courrier, à refaire un tour.

Fonctionnement on risque de heurter un mur par rapport aux homonymes (je ne compte plus le nombre de Thomas Noël à Massy). Il faudrait à mon avis afficher quelques détails (attributs) de l'usager choisi en dessous. Mais ça peut être un ticket à côté.

Oui j'y ai pensé surtout qu'on a un gabarit de rendu des usagers mais celui-ci est prévu pour fournir des bouts plus longs, avec de l'HTML et cie.

On aura un autre soucis qui est le pré-remplissage sur la première page, ça ne pourra pas fonctionner sur les attributs utilisateur. De là, je me demande si on ne devrait pas choisir l'usager avant même de rentrer dans le formulaire ? (...)

Oui le choix de l'utilisateur avant d'entrer est le parcours recommandé via le portail agent; ici il y avait justement la demande qu'il puisse être choisi après avoir entamé la saisie.