Produits Entr'ouvert » Intégrations graphiques Publik

L'intérêt de ce messages-login me parait faible (mais peut-être que le thème minint est inaltérable) :

~/wd/eo/publik-base-theme$ git grep messages-login
static/minint/initial.css:#messages.messages-login {
static/minint/initial.css:#messages.messages-login li {
templates/authentic2/base.html:  <div id="messages" class="messages-login">
templates/fargo/base.html:  <div id="messages" class="messages-login">
templates/theme.html:  <div id="messages" class="messages-login">
templates/variants/minint/theme.html:  <div id="messages" class="messages-login">
templates/variants/montpellier/base.html:  <div id="messages" class="messages-login">

une fois retiré on doit pouvoir en mutualiser 11 pareil il me semble (on peut laisser le template du minint tranquille dans ce cas).

~/wd/eo/publik-base-theme$ git grep div.*id.*messages \*.html 
templates/authentic2/base.html:  <div id="messages" class="messages-login">
templates/fargo/base.html:  <div id="messages" class="messages-login">
templates/theme.html:  <div id="messages" class="messages-login">
templates/variants/grenoble-metropole-2019/theme.html:    <div id="messages">
templates/variants/minint/theme.html:  <div id="messages" class="messages-login">
templates/variants/montpellier/authentic2/base.html:     <div id="messages">
templates/variants/montpellier/base.html:  <div id="messages" class="messages-login">
templates/variants/montpellier/theme.html:            <div id="messages">
templates/variants/nanterre/combo/cells/resultat-paiement/text-cell.html:    <div id="messages">
templates/variants/strasbourg-2018/theme.html:      <div id="messages">
templates/variants/villeurbanne-2018/theme.html:    <div id="messages">

Pour le template de base, il est sensé être passé lors de la configuration des URLs de l'application :

combo/urls.py-    urlpatterns.append(
combo/urls.py-        url(
combo/urls.py:            r'^accounts/mellon/',
combo/urls.py:            include('mellon.urls'),
combo/urls.py-            kwargs={
combo/urls.py:                'template_base': 'combo/mellon_base_template.html', <-- là
combo/urls.py:                'context_hook': mellon_page_hook,
combo/urls.py-            },
combo/urls.py-        )
combo/urls.py-    )

coté A2 rien de spécifique semble-t-il :

src/authentic2_auth_saml/urls.py:    url(r'^accounts/saml/', include('mellon.urls'), kwargs={'template_base': 'authentic/base.html'})

Pour info Minint ce sera remplacé par une intégration graphique basée sur le système de design de l'état prochainement (échéance fin novembre pour la livraison).

Benjamin Dauvergne a écrit :

L'intérêt de ce messages-login me parait faible (mais peut-être que le thème minint est inaltérable) :

C'est copié collé depuis templates/authentic2/base.html, ça fait le café, j'ai très peu envie de toucher à des inté graphiques ici (mais s'il s'agit juste d'enlever la classe je peux le faire).

Pour le template de base, il est sensé être passé lors de la configuration des URLs de l'application :

Mais ça fonctionne parce que dans django-mellon, base.html commence par {% extends template_base|default:"base.html" %}. Ça n'a jamais fonctionné dans nos déploiement puisque le base.html qui est pris est celui de publik-base-theme et qu'il n'exploite pas template base ({% extends theme_base %} à la place).
Ce patch ça pourrait être

-{% extends theme_base %}
+{% extends template_base|default:theme_base %}

J'aime quand même bien mon patch qui ne ferait pas de mal à une mouche.

Je ne comprends pas les méandres de ce ticket, mais je sais que le problème qu'il cible est vraiment pénible (#57574: messages d'erreur au retour d'un sso qui s'affichent au mauvais moment)... et donc si quelqu'un qui a pigé peu valider, ça aidera.

Allez go je comprend rien à ce que vous faites, il y avait une solution acceptée qui était de dégager les mellon/base.html et de passer template_base pour éviter de surcharger ce template¹ partout, et là ça va rendre ça obligatoire finalement.

¹

$ ls -l */templates/mellon/base.html */*/templates/mellon/base.html */*/*/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 217  8 nov.   2018 authentic2-auth-fedict/src/authentic2_auth_fedict/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 232 25 mars   2021 authentic2/src/authentic2_auth_saml/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 232 25 mars   2021 authentic/src/authentic2_auth_saml/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 236  1 mai    2020 chrono/chrono/manager/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 228 27 juil. 11:48 django-mellon/mellon/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 227 24 mars   2020 fargo/fargo/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 198  9 janv.  2018 hobo/hobo/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 232 11 mars   2020 passerelle/passerelle/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 205 29 mars   2020 publik-base-theme/templates/mellon/base.html
-rw-r--r-- 1 bdauvergne bdauvergne 177 26 janv.  2018 welco/welco/templates/mellon/base.html

Normalement on devrait en avoir aucun si chaque brique met bien template_base comme combo :

bdauvergne@revestel:~/wd/eo/combo$ git grep -B2 -A4 include.*mellon
combo/urls.py-        url(
combo/urls.py-            r'^accounts/mellon/',
combo/urls.py:            include('mellon.urls'),
combo/urls.py-            kwargs={
combo/urls.py-                'template_base': 'combo/mellon_base_template.html',
combo/urls.py-                'context_hook': mellon_page_hook,
combo/urls.py-            },
bdauvergne@revestel:~/wd/eo/combo$ cd -
/home/bdauvergne/wd/eo/passerelle
bdauvergne@revestel:~/wd/eo/passerelle$ git grep include.*mellon
passerelle/urls.py:    urlpatterns += [url(r'^accounts/mellon/', include('mellon.urls'))]

Je trouve embêtant d'avoir un fonctionnement différent entre combo et tout le reste (surtout que le problème ne concerne qu'authentic ici a priori). Le bug dans authentic j'ai l'impression que c'est surtout le chemin qui est faux dans le code :

urlpatterns = [
    url(r'^accounts/saml/', include('mellon.urls'), kwargs={'template_base': 'authentic/base.html'})
]

J'ai ouvert #59519 qui n'est qu'une redite de #57574 mais je n'avais pas assez réfléchi sur celui-là je suppose. Ici il faudrait juste virer templates/mellon/base.html. Ensuite on peut passer tranquillement sur chaque brique pour ajouter template_base et virer mellon/base.html mais ça peut se faire sans précipitation.

Automatic expiration