Fix get_queryset(request) → get_queryset() in SnippetViewSet subclasses

[Copilot]

O que esse PR faz?

Corrige a assinatura get_queryset(self, request) (padrão ModelAdmin) para get_queryset(self) (padrão SnippetViewSet) em todas as subclasses de SnippetViewSet.

No Wagtail SnippetViewSet, get_queryset() não recebe request — usa self.request internamente. Passar request fazia super().get_queryset(request) retornar None em vez de um QuerySet.

# ANTES (ModelAdmin)
def get_queryset(self, request):
    return super().get_queryset(request).filter(...)

# DEPOIS (SnippetViewSet)
def get_queryset(self):
    return super().get_queryset().filter(...)

Classes corrigidas:

• core/wagtail_hooks.py — LicenseViewSet
• journal/wagtail_hooks.py — FilteredJournalQuerysetMixin, JournalAdminOnlySnippetViewSet, SciELOJournalAdminViewSet, CrossmarkPolicyAdmin
• location/wagtail_hooks.py — LocationAdmin
• issue/wagtail_hooks.py — IssueAdminSnippetViewSet
• thematic_areas/wagtail_hooks.py — GenericThematicAreaAdmin, GenericThematicAreaFileAdmin, ThematicAreaAdmin, ThematicAreaFileAdmin
• article/models.py + article/wagtail_hooks.py — movido get_queryset do proxy model ArticlePeerReviewStats para ArticlePeerReviewStatsSnippetViewSet

django_celery_beat/wagtail_hooks.py não foi alterado pois ainda usa ModelAdmin.

Onde a revisão poderia começar?

journal/wagtail_hooks.py — contém os casos mais complexos com FilteredJournalQuerysetMixin e múltiplas ViewSets que usam self.request.user para filtragem por permissão.

Como este poderia ser testado manualmente?

1. Acessar o admin do Wagtail
2. Navegar para qualquer listagem de Snippets afetada (Journals, Issues, SciELO Journals, Licenses, Locations, Thematic Areas, Crossmark Policies, Peer Review Stats)
3. Verificar que a listagem carrega corretamente (antes retornava erro ou lista vazia por None)
4. Verificar que filtros de permissão por usuário/coleção continuam funcionando

Algum cenário de contexto que queira dar?

Na migração de ModelAdmin para SnippetViewSet, a assinatura de get_queryset mudou. O SnippetViewSet do Wagtail não recebe request como argumento — ele usa self.request internamente. Quando se chama super().get_queryset(request), o Wagtail interpreta esse request de forma inesperada e retorna None.

Screenshots

N/A — correção de backend sem impacto visual.

Quais são tickets relevantes?

N/A

Referências

• Wagtail SnippetViewSet documentation

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Trocar get_queryset(request), usado com ModelAdmin, por get_queryset() se estiver usando Snippets</issue_title>
<issue_description>### Descrição da tarefa

Corrigir o padrão get_queryset(request). usado com ModelAdmin, removendo o request para usar com Snippets.

return super().get_queryset(request).filter(**params)

O super().get_queryset(request) está retornando None em vez de um QuerySet.

Isso acontece porque, na migração de ModelAdmin para SnippetViewSet, a assinatura do método get_queryset mudou. No SnippetViewSet do Wagtail, o método get_queryset da view não recebe request como argumento — ele usa self.request internamente. Quando você chama super().get_queryset(request), o Wagtail interpreta esse request de forma inesperada e retorna None.

O fix provável é ajustar a chamada ao super(). Sem ver o código completo, mas baseado no padrão da migração que você tem feito, deve ser algo assim:

# ANTES (padrão ModelAdmin)
def get_queryset(self, request):
    return super().get_queryset(request).filter(**params)

# DEPOIS (padrão SnippetViewSet)
def get_queryset(self):
    return super().get_queryset().filter(**params)

Se você precisar do request para montar os params (por exemplo, filtrar por usuário), use self.request dentro do método.
</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Trocar get_queryset(request), usado com ModelAdmin, por get_queryset() se estiver usando Snippets #1419

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

Corrige a assinatura de get_queryset em subclasses de SnippetViewSet para o padrão do Wagtail (get_queryset(self) usando self.request), evitando que chamadas a super().get_queryset(request) retornem None e quebrem listagens de snippets no admin.

Changes:

• Atualiza get_queryset(self, request) → get_queryset(self) em vários SnippetViewSet, trocando request.user por self.request.user quando necessário.
• Ajusta chamadas a super().get_queryset() para não receber request.
• Move a lógica de queryset otimizado do proxy ArticlePeerReviewStats para o ArticlePeerReviewStatsSnippetViewSet.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
thematic_areas/wagtail_hooks.py	Ajusta get_queryset() em viewsets de Thematic Areas para assinatura correta do SnippetViewSet.
location/wagtail_hooks.py	Ajusta get_queryset() do LocationAdmin removendo request e mantendo otimizações.
journal/wagtail_hooks.py	Corrige assinatura/uso de request em mixin e viewsets (incluindo filtros por permissões via self.request.user).
issue/wagtail_hooks.py	Corrige assinatura/uso de request no queryset do admin de Issues.
core/wagtail_hooks.py	Corrige assinatura do LicenseViewSet.get_queryset() para padrão SnippetViewSet.
article/wagtail_hooks.py	Adiciona get_queryset() otimizado no ArticlePeerReviewStatsSnippetViewSet.
article/models.py	Remove get_queryset do proxy model ArticlePeerReviewStats (migração da responsabilidade para o ViewSet).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

get_queryset() is prefetching the M2M relations (doi, titles, languages) for every page of the listing. This adds extra queries and can pull additional rows even though these relations aren’t used in list_display/filters here; consider dropping these prefetch_related() calls (or limiting them to what the list actually needs) to keep the admin listing lighter.

Suggested change

	'doi',
	'titles',
	'languages',
	'titles',