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

Rewrite Updatehandler #533

Closed
JedWatson opened this issue Aug 11, 2014 · 2 comments
Closed

Rewrite Updatehandler #533

JedWatson opened this issue Aug 11, 2014 · 2 comments
Assignees

Comments

@JedWatson
Copy link
Member

Getting ready for custom field types, and as hi lighted in #526, we need to fix the "inversion of control" problems with the UpdateHandler.

It currently has hard-coded logic branches to handle the requirements of different built-in field types; these will need to be refactored into a format that field types can register hooks with (or respond to with specific, optional methods).

We're halfway there with things like the getRequestHandler methods on some fields, we just need to finish cleaning it up.

@grabbou
Copy link
Contributor

grabbou commented Aug 11, 2014

As it's not major issue, we can fix that after finishing #526. That's why I suggest switching to 0.3 milestone. What you think about that?

@creynders
Copy link
Contributor

updateHandler will be dropped entirely

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants