SonarQube PHP
  1. SonarQube PHP
  2. SONARPHP-57

PHPDepend reports invalid jdepend.xml

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.4
    • Fix Version/s: 1.1
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      2

      Description

      org.sonar.api.utils.SonarException:
      PDepend report isn't valid: jdepend.xml

      1. jdepend.xml
        6 kB
        Sebastian Marek
      2. pdepend.xml
        4 kB
        Thomas Delerm

        Issue Links

          Activity

          Hide
          Thomas Delerm added a comment - - edited

          For information, the issue in the ML is the exact same as mine (I am using Windows too).

          http://sonar-user.15.n6.nabble.com/PHP-Plugin-1-0-PDepend-report-isn-t-valid-td4375219.html

          Glad some other people reported the probleme because our "PHP quality code" projects are totally halted by this bug.

          Show
          Thomas Delerm added a comment - - edited For information, the issue in the ML is the exact same as mine (I am using Windows too). http://sonar-user.15.n6.nabble.com/PHP-Plugin-1-0-PDepend-report-isn-t-valid-td4375219.html Glad some other people reported the probleme because our "PHP quality code" projects are totally halted by this bug.
          Show
          Evgeny Mandrikov added a comment - Also see http://markmail.org/message/ooqbrj5wpxoz3kb3
          Hide
          Evgeny Mandrikov added a comment -

          I'm not sure at 100%, however maybe SONARPLUGINS-1711 provides fix for this problem.

          Show
          Evgeny Mandrikov added a comment - I'm not sure at 100%, however maybe SONARPLUGINS-1711 provides fix for this problem.
          Hide
          Sebastian Marek added a comment -

          First batch of changes committed to the trunk.

          Introduced sonar.phpDepend.reportType option to control which parser to run. Default value phpunit-xml preserves original functionality.
          summary-xml which is not implemented yet will throw an exception as of now.

          Implemented PhpDependParserSelector which is responsible for deciding which parser to use.

          Transformed PhpDependResultsParser into an abstract class, and created 2 parsers extending it:

          • PhpDependPhpUnitReportParser (preserving existing functionality)
          • PhpDependSummaryReportParser (a stub at the moment to be extended into a fully functional parser)

          Further plans:

          • implement summary-xml parsing
          • split phpdepend.xml into 2 packages - phpunitxml and summaryxml (this way it will be much easier to completely remove obsolete parser later on)
          • leave summary-xml parser as a new feature, deprecating phpunit-xml parsing in 1.1 version
          • wait for feedback, and plan for removal of phpunit-xml in 1.2 version superseding it with summary-xml
          Show
          Sebastian Marek added a comment - First batch of changes committed to the trunk. Introduced sonar.phpDepend.reportType option to control which parser to run. Default value phpunit-xml preserves original functionality. summary-xml which is not implemented yet will throw an exception as of now. Implemented PhpDependParserSelector which is responsible for deciding which parser to use. Transformed PhpDependResultsParser into an abstract class, and created 2 parsers extending it: PhpDependPhpUnitReportParser (preserving existing functionality) PhpDependSummaryReportParser (a stub at the moment to be extended into a fully functional parser) Further plans: implement summary-xml parsing split phpdepend.xml into 2 packages - phpunitxml and summaryxml (this way it will be much easier to completely remove obsolete parser later on) leave summary-xml parser as a new feature, deprecating phpunit-xml parsing in 1.1 version wait for feedback, and plan for removal of phpunit-xml in 1.2 version superseding it with summary-xml
          Hide
          Sebastian Marek added a comment -

          Closing as it got solved with SONARPLUGINS-1711.

          SONARPLUGINS-1351 is covering the fact that phpunit-xml report is a deprecated format and will be replaced with a summary-xml format.

          Show
          Sebastian Marek added a comment - Closing as it got solved with SONARPLUGINS-1711 . SONARPLUGINS-1351 is covering the fact that phpunit-xml report is a deprecated format and will be replaced with a summary-xml format.

            People

            • Assignee:
              Sebastian Marek
              Reporter:
              Anonymous
            • Votes:
              2 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: