Opened 16 years ago

Closed 16 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: no UI/UX: no


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 by Brian Rosner, 16 years ago

Triage Stage: UnreviewedAccepted

comment:2 by Brian Rosner, 16 years ago

Owner: changed from nobody to Brian Rosner
Status: newassigned

comment:3 by Brian Rosner, 16 years ago

Almost forgot to reference some discussion on django-dev:

comment:4 by sjulean, 16 years ago

Cc: sjulean@… added

comment:5 by Brian Rosner, 16 years ago

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 by Brian Rosner, 16 years ago

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

comment:7 by Brian Rosner, 16 years ago

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