13.6.21.lasares.RC5, bogue sur un ficheir js

Progress0%
WorkflowSupport request
StatusProblem has been recorded

Bonjour,

A priori aprés la mise à jour depuis une version RC4,
sélection d'onglets , ou de listes déroulantes impossible.

il semble que le problème soit dans un fichier js  de:

../included/browser/library_js_endpage.min.js

vers la ligne 66 (pas évident le js)

uniqueId:function(){return this.each(function(){this.id||(this.id="ui-id-"+a)})}      sur +a?

 

 


Alexis Raimbault
on Aug. 20 2013 at 07:54 am GMT

patch_jsmin.zip

[this file has been deleted]

mise à jour du patch car le fichier jsmin ne contenait pas le correctif !




Alexis Raimbault webmaster free-lance

Alexis Raimbault
on Jul. 17 2013 at 09:38 am GMT

voilà un patch pour corriger la syntaxe +++a, qui était effectivement dû à une double minification d'un fichier js (celui de jquery-ui).
le fichier tools/jsmin.php est donc corrigé pour ne pas re-minifier les fichiers qui le sont déjà, en se basant sur la précence de .min. dans leur noms.

Ce sujet semble néanmoins rapporter des problèmes de différentes natures. Possible qu'il y ait encore des choses à corriger.

patch-jsmin.zip

[this file has been deleted]




Alexis Raimbault webmaster free-lance
Gérard - on Jul. 18 2013 at 04:04 pm GMT

Salut,
Je confirme qu'il y a encore des problèmes.
J'ai une version RC5 (nouvellement mise à jour, une nouvelle fois) et j'ai appliqué ce patch.
Je me retrouve avec le même problème de non fonctionnement de (i) la sélection des onglets, (ii) liste déroulante impossible, (iii) layout accordéon inopérant, (iv) visualisation impossible des "index de contenu".
Après avoir essayer plusieurs solutions (cf. ci-dessous), je ne sais plus à quel saint me vouer, ou plutôt à quelle version de Yacs me fier !
Mais le soleil brille et les vacances sont là, du coup mon site peut bien jouer les fainéants.
A+ et merci à Alexis pour sa fidélité et sa pugnacité.
Gérard

dobliu - on Jul. 18 2013 at 09:28 pm GMT
Merci Alexis pour le patch, à Gérard de l'avoir testé , mais je reste comme ça, les js en RC4 , (pour moi le mode verbose marche pas en RC4), Il faut peut être mettre en cohérence de version les fichiers dans le répertoire included/brower/css/redmond, notamment le jqery.css . Les vacances venant, j'essaierai de faire plus d'essais.
Alexis Raimbault - on Jul. 19 2013 at 06:56 am GMT
Gérard, je suis allé voir ton site (polar-harboiled) et tu as plusieurs erreurs javascript. Sachant que à la moindre erreur, c'est tous les fonctionnements javascript du fichier chargé qui sont en berne.
- première erreur il ne trouve pas le fichier http://polar-hardboiled.info/s...r-2.6.2.min.js. Vérifie le chemin.
- deuxième erreur, celle corrigée par le patch ci-dessus, l'erreur de syntaxe "+++a" est toujours présente. Il faut mettre à jour /included/browser/library_js_endpage.min.js tel que je l'ai donné dans le patch OU refaire une minification avec tools/jsmin.php (après avoir patché ce dernier).


Alexis Raimbault webmaster free-lance
Alexis Raimbault - on Jul. 19 2013 at 07:02 am GMT
Dobliu, les fichier css de jquery-ui sont en cohérence avec la version de jquery-ui 1.10.3, livrée avec RC5.
RC4 avait une version antérieure, donc forcément en prenant les libraries de RC4, c'est incohérent.


Alexis Raimbault webmaster free-lance
Alexis Raimbault - on Jul. 19 2013 at 07:03 am GMT
En tout cas merci à vous deux pour vos retours, et courage, ça va marcher, ça marche bien ici sur Yacs.fr non ?


Alexis Raimbault webmaster free-lance
Gérard - on Jul. 19 2013 at 07:57 am GMT
Merci Alexis,
Mais même avec ta manip, cela ne marche toujours pas.
Y a t'il un moyen de repasser en RC4 ou une version qui marche ?
Alexis Raimbault - on Jul. 19 2013 at 10:24 am GMT
Je suis allé voir et ton site marche très bien, avec les js de RC5.
je suppose donc que tu as corrigé depuis ?


Alexis Raimbault webmaster free-lance
dobliu
on Jul. 9 2013 at 03:15 pm GMT

Aprés une recherche NET, il semble que le problème vienne du code :

(this.id="ui-id-"+ ++a ) donne (this.id="ui-id-" +++a) donc erreur

changer dans js_endpage par
(this.id="ui-id-"+ (++a))

ça c'est fait ...
 

Alexis Raimbault - on Jul. 9 2013 at 04:17 pm GMT
On dirait une erreur du à la minification.
par contre c'est pas vraiment une correction que de modifier à la main dans ce fichier minifié.

tu peux répondre à ma question au dessous ? le contenu de tes répértoire js_header et js_endpage ? merci.


Alexis Raimbault webmaster free-lance
dobliu - on Jul. 9 2013 at 09:39 pm GMT
Fichiers, c'est répondu plus bas
.
pour la minification, si ça bogue comment on fait faut corriger le minificateur? ( c'est dans YACS?)

A+
Alexis Raimbault - on Jul. 10 2013 at 07:24 am GMT
Une solution serait de ne pas minifier un fichier déjà minifié, comme c'est le cas des libs jquery et jquery-ui incluse dans yacs.
Par contre je dois vérifier si le pb est bien du au minificateur de yacs.

d'autre part yacs.fr est en RC5 ici, et on a pas tout ces problèmes.


Alexis Raimbault webmaster free-lance
dobliu - on Jul. 10 2013 at 10:22 am GMT
J'utilise le navigateur FIREFOX , pb rencontré sur W$$ et Linux , je n'ai pas essayé sur un autre navigateur dans la configuration RC5 non corrigée.
En second, si le fichier jquery-ui-1.10.3.custom.min.js est minifié 2 fois, je pense plus au problème du minificateur , il faut dire que l'écriture de code de 3+ , c'est pas une syntaxe pour éviter les bogues.
dobliu
on Jul. 8 2013 at 09:40 pm GMT

APrés avoir mis le mode verbose, j'ai  remis les 2 js de la RC4 et là pas possible ça marche pas non plus,  a à priori le mode verbose  doit  déclencher égalment une erreur js....
Bref aprés un moment de reflexion,  changement du fichier control.include.php pour mettre le mode debug sur N.

Miracle ça fonctionne ....

 

Alexis Raimbault - on Jul. 9 2013 at 04:04 pm GMT
D'accord, vous évitez ainsi d'avoir un site en erreur, mais cela ne corrige rien là : vous avez fait marche arrière.


Alexis Raimbault webmaster free-lance
dobliu
on Jul. 8 2013 at 09:14 pm GMT

Ok j'ai activé la verbosité et remis les 2 fichiers js de la RC5

Warning: Invalid argument supplied for foreach() in /mnt..//yacs/shared/js_css.php on line 83

sous Web developper :

Error: ReferenceError: $ is not defined
Source File: http://../yacs/shared/yacs.js
Line: 1770

+ d'autre warning

dobliu - on Jul. 9 2013 at 08:31 pm GMT
Ncluded/browser/js_header
il y a 3 fichiers
93637 juin 19 17:22 jquery-1.8.3.min.js
3297 oct. 31 2011 reflection.js
25560 déc. 28 2011 swfobject.js

included/browser/js_endpage
il y a 6 fichiers:
1008 sept. 20 2012 jquery.autogrow-textarea.js
2247 oct. 31 2011 jquery.json.min.js
15264 août 21 2012 jquery.livetwitter.js
5467 sept. 20 2012 jquery.masonry.min.js
7715 févr. 2 2012 jquery.tipsy.js
102110 juin 19 17:06 jquery-ui-1.10.3.custom.min.js
Alexis Raimbault - on Jul. 10 2013 at 07:30 am GMT
Merci Dobliu, niveau fichiers c'est correct.

Avec cette configuration, tu confirmes que en mode verbose tu as un warning sur shared/js_css ligne 83 ?
si tu affiche le source HTML d'une page lorsque tu es en mode verbose, et ce que tu retrouves dans le source tous les appels aux fichiers que tu as listé ci-dessus ?


Alexis Raimbault webmaster free-lance
dobliu - on Jul. 11 2013 at 12:04 pm GMT
Yes RC5 avec ce 2 js , en verbose j'ai ce warning .
la liste des fihiers du source HTML en verbose pas encore testé (pas beaucoup de temps)

Alexis Raimbault
on Jul. 8 2013 at 01:40 pm GMT
Bonjour,

peux-tu passer en mode verbose ?

dans le panneau de controle, paramètre système, onlet "sytème", verbosité :
"fournir autant d'information que possible"

dès lors ton serveur, au lieu d'utiliser le fichier js compressé, va prendre les fichiers js sources.

Dès lors on comprendra mieux d'où vient l'erreur, merci


Alexis Raimbault webmaster free-lance
Alexis Raimbault
on Jul. 8 2013 at 01:33 pm GMT
La page a été modifiée
dobliu
on Jul. 8 2013 at 10:00 am GMT
La page a été modifiée
dobliu
on Jul. 8 2013 at 09:58 am GMT
La page a été créée