Je pensai au cas ou la librairie était chargée sans passer par 'combine_script', mais c'est peut-être un faux problème.
cljosse a écrit:
Si les scripts "packed" font parti des scripts combinés oui.
Je ne comprend pas le problème...
Si les scripts "packed" font parti des scripts combinés oui.
cljosse a écrit:
Il me semble que c'est en tête de chaque script combiné et non en fin, à la suite d'un script packed.
C'est du moins ce qui m'avait semblé quand j'avais été confronté à ce problème.
Rajouter le point virgule au début ou à la fin de chaque script combiné revient exactement au même, non?
Bonjour.
Il me semble que c'est en tête de chaque script combiné et non en fin, à la suite d'un script packed.
C'est du moins ce qui m'avait semblé quand j'avais été confronté à ce problème.
A+
grum a écrit:
Faut juste avoir en tête que si quelqu'un réalise un plugin avec des librairies packée (quelque soit le nom du fichier, .min.js ou .truc.js), si le fichier ne finit pas par un point-virgule y a un fort potentiel pour que çà bloque tout.
C'est corrigé: [Subversion] r9606
Un point virgule est systématiquement ajouté à la fin de chaque script combiné s'il n'y est pas.
Bon, si on rajoute automatiquement un ; à la fin de chaque script combiné, il ne devrait plus y avoir de soucis...
pas réessayé avec la dernière RC, mais le fichier se charge parfaitement.
C'est juste qu'après une instruction eval(), tout code js merdoie.
J'ai trouvé la solution pour éviter d'intervenir sur piwigo : je modifie le fichier packé, je rajoute le point-virgule à la mano à la main à la fin du fichier, et hop c'est bon !
Faut juste avoir en tête que si quelqu'un réalise un plugin avec des librairies packée (quelque soit le nom du fichier, .min.js ou .truc.js), si le fichier ne finit pas par un point-virgule y a un fort potentiel pour que çà bloque tout.
grum a écrit:
P@t a écrit:
Est-ce que tes librairies sont bien nommées xxx.packed.js?
non.
c'est xxx.pack.js
si je met packed çà va fonctionner ?
Je ne sais pas si ca va fonctionner, mais en tout cas, combine_script minimise tous les fichiers javascripts, excepté ceux dont le nom comporte .min ou .packed
Peut-etre qu'il essait de minimiser ton fichier packed, et ca doit donner un résultat improbable.
Autre piste: est-ce que ton fichier js appelle d'autres fichiers js? Si c'est le cas, c'est mort pour utiliser le combine_script.
P@t a écrit:
Est-ce que tes librairies sont bien nommées xxx.packed.js?
non.
c'est xxx.pack.js
si je met packed çà va fonctionner ?
grum a écrit:
Je reviens sur le sujet.
Certes, çà fonctionne si on n'utilise pas de packed.
Néanmoins, pour ma part j'utilise certaines librairies qui ne sont disponibles que sous forme de js packé.
Est-ce que tes librairies sont bien nommées xxx.packed.js?
grum a écrit:
Bref, pour moi, le problème n'est pas vraiment résolu.
"Eh ! oui" (Othello, ac. 2, sc. 3)
Je reviens sur le sujet.
Certes, çà fonctionne si on n'utilise pas de packed.
Néanmoins, pour ma part j'utilise certaines librairies qui ne sont disponibles que sous forme de js packé.
A prévoir :
- soit des incompatibilités lors de l'utilisation des librairies
- soit devoir désactiver le mode combiné
- soit changer de librairie (mais dans ce cas, ayant déjà eu du mal à trouver celle que j'utilise actuellement, je n'ai aucune idée si je serais à même de migrer les plugins...)
Bref, pour moi, le problème n'est pas vraiment résolu.
jQuery et jQuery UI ont été mis à jour.
[Subversion] r9172
Il n'y a plus aucun fichier packed.
Pour info, il y a la discussion qui va bien sur le forum EN : http://piwigo.org/forum/viewtopic.php?id=16802