Opened 9 years ago

Closed 9 years ago

#5721 closed (fixed)

Objects with file fields, edited inline, throws validation errors on edit

Reported by: reidab@… Owned by: Brian Rosner
Component: contrib.admin Version: newforms-admin
Severity: Keywords: nfa-blocker
Cc: sjulean@… Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

This is similar to #26.

I have an object with multiple related file attachments, each represented by a related object that i'm editing inline (tabular).

When I edit the parent object, it displays the current value of the file field as a link above the file input.
If I save the parent object, the empty file fields (labeled Change:) throw "This field is required" errors, even though the field has a current value.

It seems that validation on file fields should check both for input (and replace if it exists) and for an existing value.

Change History (7)

comment:1 Changed 9 years ago by Brian Rosner

Triage Stage: UnreviewedAccepted

comment:2 Changed 9 years ago by Brian Rosner

Owner: changed from nobody to Brian Rosner
Status: newassigned

comment:3 Changed 9 years ago by Brian Rosner

Almost forgot to reference some discussion on django-dev: http://groups.google.com/group/django-developers/browse_frm/thread/b2a177e123e14947

comment:4 Changed 9 years ago by sjulean

Cc: sjulean@… added

comment:5 Changed 9 years ago by Brian Rosner

Keywords: nfa-blocker added; fileuploadfield filefield admin newforms-admin removed

This is something that is broken in newforms-admin and doesn't match current trunk's admin behavior. I will have a patch for this soon.

comment:6 Changed 9 years ago by Brian Rosner

#6241 really supersedes this ticket since the attached patch there fixes this problem. Won't close this ticket quite yet.

comment:7 Changed 9 years ago by Brian Rosner

Resolution: fixed
Status: assignedclosed

This was fixed in r7021 and merged over in r7022.

Note: See TracTickets for help on using tickets.
Back to Top