Skip to content

[Reference][Constraints] Added missing class to static callback example in PHP #2759

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

Merged
merged 1 commit into from
Jun 30, 2013

Conversation

vjnrv
Copy link
Contributor

@vjnrv vjnrv commented Jun 22, 2013

Q A
Doc fix? yes
New docs? no
Applies to 2.2+
Fixed tickets -

Signed-off-by: Pedro Junior <v.ju.ni.or.v@gmail.com>
@xabbuh
Copy link
Member

xabbuh commented Jun 22, 2013

👍

@wouterj
Copy link
Member

wouterj commented Jun 22, 2013

can't you use array($this, ...) instead?

@vjnrv
Copy link
Contributor Author

vjnrv commented Jun 22, 2013

No, because this section shows how to validate with static method of other class that is not the entity itself.

http://symfony.com/doc/2.2/reference/constraints/Callback.html#methods

@wouterj
Copy link
Member

wouterj commented Jun 22, 2013

Ok, I didn't look at the code. 👍

weaverryan added a commit that referenced this pull request Jun 30, 2013
[Reference][Constraints] Added missing class to static callback example in PHP
@weaverryan weaverryan merged commit 0849175 into symfony:2.2 Jun 30, 2013
@weaverryan
Copy link
Member

Great find, great fix, easy merge! Thanks Pedro!

@vjnrv vjnrv deleted the 2.2-static-callback-fix branch July 1, 2013 12:22
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

Successfully merging this pull request may close these issues.

4 participants