Development #65533
l'"authenticator id" du bloc mot de passe a changé
0%
Description
Il est utilisé dans les intégrations graphiques,
{% for id, login_block in blocks.items %}
<div class="block block-{{login_block.authenticator.id}} block-{{id}}">
Et on y obtient désormais block-password_1 (ou 2 ou 3 etc.), plutôt que block-password.
Ça fait que des sélecteurs comme
.block-password input[type="text"], .block-password input[type="password"]
ne s'appliquent désormais plus, avec des conséquences sur le rendu.
Ça éclate encore plus sévèrement les intégrations imio qui font :
<h2>Connexion avec <span>identifiant</span></h2>
{{blocks.password.content|safe}}
</div>
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Fichier 0001-authenticators-use-only-type-as-identifier-in-case-o.patch 0001-authenticators-use-only-type-as-identifier-in-case-o.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ce qui a été zappé c'est que l'identifiant du block était suffixé seulement en cas d'instances multiples :
# check if the authenticator has multiple instances
if hasattr(authenticator, 'instances'):
[...]
block['id'] = '%s_%s' % (block['id'], instance_id)
Mini patch pour reproduire ce comportement avec les authenticators migrés.
Mis à jour par Serghei Mihai il y a presque 2 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Thomas Noël il y a presque 2 ans
Le «self.internal» ça permet de savoir qu'une instance est unique ? (je suis un peu perdu dans le code..)
Mis à jour par Valentin Deniaud il y a presque 2 ans
Thomas Noël a écrit :
Le «self.internal» ça permet de savoir qu'une instance est unique ? (je suis un peu perdu dans le code..)
Oui dans le contexte actuel, ça veut dire pas le droit d'ajouter ni de supprimer. Quand on ajoutera franceconnect, il sera explosé en deux flags et ça sera plus explicite : self.unique (pas le droit d'en avoir plus d'un) et self.protected (pas le droit de supprimer).
Mis à jour par Valentin Deniaud il y a presque 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 9c8b58d782a267b491eb418082ef5d5f1d34a6c7
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Mon May 23 09:30:09 2022 +0200
authenticators: use only type as identifier in case of single instance (#65533)Mis à jour par Transition automatique il y a presque 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
authenticators: use only type as identifier in case of single instance (#65533)