Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 8, 2020. It is now read-only.

Fix returned NamespaceType for Parameters from Reflection#5193

Closed
danez wants to merge 5 commits into
zendframework:masterfrom
danez:master
Closed

Fix returned NamespaceType for Parameters from Reflection#5193
danez wants to merge 5 commits into
zendframework:masterfrom
danez:master

Conversation

@danez
Copy link
Copy Markdown
Contributor

@danez danez commented Sep 30, 2013

If a parameter inside a namespace-class has a typehint to an other namespaced-class the returned typehint lacks the beginning backslash.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add license docblock here ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@samsonasik
Copy link
Copy Markdown
Contributor

@danez maybe

php php-cs-fixer.phar  fix tests/ZendTest/Code/Generator/TestAsset/NamespaceTypeHintClass.php  

?

@danez
Copy link
Copy Markdown
Contributor Author

danez commented Oct 1, 2013

sorry, haven't had time to wait for travis yesterday.

@danez
Copy link
Copy Markdown
Contributor Author

danez commented Oct 10, 2013

Any news on this?

@Ocramius
Copy link
Copy Markdown
Member

👍

@ghost ghost assigned weierophinney Oct 23, 2013
weierophinney added a commit that referenced this pull request Oct 23, 2013
Fix returned NamespaceType for Parameters from Reflection
weierophinney added a commit that referenced this pull request Oct 23, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants