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

Recuperar o SciELO ID da versao aop para inserir na versao regular #3130

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

robertatakenaka
Copy link
Member

O que esse PR faz?

  • Recuperar o SciELO ID da versao aop para inserir na versao regular

Primeiramente, o XC identifica se o documento já foi registrado anteriormente.
Se sim, então recupera seu scielo_id, caso contrário cria um scielo_id.
Numa etapa seguinte, o XC verifica se o documento tem uma versão AOP. Se sim, o scielo_id desta versão é resgatado e atribuído ao documento.
Em resumo:

  • se o documento já teve uma versão AOP, é este scielo_id que vale.
  • se o documento foi registrado anteriormente, é o scielo_id do primeiro registro que vale.

Onde a revisão poderia começar?

  • src/scielo/bin/xml/app_modules/app/pkg_processors/pkg_processors.py:L125-127
    (Os scielo_id são recuperados do artigos anteriormente registrados)
  • src/scielo/bin/xml/app_modules/app/db/xc_models.py
    (scielo_id recuperado do registro do aop)
  • src/scielo/bin/xml/app_modules/app/pkg_processors/pkg_processors.py:L138-140
    (scielo_id é atualizado nos arquivos XML, se necessário)

Como este poderia ser testado manualmente?

Seria necessário ter a estrutura dos programas da metodologia (Title Manager, Bases, XML Converter).
Ter os pacotes aop e do fascículo.
abc-108-02.zip
abc-2017-ahead.zip

Executar o XC para o pacote aop

c:\python27\python xml_converter.py c:\scielo_data\pacotes\abc-2017-ahead

Parte do Registro do AOP, apresentando o campo 885

!v702!abc/2017nahead/0066-782X-abc-20170001.xml
!v705!S
!v706!h
!v708!1
!v709!article
!v885!XcyjWQJTWkqPpMXd8h8v8Xn

da base
Captura de Tela 2019-07-19 às 10 53 06

Executar o XC para o pacote do fascículo

c:\python27\python xml_converter.py c:\scielo_data\pacotes\abc-108-02

Parte do Registro do v108n2, apresentando o campo 885

!v702!abc/v108n2/0066-782X-abc-20170001.xml
!v705!S
!v706!h
!v708!1
!v709!article
!v881!S0066-782X2017005002103
!v885!XcyjWQJTWkqPpMXd8h8v8Xn

Captura de Tela 2019-07-19 às 11 34 14

Algum cenário de contexto que queira dar?

Por ter notado que a transferência (FTP) do pacote para o Kernel estava demorando, foi modificado para ser executado como uma tarefa assíncrona. Também foi notado que se o XC estiver rodando no mesmo servidor em que se espera disponibilizar os pacotes para o Kernel, não há necessidade de executar uma transferência, mas sim, uma cópia ou movimentação de arquivos.

Screenshots

n/a


Quais são tickets relevantes?

#3128, #3121

Referências

n/a

article_id_node.text != received.registered_scielo_id):
article_id_node.text = received.registered_scielo_id

if node is None:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

node sempre será None aqui, certo?

Copy link
Contributor

@patymori patymori left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fiz um comentário mas que não aponta um grande problema.

if article_id_node is None:
article_id_node = ET.Element("article-id")
article_id_node.set("specific-use", "scielo")
article_id_node.set("pub-type-id", "publisher-id")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deveria ser pub-id-type

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants