Opened 18 years ago
Closed 18 years ago
#3018 closed defect (worksforme)
[patch] FileField does not allow modifications with upload_to set
Reported by: | ido | Owned by: | Adrian Holovaty |
---|---|---|---|
Component: | contrib.admin | Version: | dev |
Severity: | normal | Keywords: | |
Cc: | Triage Stage: | Unreviewed | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
When you have FileField with the upload_to option; second time uploaded filename is always considered invalid.
Ie. you can create one instance, but you can't go back and edit this one and save. You always get:
Enter a valid filename.
This is very similar to ticket #2923
The issue is in: djang/db/models/fields/init__.py:def isWithinMediaRoot
this validator check explicitly if the uploaded file is in settings.MEDIA_ROOT
while part of the point of upload_to is that you can specify another directory.
i changed "settings.MEDIA_ROOT" to "self.upload_to or settings.MEDIA_ROOT"
to ensure that if upload_to is set, that location is used.
Attachments (1)
Change History (4)
by , 18 years ago
Attachment: | patch-db-models-fields-filefield.patch added |
---|
comment:1 by , 18 years ago
milestone: | → Version 1.0 |
---|---|
Version: | → SVN |
comment:3 by , 18 years ago
Resolution: | → worksforme |
---|---|
Status: | new → closed |
Works for me.
Ticket #2923 already fixed the problem.
Your patch isn't needed because the saved path in the filefield already includes the upload_to part.
patch for ticket #3018