Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Propriété d'une nuvelle fiche : confusion avec la page de création #1092

Open
J9rem opened this issue Jul 26, 2023 · 2 comments
Open

Propriété d'une nuvelle fiche : confusion avec la page de création #1092

J9rem opened this issue Jul 26, 2023 · 2 comments

Comments

@J9rem
Copy link
Contributor

J9rem commented Jul 26, 2023

Type of issue (keep only one) / Type de demande (ne garder qu'une ligne)
Bug / Bogue

Description
Après de la création d'un nouvelle fiche, la fiche est affichée dans la page de création. La barre d'édition (en particuliers pour les administrateurs) indique le propriétaire de la page de création et non celle de la fiche.
Dans le cas de la création d'un nouvelle utilisateur avec la fiche (utilisateur_wikini), le propriétaire de la fiche est ce nouvel utilisateur mais ça ne se voit pas.

Demande:

  • ping @EdmondAgate
  • faire en sorte que l'affichage soit plus explicite et permette d'indiquer à quelle page il fait référence quand on parle de propriétaire

Additionnal informations / Informations complémentaires

  • version of YesWiki / version de YesWiki : à partir de doryphore 4.4.0 (mais c'était présent auparavant aussi)
  • screenshot / capture d’écran
    • vue après création d'une fiche en tant qu'admin, confusion possible vue _apres_creation
    • barre d'édition sur la fiche concernée : tout est bon
      image
@acheype
Copy link
Contributor

acheype commented Aug 3, 2023

Cette confusion vient du fait qu'après la création, on redirige non pas vers la page mais vers /?BazaR avec certains paramètres. Je pense que ce choix à la base était motivé par le fait que c'était une façon de pouvoir afficher un message de confirmation de la création de la fiche. Je n'y vois pas une autre utilité et en effet, ce comportement apporte de la confusion et un flow de navigation pas du tout ergonomique. Cela entraîne également des soucis pour l'extension LMS, notamment le fait de perdre complètement la navigation suite à la création d'une fiche (le menu de gauche ne pouvant s'afficher correctement).

Une solution qui réglerait ces soucis serait de rediriger directement sur la fiche après sa création et que le message soit géré par le « toaster » qui en effet permet d'afficher un message à la page suivante.
Une autre solution est que le handler show puisse traiter un paramètre GET comme celui actuellement traité par la page BazaR pour afficher le message de confirmation.

Et s'il on veut d'aller au bout de cette amélioration, ce serait aussi utile que les paramètres GET avec lequels on a appelé la page à éditer soit transmis à la page d'édition (/edit) pour que la redirection s'effectue ensuite avec ces derniers. Cela permettrait à de garder complètement le flow de navigation avec l'extension LMS, et j'imagine que ça peut servir également pour d'autres cas.

@J9rem
Copy link
Contributor Author

J9rem commented Aug 3, 2023

Effectivement, je pense que ta solution @acheype est bien plus logique :

  • revenir sur la fiche après création ou modification (sauf si utilisation de incomingurl
  • ajouter un message grâce à la commande flash permettant d'indiquer le message de création de fiche (ou via GET) ou les deux (genre stockage du message dans $SESSION avec une clé unique et utilisation du paramètre GET pour en déclencher l'affichage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants