Projet

Général

Profil

Development #53144

alpes-maritimes-2018: corriger le lien de création de compte posé manuellement

Ajouté par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans. Mis à jour il y a environ 3 ans.

Statut:
Fermé
Priorité:
Normal
Version cible:
-
Début:
15 avril 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Cf. #52502


Fichiers

Révisions associées

Révision 4b141865 (diff)
Ajouté par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans

alpes-maritimes-2018: fix manual registration link url (#53144)

Historique

#2

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans

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.

#3

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>
#4

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.

#5

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.

#6

Mis à jour par Serghei Mihai (congés, retour 15/05) il y a environ 3 ans

Il n'y aura pas plusieurs liens de création de compte sur la page.
Merci pour l'astuce.

#7

Mis à jour par Benjamin Dauvergne il y a environ 3 ans

  • Statut changé de Solution proposée à Solution validée

Ok.

#8

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)
#9

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

Formats disponibles : Atom PDF