Multiple fields with plup [#1577998]

If we have several fields using plup upload, the files will go to the last one only.

As a test, create a new content type that has two image fields and that are using plup as widget. Then just upload files to the first widget and save. Go to edit age again and you will see that you actually uploaded files to the second widget.

Comment	File	Size	Author
#5	plup.js.patch	7.3 KB	justindhoffman
#5	plup.module.patch	3.12 KB	justindhoffman
#5	plup.tar.gz	18.6 KB	justindhoffman

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

BrianLewisDesign CreditAttribution: BrianLewisDesign commented 3 June 2012 at 17:30

same, all uploads go to the last one plup field.

Comment #2

justindhoffman CreditAttribution: justindhoffman commented 22 June 2012 at 13:42

The issue is that plup fields are created and identified with html id not class.

Comment #3

JessicaDenkelaar CreditAttribution: JessicaDenkelaar commented 26 June 2012 at 08:37

I've got the same problem. I think the solution lies in what Justin says. Has someone already have a distribution with classes instead of ID's?

Comment #4

Anonymous (not verified) CreditAttribution: Anonymous commented 3 July 2012 at 16:36

Status:

Active

» Closed (fixed)

Should work with the latest dev.

Comment #5

justindhoffman CreditAttribution: justindhoffman commented 4 July 2012 at 12:41

Status:

Closed (fixed)

» Needs review

File	Size
plup.tar.gz	18.6 KB
plup.module.patch	3.12 KB
plup.js.patch	7.3 KB

The posted dev (from July 3) does not work with multiple fields. It doesn't incorporate separate IDs for each field, nor separate settings for each, as has been discussed here.

I have come up with these patches from the lastest dev version. I also attached a complete working folder for those bad at patches.

I have tested them and they work for me. Hopefully someone else will find them useful.