Note: This is for the D7 version.

There's a TODO in plupload.js to get rid of the alert("Your files have not yet been uploaded") and instead just submit the form for them. The code to do that was commented out because it was causing errors in some runtimes. Patch forthcoming.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

ksenzee’s picture

Status: Active » Needs review
FileSize
1.74 KB

Patch attached. In my testing this worked in FF and IE on Ubuntu and WinXP using html5, flash, and html4.

greggles’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev

:)

effulgentsia’s picture

Status: Needs review » Reviewed & tested by the community

Code makes sense, and is getting lots of testing from drupalgardens.com users.

@greggles: If you're ok with it, please add me as a committer to the project. I promise to only commit to the D7 version. Or if you'd like to chat first, to coordinate vision for D7, please let me know. Thanks.

pwolanin’s picture

We have this in gardens - I think it's rtbc

effulgentsia’s picture

#1 was patched against a local copy containing changes not in HEAD yet. Here's the full patch, which fixes issues with the html4 runtime, as well as what is covered by the #1 patch.

effulgentsia’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

Status: Fixed » Closed (fixed)

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

  • Commit 57e0cc3 on master, 7.x-2.x, 8.x-1.x by effulgentsia:
    [#898880-5]: Workaround for bugs in html4 runtime plus an improvement of...