Bug #25514
rouen : style du bouton de connexion pas respecté sur toutes les pages
0%
Description
Sur la page d'accueil c'est bon, et puis ici https://demarches-rouen.test.entrouvert.org/10-premiers-formulaires/carte-d-acces-en-vehicule-cimetiere/
ça ne marche plus.
Fichiers
Historique
Mis à jour par Emmanuel Cazenave il y a presque 6 ans
- Fichier 0001-rouen-adapt-user-info-zone-on-wcs-25514.patch 0001-rouen-adapt-user-info-zone-on-wcs-25514.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Une duplication de plus, what else ?
Mis à jour par Frédéric Péters il y a presque 6 ans
Le bon moyen ce serait de retirer la redéfinition du bloc user-info du templates/wcs/base.html, laisser le bloc user-info partagé par combo être rendu côté w.c.s.; il y a déjà eu pas mal d'adaptations à l'objet User côté w.c.s. pour fournir les mêmes propriétés/méthodes que le modèle Django, ça pourrait bien déjà fonctionner.
Alternativement pour juste tenter ça côté rouen, simplement faire {% block user-info }{{ block.super }}{ endblock %} dans le templates/variants/rouen/wcs/base.html pourrait être ok.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Alternativement pour juste tenter ça côté rouen, simplement faire {% block user-info }{{ block.super }}{ endblock %} dans le templates/variants/rouen/wcs/base.html pourrait être ok.
Ça ne marche pas à cause des variables disponibles dans le contexte d'évaluation du template il me semble : dans le template combo il y a un {% if user.is_authenticated %} qui échoue quand il est évalué coté wcs.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Bon ça marche pas mais je trompe dans l'analyse, je poursuis.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Dans templates/wcs/base.html je retire le bloc user-info; je restart memcached pour être sûr, je vais sur wcs en n'étant pas connecté il prend bien le bloc non connecté, je me connecte il prend bien le bloc connecté. La seule différence dans le résultat devient l'adresse du lien "inscription" (/register/ en "natif" wcs, https://.../accounts/register/ en "natif" combo, mais dans la pratique ça ne change rien).
Bien sûr là ça se met à toucher tout le monde.
J'ai regardé pour ma deuxième phrase, templates/variants/rouen/wcs/base.html, mais ce template n'existe en fait actuellement pas et il n'y a pas de possibilité d'un {% extends ../../wcs/base.html %} comme on le fait avec les templates combo. On se trouve donc à devoir tout copier comme dans ton patch mais alors, quand même, le bloc user-info il peut être simplement supprimé, pour exploiter celui partagé par combo.
Pour reprendre, ma solution préférée c'est dégager le user-info de templates/wcs/base.html mais vu que ça va toucher tout le monde ça ne peut clairement pas être fait aujourd'hui.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Pour reprendre, ma solution préférée c'est dégager le user-info de templates/wcs/base.html mais vu que ça va toucher tout le monde ça ne peut clairement pas être fait aujourd'hui.
→ sur ce point, #25616.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Fichier Screenshot-2018-8-9 Compte Citoyen - Démarches - Carte d_accès en véhicule cimetière.png Screenshot-2018-8-9 Compte Citoyen - Démarches - Carte d_accès en véhicule cimetière.png ajouté
J'applique ton patch #25616, je supprime templates/variants/rouen/wcs/base.html
et je me retrouve, lorsque je suis non connecté, sur une démarche, avec la capture ci-jointe, qui n'est pas le résultat que j'attends.
Un coup de pdb dans, wcs/compat.py::render_template, et je constate que 'idp_registration_url' n'est pas dans le contexte d'évaluation du template, et que donc /templates/variants/rouen/combo/page_template.html
ne peut pas me donner le résultat attendu.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Parce que tu n'a pas rejoué hobo-deploy depuis #25116 ?
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution proposée à Rejeté
Yes, ça marche, merci.
Et donc plus rien à faire ici, juste attendre le déploiement de #25616.