scieloorg / scielo.org

Website institucional da Rede SciELO
MIT License
1 stars 7 forks source link

Mau funcionamento do filtro de periódicos ativos, descontinuados e todos #66

Open gustavofonseca opened 5 years ago

gustavofonseca commented 5 years ago
  1. Na interface que apresenta a lista completa dos periódicos, digite o termo "ágora" na caixa de busca: screen shot 2018-12-17 at 15 26 11

Serão exibidos 2 resultados de periódicos ativos.

  1. Clique no filtro "Periódicos ativos": screen shot 2018-12-17 at 15 27 40

Apenas 1 dos periódicos é exibido, o que está errado.

Atualização de 2018-12-19 A partir deste ticket foram identificadas novas atividades que estão fora do seu escopo, e deverão ser tratadas separadamente, em tickets próprios.

As atividades relativas a este ticket são:

Note que documentos com status de publicação in progress nunca devem ser exibidos.

ghost commented 5 years ago

@gustavofonseca

Gustavo, acredito que essa questão está relacionada com meu comentário na issue 63 .

Acredito que o periódico em questão possui um status diferente de current e deceased. Porém, está com o marcador verde.

Também, acredito que a solução para ambos poderia ser tratar uma propriedade ou uma lista de exclusão vinda do backend para ser usada no processamento da lista.

Ou, se não estiver relacionado, eu posso criar um condicional para persistir somente registros que contenham os dois status durante o processamento dos dados e geração do SQL. Já podendo buscar uma melhor solução do issue 2 que trata da otimização deste processo.

Atenciosamente, Juliano.

gustavofonseca commented 5 years ago

@julianomaciel, realmente aparenta relação com o issue #63.

Não me sinto apto no momento para sugerir ou discutir estratégias de implementação, mas penso que esta funcionalidade precisa estar totalmente alinhada com a que determina a cor das bolinhas (verde ou vermelha) dos periódicos.

ghost commented 5 years ago

@gustavofonseca

Gustavo, poderia fazer a gentileza de avaliar no servidor de homologação a solicitação:

esta funcionalidade precisa estar totalmente alinhada com a que determina a cor das bolinhas (verde ou vermelha) dos periódicos

A url é:

http://scielohomolog.parati.ag/scielo-org-site/pt/periodicos/listar-por-ordem-alfabetica

Adicionei este condicional em todas as queries para remover do conjunto resultado as que não possuem status:

 if ($status) {
      $this->db->where('status', $status);
  } else {
     $this->db->where('(status="current" OR status="deceased")');
  }

Atenciosamente, Juliano.

gustavofonseca commented 5 years ago

@julianomaciel, talvez a gente necessite rever toda essa funcionalidade. Considerar que os nossos periódicos se dividem apenas entre os estados "current" e "deceased" não é correto, pois estaríamos ignorando os periódicos "suspended" que na minha opinião deveriam ser identificados com a bolinha vermelha. O que você pensa @alexxxmendonca ?

gustavofonseca commented 5 years ago

Dêem uma olhada em https://github.com/scieloorg/scielo20gt6/blob/master/active_journals.ipynb

Na sessão How many had been deindexed by these reasons? vocês encontrarão uma visão dos status de publicação dos periódicos da Rede.

alexxxmendonca commented 5 years ago

@gustavofonseca estou de acordo.

Ativo é somente o que tem publicação regular na base. Todo o resto deve estar em "Descontinuado" e o motivo (deceased, suspended) dedeve aparecer ao lado.

ghost commented 5 years ago

@gustavofonseca @alexxxmendonca

Alex e Gustavo, obrigado pelo retorno!

Sobre:

Ativo é somente o que tem publicação regular na base. Todo o resto deve estar em "Descontinuado"

Irei adicionar esse critério e os 192 registros com status suspended irão aparecer na lista com o marcador vermelho.

o motivo (deceased, suspended) dedeve aparecer ao lado

Seria um texto ou label além do marcador ?

Atenciosamente, Juliano.

alexxxmendonca commented 5 years ago

Estou na dúvida se são 192 como suspended. No site SciELO Brasil tem menos (cerca de 140)

Seria um texto ou label além do marcador ?

Seria algo semelhante com isso:

image

Não sei como a Parati pode ter acesso a esses dados. @gustavofonseca pode nos ajudar?

Esse ticket está relacionado com esse, do OPAC: https://github.com/scieloorg/opac/issues/1117

alexxxmendonca commented 5 years ago

Estou na dúvida se são 192 como suspended. No site SciELO Brasil tem menos (cerca de 140)

Estamos falando do portal org, e não da coleção SciELO Brasil. Se for isso faz sentido serem mais do que 140 mesmo.

gustavofonseca commented 5 years ago

Estou na dúvida se são 192 como suspended. No site SciELO Brasil tem menos (cerca de 140)

Estamos falando do portal org, e não da coleção SciELO Brasil. Se for isso faz sentido serem mais do que 140 mesmo.

Exato.

gustavofonseca commented 5 years ago

Seria um texto ou label além do marcador ?

Eu penso que esta modificação merece um ticket exclusivo e o envolvimento da Parati. Podemos fazer assim?

ghost commented 5 years ago

@gustavofonseca @alexxxmendonca

Gustavo e Alex, nos logs do git para o arquivo da base de dados local da aplicação, a última atualização (processamento dos dados do endpoint) foi em setembro no seguinte commit:

commit 8a28729641057ef7c6ee127069714a8c4173f621
Author: Juliano Maciel <julianomfferreira@gmail.com>
Date:   Tue Sep 18 16:27:10 2018 -0300

    Busca por nome do periódico deve ser 'case insensitive' (tk04).

diff --git a/application/cache/database/data.db b/application/cache/database/data.db
index 4c5ae18..a43e3e5 100755
Binary files a/application/cache/database/data.db and b/application/cache/database/data.db differ

2018-12-19-155247_1366x768_scrot

Pode ser que seja necessário uma nova atualização no ticket a ser aberto por vocês.

Atenciosamente, Juliano.

gustavofonseca commented 5 years ago

Então eu vou editar a descrição deste ticket para deixar claro qual a atividade a ser realizada.

ghost commented 5 years ago

@gustavofonseca

Gustavo, fiz a atualização do critério de filtragem nas queries para:

        if ($status == "current") {
            $this->db->where('status = "current"');
        } elseif ($status == "deceased") {
            $this->db->where('(status = "deceased" OR status = "suspended")');
        } else {
            $this->db->where('status != "inprogress"');
        }

que fez a inclusão dos registros que faltavam na lista de periódicos descontinuados.

Podendo ser avaliado no servidor de homologação:

http://scielohomolog.parati.ag/scielo-org-site/pt/periodicos/listar-por-ordem-alfabetica

Atenciosamente, Juliano.

gustavofonseca commented 5 years ago

Aparentemente resolve este ticket

gustavofonseca commented 5 years ago

Curiosamente a "Ágora: Estudos em Teoria Psicanalítica" não aparece mais na listagem o que é um erro.