craigmccoy / jquery-csv

Automatically exported from code.google.com/p/jquery-csv
MIT License
0 stars 0 forks source link

toArrays method does not work in IE9 #18

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Try to run your the tests (http://jquery-csv.googlecode.com/git/test/test.html) 
in IE9. The defect was discovered in other browsers as well, but running the 
tests in IE9 shows the worst possible outcome. 

The problem is an extra ',' at line 659 (rev.:5eacc4afc1d5) of jquery.csv.js:

state: {
          rowNum: 1,
          colNum: 1,  //here is the problem
        }

Link to source:
http://code.google.com/p/jquery-csv/source/browse/src/jquery.csv.js?spec=svn5eac
c4afc1d55714e45e2ceb891f930e0092c6f5&r=5eacc4afc1d55714e45e2ceb891f930e0092c6f5#
659

Original issue reported on code.google.com by andrey.stepantsov@gmail.com on 19 Nov 2012 at 4:54

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 78b3312a3693.

Original comment by evanpla...@gmail.com on 19 Nov 2012 at 9:05

GoogleCodeExporter commented 9 years ago
Interesting... Google code automagically closed this issue by detecting that a 
fix was documented in the repository commit.

Quick question, what build of IE9 are you using? I tried it on mine (IE9 x64) 
and it didn't show any problems.

Anyway, I added a new release (ie. 0.71) that includes the fix. If you would 
take a moment to verify that the issue is fixed I'd appreciate it.

Thank you for taking the time to report this bug.

Original comment by evanpla...@gmail.com on 20 Nov 2012 at 7:27

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
The issue is resolved. I have discovered it in IE9 (compatibility mode) and 
IE8, both browsers on 32bit and 64bit platforms. 
Thank you!

--Andrey

Original comment by andrey.stepantsov@gmail.com on 20 Nov 2012 at 9:46