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.

CommentFileSizeAuthor
#7 1506642_parse_todo.patch629 bytesMrHaroldA
Members fund testing for the Drupal project. Drupal Association Learn more

Comments

Moloc’s picture

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?

slashrsm’s picture

Status: Active » Postponed (maintainer needs more info)

@succinct: Have you managed to fix this?

Johnny vd Laar’s picture

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...

Johnny vd Laar’s picture

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
		@*/
slashrsm’s picture

So there are no problems with Plupload?

Johnny vd Laar’s picture

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

MrHaroldA’s picture

Status: Postponed (maintainer needs more info) » Needs review
FileSize
629 bytes

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

slashrsm’s picture

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...