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

Fix: Corrige forma dos artigos serem selecionados por data. #876

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

samuelveigarangel
Copy link
Collaborator

O que esse PR faz?

Corrige forma dos artigos serem selecionados por data.

Onde a revisão poderia começar?

pelos commits

Como este poderia ser testado manualmente?

Executar carga de artigos

Algum cenário de contexto que queira dar?

N/A

Screenshots

N/A

Quais são tickets relevantes?

#869

Referências

Indique as referências utilizadas para a elaboração do pull request.

@samuelveigarangel samuelveigarangel changed the title Fix: Corrige forma dos artigos serem selecionados por data. [wip] Fix: Corrige forma dos artigos serem selecionados por data. Oct 16, 2024
@samuelveigarangel samuelveigarangel changed the title [wip] Fix: Corrige forma dos artigos serem selecionados por data. Fix: Corrige forma dos artigos serem selecionados por data. Oct 21, 2024
article/tasks.py Outdated
# Obtém a data do último artigo válido
last_valid_article = Article.objects.all().order_by("-updated").first()
if last_valid_article:
from_date = last_valid_article.updated
Copy link
Member

Choose a reason for hiding this comment

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

@samuelveigarangel pensando melhor: a data do último artigo criado ou atualizado não vai garantir a seleção adequada dos registros de PidProvider. Como o procedimento de carga de artigos não está sincronizado com o recebimento de XML, a seleção dos registros de PidProvider pode não ser completa.
Para resolver este problema, faça o seguinte:

No lugar de

        last_valid_article = Article.objects.all().order_by("-updated").first()
        if last_valid_article:
            from_date = last_valid_article.updated 

use:

        last_created_article = Article.objects.all().order_by("-created").first()
        if last_created_article:
            pid_v3 = last_created_article.pid_v3
            from_date = PidProviderXML.objects.filter(v3=pid_v3).order_by("created").first().created

Copy link
Member

@robertatakenaka robertatakenaka left a comment

Choose a reason for hiding this comment

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

@samuelveigarangel fazer a correção

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