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

pass data to afterhooks, too #386

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

pass data to afterhooks, too #386

wants to merge 1 commit into from

Conversation

ktmud
Copy link
Contributor

@ktmud ktmud commented Mar 2, 2014

see #385

@1602
Copy link
Owner

1602 commented Mar 2, 2014

It only makes sense for validations before saving (such as afterValidate), but for all other it will be misleading. Could you please take that into account?

@ktmud
Copy link
Contributor Author

ktmud commented Mar 2, 2014

For afterSave, people can receive data for some kind of double check (though they may never need it). Setting up a new limitation doesn't bring much good, meanwhile introduced more complexity, and broke the consistency.

BTW, is it possible to rewrite the hook mechanism with hooks-js or something? Currently, adding multiple handler on one hook is painful.

@1602
Copy link
Owner

1602 commented Mar 2, 2014

Let's leave standardization with breaking changes to major release. This argument was introduced after hooks was implemented, to get proof of traction before making any breaking changes.

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.

2 participants