MinELenI / CBSviewer

Webrichtlijnen compliant webmap viewer
http://mineleni.github.io/CBSviewer/
BSD 2-Clause "Simplified" License
0 stars 6 forks source link

yuicompressor faalt op jquery.layout.js #23

Closed mprins closed 11 years ago

mprins commented 11 years ago

De yuicompressor t compiler faalt met een error tijdens het compileren van jquery.layout.js (sinds SHA: 5a919913994116915541a64483c50e74be2c3dd5 ) de melding snap ik niet helemaal, maar het lijkt erop dat het hier gaat om de float: properties, misschien dat een stel aanhalingstekens eromheen dat op kan lossen? dus: { width: "auto", 'float': "left" } en { 'float': "none" } in plaats van de code zoals die in onderstaande foutmelding staat.

De fout treedt zowel op de Jenkins (http://gisdemo.agro.nl/jenkins/job/CBS-viewer/128/) als de Travis (https://travis-ci.org/#!/MinELenI/CBSviewer/builds/3097019) server op.

18$ mvn install --quiet -DskipTests=true
19[ERROR] .../src/main/webapp/lib/jquery.layout.js:line 1067:column 35:invalid property id
20                      $P.css({ width: "auto", float: "left" }); // float = FORCE pane to auto-size
21[ERROR] .../src/main/webapp/lib/jquery.layout.js:line 1069:column 20:invalid property id
22                      $P.css({ float: "none" }); // RESET
23[ERROR] .../src/main/webapp/lib/jquery.layout.js:line 1:column 0:Compilation produced 2 syntax errors.
24[ERROR] Failed to execute goal net.alchim31.maven:yuicompressor-maven-plugin:1.3.2:compress (default) on project cbsviewer: Execution default of goal net.alchim31.maven:yuicompressor-maven-plugin:1.3.2:compress failed: Compilation produced 2 syntax errors. -> [Help 1]
25[ERROR] 
26[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
27[ERROR] Re-run Maven using the -X switch to enable full debug logging.
28[ERROR] 

het gaat om de regels: https://github.com/MinELenI/CBSviewer/blob/master/src/main/webapp/lib/jquery.layout.js#L1067 en https://github.com/MinELenI/CBSviewer/blob/master/src/main/webapp/lib/jquery.layout.js#L1069

pmanders commented 11 years ago

Hmm, hier heb ik niets aan veranderd. Zal er eens naar kijken

mprins commented 11 years ago

zie: http://stackoverflow.com/questions/424469/compression-issues-with-jquery-file-in-yui-compressor

"float" is een gereserveerd woord in de standaard.

mprins commented 11 years ago

fixed met SHA: 22adf2c