SonarQube Plugins

Problem in the calculation of cyclomatic complexity

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: JAVASCRIPT-0.4
  • Fix Version/s: JAVASCRIPT-1.0
  • Component/s: JavaScript
  • Labels:
    None
  • Environment:
    linux
  • Number of attachments :
    2

Description

the calculation of cyclomatic complexity is not accurate. in the attachment, you get the code and the screenshot.

It appears to me that the number of methods is not accurate, and so does the calculation of complexity/method

  1. Text File
    code.txt
    21/Mar/12 5:09 AM
    6 kB
    pti
  1. screenshot.JPG
    68 kB
    21/Mar/12 5:09 AM

Issue Links

is superceded by

New Feature - A new feature of the product, which has yet to be developed. SONARPLUGINS-1789 Integrate JavaScript parser 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
Eriks Nukis added a comment -

Metric "Complexity/Method" seems to be correct, but metric "Methods" was saved incorrectly. Will be fixed in 0.5

Show
Eriks Nukis added a comment - Metric "Complexity/Method" seems to be correct, but metric "Methods" was saved incorrectly. Will be fixed in 0.5
Hide
Permalink
Eriks Nukis added a comment -

Fixed in r5219

Show
Eriks Nukis added a comment - Fixed in r5219
Hide
Permalink
pti added a comment -

can be a dumb question... could you please tell me how to get the patch for this issue?
should i wait for version 5, or can we get a patched version?

Show
pti added a comment - can be a dumb question... could you please tell me how to get the patch for this issue? should i wait for version 5, or can we get a patched version?
Hide
Permalink
Eriks Nukis added a comment -

you can use snapshot version where this is already fixed http://snapshots.repository.codehaus.org/org/codehaus/sonar-plugins/sonar-javascript-plugin/0.5-SNAPSHOT/

Show
Eriks Nukis added a comment - you can use snapshot version where this is already fixed http://snapshots.repository.codehaus.org/org/codehaus/sonar-plugins/sonar-javascript-plugin/0.5-SNAPSHOT/
Hide
Permalink
Evgeny Mandrikov added a comment - - edited

Note that this issue fully superseded by SONARPLUGINS-1789, because the way to compute number of functions and complexity was completely changed.

For the given example:

  • functions = 7
  • complexity = 28
Show
Evgeny Mandrikov added a comment - - edited Note that this issue fully superseded by SONARPLUGINS-1789 , because the way to compute number of functions and complexity was completely changed. For the given example: functions = 7 complexity = 28

People

  • Assignee:
    Eriks Nukis
    Reporter:
    pti
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: