Skip to content

Added tip for those using other programs (like Doxygen) that use annotati #380

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 2 commits into from
Jun 14, 2011
Merged

Conversation

RogerWebb
Copy link
Contributor

Added tip for those using other programs (like Doxygen) that use annotations letting the user know that the @IgnoreAnnotation annotation is necessary to prevent an Exception from being thrown.

…tations letting the user know that the @IgnoreAnnotation annotation is necessary to prevent an Exception from being thrown.
you must use the @IgnoreAnnotation annotation to indicate which annotations
Symfony and Doctrine should ignore. Failing to do so may result in an
Exception being thrown.
Example: @IgnoreAnnotation("param") will prevent "@param ..." from throwing an Exception.
Copy link
Member

Choose a reason for hiding this comment

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

You should use another example as @param is already ignored by default.

…example and added a note letting the reader know where the @IgnoreAnnotation annotation should be placed.
weaverryan added a commit that referenced this pull request Jun 14, 2011
Added tip for those using other programs (like Doxygen) that use annotati
@weaverryan weaverryan merged commit fb62856 into symfony:master Jun 14, 2011
@weaverryan
Copy link
Member

I think this is a great note and I've merged it in. The tricky part is that this really applies to all annotations - and a case could be made for this to be in the validation chapter as well. So, I'm not totally sure we have it in the right place yet, but it's at least in a good place.

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

Successfully merging this pull request may close these issues.

3 participants