Project

General

Profile

Bug #5257

Incorrectly rendered column data

Added by Darren Hardy almost 9 years ago. Updated almost 9 years ago.

Status:
Resolved
Priority:
Normal
Category:
morpho - dsp
Target version:
Start date:
12/15/2010
Due date:
% Done:

0%

Estimated time:
Bugzilla-Id:
5257

Description

Morpho doesn't seem to handle escaped quotes correctly in some CSV files.

In this example (from the attached CSV file):

"Fort Jefferson, FL",USA
"Kealakakua, HI",USA

Morpho renders the the columns incorrectly (uses 3 columns rather than 2).

It may be a problem with the specific EML for this example, but it looks clean to me. The docid is pisco_smr_synthesis.1.3 and the data table is called "pisco_smr_studylist.csv".

This is running Morpho 1.9 on MacOS 10.6.

pisco_smr_synthesis.2.1-co_smr_synthesis.2.1.csv (21.9 KB) pisco_smr_synthesis.2.1-co_smr_synthesis.2.1.csv Darren Hardy, 12/15/2010 01:23 PM
screenshot.png (241 KB) screenshot.png Darren Hardy, 12/15/2010 01:24 PM
screenshot2.png (562 KB) screenshot2.png Darren Hardy, 12/15/2010 01:28 PM
Capture.PNG (113 KB) Capture.PNG Darren Hardy, 12/15/2010 01:36 PM

Related issues

Has duplicate Morpho - Bug #4636: Morpho should ignore commas inside double-quoted fields (CSV import)Resolved12/15/2009

History

#3 Updated by Darren Hardy almost 9 years ago

Morpho 1.9 on Windows7 also renders the data table incorrectly.

#4 Updated by Matt Jones almost 9 years ago

This is probably just a shortcoming in our home-grown CSV parser. Since we developed this, effective CSV libraries have arisen for Java, and we could probably easily incorporate one of those. Might want to explore using, e.g., JavaCSV (http://sourceforge.net/projects/javacsv/).

#5 Updated by ben leinfelder almost 9 years ago

there is a bug for this already. I've looked into it briefly but it did not make it into the 1.9.0 release

#6 Updated by ben leinfelder almost 9 years ago

I checked that the data columns for pisco_smr_synthesis.1.3 were rendering with my new change- and they are

#7 Updated by Redmine Admin over 6 years ago

Original Bugzilla ID was 5257

Also available in: Atom PDF