#3257: [patch] added choice fields that return models
-------------------------------------------------+--------------------------
Reporter: Honza Kr�l <[EMAIL PROTECTED]> | Owner: adrian
Status: new | Component: django.newforms
Version: | Resolution:
Keywords: | Stage: Design decision needed
Has_patch: 1 | Needs_docs: 1
Needs_tests: 1 | Needs_better_patch: 1
-------------------------------------------------+--------------------------
Comment (by Honza Kral <[EMAIL PROTECTED]>):
Replying to [comment:5 [EMAIL PROTECTED]:
1. i thoguht that since it's meant to return model,a failure to do so (
non-existent pk) is error
1. my bad - if you add {{{ if not new_value: return None }}} (or {{{ []
}}} for Multi...) after the call to (Multi)ChoiceField.clean(), it should
do the trick... you don't have to worry explicitly about the {{{ required
}}} because its handled by the ChoiceField.clean()
1. always good news ;)
thanks for working on the test, I really appreciate it.
--
Ticket URL: <http://code.djangoproject.com/ticket/3257#comment:6>
Django Code <http://code.djangoproject.com/>
The web framework for perfectionists with deadlines
--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Django
updates" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at
http://groups.google.com/group/django-updates?hl=en
-~----------~----~----~----~------~----~------~--~---
