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.

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

Update form_events.rst #4031

Closed
wants to merge 2 commits into from
Closed

Update form_events.rst #4031

wants to merge 2 commits into from

Conversation

redstar504
Copy link
Contributor

EventSubscriberInterface::getSubscribedEvents() method is static.

EventSubscriberInterface::getSubscribedEvents() method is static.
@xabbuh
Copy link
Member

xabbuh commented Jul 17, 2014

👍

1 similar comment
@stof
Copy link
Member

stof commented Jul 17, 2014

👍

@redstar504
Copy link
Contributor Author

Also, why is this subscriber class called AddEmailFieldListener? Shouldn't it be AddEmailFieldSubscriber?

Remove extraneous quote.
@wouterj
Copy link
Member

wouterj commented Jul 20, 2014

@redstar504 Symfony Core calls everything listeners (in fact, a subscriber is nothing more than a group of listener functions), we just follow that convention in the docs.

@weaverryan
Copy link
Member

You're on a roll - thanks @redstar504 for all the fixes!

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.

5 participants