SonarQube Plugins
  1. SonarQube Plugins
  2. SONARPLUGINS-597

Inner classes are not properly handled by the plugin in GMetrics results parsing

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: GROOVY-0.1
    • Fix Version/s: GROOVY-0.6
    • Component/s: Groovy
    • Labels:
      None
    • Number of attachments :
      4

      Description

      An example is attached. Currently a new resource is created that just has complexity.

        Issue Links

          Activity

          Hide
          Dave Smith added a comment -

          Sorry for the delay getting back to this (requesting maven slows me down). I attached a maven project and log file showing the analysis failure when a groovy file contains an inner class. This is against 0.5 of the sonar-groovy-plugin. Applying my earlier patch allows the analysis to complete.

          Show
          Dave Smith added a comment - Sorry for the delay getting back to this (requesting maven slows me down). I attached a maven project and log file showing the analysis failure when a groovy file contains an inner class. This is against 0.5 of the sonar-groovy-plugin. Applying my earlier patch allows the analysis to complete.
          Hide
          Ned Wolpert added a comment -

          Since groovy has inner class support, will we see a fix for this? I ran into this myself with one of our projects and applied the patch given (Commented out offending lines) to get it to work here.

          Show
          Ned Wolpert added a comment - Since groovy has inner class support, will we see a fix for this? I ran into this myself with one of our projects and applied the patch given (Commented out offending lines) to get it to work here.
          Hide
          Evgeny Mandrikov added a comment -

          This is fixed, but as consequence property "sonar.groovy.gmetrics.reportPath" not available anymore.

          Show
          Evgeny Mandrikov added a comment - This is fixed, but as consequence property "sonar.groovy.gmetrics.reportPath" not available anymore.
          Hide
          David RACODON added a comment -

          Manually tested with example provided in the ticket.

          Show
          David RACODON added a comment - Manually tested with example provided in the ticket.
          Hide
          Evgeny Mandrikov added a comment -

          Integration test is in place.

          Show
          Evgeny Mandrikov added a comment - Integration test is in place.

            People

            • Assignee:
              Evgeny Mandrikov
              Reporter:
              Olivier Gaudin
            • Votes:
              24 Vote for this issue
              Watchers:
              16 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: