AtlasOfLivingAustralia / sandbox

User testing and data upload service
https://sandbox.ala.org.au
0 stars 4 forks source link

sandbox - double-quotes not handled well #13

Closed nickdos closed 9 years ago

nickdos commented 9 years ago

From @djtfmartin on August 19, 2014 10:36

migrated from: https://code.google.com/p/ala/issues/detail?id=136 date: Thu Aug 8 16:40:49 2013 author: moyesyside


Original Issue reported by Reported by john.t...@austmus.gov.au, Mar 7, 2012 - https://code.google.com/p/ala-portal/issues/detail?id=186

Reported by john.tann@austmus.gov.au, Mar 7, 2012 Sandbox doesn't cope well with a file that has (an odd number of) double-quotes in it.

What steps will reproduce the problem?

  1. Copy contents of first attached file (Double-quotes-is-a-problem.csv) with 10 records into sandbox
    • note that one record (J57153) has an odd number of double-quotes
      1. View data - only 6 records
      2. Copy contents of second attached file (Double-quotes-is-not-a-problem.csv) with 10 records into sandbox
    • note that each record has an even number of double-quotes
      1. View data - all 10 records

What is the expected output? All 10 records should appear in the viewed data

What do you see instead? Some records are missing - those after the record with the odd number of double quotes.

What version of the product are you using? On what operating system? 8 march version of sandbox Double quotes is a problem.csv 1.1 KB   View   Download Double quotes is not a problem.csv 1.2 KB   View   Download Mar 15, 2012 Project Member #1 moyesyside@gmail.com (No comment was entered for this change.) Labels: sandbox

Copied from original issue: AtlasOfLivingAustralia/biocache-hubs#27

nickdos commented 9 years ago

From @djtfmartin on August 19, 2014 10:36

date: Thu Aug 8 22:29:21 2013 author: moyesyside


(No comment was entered for this change.)


UPDATES: Labels: Sandbox

nickdos commented 9 years ago

From @mbohun on August 19, 2014 10:36

date: Mon Aug 12 12:44:06 2013 author: johntan...@gmail.com


August 2013: Still wanted

djtfmartin commented 9 years ago

this should be fixed as we replaced the underlying CSV library for this reason.