This repository was archived by the owner on Jan 29, 2020. It is now read-only.
Fix handling of FQCNs in method parameters#61
Merged
weierophinney merged 2 commits intozendframework:masterfrom Jun 27, 2016
djmattyg007:fix_methodscanner_handling_of_fqcns
Merged
Fix handling of FQCNs in method parameters#61weierophinney merged 2 commits intozendframework:masterfrom djmattyg007:fix_methodscanner_handling_of_fqcns
weierophinney merged 2 commits intozendframework:masterfrom
djmattyg007:fix_methodscanner_handling_of_fqcns
Conversation
Previously, MethodScanner did not detect a leading T_NS_SEPARATOR character for a type name. By failing to detect it, it passes it to ParameterScanner without it, who then dutifully resolves the classname as if it was relative to the current namespace. This is obviously incorrect behaviour, and it is fixed with this patch. Tests have also been added and updated.
weierophinney
added a commit
that referenced
this pull request
Jun 27, 2016
…_fqcns Fix handling of FQCNs in method parameters
weierophinney
added a commit
that referenced
this pull request
Jun 27, 2016
weierophinney
added a commit
that referenced
this pull request
Jun 27, 2016
Member
|
Thanks, @djmattyg007 ! |
basz
added a commit
to basz/zend-code
that referenced
this pull request
Sep 28, 2016
* 'master' of https://github.com/zendframework/zend-code: (39 commits) Fix Travis code convention Fix Travis code convention Fix Travis code convention Fix Travis code convention 1. Fix ClassScanner::getInterfaces() which don't work with Interface. 2. add TestCase ClassScannerTest::testGetInterfacesFromInterface(). Remove fast_finish flag Bumped to next dev version (3.0.5) Added CHANGELOG for zendframework#75 Separated new tests to separate methods testCorrectExtendNamesFromGlobalNamespace add tests Added CHANGELOG for zendframework#59 Update ClassGenerator.php fast fix Bumped to next dev version (3.0.4) Set date for 3.0.3 Added CHANGELOG for zendframework#72 Added CHANGELOG for zendframework#69 Added CHANGELOG for zendframework#61 Provide a fix for issue zendframework#70 ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Previously,
MethodScannerdid not detect a leadingT_NS_SEPARATORcharacter for a type name. By failing to detect it, it passes it to
ParameterScannerwithout it, who then dutifully resolves the classnameas if it was relative to the current namespace.
This is obviously incorrect behaviour, and it is fixed with this patch.
Tests have also been added and updated.