#11149 closed (fixed)
error with def save_form_data --
Reported by: | Roman Kalinichenko | Owned by: | Karen Tracey |
---|---|---|---|
Component: | Database layer (models, ORM) | Version: | dev |
Severity: | Keywords: | ||
Cc: | Triage Stage: | Accepted | |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description (last modified by )
Hi everyone
I use django revision 10000 and it is normal. But when I update to revision more then 10200 that I get error:
cannot identify image fileRequest Method: POST Request URL: http://127.0.0.1:8000/users/a1/ Exception Type: IOError Exception Value: cannot identify image file Exception Location: /usr/lib/python2.5/site-packages/PIL/Image.py in open, line 1917 Python Executable: /usr/bin/python2.5 Python Version: 2.5.4
My code where is problem is:
class ThumbnailImageField(models.ImageField): def save_form_data(self, instance, data): if data and isinstance(data, UploadedFile): im = Image.open(data).convert("RGB")
if I change data to constant path that it will be work without error
Attachments (1)
Change History (12)
comment:1 by , 15 years ago
comment:2 by , 15 years ago
comment:3 by , 15 years ago
#11055 is not about admin, but about upload file rewinding (without it PIL can not recognize image). Please checkout curent trunk head (use svn co, not svn up) and post full traceback if you can confirm error.
by , 15 years ago
Attachment: | trackback.txt added |
---|
comment:4 by , 15 years ago
I make use svn co and get version beta 1 SVN-10825
but problem not disapears. So if i back to version to 10189 it work there.
I attach file with TrackBack :
comment:6 by , 15 years ago
milestone: | → 1.1 |
---|---|
Owner: | changed from | to
Status: | new → assigned |
Triage Stage: | Unreviewed → Accepted |
This looks like a bug in code added in r10190 that is resulting in file-type fields having their save_form_data methods be called twice. First time through it works, 2nd time it fails. Not sure why it fails the 2nd time around (wild guess the file read pointer is at the wrong place by then) but it's not really worth tracking down since we shouldn't be calling the save method twice, and fixing that fixes the error.
comment:7 by , 15 years ago
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
follow-up: 9 comment:8 by , 15 years ago
After update it not import
"from django.core.files.utils import FileProxyMixin"
comment:9 by , 15 years ago
Replying to romankrv:
After update it not import
"from django.core.files.utils import FileProxyMixin"
That would seem to indicate your checkout is not complete. Do you have the file django/core/files/utils.py? It was added quite recently and the only thing in it is FileProxyMixin:
http://code.djangoproject.com/browser/django/trunk/django/core/files/utils.py
comment:10 by , 15 years ago
You are right. I haven't it but I reinstall it and it work. Bug is fixed.
Good work cooparete.
Looks like #11055 duplicate. Does trunk head also has this error? And please post full traceback.