Comments (8)
Have you tried the latest release (ie 0.62)?
Original comment by [email protected]
on 3 Oct 2012 at 2:47
from jquery-csv.
Well, I was successfully able to reproduce the bug.
The one caused by the data set you posted was fixed in 0.62:
"Del","Trotter",,"0123 456 789"
But there's another elusive one where quoted but empty values get skipped.
That would look like this:
"Del", "Trotter","", "0123 456 789"
The first step is to pinpoint if your use case still fails in IE8 with the
latest code. I don't currently have an IE8 environment setup (I run IE9) but I
have included a null value test in the latest release (0.63) that should
pinpoint the specific problem.
What I need from you is to run the test runner and post the results.
Fortunately, it's very easy.
The link for the test runner is:
http://jquery-csv.googlecode.com/git/test/test.html
With the following data:
"Are you thinking what I\'m thinking?",,"aim for the bushes"
"Are you thinking what I\'m thinking?","","aim for the bushes"
The test should fail with the result:
[
[
"Are you thinking what I'm thinking?",
"",
"aim for the bushes"
],
[
"Are you thinking what I'm thinking?",
"aim for the bushes"
]
]
IE. the first entry makes passes but the second one fails because the null
value is skipped during the parsing phase.
Original comment by [email protected]
on 3 Oct 2012 at 5:15
- Changed state: Accepted
from jquery-csv.
I just use Virtualbox
Original comment by [email protected]
on 4 Oct 2012 at 8:57
Attachments:
from jquery-csv.
OK, that means it's currently working consistently with the rest of the
browsers.
So, this:
"Del","Trotter",,"0123 456 789"
Should parse properly in 0.63.
If you were to write it as:
"Del", "Trotter","","0123 456 789"
It will fail.
*Note: I use VirtualBox too but am too lazy to spend the time setting up a new
windows image.*
Also, it's a good idea to remove the spaces before/after the commas. They're
allowed for now but the next parser implementation will throw errors on 'dirty'
data.
Original comment by [email protected]
on 4 Oct 2012 at 1:26
from jquery-csv.
OK, it's fixed...
It turns out that one of the previous bugfixes needed to skip empty delimited
matches for a good reason but I neglected to notice that another check was
needed to add them back in.
If you could run the test runner again to verify that it's working, I would
appreciate it. Then I'll be able to close this once and for all.
Note: The fix will be incorporated into 0.64 when it's released. Until then you
can get the working source from the repository.
Original comment by [email protected]
on 6 Oct 2012 at 3:01
from jquery-csv.
Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1)
Tests completed in 221 milliseconds.
11 tests of 11 passed, 0 failed.
Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 5.1)
Tests completed in 131 milliseconds.
11 tests of 11 passed, 0 failed.
Mozilla/4.0 (compatible; MSIE 8.0; Windows NT 5.1; Trident/4.0)
Tests completed in 132 milliseconds.
11 tests of 11 passed, 0 failed.
Excellent :-)
Original comment by [email protected]
on 6 Oct 2012 at 10:20
from jquery-csv.
Original comment by [email protected]
on 7 Oct 2012 at 2:55
- Changed state: Verified
from jquery-csv.
Original comment by [email protected]
on 11 Oct 2012 at 4:08
from jquery-csv.
Related Issues (20)
- doesn't seem to recoginse end of line properly in some cases. HOT 1
- CR line ending are not well treated HOT 1
- Typo in basic-usage.html (fixed in 0.8.0) HOT 2
- Enhancement: Parse CSV in a thread using settimeout HOT 3
- 배열의 기본 속성값(remove)이 포함 됩니다. HOT 1
- Karma test fails - HOT 2
- Failures on test.html file HOT 2
- colNum not reset when onParseValue callback called
- Export $ globally in browser when not already set
- CSVDataError: Illegal State - Only in IE HOT 3
- Using Zurb Foundation 3 causing an error HOT 3
- $.csv.toObjects return garbage value HOT 3
- Parse doesn't handle quotes in the CSV, even if escaped HOT 3
- Hooks relying on a return value of false to skip entries makes parsing values that resolve to false impossible
- CSV file created with Excel for Mac fails HOT 3
- Uncaught ReferenceError: require is not defined
- Generate Motion chart HOT 1
- Options arg not really optional? HOT 2
- TypeError: csv.replace is not a function
- Data Format
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from jquery-csv.