Produits Entr'ouvert » Authentic 2

Peut-être un nom plus évocateur que block-name pour ce nouveau bloc ?
Aussi, si je comprends la remarque de Frédéric dans le ticket lié #65803 côté base-theme, il y a déjà des intégrations où ce nom est déjà défini dans un gabarit custom, et où ce patch apporterait un doublon du titre, c’est bien ça ? (auquel cas il faudrait passer ça en même temps qu’un dédoublonnage des titres dans chacune des intégrations concernées.)

Paul Marillonnet a écrit :

Peut-être un nom plus évocateur que block-name pour ce nouveau bloc ?

(Et je réalise trop tard qu’il y a un second patche où ce nom de bloc diffère. Cependant la remarque s’applique toujours, je crois. Peut-être un login-block-title ou login-block-name ?)

Serghei Mihai a écrit :

Ok, go pour login-block-name, pour suivre le nom de la variable.

Pour ma part oui ça me va très bien, mais est-ce qu’on a la liste des intégrations pour lesquelles ce patch pourrait poser problème (je ne suis pas sûr d’avoir compris le souci, mes interrogations sont restées sans réponse ici (#53264-6)).

Pour ma part oui ça me va très bien, mais est-ce qu’on a la liste des intégrations pour lesquelles ce patch pourrait poser problème (je ne suis pas sûr d’avoir compris le souci, mes interrogations sont restées sans réponse ici (#53264-6)).

Je n'ai pas relu ni testé, il faut par exemple regarder imio, qui fait :

<div id="Mot_de_passe">
<h2>Connexion avec <span>identifiant</span></h2>
{{blocks.password.content|safe}}
</div>

J'ai collé un patch pourri, désolé.
Je rebase et colle un truc propre.

C'est nécessaire d'avoir ce block_title dans le contexte ? Au gré de l'évolution du code ces paramètres ultra spécifiques au nom vague ne sont jamais nettoyés, ensuite on se retrouve à git grep partout pour savoir si on peut les virer et au final on ne peut que croiser les doigts.

Vu l'usage qui en est fait dans le patch, je serais plutôt pour (par ordre de préférence) 1/ hardcoder la chaîne « FranceConnect » dans le template 2/ utiliser {{ authenticator }} qui produit cette chaîne 3/ utiliser direct {{ authenticator._meta.verbose_name }}.

J'ai fait un micmac dans le nom de la branche. C'est corrigé et voici le dernier patch.

Je bloque là dessus

        <div class="block block-{% firstof login_block.authenticator.type login_block.authenticator.id %} block-{{id}}">
          {% if blocks|length == 1 %}
            <h2>Connexion</h2>
          {% endif %}
          <div {% if login_block.extra_css_class %} class="{{ login_block.extra_css_class }}"{% endif %}>
            {{ login_block.content|safe }}
          </div>
        </div>

Ça produit une structure de balises différentes :

Si 1 seul block de connexion

.block
  h2 Default title
  div
    block.content

Si 2 blocks ou plus avec titre personnalisé :

.block
  div
    h2 Custom title
    block.content

J'ai l'impression que la div {{ login_block.extra_css_class }}` n'est pas vraiment utile.
On pourrait faire sauter la div à cet endroit et bouger la class sur la div du block,

      {% for id, login_block in blocks.items %}
        <div class="block block-{% firstof login_block.authenticator.type login_block.authenticator.id %} block-{{id}} {% if login_block.extra_css_class %}{{ login_block.extra_css_class }}">
          {% if blocks|length == 1 %}
            <h2>Connexion</h2>
          {% endif %}
          {{ login_block.content|safe }}
        </div>
      {% endfor %}

Et ajouter cette div au sein des block.content :

{% block content||login %}
  <div>
    // code content
  </div>
{% endblock %}

et y'aura plus qu'à ajouter la class .cell--body à ces div ensuite :) via #70451

J'ai poussé un commit dans ce sens sur les 2 dépots

Tu as raison.
J'ai corrigé l'indentation et permis une petite correction dans le message de commit.

Avec ça Jenkins doit être content.

Super je vais maintenant regarder les thèmes #65803

J'ai aussi poussé des modifs des templates d'inscription pour respecter le même DOM.

J'ai ajouté un block titre au template saml login (besoin pour Roanne)

Côté a2 on est bons. C'est dans les thèmes qu'il reste du taf.

Automatic expiration