scieloorg / Web

SciELO Web
www.scielo.br
6 stars 21 forks source link

Apresenta todos os resumos e palavras-chaves no texto #700

Closed robertatakenaka closed 4 years ago

robertatakenaka commented 4 years ago

O que esse PR faz?

Apresenta todos os resumos e palavras-chaves no texto, independentemente do idioma do texto, ou de onde está declarado na estrutura do XML.

Onde a revisão poderia começar?

Revisar por commits.

Como este poderia ser testado manualmente?

ter uma instância com a correção.

Algum cenário de contexto que queira dar?

A lógica anterior, somente apresentava o resumo no contexto do article-meta ou sub-article. Então, o texto completo era proveniente do article/body, os resumos que estavam em article-meta eram apresentados, excluindo os resumos encontrados nos sub-articles. Caso o texto completo era proveniente do sub-article/body, eram apresentados os resumos do contexto do sub-article.

Em consequência, desta mudança para quaisquer textos, serão apresentados todos os resumos.

Screenshots

Como estava:

No texto em Português

Captura de Tela 2019-11-12 às 17 48 16

No texto em Inglês

Captura de Tela 2019-11-12 às 17 48 05

Como ficou: No texto em Português:

Captura de Tela 2019-11-12 às 17 48 55

No texto em Inglês:

Captura de Tela 2019-11-12 às 17 49 12

Quais são tickets relevantes?

698

Referências

n/a

robertatakenaka commented 4 years ago

@robertatakenaka testando o artigo S0034-70942019000300227 o sub-article não exibe o texto completo.

Segue a imagem:

Screen Shot 2019-11-25 at 10 00 07

O problema era a ausência da definição da variável lang.

robertatakenaka commented 4 years ago

@joffilyfe estava incorreto como eu estava identificando o idioma do abstract que não contém @xml:lang, então tem-se que buscar nos elementos superiores, e esta parte estava com erro.