Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#7 | 1506642_parse_todo.patch | 629 bytes | MrHaroldA |
Comments
Comment #1
Moloc CreditAttribution: Moloc commentedWith 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?
Comment #2
slashrsm CreditAttribution: slashrsm commented@succinct: Have you managed to fix this?
Comment #3
Johnny vd Laar CreditAttribution: Johnny vd Laar commentedI'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...
Comment #4
Johnny vd Laar CreditAttribution: Johnny vd Laar commentedMy problem is related to the use of shadowbox module that does this:
Comment #5
slashrsm CreditAttribution: slashrsm commentedSo there are no problems with Plupload?
Comment #6
Johnny vd Laar CreditAttribution: Johnny vd Laar commentedI guess not. Another module is doing a cc_on which breaks plupload
Comment #7
MrHaroldA CreditAttribution: MrHaroldA commentedIt's strictly not a plupload issue, but patching it anyway to prevent clashes with IE's weird comment parsing ...
Comment #8
slashrsm CreditAttribution: slashrsm commentedThanks. Committed to 7.x-1.x and 7.x-2.x.