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

PHPMD library has PHP7 error #5

Closed
davidscotson opened this issue Nov 18, 2015 · 4 comments
Closed

PHPMD library has PHP7 error #5

davidscotson opened this issue Nov 18, 2015 · 4 comments

Comments

@davidscotson
Copy link

You can see on this log that the pdepend library that phpmd uses is throwing errors to do with undefined constants on PHP7:

https://travis-ci.org/danmarsden/moodle-plagiarism_urkund/jobs/91730002

This is the upstream bug, which seems to have been fixed and released already:

pdepend/pdepend#213

Not clear if it's made it's way to PHPMD or not yet

@polothy
Copy link
Contributor

polothy commented Nov 18, 2015

I think if I run composer update on this project, it might pull it down.

@ecampbell
Copy link

Not sure if this is a separate bug, but I encountered an error with PHP7 while checking my plugin too, cf. https://travis-ci.org/ecampbell/moodle-qformat_wordtable/jobs/92040018

@davidscotson
Copy link
Author

That looks like it might be a problem in the Travis system, seems to have happened to a few different people in the last day or so, but worked before that.

Ah, found it:

travis-ci/travis-ci#5111

@polothy
Copy link
Contributor

polothy commented Dec 31, 2015

This has been fixed in 1.2.0. The dependencies have been updated, which support PHP7.

@polothy polothy closed this as completed Dec 31, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants