Merge branch '3.1.x' into script_remove_multiplos_autores

sapl/protocoloadm/forms.py

@@ -221,7 +221,7 @@ class DocumentoAdministrativoFilterSet(django_filters.FilterSet):
         )
 
 
-class AnularProcoloAdmForm(ModelForm):
+class AnularProtocoloAdmForm(ModelForm):
 
     logger = logging.getLogger(__name__)
 
@@ -240,7 +240,7 @@ class AnularProcoloAdmForm(ModelForm):
         widget=forms.Textarea)
 
     def clean(self):
-        super(AnularProcoloAdmForm, self).clean()
+        super(AnularProtocoloAdmForm, self).clean()
 
         cleaned_data = self.cleaned_data
 
@@ -313,7 +313,7 @@ class AnularProcoloAdmForm(ModelForm):
                      form_actions(label='Anular')
                      )
         )
-        super(AnularProcoloAdmForm, self).__init__(
+        super(AnularProtocoloAdmForm, self).__init__(
             *args, **kwargs)
 
 

sapl/protocoloadm/tests/test_protocoloadm.py

@@ -9,7 +9,7 @@ import pytest
 
 from sapl.base.models import AppConfig
 from sapl.materia.models import UnidadeTramitacao
-from sapl.protocoloadm.forms import (AnularProcoloAdmForm,
+from sapl.protocoloadm.forms import (AnularProtocoloAdmForm,
                                      DocumentoAdministrativoForm,
                                      MateriaLegislativa, ProtocoloDocumentForm,
                                      ProtocoloMateriaForm)
@@ -51,7 +51,7 @@ def test_anular_protocolo_submit(admin_client):
 
 @pytest.mark.django_db(transaction=False)
 def test_form_anular_protocolo_inexistente():
-    form = AnularProcoloAdmForm({'numero': '1',
+    form = AnularProtocoloAdmForm({'numero': '1',
                                  'ano': '2016',
                                  'justificativa_anulacao': 'TESTE'})
 
@@ -64,7 +64,7 @@ def test_form_anular_protocolo_inexistente():
 @pytest.mark.django_db(transaction=False)
 def test_form_anular_protocolo_valido():
     mommy.make(Protocolo, numero='1', ano='2016', anulado=False)
-    form = AnularProcoloAdmForm({'numero': '1',
+    form = AnularProtocoloAdmForm({'numero': '1',
                                  'ano': '2016',
                                  'justificativa_anulacao': 'TESTE'})
     if not form.is_valid():
@@ -74,7 +74,7 @@ def test_form_anular_protocolo_valido():
 @pytest.mark.django_db(transaction=False)
 def test_form_anular_protocolo_anulado():
     mommy.make(Protocolo, numero='1', ano='2016', anulado=True)
-    form = AnularProcoloAdmForm({'numero': '1',
+    form = AnularProtocoloAdmForm({'numero': '1',
                                  'ano': '2016',
                                  'justificativa_anulacao': 'TESTE'})
     assert form.errors['__all__'] == \
@@ -88,7 +88,7 @@ def test_form_anular_protocolo_campos_obrigatorios():
     # TODO: generalizar para diminuir o tamanho deste método
 
     # numero ausente
-    form = AnularProcoloAdmForm({'numero': '',
+    form = AnularProtocoloAdmForm({'numero': '',
                                  'ano': '2016',
                                  'justificativa_anulacao': 'TESTE'})
     if form.is_valid():
@@ -98,7 +98,7 @@ def test_form_anular_protocolo_campos_obrigatorios():
     assert form.errors['numero'] == [_('Este campo é obrigatório.')]
 
     # ano ausente
-    form = AnularProcoloAdmForm({'numero': '1',
+    form = AnularProtocoloAdmForm({'numero': '1',
                                  'ano': '',
                                  'justificativa_anulacao': 'TESTE'})
     if form.is_valid():
@@ -108,7 +108,7 @@ def test_form_anular_protocolo_campos_obrigatorios():
     assert form.errors['ano'] == [_('Este campo é obrigatório.')]
 
     # justificativa_anulacao ausente
-    form = AnularProcoloAdmForm({'numero': '1',
+    form = AnularProtocoloAdmForm({'numero': '1',
                                  'ano': '2016',
                                  'justificativa_anulacao': ''})
     if form.is_valid():
@@ -261,7 +261,7 @@ def test_create_tramitacao(admin_client):
 @pytest.mark.django_db(transaction=False)
 def test_anular_protocolo_dados_invalidos():
 
-    form = AnularProcoloAdmForm(data={})
+    form = AnularProtocoloAdmForm(data={})
 
     assert not form.is_valid()
 
@@ -276,10 +276,10 @@ def test_anular_protocolo_dados_invalidos():
 
 @pytest.mark.django_db(transaction=False)
 def test_anular_protocolo_form_anula_protocolo_inexistente():
-    form = AnularProcoloAdmForm(data={'numero': '1',
+    form = AnularProtocoloAdmForm(data={'numero': '1',
                                       'ano': '2017',
                                       'justificativa_anulacao': 'teste'
-                                      })
+                                        })
 
     assert not form.is_valid()
 
@@ -291,10 +291,10 @@ def test_anular_protocolo_form_anula_protocolo_inexistente():
 def test_anular_protocolo_form_anula_protocolo_anulado():
     mommy.make(Protocolo, numero=1, ano=2017, anulado=True)
 
-    form = AnularProcoloAdmForm(data={'numero': '1',
+    form = AnularProtocoloAdmForm(data={'numero': '1',
                                       'ano': '2017',
                                       'justificativa_anulacao': 'teste'
-                                      })
+                                        })
 
     assert not form.is_valid()
 
@@ -316,10 +316,10 @@ def test_anular_protocolo_form_anula_protocolo_com_doc_vinculado():
                ano=2017,
                numero_protocolo=1)
 
-    form = AnularProcoloAdmForm(data={'numero': '1',
+    form = AnularProtocoloAdmForm(data={'numero': '1',
                                       'ano': '2017',
                                       'justificativa_anulacao': 'teste'
-                                      })
+                                        })
 
     assert not form.is_valid()
 
@@ -338,10 +338,10 @@ def test_anular_protocolo_form_anula_protocolo_com_doc_vinculado():
     mommy.make(DocumentoAdministrativo,
                protocolo=protocolo_documento)
 
-    form = AnularProcoloAdmForm(data={'numero': '2',
+    form = AnularProtocoloAdmForm(data={'numero': '2',
                                       'ano': '2017',
                                       'justificativa_anulacao': 'teste'
-                                      })
+                                        })
 
     assert not form.is_valid()
 

sapl/protocoloadm/views.py

@@ -37,7 +37,7 @@ from sapl.utils import (create_barcode, get_base_url, get_client_ip,
                         get_mime_type_from_file_extension,
                         show_results_filter_set, mail_service_configured)
 
-from .forms import (AcompanhamentoDocumentoForm, AnularProcoloAdmForm,
+from .forms import (AcompanhamentoDocumentoForm, AnularProtocoloAdmForm,
                     DocumentoAcessorioAdministrativoForm,
                     DocumentoAdministrativoFilterSet,
                     DocumentoAdministrativoForm, FichaPesquisaAdmForm, FichaSelecionaAdmForm, ProtocoloDocumentForm,
@@ -484,7 +484,7 @@ class ProtocoloListView(PermissionRequiredMixin, ListView):
 
 class AnularProtocoloAdmView(PermissionRequiredMixin, CreateView):
     template_name = 'protocoloadm/anular_protocoloadm.html'
-    form_class = AnularProcoloAdmForm
+    form_class = AnularProtocoloAdmForm
     form_valid_message = _('Protocolo anulado com sucesso!')
     permission_required = ('protocoloadm.action_anular_protocolo', )
 

sapl/sessao/migrations/0037_auto_20190415_1324.py

@@ -0,0 +1,33 @@
+# -*- coding: utf-8 -*-
+# Generated by Django 1.11.20 on 2019-04-15 16:24
+from __future__ import unicode_literals
+
+from django.conf import settings
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+        ('sessao', '0036_auto_20190412_1106'),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name='registrovotacao',
+            name='data_hora',
+            field=models.DateTimeField(auto_now_add=True, null=True, verbose_name='Data/Hora'),
+        ),
+        migrations.AddField(
+            model_name='registrovotacao',
+            name='ip',
+            field=models.CharField(blank=True, default='', max_length=30, verbose_name='IP'),
+        ),
+        migrations.AddField(
+            model_name='registrovotacao',
+            name='user',
+            field=models.ForeignKey(blank=True, null=True, on_delete=django.db.models.deletion.PROTECT, to=settings.AUTH_USER_MODEL),
+        ),
+    ]

sapl/sessao/models.py

@@ -456,6 +456,19 @@ class RegistroVotacao(models.Model):
         verbose_name=_('Abstenções'))
     observacao = models.TextField(
         blank=True, verbose_name=_('Observações'))
+    user = models.ForeignKey(get_settings_auth_user_model(),
+                             on_delete=models.PROTECT,
+                             null=True,
+                             blank=True)
+    ip = models.CharField(verbose_name=_('IP'),
+                          max_length=30,
+                          blank=True,
+                          default='')
+    data_hora = models.DateTimeField(
+        verbose_name=_('Data/Hora'),
+        auto_now_add=True,
+        blank=True,
+        null=True)
 
     class Meta:
         verbose_name = _('Votação')

sapl/sessao/views.py

@@ -35,7 +35,7 @@ from sapl.parlamentares.models import (Filiacao, Legislatura, Mandato,
                                        Parlamentar, SessaoLegislativa)
 from sapl.sessao.apps import AppConfig
 from sapl.sessao.forms import ExpedienteMateriaForm, OrdemDiaForm
-from sapl.utils import show_results_filter_set, remover_acentos
+from sapl.utils import show_results_filter_set, remover_acentos, get_client_ip
 
 from .forms import (AdicionarVariasMateriasFilterSet, BancadaForm, BlocoForm,
                     ExpedienteForm, JustificativaAusenciaForm, OcorrenciaSessaoForm, ListMateriaForm,
@@ -2163,6 +2163,8 @@ class VotacaoView(SessaoPermissionMixin):
                     votacao.ordem_id = ordem_id
                     votacao.tipo_resultado_votacao_id = int(
                         request.POST['resultado_votacao'])
+                    votacao.user = request.user
+                    votacao.ip = get_client_ip(request)
                     votacao.save()
                 except Exception as e:
                     username = request.user.username
@@ -2383,6 +2385,8 @@ class VotacaoNominalAbstract(SessaoPermissionMixin):
             votacao.numero_votos_nao = votos_nao
             votacao.numero_abstencoes = abstencoes
             votacao.observacao = request.POST.get('observacao', None)
+            votacao.user = request.user
+            votacao.ip = get_client_ip(request)
 
             votacao.materia_id = materia_votacao.materia.id
             if self.ordem:
@@ -2410,6 +2414,8 @@ class VotacaoNominalAbstract(SessaoPermissionMixin):
                 voto_parlamentar.voto = voto
                 voto_parlamentar.parlamentar_id = parlamentar_id
                 voto_parlamentar.votacao_id = votacao.id
+                voto_parlamentar.user = request.user
+                voto_parlamentar.ip = get_client_ip(request)
                 voto_parlamentar.save()
 
                 resultado = form.cleaned_data['resultado_votacao']
@@ -2847,10 +2853,10 @@ class VotacaoExpedienteView(SessaoPermissionMixin):
             if (int(request.POST['voto_presidente']) == 0):
                 qtde_presentes -= 1
 
-            if (qtde_votos > qtde_presentes or qtde_votos < qtde_presentes):
+            if qtde_votos != qtde_presentes:
                 form._errors["total_votos"] = ErrorList([u""])
                 return self.render_to_response(context)
-            elif (qtde_presentes == qtde_votos):
+            else:
                 try:
                     votacao = RegistroVotacao()
                     votacao.numero_votos_sim = int(request.POST['votos_sim'])
@@ -2861,6 +2867,8 @@ class VotacaoExpedienteView(SessaoPermissionMixin):
                     votacao.expediente_id = expediente_id
                     votacao.tipo_resultado_votacao_id = int(
                         request.POST['resultado_votacao'])
+                    votacao.user = request.user
+                    votacao.ip = get_client_ip(request)
                     votacao.save()
                 except Exception as e:
                     username = request.user.username
@@ -3640,6 +3648,8 @@ class VotacaoEmBlocoSimbolicaView(PermissionRequiredForAppCrudMixin, TemplateVie
                             resultado = TipoResultadoVotacao.objects.get(
                                 id=request.POST['resultado_votacao'])
                             votacao.tipo_resultado_votacao = resultado
+                            votacao.user = request.user
+                            votacao.ip = get_client_ip(request)
                             votacao.save()
                         except Exception as e:
                             username = request.user.username
@@ -3671,6 +3681,8 @@ class VotacaoEmBlocoSimbolicaView(PermissionRequiredForAppCrudMixin, TemplateVie
                             resultado = TipoResultadoVotacao.objects.get(
                                 id=request.POST['resultado_votacao'])
                             votacao.tipo_resultado_votacao = resultado
+                            votacao.user = request.user
+                            votacao.ip = get_client_ip(request)
                             votacao.save()
                         except Exception as e:
                             username = request.user.username
@@ -3862,6 +3874,8 @@ class VotacaoEmBlocoNominalView(PermissionRequiredForAppCrudMixin, TemplateView)
                             voto_parlamentar.voto = voto
                             voto_parlamentar.parlamentar_id = parlamentar_id
                             voto_parlamentar.votacao_id = votacao.id
+                            voto_parlamentar.user = request.user
+                            voto_parlamentar.ip = get_client_ip(request)
                             voto_parlamentar.save()
 
                             ordem.resultado = form.cleaned_data['resultado_votacao'].nome
@@ -3889,6 +3903,8 @@ class VotacaoEmBlocoNominalView(PermissionRequiredForAppCrudMixin, TemplateView)
                         votacao.materia = expediente.materia
                         votacao.expediente = expediente
                         votacao.tipo_resultado_votacao = form.cleaned_data['resultado_votacao']
+                        votacao.user = request.user
+                        votacao.ip = get_client_ip(request)
                         votacao.save()
 
                         # Salva os votos de cada parlamentar
@@ -3904,6 +3920,8 @@ class VotacaoEmBlocoNominalView(PermissionRequiredForAppCrudMixin, TemplateView)
                             voto_parlamentar.voto = voto
                             voto_parlamentar.parlamentar_id = parlamentar_id
                             voto_parlamentar.votacao_id = votacao.id
+                            voto_parlamentar.user = request.user
+                            voto_parlamentar.ip = get_client_ip(request)
                             voto_parlamentar.save()
 
                             expediente.resultado = form.cleaned_data['resultado_votacao'].nome

sapl/templates/base/layouts.yaml

@@ -18,8 +18,8 @@ AppConfig:
   - esfera_federacao
 
   {% trans 'Proposições e Protocolo' %}:
-  - sequencia_numeracao proposicao_incorporacao_obrigatoria receber_recibo_proposicao
+  - sequencia_numeracao protocolo_manual receber_recibo_proposicao
-  - escolher_numero_materia_proposicao protocolo_manual
+  - proposicao_incorporacao_obrigatoria escolher_numero_materia_proposicao
 
   {% trans 'Textos Articulados' %}:
   - texto_articulado_proposicao texto_articulado_materia texto_articulado_norma