andoaki / jslint4java

Automatically exported from code.google.com/p/jslint4java
Other
0 stars 0 forks source link

Does not work with jslint.js >= 2012-05-09 #81

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?

1. git clone git://github.com/douglascrockford/JSLint.git 
2. add this config to the pom in maven: 
<jslintSource>${basedir}/src/tests/jslint/jslint.js</jslintSource>
3. mvn jslint4java:lint

What is the expected output? What do you see instead?

[ERROR] Failed to execute goal 
com.googlecode.jslint4java:jslint4java-maven-plugin:2.0.2:lint (default-cli) on 
project: Execution default-cli of goal 
com.googlecode.jslint4java:jslint4java-maven-plugin:2.0.2:lint failed: 
TypeError: Cannot read property "length" from undefined 
(/src/tests/jslint/jslint.js#6336) -> [Help 1]

What version of the product are you using? On what operating system?

jslint4java-2.0.2 on Mac OS X 10.7.3 with jslint.js-2012-05-09

Please provide any additional information below.

Works great with jslint.js-2012-04-15, which is the previous version.
Once cloned, you can get it at:
git checkout 4f2bf65ed59a0ba16fb22a6a90f76fc0060a2c22

Fails with jslint.js-2012-05-09 which you can get at:
git checkout 097ba532870c9c358f6c535d0712171d56a3f2a6

It appears the main difference between the two versions is the HTML report 
changed from JSLINT.report() to JSLINT.error_report() and now requires 
JSLINT.data() to be passed into it vs. the bool for errors_only.

https://github.com/douglascrockford/JSLint/commit/097ba532870c9c358f6c535d071217
1d56a3f2a6#diff-3

Original issue reported on code.google.com by jmeyo...@gmail.com on 2 Jun 2012 at 6:25

GoogleCodeExporter commented 8 years ago
Good catch!  I really need to get a CI build up and running…

Original comment by d...@happygiraffe.net on 5 Jun 2012 at 7:19

GoogleCodeExporter commented 8 years ago
https://github.com/happygiraffe/jslint4java/compare/34e3f83319...a490ae118e

Original comment by d...@happygiraffe.net on 6 Jun 2012 at 5:37