Development #53144
alpes-maritimes-2018: corriger le lien de création de compte posé manuellement
0%
Description
Cf. #52502
Fichiers
Révisions associées
Historique
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans
- Fichier 0001-alpes-maritimes-2018-fix-manual-registration-link-ur.patch 0001-alpes-maritimes-2018-fix-manual-registration-link-ur.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Exploitation du bon lien généré par authentic (via #53070).
Il faudrait juste donner un id "manual-registration-link" au lien posé dans Combo.
Mis à jour par Thomas Jund il y a environ 3 ans
Je vois plusieurs défauts à ton script :
- Trop de requêtes DOM.
- Des conditions complexes.
- être lancé sur toutes les pages.
- une boucle sur un #id (unique par essence)
Si tu n'as pas d'autres choix que de récupérer l'url via JS pour l'utiliser sur un lien, voilà ce que je ferais :
Dans 'user-info.html' j'ajouterais une fonction JS :
<script> function goto-registration(e) { e.preventDefault(); window.location = '{{registration_url}}' } </script>
Comme ça pas de requête vers le DOM.
et dans la cellule combo appeler la function avec un attr `onclik`
<a class="tip-button" href="{{idp_registration_url}}" onclick="goto-registration(event)" id="manual-registration-link">CREER VOTRE PROPRE COMPTE</a>
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans
- Statut changé de Solution proposée à En cours
J'aimerais éviter de taper du HTML qui n'est pas pris en compte par CKEditor, typiquement: onclick="goto-registration(event)"
.
Pour éviter d'éxecuter le script partout je peux l'ajouter dans login.html
surchargé du thème.
Mis à jour par Thomas Jund il y a environ 3 ans
J'aimerais éviter de taper du HTML qui n'est pas pris en compte par CKEditor.
ok, conserve alors le script dans `user-info` (qui évite de chercher via le dom le lien de connexion)
et exécute la fonction au click sur "#manual-registration-link" depuis l'endroit qui te parait le plus opportun.
Si sur un id, inutile de boucler.
const goto_registration_link = document.getElementById('manual-registration-link'); if (goto_registration_link) $(goto_registration_link).click(goto-registration);
Et si tu souhaites leur laisser la possibilité de mettre plusieurs liens sur une page, utilise une class.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans
- Fichier 0001-alpes-maritimes-2018-fix-manual-registration-link-ur.patch 0001-alpes-maritimes-2018-fix-manual-registration-link-ur.patch ajouté
- Statut changé de En cours à Solution proposée
Il n'y aura pas plusieurs liens de création de compte sur la page.
Merci pour l'astuce.
Mis à jour par Benjamin Dauvergne il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Ok.
Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 4b141865ec469da170d5865e0d589f605d864dd2 (origin/main) Author: Serghei Mihai <smihai@entrouvert.com> Date: Tue Apr 13 17:28:00 2021 +0200 alpes-maritimes-2018: fix manual registration link url (#53144)
Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
alpes-maritimes-2018: fix manual registration link url (#53144)