Project

General

Profile

Bug #26277

jquery inclut 2 fois dans le template de login

Added by Serghei Mihai over 1 year ago. Updated over 1 year ago.

Status:
Nouveau
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
10 Sep 2018
Due date:
% Done:

0%

Patch proposed:
No
Planning:
No

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

History

#3 Updated by Serghei Mihai over 1 year ago

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.

#4 Updated by Benjamin Dauvergne over 1 year ago

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).

#5 Updated by Benjamin Dauvergne over 1 year ago

Pour l'instant pour moi ça reste une fausse bonne idée qui ne corrige pas le fond du problème.

#6 Updated by Serghei Mihai over 1 year ago

Je suis d'accord.

Je vais chercher une solution du côté du thème.

Also available in: Atom PDF