Project

General

Profile

Actions

Bug #5257

closed

Incorrectly rendered column data

Added by Darren Hardy about 14 years ago. Updated about 14 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.


Files

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)Resolvedben leinfelder12/15/2009

Actions
Actions #3

Updated by Darren Hardy about 14 years ago

Morpho 1.9 on Windows7 also renders the data table incorrectly.

Actions #4

Updated by Matt Jones about 14 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/).

Actions #5

Updated by ben leinfelder about 14 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

Actions #6

Updated by ben leinfelder about 14 years ago

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

Actions #7

Updated by Redmine Admin over 11 years ago

Original Bugzilla ID was 5257

Actions

Also available in: Atom PDF