SonarQube Plugins

Rule: Generic JavaScript rule to define some homemade checks with an XPath expression

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: JAVASCRIPT-1.0
  • Component/s: JavaScript
  • Labels:
    None
  • Number of attachments :
    0

Issue Links

depends upon

New Feature - A new feature of the product, which has yet to be developed. SONARPLUGINS-1788 Provide JavaScript rule engine based on SSLR

  • 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
Hide
Permalink
Freddy Mallet added a comment - - edited

Works well evgeny could you just update the description as follows :

<p>
This rule allows to define some homemade Javascript rules with help of an XPath expression.

Violations are created depending on the return value of the XPath expression. If the XPath expression returns:

a list of AST nodes, then a line violation with the given message is created on each the line of each node
a boolean, then a a file violation with the given message is created only if the boolean is true
anything else, no violation is created
Here is an example of an XPath expression to log a violation on each if statement : //ifStatement</p>
Show
Freddy Mallet added a comment - - edited Works well evgeny could you just update the description as follows : <p> This rule allows to define some homemade Javascript rules with help of an XPath expression. Violations are created depending on the return value of the XPath expression. If the XPath expression returns: a list of AST nodes, then a line violation with the given message is created on each the line of each node a boolean, then a a file violation with the given message is created only if the boolean is true anything else, no violation is created Here is an example of an XPath expression to log a violation on each if statement : //ifStatement</p>
Hide
Permalink
Evgeny Mandrikov added a comment -

Done.

Show
Evgeny Mandrikov added a comment - Done.
Hide
Permalink
Freddy Mallet added a comment -

Manually tested !

Show
Freddy Mallet added a comment - Manually tested !

People

  • Assignee:
    Evgeny Mandrikov
    Reporter:
    Evgeny Mandrikov
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: