scieloorg / PC-Programs

SciELO PC Programs
7 stars 24 forks source link

Corrige leitura de arquivos de configuração no Windows #3301

Closed robertatakenaka closed 4 years ago

robertatakenaka commented 4 years ago

O que esse PR faz?

Corrige leitura de arquivos de configuração no Windows que "grudava" o conteúdo de um arquivo com o contéudo do seguinte.

Onde a revisão poderia começar?

por commits

Como este poderia ser testado manualmente?

No Windows, executando o XC ou instanciando a classe prodtools.config.config.Configuration.

Algum cenário de contexto que queira dar?

Diferentemente do XC do Linux, o XC do Windows pode usar mais de 1 arquivo de configuração. Originalmente o pacote de programas PC Programs já usava o arquivo scielo_paths.ini do qual se pode obter vários dados necessários para que o XC funcione, no entanto, com as novas versões vieram novas demandas de configuração, no entanto, opcionais, que foram resolvidas em arquivos de configuração à parte: scielo_collection.ini com configurações específicas da coleção e scielo_env.ini específicas de ambiente como proxy, acesso à internet etc.

Screenshots

Antes

O arquivo não contém quebra de linha no final, então o último valor "gruda" com ;

Captura de Tela 2020-07-07 às 11 09 10

O arquivo com quebra de linha no final, o o último valor fica correto

Captura de Tela 2020-07-07 às 11 10 59

Depois da correção

Captura de Tela 2020-07-07 às 11 23 41

Quais são tickets relevantes?

3299

Referências

n/a

robertatakenaka commented 4 years ago

desconsiderado e vou abrir um outro #3306