Sonar Plugins

Unable to parse phpmd result file when workspace path contains spaces

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: PHP-0.4
  • Fix Version/s: PHP-0.5
  • Component/s: PHP
  • Labels:
    None
  • Number of attachments :
    0

Description

Here is the stack

Embedded error: Unable to parse the  XML Report
/home/s27697/.hudson/jobs/GEC%20-%20TestPHP/workspace/target/logs/pmd.xml (No such file or directory)
[INFO] ------------------------------------------------------------------------
[INFO] Trace
org.apache.maven.lifecycle.LifecycleExecutionException: Can not execute Sonar
	at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoals(DefaultLifecycleExecutor.java:719)
	at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeStandaloneGoal(DefaultLifecycleExecutor.java:569)
	at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoal(DefaultLifecycleExecutor.java:539)
	at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoalAndHandleFailures(DefaultLifecycleExecutor.java:387)
	at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeTaskSegments(DefaultLifecycleExecutor.java:284)
	at org.apache.maven.lifecycle.DefaultLifecycleExecutor.execute(DefaultLifecycleExecutor.java:180)
	at org.apache.maven.DefaultMaven.doExecute(DefaultMaven.java:328)
	at org.apache.maven.DefaultMaven.execute(DefaultMaven.java:138)
	at org.apache.maven.cli.MavenCli.main(MavenCli.java:362)
	at org.apache.maven.cli.compat.CompatibleMain.main(CompatibleMain.java:60)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
	at java.lang.reflect.Method.invoke(Method.java:592)
	at org.codehaus.classworlds.Launcher.launchEnhanced(Launcher.java:315)
	at org.codehaus.classworlds.Launcher.launch(Launcher.java:255)
	at org.codehaus.classworlds.Launcher.mainWithExitCode(Launcher.java:430)
	at org.codehaus.classworlds.Launcher.main(Launcher.java:375)
Caused by: org.apache.maven.plugin.MojoExecutionException: Can not execute Sonar
	at org.codehaus.mojo.sonar.Bootstraper.executeMojo(Bootstraper.java:103)
	at org.codehaus.mojo.sonar.Bootstraper.start(Bootstraper.java:79)
	at org.codehaus.mojo.sonar.SonarMojo.execute(SonarMojo.java:88)
	at org.apache.maven.plugin.DefaultPluginManager.executeMojo(DefaultPluginManager.java:490)
	at org.apache.maven.lifecycle.DefaultLifecycleExecutor.executeGoals(DefaultLifecycleExecutor.java:694)
	... 17 more
Caused by: org.sonar.api.utils.XmlParserException: Unable to parse the  XML Report
	at org.sonar.plugins.php.pmd.PhpmdViolationsXmlParser.getViolations(PhpmdViolationsXmlParser.java:107)
	at org.sonar.plugins.php.pmd.PhpmdSensor.analyse(PhpmdSensor.java:88)
	at org.sonar.batch.phases.SensorsExecutor.execute(SensorsExecutor.java:64)
	at org.sonar.batch.phases.Phases.execute(Phases.java:93)
	at org.sonar.batch.bootstrap.ProjectModule.doStart(ProjectModule.java:143)
	at org.sonar.batch.bootstrap.Module.start(Module.java:89)
	at org.sonar.batch.bootstrap.BatchModule.analyze(BatchModule.java:110)
	at org.sonar.batch.bootstrap.BatchModule.doStart(BatchModule.java:100)
	at org.sonar.batch.bootstrap.Module.start(Module.java:89)
	at org.sonar.batch.bootstrap.BootstrapModule.doStart(BootstrapModule.java:96)
	at org.sonar.batch.bootstrap.Module.start(Module.java:89)
	at org.sonar.batch.Batch.execute(Batch.java:74)
	at org.sonar.maven.SonarMojo.executeBatch(SonarMojo.java:154)
	at org.sonar.maven.SonarMojo.execute(SonarMojo.java:143)
	at org.apache.maven.plugin.DefaultPluginManager.executeMojo(DefaultPluginManager.java:490)
	at org.codehaus.mojo.sonar.Bootstraper.executeMojo(Bootstraper.java:98)
	... 21 more
Caused by: com.ctc.wstx.exc.WstxIOException: /home/s27697/.hudson/jobs/GEC%20-%20TestPHP/workspace/target/logs/pmd.xml (No such file or directory)
	at com.ctc.wstx.stax.WstxInputFactory.createSR(WstxInputFactory.java:621)
	at com.ctc.wstx.stax.WstxInputFactory.createXMLStreamReader(WstxInputFactory.java:443)
	at org.codehaus.staxmate.SMInputFactory.createStax2Reader(SMInputFactory.java:150)
	at org.codehaus.staxmate.SMInputFactory.rootElementCursor(SMInputFactory.java:257)
	at org.sonar.plugins.php.pmd.PhpmdViolationsXmlParser.getViolations(PhpmdViolationsXmlParser.java:80)
	... 36 more
Caused by: java.io.FileNotFoundException: /home/s27697/.hudson/jobs/GEC%20-%20TestPHP/workspace/target/logs/pmd.xml (No such file or directory)
	at java.io.FileInputStream.open(Native Method)
	at java.io.FileInputStream.<init>(FileInputStream.java:106)
	at java.io.FileInputStream.<init>(FileInputStream.java:66)
	at com.ctc.wstx.util.URLUtil.inputStreamFromURL(URLUtil.java:108)
	at com.ctc.wstx.stax.WstxInputFactory.createSR(WstxInputFactory.java:618)
	... 40 more

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Julien HENRY added a comment -

Issue come from woodstock-core-lgpl-4.0.4.

In URLUtil.inputStreamFromURL(URL url):

return new FileInputStream(url.getPath());

should be

return new FileInputStream(new File(url.toURI()));

Workaround is to do the job yourself and use the SMInputFactory.rootElementCursor(File) constructor.

In PhpmdViolationsXmlParser, replace:

SMInputCursor rootNodeCursor = inputFactory.rootElementCursor(url).advance();

by

SMInputCursor rootNodeCursor = inputFactory.rootElementCursor(new File(url.toURI())).advance();
Show
Julien HENRY added a comment - Issue come from woodstock-core-lgpl-4.0.4. In URLUtil.inputStreamFromURL(URL url): return new FileInputStream(url.getPath()); should be return new FileInputStream( new File(url.toURI())); Workaround is to do the job yourself and use the SMInputFactory.rootElementCursor(File) constructor. In PhpmdViolationsXmlParser, replace: SMInputCursor rootNodeCursor = inputFactory.rootElementCursor(url).advance(); by SMInputCursor rootNodeCursor = inputFactory.rootElementCursor( new File(url.toURI())).advance();
Hide
Permalink
Fabrice Bellingard added a comment -

Actually, I even wonder why a URL is passed instead of directly a File object (from which the URL is computed in the PhpmdSensor), like this is the case in every other tool of the PHP plugin.
I think I don't even need this workaround, I'll just use directly a File object (I don't see why a PHPMD report URL should be required to get identified).

Show
Fabrice Bellingard added a comment - Actually, I even wonder why a URL is passed instead of directly a File object (from which the URL is computed in the PhpmdSensor), like this is the case in every other tool of the PHP plugin. I think I don't even need this workaround, I'll just use directly a File object (I don't see why a PHPMD report URL should be required to get identified).
Hide
Permalink
Julien HENRY added a comment -

Would be better and avoid conversion.
+1

Show
Julien HENRY added a comment - Would be better and avoid conversion. +1

People

  • Assignee:
    Fabrice Bellingard
    Reporter:
    Julien HENRY
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: