up.settings.container is a DOM element so we don't need to prefix it with '#'.

CommentFileSizeAuthor
#2 plupload_widget-2782671-fix-js-1.patch1.08 KByannickoo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

yannickoo created an issue. See original summary.

yannickoo’s picture

yannickoo
Location Berlin
CreditAttribution: yannickoo at Wunder commented
Status: Active » Needs review
FileSize
plupload_widget-2782671-fix-js-1.patch1.08 KB
budalokko’s picture

budalokko CreditAttribution: budalokko commented
Status: Needs review » Reviewed & tested by the community

The problem appears for plupload library >= 2.1.9

The patch by yannickoo solves it.

NWOM’s picture

NWOM CreditAttribution: NWOM commented

This also fixed the ajax issue where the file description would not show after uploading a file (while in the same form). Before this patch, editing the file description was only possible when re-editing an existing node.

The patch applied cleanly and works great. Thank you.

Chris Charlton’s picture

Chris Charlton
Primary language English
Location Los Angeles
CreditAttribution: Chris Charlton as a volunteer commented

+1 patch #3.

cweagans’s picture

cweagans
He/Him
Primary language English
Location Boise, ID, USA
CreditAttribution: cweagans as a volunteer and at Digital Deployment, Inc. commented
Related issues: +#3107637: Known good configuration
budalokko’s picture

budalokko CreditAttribution: budalokko commented
Status: Reviewed & tested by the community » Fixed

Commited to 2.0.x-dev branch. Thanks. I had to add some checks on the PHP side for the ajax issue cited by NWOM to be solved: the widget value method is called multiple times.

Status: Fixed » Closed (fixed)

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