Fix #1183 votacao nominal intuitiva (#1188)

* Inicia a logica

* Adaptação da lógica da votação interativa

* Faz mudanças sugeridas no code review

* Ajusta separador de msg_abertas.

Ajusta separador de msg_abertas.

* Ajusta listagem de votações abertas

Ajusta listagem de votações abertas

sapl/painel/urls.py

@@ -20,6 +20,6 @@ urlpatterns = [
     url(r'^painel/cronometro$', cronometro_painel, name='cronometro_painel'),
     # url(r'^painel/cronometro$', include(CronometroPainelCrud.get_urls())),
 
-    url(r'^voto-individual/(?P<pk>\d+)$', votante_view,
+    url(r'^voto-individual/$', votante_view,
         name="voto_individual"),
 ]

sapl/painel/views.py

@@ -1,7 +1,9 @@
 from datetime import date
 
+from django.contrib import messages
 from django.contrib.auth.decorators import user_passes_test
-from django.core.exceptions import ObjectDoesNotExist
+from django.core.exceptions import ObjectDoesNotExist, MultipleObjectsReturned
+from django.db.models import Q
 from django.core.urlresolvers import reverse
 from django.http import HttpResponse, JsonResponse
 from django.http.response import Http404, HttpResponseRedirect
@@ -30,18 +32,66 @@ def check_permission(user):
     return user.has_module_perms(AppConfig.label)
 
 
-def votante_view(request, pk):
-    if not Votante.objects.filter(user=request.user).exists():
-        raise Http404('Você não tem permissão para votar')
-
-    context = {'head_title': str(_('Votação Individual')), 'sessao_id': pk}
-
-    # Pega sessão
+def votacao_aberta(request):
+    votacoes_abertas = SessaoPlenaria.objects.filter(
+        Q(ordemdia__votacao_aberta=True) |
+        Q(expedientemateria__votacao_aberta=True)).distinct()
+
+    if len(votacoes_abertas) > 1:
+        msg_abertas = []
+        for v in votacoes_abertas:
+            msg_abertas.append('''<li><a href="%s">%s</a></li>''' % (
+                reverse('sapl.sessao:sessaoplenaria_detail',
+                        kwargs={'pk': v.id}),
+                v.__str__()))
+
+        msg = _('Existe mais de uma votações aberta. Elas se encontram '
+                'nas seguintes Sessões: ' + ', '.join(msg_abertas) + '. '
+                'Para votar, peça para que o Operador feche-as.')
+        messages.add_message(request, messages.INFO, msg)
+        return None, msg
+
+    elif len(votacoes_abertas) == 1:
+        ordens = OrdemDia.objects.filter(
+            sessao_plenaria=votacoes_abertas.first(),
+            votacao_aberta=True)
+        expedientes = ExpedienteMateria.objects.filter(
+            sessao_plenaria=votacoes_abertas.first(),
+            votacao_aberta=True)
+
+        numero_materias_abertas = len(ordens) + len(expedientes)
+        if numero_materias_abertas > 1:
+            msg = _('Existe mais de uma votação aberta na Sessão: ' +
+                    ('''<li><a href="%s">%s</a></li>''' % (
+                        reverse('sapl.sessao:sessaoplenaria_detail',
+                                kwargs={'pk': votacoes_abertas.first().id}),
+                        votacoes_abertas.first().__str__())) +
+                    'Para votar, peça para que o Operador as feche.')
+            messages.add_message(request, messages.INFO, msg)
+            return None, msg
+
+    return votacoes_abertas.first(), None
+
+
+def votante_view(request):
+    # Pega o votante relacionado ao usuário
     try:
-        sessao = SessaoPlenaria.objects.get(pk=pk)
+        votante = Votante.objects.get(user=request.user)
     except ObjectDoesNotExist:
         raise Http404()
 
+    context = {'head_title': str(_('Votação Individual'))}
+
+    # Verifica se usuário possui permissão para votar
+    if 'parlamentares.can_vote' in request.user.get_all_permissions():
+        context.update({'permissao': True})
+
+        # Pega sessão
+        sessao, msg = votacao_aberta(request)
+
+        if sessao and not msg:
+            pk = sessao.pk
+            context.update({'sessao_id': pk})
             context.update({'sessao': sessao,
                             'data': sessao.data_inicio,
                             'hora': sessao.hora_inicio})
@@ -53,54 +103,34 @@ def votante_view(request, pk):
             # Se aberta, verifica se é nominal. ID nominal == 2
             ordem_dia = get_materia_aberta(pk)
             expediente = get_materia_expediente_aberta(pk)
-    materia = None
 
+            materia_aberta = None
             if ordem_dia:
-        materia = ordem_dia.materia
-        if ordem_dia.tipo_votacao == VOTACAO_NOMINAL:
-            context.update({'materia': materia, 'ementa': materia.ementa})
-            presentes = PresencaOrdemDia.objects.filter(sessao_plenaria_id=pk)
-        else:
-            context.update(
-                {'materia': 'A matéria aberta não é votação nominal.'})
+                materia_aberta = ordem_dia
+                presentes = PresencaOrdemDia.objects.filter(
+                    sessao_plenaria_id=pk).values_list(
+                    'parlamentar_id', flat=True).distinct()
             elif expediente:
-        materia = expediente.materia
-        if expediente.tipo_votacao == VOTACAO_NOMINAL:
-            context.update({'materia': materia, 'ementa': materia.ementa})
+                materia_aberta = expediente
                 presentes = SessaoPlenariaPresenca.objects.filter(
-                sessao_plenaria_id=pk)
-        else:
-            context.update(
-                {'materia': 'A matéria aberta não é votação nominal.'})
-    else:
-        context.update(
-            {'materia': 'Nenhuma matéria com votação nominal aberta.'})
+                    sessao_plenaria_id=pk).values_list(
+                    'parlamentar_id', flat=True).distinct()
 
-    # Verifica se usuário possui permissão para votar
-    if 'parlamentares.can_vote' in request.user.get_all_permissions():
-        context.update({'permissao': True})
-    else:
-        context.update({'permissao': False})
+            if materia_aberta:
+                if materia_aberta.tipo_votacao == VOTACAO_NOMINAL:
+                    context.update({'materia': materia_aberta.materia,
+                                    'ementa': materia_aberta.materia.ementa})
 
-    # Verifica se usuário está presente na sessão
-    try:
-        votante = Votante.objects.get(user=request.user)
-    except ObjectDoesNotExist:
-        context.update({'error_message':
-                        'Erro ao recuperar parlamentar ligado ao usuário'})
-    else:
                     parlamentar = votante.parlamentar
-        context.update({'presente': False})
-        if len(presentes) > 0:
-            for p in presentes:
-                if p.parlamentar.id == parlamentar.id:
-                    context.update({'presente': True})
-                    break
+                    parlamentar_presente = False
+                    if parlamentar.id in presentes:
+                        parlamentar_presente = True
                     else:
                         context.update({'error_message':
-                            'Nenhuma matéria com votação nominal aberta.'})
+                                        'Não há presentes na Sessão com a '
+                                        'matéria em votação.'})
 
-    # Recupera o voto do parlamentar logado
+                    if parlamentar_presente:
                         voto = []
                         if ordem_dia:
                             voto = VotoParlamentar.objects.filter(
@@ -112,10 +142,33 @@ def votante_view(request, pk):
                         if voto:
                             try:
                                 voto = voto.get(parlamentar=parlamentar)
+                                context.update({'voto_parlamentar': voto.voto})
                             except ObjectDoesNotExist:
-            context.update({'voto_parlamentar': 'Voto não computado.'})
+                                context.update(
+                                    {'voto_parlamentar': 'Voto não '
+                                     'computado.'})
                     else:
-            context.update({'voto_parlamentar': voto.voto})
+                        context.update({'error_message':
+                                        'Você não está presente na '
+                                        'Ordem do Dia/Expediente em votação.'})
+                else:
+                    context.update(
+                        {'error_message': 'A matéria aberta não é votação '
+                         'nominal.'})
+            else:
+                context.update(
+                    {'error_message': 'Nenhuma matéria aberta.'})
+
+        elif not sessao and msg:
+            return HttpResponseRedirect('/')
+
+        else:
+            context.update(
+                {'error_message': 'Nenhuma sessão com matéria aberta.'})
+
+    else:
+        context.update({'permissao': False,
+                        'error_message': 'Usuário sem permissão para votar.'})
 
     # Salva o voto
     if request.method == 'POST':
@@ -156,7 +209,7 @@ def votante_view(request, pk):
                 voto.save()
 
         return HttpResponseRedirect(
-            reverse('sapl.painel:voto_individual', kwargs={'pk': pk}))
+            reverse('sapl.painel:voto_individual'))
 
     return render(request, 'painel/voto_nominal.html', context)
 

sapl/sessao/views.py

@@ -6,6 +6,7 @@ from django.contrib.auth.decorators import permission_required
 from django.contrib.auth.mixins import PermissionRequiredMixin
 from django.core.exceptions import MultipleObjectsReturned, ObjectDoesNotExist
 from django.core.urlresolvers import reverse
+from django.db.models import Q
 from django.forms.utils import ErrorList
 from django.http import JsonResponse
 from django.http.response import Http404, HttpResponseRedirect
@@ -81,49 +82,44 @@ def reordernar_materias_ordem(request, pk):
         reverse('sapl.sessao:ordemdia_list', kwargs={'pk': pk}))
 
 
-@permission_required('sessao.change_expedientemateria')
-def abrir_votacao_expediente_view(request, pk, spk):
-    existe_expediente_aberto = ExpedienteMateria.objects.filter(
-        sessao_plenaria_id=spk, votacao_aberta=True
-    ).exists()
-    existe_ordem_aberta = OrdemDia.objects.filter(
-        sessao_plenaria_id=spk, votacao_aberta=True
-    ).exists()
-
-    if existe_expediente_aberto or existe_ordem_aberta:
-        msg = _('Já existe uma matéria com votação aberta. Para abrir '
-                'outra, termine ou feche a votação existente.')
+def verifica_votacoes_abertas(request, model, pk):
+    votacoes_abertas = SessaoPlenaria.objects.filter(
+        Q(ordemdia__votacao_aberta=True) |
+        Q(expedientemateria__votacao_aberta=True)).distinct()
+
+    if votacoes_abertas:
+        msg_abertas = []
+        for v in votacoes_abertas:
+            msg_abertas.append('''<li><a href="%s">%s</a></li>''' % (
+                reverse('sapl.sessao:sessaoplenaria_detail',
+                        kwargs={'pk': v.id}),
+                v.__str__()))
+
+        msg = _('Já existem votações abertas nas seguintes Sessões: ' +
+                ', '.join(msg_abertas) + '. Para abrir '
+                'outra, termine ou feche as votações abertas.')
         messages.add_message(request, messages.INFO, msg)
+
     else:
-        expediente = ExpedienteMateria.objects.get(id=pk)
-        expediente.votacao_aberta = True
-        expediente.save()
+        materia_votacao = model.objects.get(id=pk)
+        materia_votacao.votacao_aberta = True
+        materia_votacao.save()
+
+
+@permission_required('sessao.change_expedientemateria')
+def abrir_votacao_expediente_view(request, pk, spk):
+    verifica_votacoes_abertas(request, ExpedienteMateria, pk)
     return HttpResponseRedirect(
         reverse('sapl.sessao:expedientemateria_list', kwargs={'pk': spk}))
 
 
 @permission_required('sessao.change_ordemdia')
 def abrir_votacao_ordem_view(request, pk, spk):
-    existe_ordem_aberta = OrdemDia.objects.filter(
-        sessao_plenaria_id=spk, votacao_aberta=True
-    ).exists()
-    existe_expediente_aberto = ExpedienteMateria.objects.filter(
-        sessao_plenaria_id=spk, votacao_aberta=True
-    ).exists()
-
-    if existe_ordem_aberta or existe_expediente_aberto:
-        msg = _('Já existe uma matéria com votação aberta. Para abrir '
-                'outra, termine ou feche a votação existente.')
-        messages.add_message(request, messages.INFO, msg)
-    else:
-        ordem = OrdemDia.objects.get(id=pk)
-        ordem.votacao_aberta = True
-        ordem.save()
+    verifica_votacoes_abertas(request, OrdemDia, pk)
     return HttpResponseRedirect(
         reverse('sapl.sessao:ordemdia_list', kwargs={'pk': spk}))
 
 
-
 def put_link_materia(context):
     for i, row in enumerate(context['rows']):
         materia = context['object_list'][i].materia
@@ -133,6 +129,7 @@ def put_link_materia(context):
         context['rows'][i][1] = (row[1][0], url_materia)
     return context
 
+
 def get_presencas_generic(model, sessao, legislatura):
     presencas = model.objects.filter(
         sessao_plenaria=sessao)
@@ -1753,6 +1750,18 @@ class VotacaoNominalAbstract(SessaoPermissionMixin):
                 materia_votacao.votacao_aberta = False
                 materia_votacao.save()
 
+            # Verifica se existe algum VotoParlamentar sem RegistroVotacao
+            # Por exemplo, se algum parlamentar votar e sua presença for
+            # removida da ordem do dia/expediente antes da conclusão da
+            # votação
+            if self.ordem:
+                VotoParlamentar.objects.filter(
+                    ordem=ordem,
+                    votacao__isnull=True).delete()
+            elif self.expediente:
+                VotoParlamentar.objects.filter(
+                    expediente=expediente,
+                    votacao__isnull=True).delete()
             return self.form_valid(form)
         else:
             return self.form_invalid(form)

sapl/templates/base.html

@@ -63,6 +63,11 @@
                   </a>
                   <ul class="dropdown-menu">
                     <li><a>{{user.username}}</a></li>
+                    {% if 'parlamentares.can_vote' in request.user.get_all_permissions %}
+                      <li><a href="{% url 'sapl.painel:voto_individual' %}">
+                        Votar Matéria
+                      </a></li>
+                    {% endif %}
                     <li><a href="{% url 'sapl.base:logout' %}">Sair</a></li>
                   </ul>
                 </li>

sapl/templates/painel/voto_nominal.html

@@ -21,7 +21,7 @@
       }
     </style>
   </head>
-  {% if permissao and presente %}
+  {% if not error_message %}
     <body>
       <h1><b><font color="#4FA64D"><p align="center">{{sessao}}</p></font></b></h1>
       <table style="width:100%">
@@ -74,23 +74,7 @@
 
       </form>
     </body>
-  {% elif not permissao %}
-    {% if error_message %}
-      <h2><font color="red"><p align="center" style="font-family:Verdana">{{error_message}}</p></font></h2>
-    {% else %}
-      <h2><font color="red"><p align="center" style="font-family:Verdana">Usuário sem permissão para participar de votações.</p></font></h2>
-    {% endif %}
-  {% elif not presente %}
-    {% if error_message %}
-      <h2><font color="red"><p align="center" style="font-family:Verdana">{{error_message}}</p></font></h2>
     {% else %}
-      <h2><font color="red"><p align="center" style="font-family:Verdana">Usuário não presente na Sessão Plenária.</p></font></h2>
-    {% endif %}
-  {% else %}
-    {% if error_message %}
       <h2><font color="red"><p align="center" style="font-family:Verdana">{{error_message}}</p></font></h2>
-    {% else %}
-      <h2><font color="red"><p align="center" style="font-family:Verdana">Usuário não presente na Sessão Plenária e sem permissão para votações.</p></font></h2>
-    {% endif %}
     {% endif %}
 </html>

sapl/templates/sessao/sessaoplenaria_detail.html

@@ -1,14 +0,0 @@
-{% extends "crud/detail.html" %}
-{% load i18n %}
-{% load crispy_forms_tags %}
-
-
-{% block extra_actions %}
-{% if 'parlamentares.can_vote' in request.user.get_all_permissions %}
-  <div class="actions btn-group btn-group-sm" role="group">
-    <a href="{% url 'sapl.painel:voto_individual' object.pk %}" class="btn btn-default">
-      Votar Matéria
-    </a>
-  </div>
-{% endif %}
-{% endblock extra_actions %}