Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Corrige data de atualização para registros do tipo issue #45

Merged
merged 2 commits into from
Jun 11, 2020

Conversation

joffilyfe
Copy link

O que esse PR faz?

Este pull request corrige a criação de arquivos id para as entidades do tipo issue. O campo v91 deve ter seu conteúdo preenchido com a data de atualização do registro.

Onde a revisão poderia começar?

  • delorean/templates/issue_db_entry.txt

Como este poderia ser testado manualmente?

Para testar este pull request manualmente, deve-se:

  • Configurar o arquivo development.ini ou production.ini com credenciais de acesso ao SciELO Manager;
  • Iniciar a aplicação (pserve development.ini);
  • Acessar o endereço para gerar o arquivo id (http://0.0.0.0:6543/generate/title?collection=saude-publica);
  • Descompacte o arquivo .tar;
  • Abra o arquivo issue.id;
  • Procure pelo registro de mfn=2312;
  • Verifique a chave v91 com valor igual a 20200527;

Algum cenário de contexto que queira dar?

Este BUG ocasiona a não atualização das bases do ArticlesMeta. Em testes realizados, nós temos pelo menos 1900 registros de um total de 2357 que serão atualizados nos próximos processamentos.

Exemplo

Como exemplo de atualização nós temos a issue de id 0102-311X20200006 com as seguintes atualizações pendentes https://gist.github.com/joffilyfe/95ecea7b004441f227e4e3f0b2396651

Quais são tickets relevantes?

scieloorg/isis2mongo/issues/19

Referências

N/A

O campo 091 das entidades do tipo `issue` representa a data de atualização
e não a data de criação. Esta troca de semântica acaba por inviabilizar
a atualização de registros por outras aplicações.

Com essa modificação o arquivo `id` passa a ser mais correto e fidedigno
com as especificações da base MST.
@joffilyfe joffilyfe added the bug label Jun 11, 2020
@joffilyfe joffilyfe merged commit 7fc8089 into scieloorg:master Jun 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants