PHPDepend reports invalid jdepend.xml

Same error for me.
at org.sonar.plugins.php.phpdepend.PhpDependResultsParser.getMetrics(Php
DependResultsParser.java:287) [sonar-php-plugin-0.3.jar:na]
"Embedded error: org.sonar.api.utils.SonarException: PDepend report isn't valid:
pdepend.xml
Cannot construct org.sonar.plugins.php.phpdepend.xml.MetricsNode as it does not
have a no-args constructor
"
Error reproduced with those 3 configs :
Sonar 2.10 + PHPSonar 0.4
Sonar 2.7 + PHPSonar 0.4
Sonar 2.7 + PHPSonar 0.3
All the pear packages have the versions said in the documentation

This error is blocking and prevents from using the SonarPHP plugin totally !

Thomas Delerm Can you supply the report file that causes you problems?

Here it is.

This issue can be closed. The problem is fixed by using --phpunit-xml instead of --jdepend-xml to generate the PHP_Depend XML logfile.

However, this sounds broken to me as --phpunit-xml is likely to go away at some point. Back in 2007 [1] I added functionality to PHPUnit to calculate, collect, and report software metrics. This was, of course, wrong. Half a year later [2] I removed that functionality from PHPUnit. At the time some people were relying on PHPUnit's XML logfile for these software metrics which is why Manuel Pichler added the --phpunit-xml argument (and respective logfile format) to PHP_Depend.

Long story short: pdepend + --jdepend-xml gives you a logfile that follows a de-facto standard while pdepend + --phpunit-xml gives you something that is deprecated.

Now if the logfile generated by --jdepend-xml does not contain all the information you need please have a look at --summary-xml. This is the actively maintained logfiles format of PHP_Depend.

–
[1] http://sebastian-bergmann.de/archives/689-PHPUnit-and-Software-Metrics.html
[2] http://sebastian-bergmann.de/archives/744-On-PHPUnit-and-Software-Metrics.html

Ok, so it looks like the root reason is that the Sonar Plugin uses an antique version of the Pdepend log file.
However, the error I got was obtained by using the "standard" PHP Sonar plugin, and the command to call pdepend was :

pdepend.bat --phpunit-xml=D:\code\php\ProjectName\target\logs\pdepend.xml --suffix=php5,phtml D:\code\php\ProjectName

So there is something unclear because the 1st person got this error while generating a jdepend.xml file, and I got it with a pdepend.xml (old format) ????

The issue is still valid. I have to look at what's causing the issue with your file Thomas. Will let you know as soon as I know.

For confirmation, this issue is still happening with plugin PHP-0.6

As Sebastian wrote, the pdepend.xml that I get is not in the format expected by PHP-Sonar.
I can provide additional info if needed.

Caused by: com.thoughtworks.xstream.converters.ConversionException: Cannot const
ruct org.sonar.plugins.php.phpdepend.xml.MetricsNode as it does not have a no-ar
gs constructor : Cannot construct org.sonar.plugins.php.phpdepend.xml.MetricsNod
e as it does not have a no-args constructor

Hi,

I just get the plugin 1.0 and the issue is still there. It's quite annoying because pdepend provides all metrics so the plugin is unusable. It seems version 1.1 fix the problem but where can I get this version or the sources of this version ?

It's not fixed yet, it will be fixed in version 1.1

@Samuel: just for your information, if you let Sonar run PHPDepend (i.e. not reuse existing reports), then this works perfectly.
I'd also just want to mention that the idea behind Sonar is that everything related to quality should be centralized in Sonar (most notably quality profiles that define rule sets), which in turn means that reusing reports should disappear some day - as this is done for Java or other languages in Sonar, as this will often lead to inconsistant results (=> rulesets used to generate the PHPDepend result file that you're reusing are probably not the same as the one that are defined in Sonar - and that you can adapt to meet your needs).

Ok, i figured out that i'm not in the good bug description but it's hard to find the good one. I'm not reusing an existing generated file. I just get the plugin 1.0 and launch the mvn sonar:sonar command. But it didn't work for me (it was the same thing using the plugin 0.6).
I cannot imagine that the plugin doesn't work at all so I need your help. Is the content of the file wrong for some reasons ?
Before going further, is there a better place (issue number) related to my case ?

Thanks a lot. I'm a big fan of Sonar in the java world and I'd like to use it with the PHP language

@Samuel: the first place to ask questions is always the user mailing-list.
So I suggest that we simply keep on discussing on the user ML!

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.

Also see http://markmail.org/message/ooqbrj5wpxoz3kb3

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

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

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.