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.

Sign up for GitHub

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

Jump to bottom

Support default parameter values in @method #1194

Closed
Ocramius opened this issue Apr 3, 2014 · 4 comments
Closed

Support default parameter values in @method #1194

Ocramius opened this issue Apr 3, 2014 · 4 comments
Labels
enhancement
Milestone
2.6

Comments

@Ocramius
Copy link

Ocramius commented Apr 3, 2014

As of zendframework/zendframework#6038, it seems like default parameters are not considered in @method annotations:

/**
 * @method FooBar gimmeFooBar(int $baz = null)
 */

Note that these already work in some IDEs
@stefanotorresi
Copy link

also somewhat related to #1085

@toxalot
Copy link

toxalot commented Apr 6, 2014

Related to phpDocumentor/template.clean#29

@weierophinney weierophinney mentioned this issue Apr 14, 2014
Merged
@mvriel mvriel modified the milestones: 2.6, 2.5 May 17, 2014
mvriel added a commit to mvriel/phpDocumentor2 that referenced this issue May 20, 2014
@mvriel
phpDocumentor#1194: Add support for default argument values
417cd72 
Magic methods did not support default values for their arguments.
With this commit we improve the argument algorithm to detect, and
set, these and in the process make the algorithm more foolproof.
@mvriel
Copy link
Member

mvriel commented May 20, 2014

I have issued PR #1271 to add this improvement; the fix will be included in the next release (2.6)

@mvriel mvriel closed this as completed May 20, 2014
@Ocramius
Copy link
Author

@mvriel thanks a lot!

Ocramius added a commit to zendframework/zendframework that referenced this issue May 20, 2014
@Ocramius
Merge branch 'cs/#6038-headlink-docblock-optional-params-hinting'
65f48a0 
Type-hints as of phpDocumentor/phpDocumentor#1194

Close #6038
Ocramius added a commit to zendframework/zendframework that referenced this issue May 20, 2014
@Ocramius
Merge branch 'cs/#6038-headlink-docblock-optional-params-hinting' int…
40148c3 
…o develop

Optional type hints as of phpDocumentor/phpDocumentor#1194

Close #6038
Forward Port #6038
ashnazg added a commit that referenced this issue May 22, 2014
@ashnazg
Merge pull request #1271 from mvriel/feature/1194
9b80982 
#1194: Add support for default argument values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
2.6
Development

No branches or pull requests
4 participants
@Ocramius @mvriel @stefanotorresi @toxalot