In the plupload module's plupload.js, IE throws the following error:

SCRIPT1004: Expected ';'
plupload.js?m1kcz7, line 84 character 21

This is due to the following code:

//@todo Implement a setting for whether the field is required, rather
//than assuming that all are.

The culprit is that IE is trying to interpret @todo even though it is inside a comment. Removing the @ solves this issue.

Files: 
CommentFileSizeAuthor
#7 1506642_parse_todo.patch629 bytesMrHaroldA

Comments

With which version of IE (and which OS) are you able to reproduce this issue? I am not able to reproduce this with IE8 on XP.
When does the error occur? On page load?
Do you have JavaScript-File aggregation in ?q=admin/config/development/performance enabled or disabled?

Status:Active» Postponed (maintainer needs more info)

@succinct: Have you managed to fix this?

I'm getting the same error in IE 9 / Windows 7

I think it has to do with this:
http://stackoverflow.com/questions/3091802/reason-behind-a-javascript-pa...

My problem is related to the use of shadowbox module that does this:

/*@cc_on
ie = true;
@if (@_win32)
windows = true;
@elif (@_mac)
mac = true;
@end
@*/

So there are no problems with Plupload?

I guess not. Another module is doing a cc_on which breaks plupload

Status:Postponed (maintainer needs more info)» Needs review
StatusFileSize
new629 bytes

It's strictly not a plupload issue, but patching it anyway to prevent clashes with IE's weird comment parsing ...

Status:Needs review» Fixed

Thanks. Committed to 7.x-1.x and 7.x-2.x.

Status:Fixed» Closed (fixed)

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

  • Commit 9c092bb on 7.x-1.x, 7.x-2.x, 8.x-1.x authored by MrHaroldA, committed by slashrsm: Issue #1506642 by MrHaroldA | succinct: Fixed plupload.js issue with...
  • Commit 417cd7d on 7.x-1.x, 8.x-1.x authored by MrHaroldA, committed by slashrsm: Issue #1506642 by MrHaroldA | succinct: Fixed plupload.js issue with...