When using plupload, and when adding several tags at once (for several atoms) the following problem occurs:
Only one tag is added. It is the concatenation off all tags with double quotes around them.
If I fill tag1, tag2 in the scald tag's field, I would expect to add those 2 new terms in the vocabulary. But in fact only one therm will be added: "tag1, tag2".
Note that when adding only one tag, no double quotes are added

CommentFileSizeAuthor
#4 2279965-plupload-tags.patch1.98 KBjcisio
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DuneBL’s picture

DuneBL
Primary language French
CreditAttribution: DuneBL commented
Title: Plupload can't add multiple tags at one » Plupload can't add multiple tags at once
DuneBL’s picture

DuneBL
Primary language French
CreditAttribution: DuneBL commented
Issue summary: View changes
davidhk’s picture

davidhk CreditAttribution: davidhk commented

I can recreate this on simplytest.me:

1. Click "Create atom"
2. Click "Image"
3. Click "add files" and select two jpg files
4. On the Add screen, in the Tags field, type tag1,tag2
5. Click Continue
6. On the Options screen, each image's Tags field shows "tag1,tag2"
7. Click Finish

The atom is created with a single tag tag1,tag2, instead of two tags tag1 and tag2

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Active » Needs review
FileSize
2279965-plupload-tags.patch1.98 KB
jcisio’s picture

jcisio CreditAttribution: jcisio commented
Component: Scald core » Providers
DuneBL’s picture

DuneBL
Primary language French
CreditAttribution: DuneBL commented

Hello jcisio, I confirm that your patch in #4 is working!!!
Many thanks for this great job.

jcisio’s picture

jcisio CreditAttribution: jcisio commented
Status: Needs review » Fixed

Committed and pushed! Thanks.

  • Commit 8869de1 on 7.x-1.x by jcisio: 
    Issue #2279965 by jcisio | DuneBL: Fixed Plupload can't add multiple...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.