Sonar Plugins

Rule: Avoid usage of eval

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
Evgeny Mandrikov made changes -
Field Original Value New Value
Link This issue depends upon SONARPLUGINS-1788 [ SONARPLUGINS-1788 ]
Evgeny Mandrikov made changes -
Status Open [ 1 ] Resolved [ 5 ]
Assignee Evgeny Mandrikov [ godin ] Freddy Mallet [ freddy.mallet ]
Resolution Fixed [ 1 ]
Hide
Permalink
Freddy Mallet added a comment - - edited

Works well Evgeny.

Could you just update the description as follows :

<p>The eval() function in javascript is a way to run arbitrary code at run-time. In almost all cases, eval should never be used. If it exists in your page, there is almost always a more correct way to accomplish what you are doing.

The rule is, "Eval is evil." Don't use it unless you are an experienced developer and know that your case is an exception.</p>

<p>
The following code snippet illustrates this rule :
</p>

<pre>
var myCode = 'alert("Howdy?");';
eval(myCode);     //NOK
</pre>
Show
Freddy Mallet added a comment - - edited Works well Evgeny. Could you just update the description as follows : <p>The eval() function in javascript is a way to run arbitrary code at run-time. In almost all cases, eval should never be used. If it exists in your page, there is almost always a more correct way to accomplish what you are doing. The rule is, "Eval is evil." Don't use it unless you are an experienced developer and know that your case is an exception.</p> <p> The following code snippet illustrates this rule : </p> <pre> var myCode = 'alert("Howdy?");'; eval(myCode); //NOK </pre>
Freddy Mallet made changes -
Resolution Fixed [ 1 ]
Status Resolved [ 5 ] Reopened [ 4 ]
Assignee Freddy Mallet [ freddy.mallet ] Evgeny Mandrikov [ godin ]
Evgeny Mandrikov made changes -
Status Reopened [ 4 ] In Progress [ 3 ]
Hide
Permalink
Evgeny Mandrikov added a comment -

Done.

Show
Evgeny Mandrikov added a comment - Done.
Evgeny Mandrikov made changes -
Status In Progress [ 3 ] Resolved [ 5 ]
Assignee Evgeny Mandrikov [ godin ] Freddy Mallet [ freddy.mallet ]
Resolution Fixed [ 1 ]
Hide
Permalink
Freddy Mallet added a comment -

Manually tested !

Show
Freddy Mallet added a comment - Manually tested !
Freddy Mallet made changes -
Assignee Freddy Mallet [ freddy.mallet ] Evgeny Mandrikov [ godin ]
Evgeny Mandrikov made changes -
Status Resolved [ 5 ] Closed [ 6 ]

People

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

Dates

  • Created:
    Updated:
    Resolved: