Bug #26277
jquery inclut 2 fois dans le template de login
0%
Description
La page de connexion ajoute jquery.js dans les médias, or il est déjà par l'intégration dans publik-base-theme.
On se retrouve donc avec:
<script src="https://compte-orleans.test.entrouvert.org/static/xstatic/jquery.js"></script> <link href="/static/authentic2/css/password.css" type="text/css" media="all" rel="stylesheet" /> <script type="text/javascript" src="/static/xstatic/jquery.min.js"></script> <script type="text/javascript" src="/static/authentic2/js/password.js"></script> <script type="text/javascript" src="/static/authentic2/js/js_seconds_until.js"></script>
sur la page de connexion.
Dans le cas d'une intégration personnalisée, avec un template qui inclut plein de plugins, genre jquery-ui, jquery-colorbox, ou autres, cela provoque des erreurs du genre:
TypeError: $(...).colorbox is not a function
Historique
Mis à jour par Serghei Mihai il y a plus de 5 ans
Authentic en standalone depend de gadjo, qui inclut déjà jquery et jquery-ui.
Je pense qu'il n'y a pas besoin de redéclarer jquery dans les media d'un formulaire, mais uniquement les JS qui font vraiment nécessaires pour les vérifications en front.
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Serghei Mihai a écrit :
Authentic en standalone depend de gadjo, qui inclut déjà jquery et jquery-ui.
Je pense qu'il n'y a pas besoin de redéclarer jquery dans les media d'un formulaire, mais uniquement les JS qui font vraiment nécessaires pour les vérifications en front.
Oui mais non il faut tout déclarer, je n'ai pas envie de dépendre des intégrations au niveau du code, le jour où on arrête jquery dans les intégrations on a tout qui pette (et le template de base de gadjo n'est pas utilisé dans publik-base-theme, il se trouve juste que c'est importé aussi dans combo/page-template.html, mais pas jquery-ui par exemple, dont on ne se sert pas pour l'instant). gadjo c'est uniquement dans le thème de base qu'on utilise quasiment jamais, sauf chez les clients qui utilisent a2 en standalone (PSL, Formiris, Vinci&co).
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Pour l'instant pour moi ça reste une fausse bonne idée qui ne corrige pas le fond du problème.
Mis à jour par Serghei Mihai il y a plus de 5 ans
Je suis d'accord.
Je vais chercher une solution du côté du thème.
Mis à jour par Frédéric Péters il y a presque 3 ans
- Planning mis à Non
On pourrait revenir là-dessus ? Ça fait trois ans et encore aujourd'hui ça m'a emmerdé (#55469). Qu'on supprime les déclarations de jquery des classes Media, l'affaire étant posée que jquery est disponible.