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

Class File_Reflector should be compatible... #177

Closed
kallookoo opened this issue May 14, 2016 · 6 comments
Closed

Class File_Reflector should be compatible... #177

kallookoo opened this issue May 14, 2016 · 6 comments

Comments

@kallookoo
Copy link

...
Warning: Declaration of WP_Parser\File_Reflector::isNodeDocumentable(PHPParser_Node $node) should be compatible with phpDocumentor\Reflection\FileReflector::isNodeDocumentable(PhpParser\Node $node) in phpdoc-parser/lib/class-file-reflector.php on line 14
Fatal error: Declaration of WP_Parser\File_Reflector::enterNode(PHPParser_Node $node) must be compatible with PhpParser\NodeVisitor::enterNode(PhpParser\Node $node) in phpdoc-parser/lib/class-file-reflector.php on line 14

@JDGrimes
Copy link
Contributor

Sounds like you have the wrong version of phpDocumentor/PhpParser.

@kallookoo
Copy link
Author

yes, but composer install 1.4.1 version of nikic/php-parser and change names for namespace, classes, ... see https://github.com/nikic/PHP-Parser/blob/master/UPGRADE-1.0.md

@JDGrimes
Copy link
Contributor

Apparently phpDocumentor/Reflection doesn't follow semantic versioning, since it updated it sdependency from PHP-Parser 0.9 to 1.0 in version 1.1.0. Actually, I think this was an unintentional break on their part. I'll see if maybe they can fix this somehow. In the mean time, try installing version 1.0.7 of phpDocumentor/Reflection instead, which will have the correct dependency versions that are compatible with this plugin.

@JDGrimes
Copy link
Contributor

I've opened phpDocumentor/Reflection#96.

@JDGrimes
Copy link
Contributor

@23r9i0 phpDocumentor has resolved this issue by removing the 1.1.0 release. I think you will be able to re-install using composer now and it should install the correct version of PHP-Parser.

@kallookoo
Copy link
Author

ok, thanks!!

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

2 participants