Bug #7225
misc mode squelette
0%
Fichiers
Révisions associées
skeleton: select correct menu item (#7225)
Historique
Mis à jour par Frédéric Péters il y a presque 9 ans
- Fichier 0001-skeleton-don-t-consider-access-rights-when-building-.patch 0001-skeleton-don-t-consider-access-rights-when-building-.patch ajouté
- Fichier 0002-skeleton-select-correct-menu-item-7225.patch 0002-skeleton-select-correct-menu-item-7225.patch ajouté
- Statut changé de Nouveau à En cours
- Assigné à mis à Frédéric Péters
- Patch proposed changé de Non à Oui
Pour le premier faudrait sans doute à terme pouvoir marquer des pages comme privées mais devant quand même apparaître dans les menus.
Mis à jour par Frédéric Péters il y a presque 9 ans
0001-skeleton-don-t-consider-access-rights-when-building-.patch
Dans une configuration classique le menu de navigation principal va contenir des pages demandant identification, mais celles-ci vont donc être zappées lors de la construction du menu dans le mode anonyme utilisé par le webservice, et on se retrouverait alors avec un menu vide ou presque; là-dessus une possibilité c'est de charger le menu par ajax (mais c'est un peu fragile et pour un élément essentiel comme un menu je n'aime pas l'idée); une autre possibilité que j'évoque dans le commentaire précédent, c'est de pouvoir marquer des pages comme devant apparaître dans les menus pour tout le monde (comme on peut dans wcs marquer des formulaires pour qu'ils apparaissent toujours).
Mais en attendant les réflexions là-dessus, la correction rapide c'était d'ignorer la gestion d'accès pour la génération de menu lors d'appel par webservice.
0002-skeleton-select-correct-menu-item-7225.patch
C'est une bête erreur dans le code qui boucle sur toutes les pages pour trouver celle qui matche (variable "selected_page") mais qui ensuite lors de la génération de la page déclare que la page courante est dans la variable "page", qui était la variable servant dans la boucle. Et du coup c'est toujours le dernier élément de menu qui apparait actif.
Mis à jour par Serghei Mihai il y a presque 9 ans
Frédéric Péters a écrit :
C'est une bête erreur dans le code qui boucle sur toutes les pages pour trouver celle qui matche (variable "selected_page") mais qui ensuite lors de la génération de la page déclare que la page courante est dans la variable "page", qui était la variable servant dans la boucle. Et du coup c'est toujours le dernier élément de menu qui apparait actif.
J'avais ce cas en modifiant le thème de Montpellier: la derniere page était toujours considérée comme active(la page "Aide").
Merci
Mis à jour par Thomas Noël il y a presque 9 ans
De fait, on a les deux soucis sur la recette Montpellier, et les deux patches me vont (y compris le raccourci dans le premier, bien pour l'istant).
Ack.
Mis à jour par Frédéric Péters il y a presque 9 ans
- Statut changé de En cours à Résolu (à déployer)
commit b8042003581f80eabd34657c585c437532b8e5ce Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue May 12 18:47:19 2015 +0200 skeleton: select correct menu item (#7225) commit 0b7b8f7884756bab90111ea45c13cf75cea547e4 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue May 12 17:52:36 2015 +0200 skeleton: don't consider access rights when building the menu (#7225)
Mis à jour par Frédéric Péters il y a plus de 8 ans
- Statut changé de Résolu (à déployer) à Solution déployée
skeleton: don't consider access rights when building the menu (#7225)