Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Warning: DOMNode::cloneNode(): ID <id> already defined in phar: #197

Closed
localgod opened this issue Apr 6, 2015 · 13 comments
Closed

Warning: DOMNode::cloneNode(): ID <id> already defined in phar: #197

localgod opened this issue Apr 6, 2015 · 13 comments
Assignees
Labels
Milestone

Comments

@localgod
Copy link

localgod commented Apr 6, 2015

I get a couple of these warnings:

Warning: DOMNode::cloneNode(): ID jdepend.good already defined in phar:///Users/.../pdepend.phar/src/main/php/PDepend/Report/Jdepend/Chart.php on line 199
Warning: DOMNode::cloneNode(): ID jdepend.legend already defined in phar:///Users/.../pdepend.phar/src/main/php/PDepend/Report/Jdepend/Chart.php on line 224
Warning: DOMNode::cloneNode(): ID jdepend.bad already defined in phar:///Users/.../pdepend.phar/src/main/php/PDepend/Report/Jdepend/Chart.php on line 201

Running the phar version (2.0.6)

@HugoGuiroux
Copy link

Confirmed.

@vikingjs
Copy link

If it helps, I'm getting the same warnings using php 5.6. Looks like several cases where cloning a node with an ID is raising the warning.

@uirapuru
Copy link

Same here.

@manuelpichler manuelpichler self-assigned this May 21, 2015
@manuelpichler
Copy link
Contributor

I cannot reproduce that error with php 5.6.8 under ubuntu linux. I would expect it has something todo with the underlying libxml2 version. Will try to fix it blind

@manuelpichler
Copy link
Contributor

Could you please test if the following phar release solves your problem?

http://manuel-pichler.de/stuff/pdepend-2.1.0snapshot201505210758.phar

@vikingjs
Copy link

Happy to test, but I used Composer to install pdepend. The way of the future, they say. Going to have to find a quiet moment to get my build working with the phar.

If it helps, the libxml2 that's installed on my machine is 2.9.2

@sataris
Copy link

sataris commented Jun 19, 2015

I just managed to test the snapshot above and can confirm it runs with no errors

@sebastianbergmann
Copy link
Contributor

I was experiencing the same issue on Fedora 22 and can confirm that PDepend 2.1.0snapshot201505210758 solves the issue.

@localgod
Copy link
Author

PDepend 2.1.0snapshot201505210758 solves this issue on osx with php version 5.6.11

@hackel
Copy link

hackel commented Jul 22, 2015

I tried the 201505210758 snapshot and it also solved the issue for me. Unfortunately it looks like this change didn't make it into 2.1. Any chance we can get a new release soon? It's easier that trying to figure out how to add this one phar to my CI setup.

@mintbridge
Copy link

I have the same problem on 5.6.4 but not 5.6.10 or 5.6.11 all pdepend 2.10 without the snapshot mentioned above

@topdk-system
Copy link

The update fixes issue running on Windows 7, php 5.6.11.

@manuelpichler
Copy link
Contributor

Fixed in 2221f74

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

10 participants