Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

toArrays method does not work in IE9 #19

Closed
evanplaice opened this issue Sep 9, 2015 · 3 comments
Closed

toArrays method does not work in IE9 #19

evanplaice opened this issue Sep 9, 2015 · 3 comments
Labels
Milestone

Comments

@evanplaice
Copy link
Owner

From andrey.stepantsov@gmail.com on November 19, 2012 08:54:57

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: https://code.google.com/p/jquery-csv/source/browse/src/jquery.csv.js?spec=svn5eacc4afc1d55714e45e2ceb891f930e0092c6f5&r=5eacc4afc1d55714e45e2ceb891f930e0092c6f5#659

Original issue: http://code.google.com/p/jquery-csv/issues/detail?id=18

@evanplaice evanplaice added this to the backlog milestone Sep 9, 2015
@evanplaice
Copy link
Owner Author

From evanpla...@gmail.com on November 19, 2012 13:05:33

This issue was closed by revision 78b3312 .

Status: Fixed

@evanplaice
Copy link
Owner Author

From evanpla...@gmail.com on November 20, 2012 11:27:51

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.

Owner: evanpla...@gmail.com

@evanplaice
Copy link
Owner Author

From andrey.stepantsov@gmail.com on November 20, 2012 13:46:13

The issue is resolved. I have discovered it in IE9 (compatibility mode) and IE8, both browsers on 32bit and 64bit platforms.
Thank you!

--Andrey

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant