mholt/PapaParse

Quote only strings and not numbers

Open

#532 aperta il 5 lug 2018

Vedi su GitHub
 (3 commenti) (0 reazioni) (0 assegnatari)JavaScript (1157 fork)batch import
enhancementhelp wanted

Metriche repository

Star
 (11.877 star)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

I'm parsing the following file (quoted strings, non-quotes numbers):

name,age,city
"Chirstian",30,"Hamburg"

The option dynamicTyping detects name and city as strings and the age as number. If I unparse, I have two options.

Option 1) set quoteChar: '"', which results in:
name,age,city
"Chirstian","30","Hamburg"
Option 1) Skip the option `quoteChar, which results in:
name,age,city
Chirstian,30,Hamburg

I recommend an option onlyQuoteStrings, which applies the quoteChar: '"' only to strings, but not to numbers, similar to libre office save as CSV dialogue (option quote all text cells):

bildschirmfoto vom 2018-07-05 23 59 54

Guida contributor

Direzione di ricerca
Esamina la funzione unparse nel codice sorgente di PapaParse (probabilmente papaparse.js o src/unparse.js) per capire come viene attualmente applicato quoteChar. Considera l'aggiunta di una nuova opzione onlyQuoteStrings che, quando true, applica le virgolette solo ai valori stringa lasciando i numeri senza virgolette. Rivedi i test esistenti per unparse per garantire che il nuovo comportamento sia testato.
Tech stack
javascript
Dominio
data
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
JavaScript proficiencyFamiliarity with PapaParse
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60