View Issue Details

IDProjectCategoryView StatusLast Update
0001589HTML & PERLBug Report - Interfacepublic2009-06-19 20:08
Reportertwpol Assigned Tofahrenheit  
PrioritynormalSeverityminorReproducibilityalways
Status resolvedResolutionfixed 
Summary0001589: [IE] anime.js: changeMylistState(): fails with "Object required" on line 1139 because indexOf is enumerated
DescriptionThis is the same problem as in http://tracker.anidb.net/view.php?id=1498, namely that "indexOf" is enumerated on arrays when using for...in in IE because it is added by the script framework rather than being a built-in.

1133 for (var eid in file.epRelations) eids[eid] = 1;

The above line should be replaced by the same code as for changeWatchedState():

1006 for (var eid in file.epRelations) {
1007 if (typeof file.epRelations[eid] != "function") eids[eid] = 1;
1008 }

I checked for other uses in the file that might suffer the same fate, but they all look like they ultimately skip the item thanks to sufficient checks.
Additional InformationMozilla/4.0 (compatible; MSIE 8.0; Windows NT 6.1; WOW64; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; OfficeLiveConnector.1.3; OfficeLivePatch.0.0)
TagsNo tags attached.

Activities

fahrenheit

2009-06-19 20:08

reporter   ~0002837

should be fixed, also commited the same fix to mylist.js.
thanks for the report

Issue History

Date Modified Username Field Change
2009-06-15 23:19 twpol New Issue
2009-06-19 20:08 fahrenheit Note Added: 0002837
2009-06-19 20:08 fahrenheit Status new => resolved
2009-06-19 20:08 fahrenheit Resolution open => fixed
2009-06-19 20:08 fahrenheit Assigned To => fahrenheit