Skip to content

Conversation

@robertatakenaka
Copy link
Member

@robertatakenaka robertatakenaka commented Jan 27, 2023

O que esse PR faz?

Faz correções de XSL:

  • texto de relacionamento de adendos

Faz correções em classe que obtém dados de XML:

  • sobre financiamento
  • ArticleIds
  • mimetype

Onde a revisão poderia começar?

por commits

Como este poderia ser testado manualmente?

Texto do adendo

python packtools/htmlgenerator.py --nonetwork --nochecks --loglevel DEBUG   tests/fixtures/htmlgenerator/related-article/addendum.xml

Algum cenário de contexto que queira dar?

Sobre o texto do adendo o valor de related-article-type="addendum" em um article/@article-type="addendum" está inadequado, pois está indicando que é um adendo relacionado com outro. No entanto, foi feita esta decisão para evitar criar novos valores para este atributo, mas causa "conflito". Então, para a solução apropriado, considerou o valor de article/@article-type="addendum" para decidir se o texto é This addendum adds information to the following document ou This document has an addendum. Pois no caso de article/@article-type='research-article', por ex, ter um adendo relacionado o related-article-type="addendum", estaria apropriado e o texto seria This document has an addendum.

Screenshots

n/a

Quais são tickets relevantes?

#361 (comment)
scieloorg/opac#2589

Referências

n/a

@robertatakenaka robertatakenaka changed the title Faz correções de XSL e na obtenção de dados de XML sobre financiamento Faz na apresentação de relacionamento de adendo e correções nas classes de obtenção de dados de XML SPS Feb 2, 2023
@gitnnolabs
Copy link
Collaborator

@robertatakenaka
Copy link
Member Author

@gitnnolabs Já tive este problema o segundo funcionou porque na verdade está rodando a versão anterior.
A que não está funcionando é algum problema no ambiente virtual.
Tente criar o ambiente virtual do zero e instalar tudo novamente.

@gitnnolabs
Copy link
Collaborator

Deu certo!

@gitnnolabs gitnnolabs merged commit eeec283 into scieloorg:master Feb 2, 2023
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.

2 participants