#3388: [patch] newforms: form.save() fails with DoesNotExist exception when form
contains ForeignKey field
---------------------------------+------------------------------------------
Reporter: [EMAIL PROTECTED] | Owner: adrian
Status: closed | Component: django.newforms
Version: SVN | Resolution: invalid
Keywords: | Stage: Accepted
Has_patch: 1 | Needs_docs: 0
Needs_tests: 1 | Needs_better_patch: 0
---------------------------------+------------------------------------------
Changes (by [EMAIL PROTECTED]):
* status: new => closed
* resolution: => invalid
Comment:
Adrian, sorry, it seems to be my mistake.
#3257 required clean_data to contain ForeignKey object (for example Writer
object for Article, if we speak about unittests in
tests/modeltests/model_forms/models.py), your current changes require it
to contain ForeignKey object's id, so my code, based on #3257, failed and
I sent this patch to fix it.
Is there any page that could explain your vision of future features to not
make such mistakes again ? For example, I sent the patch #3370 (mysql
should set 'charset': 'utf8' to connection attributes), with this patch my
app works ok but 8 tests fail (db returns unicode), without it all tests
passed but in my app i see a mess of national charset and unicode strings
on one page.
--
Ticket URL: <http://code.djangoproject.com/ticket/3388#comment:4>
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
-~----------~----~----~----~------~----~------~--~---
