Conserta testes e verificação nos forms para se adequar ao Django 1.10

sapl/base/tests/teststub_urls.py

@@ -1,11 +1,10 @@
-from django.conf.urls import patterns, url
+from django.conf.urls import url
 from django.views.generic.base import TemplateView
 
 from sapl.urls import urlpatterns as original_patterns
 
-ptrn = patterns('',
-                url(r'^zzzz$',
+ptrn = [            url(r'^zzzz$',
                     TemplateView.as_view(
-                        template_name='index.html'), name='zzzz'))
+                        template_name='index.html'), name='zzzz')]
 
 urlpatterns = original_patterns + ptrn

sapl/comissoes/forms.py

@@ -65,6 +65,11 @@ class ParticipacaoCreateForm(forms.ModelForm):
 
     def clean(self):
         super(ParticipacaoCreateForm, self).clean()
+        cleaned_data = self.cleaned_data
+
+        if self.errors:
+            return cleaned_data
+
         return self.cleaned_data
 
     def verifica(self):
@@ -123,6 +128,10 @@ class ComissaoForm(forms.ModelForm):
 
     def clean(self):
         super(ComissaoForm, self).clean()
+        cleaned_data = self.cleaned_data
+
+        if self.errors:
+            return cleaned_data
 
         if self.cleaned_data['data_extincao']:
             if (self.cleaned_data['data_extincao'] <
@@ -162,9 +171,10 @@ class ReuniaoForm(ModelForm):
 
     def clean(self):
         super(ReuniaoForm, self).clean()
+        cleaned_data = self.cleaned_data
 
         if self.errors:
-            return
+            return cleaned_data
 
         if self.cleaned_data['hora_fim'] < self.cleaned_data['hora_inicio']:
             msg = _('A hora de término da reunião não pode '

sapl/comissoes/tests/test_comissoes.py

@@ -72,7 +72,9 @@ def test_incluir_comissao_submit(admin_client):
                                   'nome': 'Comissão Teste',
                                   'sigla': 'CT',
                                   'data_criacao': '2016-03-22',
-                                  'salvar': 'salvar'},
+                                  'unidade_deliberativa': True,
+                                  'salvar': 'salvar'
+                                  },
                                  follow=True)
     assert response.status_code == 200
 

sapl/materia/forms.py

@@ -166,9 +166,11 @@ class MateriaLegislativaForm(ModelForm):
 
     def clean(self):
         super(MateriaLegislativaForm, self).clean()
-
         cleaned_data = self.cleaned_data
 
+        if self.errors:
+            return cleaned_data
+
         data_apresentacao = cleaned_data['data_apresentacao']
         ano = cleaned_data['ano']
 
@@ -199,6 +201,9 @@ class UnidadeTramitacaoForm(ModelForm):
 
         cleaned_data = self.cleaned_data
 
+        if self.errors:
+            return cleaned_data
+
         for key in list(cleaned_data.keys()):
             if cleaned_data[key] is None:
                 del cleaned_data[key]
@@ -297,7 +302,7 @@ class TramitacaoForm(ModelForm):
             data_tram_form = cleaned_data['data_tramitacao']
 
         if self.errors:
-            return self.errors
+            return cleaned_data
 
         ultima_tramitacao = Tramitacao.objects.filter(
             materia_id=self.instance.materia_id).exclude(
@@ -367,6 +372,12 @@ class TramitacaoUpdateForm(TramitacaoForm):
         }
 
     def clean(self):
+        super(TramitacaoUpdateForm, self).clean()
+        cleaned_data = self.cleaned_data
+
+        if self.errors:
+            return cleaned_data
+
         ultima_tramitacao = Tramitacao.objects.filter(
             materia_id=self.instance.materia_id).order_by(
             '-data_tramitacao',
@@ -423,11 +434,11 @@ class LegislacaoCitadaForm(ModelForm):
 
     def clean(self):
         super(LegislacaoCitadaForm, self).clean()
+        cleaned_data = self.cleaned_data
 
         if self.errors:
-            return self.errors
+            return cleaned_data
 
-        cleaned_data = self.cleaned_data
 
         try:
             norma = NormaJuridica.objects.get(
@@ -486,9 +497,10 @@ class NumeracaoForm(ModelForm):
 
     def clean(self):
         super(NumeracaoForm, self).clean()
+        cleaned_data = self.cleaned_data
 
         if self.errors:
-            return self.errors
+            return cleaned_data
 
         try:
             MateriaLegislativa.objects.get(
@@ -531,11 +543,11 @@ class AnexadaForm(ModelForm):
 
     def clean(self):
         super(AnexadaForm, self).clean()
+        cleaned_data = self.cleaned_data
 
         if self.errors:
-            return self.errors
+            return cleaned_data
 
-        cleaned_data = self.cleaned_data
 
         try:
             materia_anexada = MateriaLegislativa.objects.get(
@@ -724,9 +736,10 @@ class DespachoInicialForm(ModelForm):
 
     def clean(self):
         super(DespachoInicialForm, self).clean()
+        cleaned_data = self.cleaned_data
 
         if self.errors:
-            return self.errors
+            return cleaned_data
 
         if DespachoInicial.objects.filter(
             materia=self.instance.materia,
@@ -769,9 +782,10 @@ class AutoriaForm(ModelForm):
 
     def clean(self):
         cd = super(AutoriaForm, self).clean()
+        cleaned_data = self.cleaned_data
 
         if self.errors:
-            return self.errors
+            return cleaned_data
 
         autorias = Autoria.objects.filter(
             materia=self.instance.materia, autor=cd['autor'])
@@ -992,6 +1006,10 @@ class TipoProposicaoForm(ModelForm):
 
     def clean(self):
         super(TipoProposicaoForm, self).clean()
+        cleaned_data = self.cleaned_data
+
+        if self.errors:
+            return cleaned_data
 
         cd = self.cleaned_data
 
@@ -1067,7 +1085,7 @@ class TipoProposicaoSelect(Select):
             str(data_has_perfil),
             force_text(option_label))
 
-    def render_options(self, choices, selected_choices):
+    def render_options(self, selected_choices):
         # Normalize to strings.
         selected_choices = set(force_text(v) for v in selected_choices)
         output = []
@@ -1226,6 +1244,10 @@ class ProposicaoForm(forms.ModelForm):
 
     def clean(self):
         super(ProposicaoForm, self).clean()
+        cleaned_data = self.cleaned_data
+
+        if self.errors:
+            return cleaned_data
 
         cd = self.cleaned_data
 
@@ -1317,6 +1339,10 @@ class DevolverProposicaoForm(forms.ModelForm):
 
     def clean(self):
         super(DevolverProposicaoForm, self).clean()
+        cleaned_data = self.cleaned_data
+
+        if self.errors:
+            return cleaned_data
 
         cd = self.cleaned_data
 
@@ -1499,6 +1525,10 @@ class ConfirmarProposicaoForm(ProposicaoForm):
 
     def clean(self):
         super(ConfirmarProposicaoForm, self).clean()
+        cleaned_data = self.cleaned_data
+
+        if self.errors:
+            return cleaned_data
 
         numeracao = sapl.base.models.AppConfig.attr('sequencia_numeracao')
 

sapl/norma/forms.py

@@ -192,10 +192,10 @@ class NormaRelacionadaForm(ModelForm):
 
     def clean(self):
         super(NormaRelacionadaForm, self).clean()
+        cleaned_data = self.cleaned_data
 
         if self.errors:
-            return self.errors
-        cleaned_data = self.cleaned_data
+            return cleaned_data
 
         try:
             norma_relacionada = NormaJuridica.objects.get(

sapl/parlamentares/forms.py

@@ -257,9 +257,10 @@ class FiliacaoForm(ModelForm):
 
     def clean(self):
         super(FiliacaoForm, self).clean()
+        cleaned_data = self.cleaned_data
 
         if self.errors:
-            return self.errors
+            return cleaned_data
 
         filiacao = super(FiliacaoForm, self).save(commit=False)
         validacao = validar_datas(self.cleaned_data['data'],