Sonar Plugins

Update jslint4java to version 2.0.2

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Won't Fix
  • Affects Version/s: JAVASCRIPT-0.3, JAVASCRIPT-0.4
  • Fix Version/s: None
  • Component/s: JavaScript
  • Labels:
    None
  • Number of attachments :
    0

Description

Plugin should be updated to use latest version of http://code.google.com/p/jslint4java/. Major changes in jslint4java:

Version 1.4.7

  • changes in options: adds the continue option, whilst removing eqeqeq, immed and laxbreak options
  • warning messages are changed and should be updated

Version 2.0.0

  • The nature of the boolean options has been inverted--beware!
    • bitwise now enables the use of bitwise operators.
    • newcap now means "ignore capitalisation of constructors"
    • plusplus now means "incr" and "decr" operators are tolerated.
    • regexp now means allow the use of "." in regexes.
    • undef now means you can declare variables out of order.
    • white now means acceptance of non-compliant whitespace.
  • New options:
    • sloppy, which is the inverse of strict.
    • confusion, allows inconsistent types.
    • vars, to tolerate multiple var declarations per function (replaces onevar).
    • eqeq, to tolerate "==" and "!=".
    • node, to predefine node.js globals.
    • properties, to require declaration of all properties.
    • unparam, to allow unused parameters.
  • Removed options:
    • onevar
    • strict

Version 2.0.2

  • Removed option:
    • adsafe
    • confusion
    • safe
  • New Options
    • anon

Issue Links

is depended upon by

New Feature - A new feature of the product, which has yet to be developed. SONARPLUGINS-1669 Would like a way to set JSLint Rules

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is duplicated by

New Feature - A new feature of the product, which has yet to be developed. SONARPLUGINS-1669 Would like a way to set JSLint Rules

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.
is superceded by

Task - A task that needs to be done. SONARPLUGINS-1829 Do not use jslint4java

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Eriks Nukis made changes -
Field Original Value New Value
Summary jslint4java should be updated to latest version 1.4.7 jslint4java should be updated to latest version
Description Plugin should be updated to use latest version of [http://code.google.com/p/jslint4java/]. Major changes:
# changes in options: adds the continue option, whilst removing eqeqeq, immed and laxbreak options
# warning messages are changed and should be updated 		Plugin should be updated to use latest version of [http://code.google.com/p/jslint4java/]. Major changes in jslint4java:
h4. Version 1.4.7
* changes in options: adds the continue option, whilst removing eqeqeq, immed and laxbreak options
* warning messages are changed and should be updated

h4. Version 2.0.0
* The nature of the boolean options has been inverted--beware!
** bitwise now enables the use of bitwise operators.
** newcap now means "ignore capitalisation of constructors"
** plusplus now means "incr" and "decr" operators are tolerated.
** regexp now means allow the use of "." in regexes.
** undef now means you can declare variables out of order.
** white now means acceptance of non-compliant whitespace.
* New options:
** sloppy, which is the inverse of strict.
** confusion, allows inconsistent types.
** vars, to tolerate multiple var declarations per function (replaces onevar).
** eqeq, to tolerate "==" and "!=".
** node, to predefine node.js globals.
** properties, to require declaration of all properties.
** unparam, to allow unused parameters.
* Removed options:
** onevar
** strict
Eriks Nukis made changes -
Fix Version/s JAVASCRIPT-0.5 [ 18248 ]
Fix Version/s JAVASCRIPT-0.4 [ 17247 ]
Affects Version/s JAVASCRIPT-0.4 [ 17247 ]
Hide
Permalink
Jakob Kruse added a comment -

This is really, really important! Current JSLint is from 2010, and uses a completely different in-file options set than current versions.

Show
Jakob Kruse added a comment - This is really, really important! Current JSLint is from 2010, and uses a completely different in-file options set than current versions.
Hide
Permalink
Fabio Gavilondo added a comment -

Another vote. Really, really important!

It's not just about the new options. jslint4java 1.4.6 (and 1.4.7) fail to parse JavaScript files encoded in UTF-8Y (aka UTF-8 BOM) as these versions don't recognize the byte order mark at the beginning of the file and give up, with the rather cryptic message:

Unexpected ".

I'm not sure if this is a problem with the old version of Rhino, or JSLint itself, but jslint4java 2.0.2 does not have that problem.

Given that nearly all Windows tools (Visual Studio, Notepad...) use UTF-8Y as their default encoding, and a lot of people do JS development on Windows it would be really important to upgrade the plugin to a JSLint version that can deal with this encoding.

Show
Fabio Gavilondo added a comment - Another vote. Really, really important! It's not just about the new options. jslint4java 1.4.6 (and 1.4.7) fail to parse JavaScript files encoded in UTF-8Y (aka UTF-8 BOM) as these versions don't recognize the byte order mark at the beginning of the file and give up, with the rather cryptic message: Unexpected ". I'm not sure if this is a problem with the old version of Rhino, or JSLint itself, but jslint4java 2.0.2 does not have that problem. Given that nearly all Windows tools (Visual Studio, Notepad...) use UTF-8Y as their default encoding, and a lot of people do JS development on Windows it would be really important to upgrade the plugin to a JSLint version that can deal with this encoding.
Eriks Nukis made changes -
Status Open [ 1 ] In Progress [ 3 ]
Evgeny Mandrikov made changes -
Link This issue is depended upon by SONARPLUGINS-1669 [ SONARPLUGINS-1669 ]
Hide
Permalink
Eddie Webb added a comment -

Anyone using jslint in a Java project? I can't understand how to allow both static analysis to occur on same project - SONARPLUGINS-1802

Show
Eddie Webb added a comment - Anyone using jslint in a Java project? I can't understand how to allow both static analysis to occur on same project - SONARPLUGINS-1802
Eriks Nukis made changes -
Description Plugin should be updated to use latest version of [http://code.google.com/p/jslint4java/]. Major changes in jslint4java:
h4. Version 1.4.7
* changes in options: adds the continue option, whilst removing eqeqeq, immed and laxbreak options
* warning messages are changed and should be updated

h4. Version 2.0.0
* The nature of the boolean options has been inverted--beware!
** bitwise now enables the use of bitwise operators.
** newcap now means "ignore capitalisation of constructors"
** plusplus now means "incr" and "decr" operators are tolerated.
** regexp now means allow the use of "." in regexes.
** undef now means you can declare variables out of order.
** white now means acceptance of non-compliant whitespace.
* New options:
** sloppy, which is the inverse of strict.
** confusion, allows inconsistent types.
** vars, to tolerate multiple var declarations per function (replaces onevar).
** eqeq, to tolerate "==" and "!=".
** node, to predefine node.js globals.
** properties, to require declaration of all properties.
** unparam, to allow unused parameters.
* Removed options:
** onevar
** strict 		Plugin should be updated to use latest version of [http://code.google.com/p/jslint4java/]. Major changes in jslint4java:
h4. Version 1.4.7
* changes in options: adds the continue option, whilst removing eqeqeq, immed and laxbreak options
* warning messages are changed and should be updated

h4. Version 2.0.0
* The nature of the boolean options has been inverted--beware!
** bitwise now enables the use of bitwise operators.
** newcap now means "ignore capitalisation of constructors"
** plusplus now means "incr" and "decr" operators are tolerated.
** regexp now means allow the use of "." in regexes.
** undef now means you can declare variables out of order.
** white now means acceptance of non-compliant whitespace.
* New options:
** sloppy, which is the inverse of strict.
** confusion, allows inconsistent types.
** vars, to tolerate multiple var declarations per function (replaces onevar).
** eqeq, to tolerate "==" and "!=".
** node, to predefine node.js globals.
** properties, to require declaration of all properties.
** unparam, to allow unused parameters.
* Removed options:
** onevar
** strict

h4. Version 2.0.2
* Removed option:
** adsafe
** confusion
** safe
* New Options
** anon
Eriks Nukis made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Resolution Fixed [ 1 ]
Evgeny Mandrikov made changes -
Summary jslint4java should be updated to latest version Update jslint4java to version 2.0.2
Evgeny Mandrikov made changes -
Link This issue is duplicated by SONARPLUGINS-1669 [ SONARPLUGINS-1669 ]
Evgeny Mandrikov made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Evgeny Mandrikov made changes -
Link This issue is superceded by SONARPLUGINS-1829 [ SONARPLUGINS-1829 ]
Evgeny Mandrikov made changes -
Status Reopened [ 4 ] Closed [ 6 ]
Resolution Won't Fix [ 2 ]
Evgeny Mandrikov made changes -
Resolution Won't Fix [ 2 ]
Status Closed [ 6 ] Reopened [ 4 ]
Evgeny Mandrikov made changes -
Fix Version/s JAVASCRIPT-0.5 [ 18248 ]
Evgeny Mandrikov made changes -
Status Reopened [ 4 ] Closed [ 6 ]
Resolution Won't Fix [ 2 ]

People

  • Assignee:
    Eriks Nukis
    Reporter:
    Eriks Nukis
Vote (4)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: