scieloorg / articles_meta

Webservices para recuperar metadados de artigos SciELO armazenados no MongoDB
BSD 2-Clause "Simplified" License
7 stars 17 forks source link

Adiciona fpage/@seq ao formato xmlrsps #216

Closed gustavofonseca closed 4 years ago

gustavofonseca commented 4 years ago

Adiciona o atributo fpage/@seq no front do formato xmlrsps quando este valor estiver disponível.

O valor do atributo @seq é importante para desambiguar documentos que iniciam numa mesma página de um mesmo fascículo.

Onde a revisão poderia começar?

n/a

Como este poderia ser testado manualmente?

Após aplicar o patch, teste com o documento /api/v1/article/?body=false&collection=scl&code=S0001-37652000000400034&format=xmlrsps, conforme descrito no ticket #213.

Algum cenário de contexto que queira dar?

n/a

Screenshots

n/a

Quais são tickets relevantes?

213

Referências

n/a

robertatakenaka commented 4 years ago

@gustavofonseca @joffilyfe @jamilatta @patymori isso resolverá somente para os documento que contém o fpage/@seq. Mas não para os HTML. No caso do HTML, não existe o v14^s, mas os artigos são desambiguados pelo order.

joffilyfe commented 4 years ago

@gustavofonseca @joffilyfe @jamilatta @patymori isso resolverá somente para os documento que contém o fpage/@seq. Mas não para os HTML. No caso do HTML, não existe o v14^s, mas os artigos são desambiguados pelo order.

@robertatakenaka nós observamos o fato para o artigo (http://articlemeta.scielo.org/api/v1/article/?body=false&format=xmlrsps&collection=scl&code=S0001-37652000000400034), ele é um artigo em HTML e foi publicado em 2000. Esse artigo em específico possui o campo v14:

Screen Shot 2020-03-19 at 18 23 21

Isso é uma exclusividade apenas desse artigo?

robertatakenaka commented 4 years ago

@gustavofonseca @joffilyfe @jamilatta @patymori isso resolverá somente para os documento que contém o fpage/@seq. Mas não para os HTML. No caso do HTML, não existe o v14^s, mas os artigos são desambiguados pelo order.

@robertatakenaka nós observamos o fato para o artigo (http://articlemeta.scielo.org/api/v1/article/?body=false&format=xmlrsps&collection=scl&code=S0001-37652000000400034), ele é um artigo em HTML e foi publicado em 2000. Esse artigo em específico possui o campo v14:

Screen Shot 2020-03-19 at 18 23 21

Isso é uma exclusividade apenas desse artigo?

hum... Não... pelo jeito o xylose está pegando o v121 (order) para garantir o v14^s, o que está tudo bem!

gustavofonseca commented 4 years ago

v121

No código do start_page_sequence, no xylose, não há nenhuma lógica envolvendo v121 ou order. É algo para nos preocuparmos?

https://github.com/scieloorg/xylose/blob/56ea916556da5fd743b2c2105247d98a72c79314/xylose/scielodocument.py#L2035-L2053

gustavofonseca commented 4 years ago

O time de desenvolvimento concorda que a mudança deste PR está correta, e se por ventura for necessária outra esta será no contexto do xylose.