Conserta bug em presenca (#1185)

Author: edwardoliveira

.codeclimate.yml codeclimate.yml

@@ -87,8 +87,8 @@ def proposicao_texto(request, pk):
     proposicao = Proposicao.objects.get(pk=pk)
 
     if proposicao.texto_original:
-        if not proposicao.data_recebimento:
-            if proposicao.autor.user_id != request.user.id:
+        if (not proposicao.data_recebimento and
+            proposicao.autor.user_id != request.user.id):
                 raise Http404
 
         arquivo = proposicao.texto_original

sapl/materia/views.py

@@ -599,16 +599,19 @@ def post(self, request, *args, **kwargs):
         if form.is_valid():
             # Pegar os presentes salvos no banco
             presentes_banco = SessaoPlenariaPresenca.objects.filter(
-                sessao_plenaria_id=self.object.id)
+                                sessao_plenaria_id=self.object.id).values_list(
+                                'parlamentar_id', flat=True).distinct()
 
             # Id dos parlamentares presentes
-            marcados = request.POST.getlist('presenca_ativos') + request.POST.getlist('presenca_inativos')
+            marcados = request.POST.getlist('presenca_ativos') \
+                     + request.POST.getlist('presenca_inativos')
+            marcados = list(map(lambda x: int(x), marcados))
 
-            # Deletar os que foram desmarcadors
-            deletar = set(set(presentes_banco) - set(marcados))
-            for d in deletar:
-                SessaoPlenariaPresenca.objects.filter(
-                    parlamentar_id=d.parlamentar_id, sessao_plenaria_id=self.object.id).delete()
+            # Deletar os que foram desmarcados
+            deletar = set(presentes_banco) - set(marcados)
+            SessaoPlenariaPresenca.objects.filter(
+                parlamentar_id__in=deletar,
+                sessao_plenaria_id=self.object.id).delete()
 
             for p in marcados:
                 sessao = SessaoPlenariaPresenca()
@@ -707,16 +710,19 @@ def post(self, request, *args, **kwargs):
         if form.is_valid():
             # Pegar os presentes salvos no banco
             presentes_banco = PresencaOrdemDia.objects.filter(
-                sessao_plenaria_id=pk)
+                                sessao_plenaria_id=self.object.id).values_list(
+                                'parlamentar_id', flat=True).distinct()
 
             # Id dos parlamentares presentes
-            marcados = request.POST.getlist('presenca_ativos') + request.POST.getlist('presenca_inativos')
-
-            # Deletar os que foram desmarcadors
-            deletar = set(set(presentes_banco) - set(marcados))
-            for d in deletar:
-                PresencaOrdemDia.objects.filter(
-                    parlamentar_id=d.parlamentar_id, sessao_plenaria_id=self.object.id).delete()
+            marcados = request.POST.getlist('presenca_ativos') \
+                     + request.POST.getlist('presenca_inativos')
+            marcados = list(map(lambda x: int(x), marcados))
+
+            # Deletar os que foram desmarcados
+            deletar = set(presentes_banco) - set(marcados)
+            PresencaOrdemDia.objects.filter(
+                parlamentar_id__in=deletar,
+                sessao_plenaria_id=self.object.id).delete()
 
             for p in marcados:
                 ordem = PresencaOrdemDia()