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
Support from Acquia helps fund testing for Drupal Acquia logo

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