Issue#3396 Obter Integrantes da mesa pela API #3415

AlGouvea · 2021-06-22T14:37:16Z

Descrição

Definida nova url e view na API para acessar dados da mesa diretora

Issue Relacionada

#3396

Motivação e Contexto

Como Isso Foi Testado?

Capturas de Tela (se apropriado):

Tipos de Mudanças

Bug fix (alteração que corrige uma issue e não altera funcionalidades já existentes)
Nova feature (alteração que adiciona uma funcionalidade e não altera funcionalidades já existentes)
Alteração disruptiva (Breaking change) (Correção ou funcionalidade que causa alteração nas funcionalidades existentes)

Checklist:

Eu li o documento de Contribuição (CONTRIBUTING).
Meu código segue o estilo de código deste projeto.
Minha alteração requer uma alteração na documentação.
Eu atualizei a documentação de acordo.
Eu adicionei testes para cobrir minhas mudanças.
Todos os testes novos e existentes passaram.

edwardoliveira · 2021-06-22T15:03:21Z

sapl/api/views.py

+
+
+


Remover essas linhas.

edwardoliveira · 2021-06-22T15:04:12Z

sapl/api/views.py

+
+    sessao_legislativa = SessaoLegislativa.objects.filter(**kwargs).order_by('-data_inicio').first()
+
+    query_legislatura = Legislatura.objects.filter(id=legislatura).order_by('-data_inicio').first()


E se não for informada a legislatura?

Vc tem certeza de que precisa dessa query????

legislatura = sessao_legislativa.legislatura

vai recuperar o objeto Legislatura

edwardoliveira · 2021-06-22T15:05:11Z

sapl/api/views.py

+
+    legislatura = request.GET.get('legislatura')
+    if legislatura:
+        kwargs['legislatura_id'] = legislatura


Acho que é o caso de se colocar a última legislatura se o usuário não informou. Um else aqui, por exemplo, visto que usamos isso na linha 71.

Verificar a possibilidade do BD estar vazio e sem legislatura.

edwardoliveira · 2021-06-22T15:09:15Z

sapl/api/views.py

+    if sessao:
+        kwargs['id'] = sessao
+
+    sessao_legislativa = SessaoLegislativa.objects.filter(**kwargs).order_by('-data_inicio').first()


Temos que pensar o que fazer se sessao_legislativa vier None.

Colocar um select_related('legislatura') após o objects na linha 69.

sapl/api/views.py

edwardoliveira · 2021-06-22T15:14:42Z

sapl/api/views.py

+    if sessao:
+        kwargs['id'] = sessao
+
+    sessao_legislativa = SessaoLegislativa.objects.filter(**kwargs).order_by('-data_inicio').first()


Colocar um select_related('legislatura') após o objects na linha 69.

edwardoliveira · 2021-06-25T12:50:19Z

sapl/api/views.py

+
+    legislatura = request.GET.get('legislatura')
+    if legislatura:
+        kwargs['legislatura_id'] = legislatura        


Coloca um else aqui: se não tiver legislatura informada pega a última ("-data_inicio")

edwardoliveira · 2021-06-25T12:51:08Z

sapl/api/views.py

+
+    if sessao_legislativa is None:
+        logger.error("Sessão ou legislatura não encontrada!")
+        return JsonResponse({})


Retorna uma mensagem de erro no json. Algo como:

{"error": "Sessão ou legislatura não encontrada!"}

edwardoliveira · 2021-06-25T12:51:20Z

sapl/api/views.py

+
+    if composicao_mesa is None:
+        logger.error("Nenhuma mesa não encontrada!")
+        return JsonResponse({})


Mesma coisa aqui: retorna uma mensagem de erro.

edwardoliveira · 2021-06-25T12:51:56Z

sapl/api/views.py

+        logger.error("Sessão ou legislatura não encontrada!")
+        return JsonResponse({})
+
+    composicao_mesa = ComposicaoMesa.objects.select_related('parlamentar', 'cargo').all().filter(


Não precisa colocar o all() e o filter() ao mesmo tempo. É desnecessário. Ou um ou outro. No caso fica o filter().

edwardoliveira · 2021-06-25T12:53:07Z

sapl/api/views.py

@@ -703,4 +747,4 @@ def get(self, request):
            'user': request.user.username,
            'is_authenticated': request.user.is_authenticated,
        }
-        return Response(content)
+        return Response(content)


Pq vc tá modificando essa linha? Não precisa. Reverter essa mudança.

… parlamentar

AlGouvea added 4 commits June 18, 2021 12:51

Adicionado rota para pagina da info de mesa diretora na API

f47f256

Arrumada a url e definida função básica da issue

af02725

Apresentando maior parte das informacoes da mesa diretora

86ddedf

Refatoração do código fonte

4d08daf

AlGouvea requested a review from edwardoliveira June 22, 2021 14:37

edwardoliveira reviewed Jun 22, 2021

View reviewed changes

sapl/api/views.py Outdated

Comment on lines 743 to 745

Copy link

Contributor

edwardoliveira Jun 22, 2021

Choose a reason for hiding this comment

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

Remover essas linhas.

edwardoliveira reviewed Jun 22, 2021

View reviewed changes

sapl/api/views.py Show resolved Hide resolved

edwardoliveira requested changes Jun 22, 2021

View reviewed changes

AlGouvea added 3 commits June 22, 2021 12:18

Implementação de condicional para legislatura não informada

f678f12

Condicionais para caso de informações não disponíveis

7c3820d

Refatoração das condicionais de informações

f5c089d

edwardoliveira reviewed Jun 25, 2021

View reviewed changes

AlGouvea added 8 commits June 25, 2021 09:57

Criado o arquivo de testes

fc8b61c

Iniciada a função de teste

b08f537

Alterada condicional de legislatura e adicionadas mensagens de erro

03668fe

Removida redundancia de query

2729e38

Desempacotamento da tupla para melhorar legibilidade

efcb0c0

Adicionada condicional para inexistencia do arquivos de fotografia do…

810744d

… parlamentar

Alterada a condicional para inexistencia do arquivos de fotografia do…

04446ac

… parlamentar

Inicio da configuração do arquivo de testes

e1acbff

edwardoliveira force-pushed the 3.1.x branch from 778b30b to c2748c0 Compare April 23, 2022 13:28

edwardoliveira force-pushed the 3.1.x branch from 7f380a6 to e20b980 Compare August 4, 2022 16:39

edwardoliveira force-pushed the 3.1.x branch from 7f1daea to 78456b6 Compare September 21, 2022 14:16

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Issue#3396 Obter Integrantes da mesa pela API #3415

Issue#3396 Obter Integrantes da mesa pela API #3415

AlGouvea commented Jun 22, 2021

edwardoliveira Jun 22, 2021

edwardoliveira Jun 22, 2021

edwardoliveira Jun 22, 2021 •

edited

Loading

edwardoliveira Jun 22, 2021

edwardoliveira Jun 22, 2021 •

edited

Loading

edwardoliveira Jun 22, 2021

edwardoliveira Jun 22, 2021

edwardoliveira Jun 22, 2021

edwardoliveira Jun 25, 2021

edwardoliveira Jun 25, 2021

edwardoliveira Jun 25, 2021

edwardoliveira Jun 25, 2021

edwardoliveira Jun 25, 2021


		sessao_legislativa = SessaoLegislativa.objects.filter(**kwargs).order_by('-data_inicio').first()

		query_legislatura = Legislatura.objects.filter(id=legislatura).order_by('-data_inicio').first()

Issue#3396 Obter Integrantes da mesa pela API #3415

Are you sure you want to change the base?

Issue#3396 Obter Integrantes da mesa pela API #3415

Conversation

AlGouvea commented Jun 22, 2021

Descrição

Issue Relacionada

Motivação e Contexto

Como Isso Foi Testado?

Capturas de Tela (se apropriado):

Tipos de Mudanças

Checklist:

Choose a reason for hiding this comment

Choose a reason for hiding this comment

edwardoliveira Jun 22, 2021 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

edwardoliveira Jun 22, 2021 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

edwardoliveira Jun 22, 2021 •

edited

Loading

edwardoliveira Jun 22, 2021 •

edited

Loading