On line 59 in js/video.js, there's the following code at the end of the onComplete function:

},

The comma throws an error in Internet Explorer 7 and 8 (likely due to buggy JSON support):

Expected identifier, string or number
video.js
Code 0
Line 60
Char: 3

Removing the comma gets rid of the error, and doesn't affect other browsers.

Comments

senpai’s picture

senpai commented
Title: JavaScript error in Internet Explorer » "Expected identifier, string or number" JavaScript error in Internet Explorer
Version: 6.x-4.0-rc1 » 6.x-4.0-rc2

I'm working on this module right now for a client. I had already found and fixed this bug locally, but hadn't posted an issue. Glad you did. :)

I'll keep this tab open and roll a patch for this fix tomorrow.

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented
Status: Active » Fixed

Fixed in Dev

senpai’s picture

senpai commented

Sweet, thanks. Now I don't have to generate a 15 line text file that removes a single character from another text file.

I feel like we've saved the planet here today, even just a little bit. ;)

hypertext200’s picture

hypertext200
he/him
Primary language Singhalese
Location New York City
commented

he he, Thanks for your contributions :).

Status: Fixed » Closed (fixed)

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