Development #27119
ne pas afficher le nom de l'agent traitant dans l'historique de la demande
0%
Description
Demandé dans #27052
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-villeurbanne-2018-don-t-show-agent-name-in-form-hist.patch 0001-villeurbanne-2018-don-t-show-agent-name-in-form-hist.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Par surcharge de template.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Nope, c'est identique à templates/variants/nanterre/wcs/front/formdata_history.html, faut faire autre chose que multiplier les copies pas maintenues. (surtout qu'ici, dès ce commit c'est en retard par rapport à wcs).
Je serais plutôt pour convenir d'une option (ça peut simplement être une variable posée via hobo, en documentant l'affaire) et la prendre en compte dans le template dans wcs.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Projet changé de Intégrations graphiques Publik à w.c.s.
- Sujet changé de villeurbanne-2018: ne pas afficher le nom de l'agent traitant dans l'historique de la demande à ne pas afficher le nom de l'agent traitant dans l'historique de la demande
- Statut changé de Solution proposée à En cours
En effet c'est mieux de pouvoir généraliser cela dans le template de wcs.
En revanche si on cache le nom dans l'historique pour l'usager il sera aussi caché dans le backoffice. Je pense qu'on devrait garder les noms des agents dans la vue de la demande depuis le backoffice.
On peut traiter cela en deux temps.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-templates-add-option-to-hide-receiver-from-form-hist.patch 0001-templates-add-option-to-hide-receiver-from-form-hist.patch ajouté
- Statut changé de En cours à Solution proposée
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à En cours
En revanche si on cache le nom dans l'historique pour l'usager il sera aussi caché dans le backoffice. Je pense qu'on devrait garder les noms des agents dans la vue de la demande depuis le backoffice.
On peut traiter cela en deux temps.
Le deuxième temps, c'est un patch en plus ici ?
Je pense que c'est nécessaire de continuer à afficher les noms en backoffice.
Par contre en front, l'option pour moi pourrait très bien être un total "ne pas afficher", sur l'idée qu'on arrive à distinguer graphiquement ce qui est fait par l'usager de ce qui est fait par un agent, sans avoir besoin de mention explicite.
{% if evolution.who %}
{% if evolution.who != '_submitter' and hide_receiver %}
{% else %}
quelque chose
{% endif %}
{% endif %}
Il doit être possible de construire ça d'une manière plus lisible.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-templates-hide-receiver-in-form-s-front-office-histo.patch 0001-templates-hide-receiver-in-form-s-front-office-histo.patch ajouté
- Statut changé de En cours à Solution proposée
Ça me va de ne pas afficher du tout le nom de l'agent traitant en front-office.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Ça me va de ne pas afficher du tout le nom de l'agent traitant en front-office.
Ce n'était pas mon propos, peu clair sans doute. Mon commentaire était que l'historique continuait à afficher le nom de l'usager, même quand le nom de l'agent n'était pas affiché, et que je trouvais qu'à ne pas afficher l'un on ne devrait pas non plus afficher l'autre.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-templates-hide-evolution-author-in-form-s-front-offi.patch 0001-templates-hide-evolution-author-in-form-s-front-offi.patch ajouté
Je suis d'accord.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Nope, plutôt que cet empilage de {% if dans le template, que choisir d'afficher ou pas soit dans le code Python. Que dans le template on n'ait plus que {% if evolution.who and display_authors %}, genre.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-templates-allow-evolution-s-author-hide-in-form-s-fr.patch 0001-templates-allow-evolution-s-author-hide-in-form-s-fr.patch ajouté
Ok.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Il y a un changement inutile d'indentation. display_authorS.
(+ le message de commit ressemble à une soupe de mots).
proposition : misc: add option to exclude user names from frontoffice form history (#27119).
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-misc-add-option-to-exclude-user-names-from-frontoffi.patch 0001-misc-add-option-to-exclude-user-names-from-frontoffi.patch ajouté
Ok, hop.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à En cours
J'étais sur le point de dire oui, quand, raté dans le changement précédent, ça passe de
get_site_option('hide_author_in_form_history', 'variables')à
has_site_option('hide_author_in_form_history')Ce qui fait, de manière importante, que ce n'est plus contrôlable via l'UI (en posant une variable dans Hobo) ou via l'intégration graphique (en posant ça dans les variables de l'intégration).
Donc, non, et tant qu'à être à refaire une passe, j'éviterais l'inversion de sens entre configuration et gabarit (hide vs display), je propose ainsi d'avoir include_authors_in_form_history (et include_authors en version raccourcie pour le gabarit).
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-misc-add-option-to-exclude-user-names-from-frontoffi.patch 0001-misc-add-option-to-exclude-user-names-from-frontoffi.patch ajouté
- Statut changé de En cours à Solution proposée
Ok, mais on la variable attendue de Hobo serait bien "hide_authors_..." pour expliciter son effet.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Ok, mais on la variable attendue de Hobo serait bien "hide_authors_..." pour expliciter son effet.
Tu ne peux pas dire "ok" et puis ne pas faire du tout ce que je demande (i.e. pas d'inversion de sens).
Dans Hobo, il sera mis include_authors_in_form_history à false, et ça dira les choses correctement et d'une seule manière.
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Fichier 0001-misc-add-option-to-exclude-user-names-from-frontoffi.patch 0001-misc-add-option-to-exclude-user-names-from-frontoffi.patch ajouté
Je me suis un peu mêlé les pinceaux.
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
ok
Mis à jour par Serghei Mihai il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 82e6d51897615ff7999992ea5e6d4301232306f4 (origin/master, origin/HEAD)
Author: Serghei Mihai <smihai@entrouvert.com>
Date: Wed Oct 24 12:10:14 2018 +0200
misc: add option to exclude user names from frontoffice form history (#27119)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: add option to exclude user names from frontoffice form history (#27119)