mholt/PapaParse

Quote only strings and not numbers

Open

#532 aberto em 5 de jul. de 2018

Ver no GitHub
 (3 comments) (0 reactions) (0 assignees)JavaScript (1.157 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (11.877 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Examine a função unparse no código fonte do PapaParse (provavelmente papaparse.js ou src/unparse.js) para entender como quoteChar é atualmente aplicado. Considere adicionar uma nova opção onlyQuoteStrings que, quando true, aplica aspas apenas a valores de string, deixando números sem aspas. Revise os testes existentes para unparse para garantir que o novo comportamento seja testado.
Pilha de tecnologia
javascript
Domain
data
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScript proficiencyFamiliarity with PapaParse
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60