Export to GitHub

jslint4java - issue #81

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


Posted on Jun 2, 2012 by Happy Bear

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/097ba532870c9c358f6c535d0712171d56a3f2a6#diff-3

Comment #1

Posted on Jun 5, 2012 by Happy Bear

Good catch! I really need to get a CI build up and running…

Status: Fixed

Labels:
Type-Defect Priority-Medium Milestone-Release2.0.3