Conserta alguns testes e retira a tabela de VotoNominal

sapl/painel/views.py

@@ -14,8 +14,7 @@ from sapl.painel.models import Painel
 from sapl.parlamentares.models import Filiacao, Votante
 from sapl.sessao.models import (ExpedienteMateria, OrdemDia, PresencaOrdemDia,
                                 RegistroVotacao, SessaoPlenaria,
-                                SessaoPlenariaPresenca, VotoNominal,
+                                SessaoPlenariaPresenca, VotoParlamentar)
-                                VotoParlamentar)
 from sapl.utils import get_client_ip
 
 from .models import Cronometro

sapl/rules/map_rules.py

@@ -152,8 +152,6 @@ rules_group_sessao = {
         (sessao.PresencaOrdemDia, __base__),
         (sessao.RegistroVotacao, __base__),
         (sessao.VotoParlamentar, __base__),
-
-        (sessao.VotoNominal, __base__),
     ]
 }
 

sapl/sessao/forms.py

@@ -35,6 +35,7 @@ def recupera_anos():
 def ANO_CHOICES():
     return [('', '---------')] + recupera_anos()
 
+
 MES_CHOICES = [('', '---------')] + RANGE_MESES
 DIA_CHOICES = [('', '---------')] + RANGE_DIAS_MES
 
@@ -151,14 +152,12 @@ class OrdemDiaForm(ExpedienteMateriaForm):
     def clean_data_ordem(self):
         return self.instance.sessao_plenaria.data_inicio
 
-
     def clean_numero_ordem(self):
         sessao = self.instance.sessao_plenaria
 
         numero_ordem_exists = OrdemDia.objects.filter(
-                                sessao_plenaria=sessao,
+            sessao_plenaria=sessao,
-                                numero_ordem=self.cleaned_data[
+            numero_ordem=self.cleaned_data['numero_ordem']).exists()
-                                    'numero_ordem']).exists()
 
         if numero_ordem_exists:
             msg = _('Esse número de ordem já existe.')
@@ -166,7 +165,6 @@ class OrdemDiaForm(ExpedienteMateriaForm):
 
         return self.cleaned_data['numero_ordem']
 
-
     def clean(self):
         cleaned_data = self.cleaned_data
         sessao = self.instance.sessao_plenaria

sapl/sessao/migrations/0008_auto_20170607_1220.py

@@ -0,0 +1,38 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.9.7 on 2017-06-07 12:20
+from __future__ import unicode_literals
+
+from django.db import migrations
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('sessao', '0007_auto_20170606_1238'),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name='votonominal',
+            name='materia',
+        ),
+        migrations.RemoveField(
+            model_name='votonominal',
+            name='parlamentar',
+        ),
+        migrations.RemoveField(
+            model_name='votonominal',
+            name='sessao',
+        ),
+        migrations.RemoveField(
+            model_name='votonominal',
+            name='user',
+        ),
+        migrations.RemoveField(
+            model_name='votonominal',
+            name='votacao',
+        ),
+        migrations.DeleteModel(
+            name='VotoNominal',
+        ),
+    ]

sapl/sessao/models.py

@@ -86,7 +86,8 @@ def get_sessao_media_path(instance, subpath, filename):
 
 
 def pauta_upload_path(instance, filename):
-    return texto_upload_path(instance, filename, subpath='pauta', pk_first=True)
+    return texto_upload_path(
+        instance, filename, subpath='pauta', pk_first=True)
     # return get_sessao_media_path(instance, 'pauta', filename)
 
 
@@ -96,7 +97,8 @@ def ata_upload_path(instance, filename):
 
 
 def anexo_upload_path(instance, filename):
-    return texto_upload_path(instance, filename, subpath='anexo', pk_first=True)
+    return texto_upload_path(
+        instance, filename, subpath='anexo', pk_first=True)
     # return get_sessao_media_path(instance, 'anexo', filename)
 
 
@@ -454,31 +456,6 @@ class VotoParlamentar(models.Model):  # RegistroVotacaoParlamentar
             'votacao': self.votacao, 'parlamentar': self.parlamentar}
 
 
-@reversion.register()
-class VotoNominal(models.Model):
-    parlamentar = models.ForeignKey(Parlamentar, on_delete=models.PROTECT)
-    voto = models.CharField(verbose_name=_('Voto'), max_length=10)
-    votacao = models.ForeignKey(RegistroVotacao,
-                                blank=True,
-                                null=True)
-
-    sessao = models.ForeignKey(SessaoPlenaria, on_delete=models.PROTECT)
-    materia = models.ForeignKey(MateriaLegislativa, on_delete=models.PROTECT)
-
-    user = models.ForeignKey(get_settings_auth_user_model(),
-                             on_delete=models.PROTECT)
-    ip = models.CharField(verbose_name=_('IP'), max_length=30)
-    data_hora = models.DateTimeField(
-        verbose_name=_('Data/Hora'), auto_now_add=True)
-
-    class Meta:
-        verbose_name = _('Registro do Voto do Parlamentar')
-        verbose_name_plural = _('Registros dos Votos dos Parlamentares')
-
-    def __str__(self):
-        return '%s - %s' % (self.parlamentar.nome_parlamentar, self.voto)
-
-
 @reversion.register()
 class SessaoPlenariaPresenca(models.Model):
     sessao_plenaria = models.ForeignKey(SessaoPlenaria,

sapl/sessao/views.py

@@ -44,7 +44,7 @@ from .models import (Bancada, Bloco, CargoBancada, CargoMesa,
                      MateriaLegislativa, Orador, OradorExpediente, OrdemDia,
                      PresencaOrdemDia, RegistroVotacao, ResumoOrdenacao,
                      SessaoPlenaria, SessaoPlenariaPresenca, TipoExpediente,
-                     TipoResultadoVotacao, TipoSessaoPlenaria, VotoNominal,
+                     TipoResultadoVotacao, TipoSessaoPlenaria,
                      VotoParlamentar)
 
 TipoSessaoCrud = CrudAux.build(TipoSessaoPlenaria, 'tipo_sessao_plenaria')
@@ -369,24 +369,25 @@ class ExpedienteMateriaCrud(MasterDetailCrud):
                                               'oid': obj.pk,
                                               'mid': obj.materia_id})
                         obj.resultado = ('<a href="%s">%s</a><br/>%s' %
-                                           (url,
+                                         (url,
-                                            resultado_descricao,
+                                          resultado_descricao,
-                                            resultado_observacao))
+                                          resultado_observacao))
                     else:
                         if obj.tipo_votacao == 2:
-                            url = reverse('sapl.sessao:votacaonominalexpdetail',
+                            url = reverse(
-                                            kwargs={
+                                'sapl.sessao:votacaonominalexpdetail',
-                                                  'pk': obj.sessao_plenaria_id,
+                                kwargs={
-                                                  'oid': obj.pk,
+                                    'pk': obj.sessao_plenaria_id,
-                                                  'mid': obj.materia_id})
+                                    'oid': obj.pk,
+                                    'mid': obj.materia_id})
                             obj.resultado = ('<a href="%s">%s</a><br/>%s' %
-                                           (url,
+                                             (url,
-                                            resultado_descricao,
+                                              resultado_descricao,
-                                            resultado_observacao))
+                                              resultado_observacao))
                         else:
                             obj.resultado = ('%s<br/>%s' %
-                                                (resultado_descricao,
+                                             (resultado_descricao,
-                                                 resultado_observacao))
+                                              resultado_observacao))
             return [self._as_row(obj) for obj in object_list]
 
     class CreateView(MasterDetailCrud.CreateView):