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

Ensure onFlush and postFlush events are always called #525

Closed
wants to merge 2 commits into from

Conversation

frosas
Copy link
Contributor

@frosas frosas commented Nov 27, 2012

See #398 (has been closed as it wasn't against the master branch)

@doctrinebot
Copy link

Hello,

thank you for positing this Pull Request. I have automatically opened an issue on our Jira Bug Tracker for you with the details of this Pull-Request. See the Link:

http://doctrine-project.org/jira/browse/DDC-2173

@beberlei
Copy link
Member

beberlei commented Jan 6, 2013

@guilhermeblanco whats your opinion? I think this is a +1 from me

@guilhermeblanco
Copy link
Member

@beberlei +1

@beberlei
Copy link
Member

beberlei commented Jan 6, 2013

Merged in c20cfed, added tests and UPGRADE.md entry.

@beberlei beberlei closed this Jan 6, 2013
@frosas
Copy link
Contributor Author

frosas commented Jan 7, 2013

Nice, and thank you for testing it!

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