Ticket #5951 (confirmed Bug)

Opened 9 years ago

Last modified 4 years ago

Uploaded images / files lost on validation error

Reported by: deichi Owned by: alecm
Priority: minor Milestone: 4.x
Component: General Version: 4.2
Keywords: Cc:

Description

Plone loses files or images that were uploaded in context of a form that throws a valdiation error.

Steps to repoduce: 1) add a newsitem on your site 2) select an image to upload from your computer 3) leave the title empty 4) click save 5) look at the shiny "Title is required" error message 6) scroll down to the image field: it is empty!

( Do that with a 100MB image again to find out how annoying that can be ;) )

Actually this is one of the most important features of a CMS... (keeping form data on errors) I would mark this as blocker, but since it is mostly annoying only and a major UI flaw, major might be enough.

Change History

comment:1 Changed 9 years ago by hannosch

  • Owner changed from somebody to alecm
  • Component changed from Unknown to Content Types

comment:2 Changed 9 years ago by alecm

  • Priority changed from major to minor
  • Milestone changed from 2.5.x to 3.0

Is it even possible to postback values to posted file fields? JS validation will help a great deal here. I have serious doubts that this is fixable for 2.5 unless there's some HTTP magic I don't know about. Anyway you look at it (even if the file field were to retain its data) you are going to have to upload the file twice, that's just the nature of HTTP and the Zope transaction machinery.

comment:3 Changed 9 years ago by wichert

  • Milestone changed from 3.0 to Future

It is not. The online validation in Plone 3.0 should fix this for most cases. To fully fix this we will need to create an object even if the add form does not validate.

comment:4 Changed 7 years ago by hannosch

  • Component changed from Content Types to Infrastructure

comment:5 Changed 4 years ago by davisagli

  • Component changed from Infrastructure to General

comment:6 Changed 4 years ago by kleist

  • Status changed from new to confirmed

Still an issue after 6 years?

comment:7 Changed 4 years ago by tills

  • Version set to 4.1

Yes, it is. I just tried it on Plone 4.2b2 with a custom dexterity type using the NamedFileFieldWidget from plone.formwidget.namedfile.

comment:8 Changed 4 years ago by kleist

  • Version changed from 4.1 to 4.2
  • Milestone changed from Future to 4.x
Note: See TracTickets for help on using tickets.