Fix qa_check

9 years ago · 447dd1b276
6 changed files with 42 additions and 42 deletions
--- a/materia/views.py
+++ b/materia/views.py
@ -5,6 +5,7 @@ from crispy_forms.helper import FormHelper
 from crispy_forms.layout import ButtonHolder, Column, Fieldset, Layout, Submit
 from django import forms
 from django.core.exceptions import ObjectDoesNotExist
+from django.core.mail import send_mail
 from django.core.urlresolvers import reverse
 from django.forms import ModelForm
 from django.shortcuts import redirect
@ -14,7 +15,6 @@ from django.utils.translation import ugettext_lazy as _
 from django.views.generic import ListView
 from django.views.generic.edit import FormMixin
 from vanilla.views import GenericView
-from django.core.mail import send_mail

 import sapl
 from comissoes.models import Comissao, Composicao
@ -22,6 +22,7 @@ from compilacao.views import IntegracaoTaView
 from norma.models import LegislacaoCitada, NormaJuridica, TipoNormaJuridica
 from parlamentares.models import Parlamentar
 from sapl.crud import build_crud
+from sessao.models import AcompanharMateria

 from .models import (Anexada, Autor, Autoria, DespachoInicial,
                     DocumentoAcessorio, MateriaLegislativa, Numeracao, Orgao,
@ -29,7 +30,6 @@ from .models import (Anexada, Autor, Autoria, DespachoInicial,
                     StatusTramitacao, TipoAutor, TipoDocumento,
                     TipoFimRelatoria, TipoMateriaLegislativa, TipoProposicao,
                     Tramitacao, UnidadeTramitacao)
-from sessao.models import AcompanharMateria

 origem_crud = build_crud(
    Origem, 'origem', [
@ -1488,7 +1488,8 @@ class TramitacaoView(FormMixin, GenericView):
                     'error': 'A origem da nova tramitação \
                        deve ser igual ao destino da última adicionada!'})

-            corpo_email = ('A tramitação da matéria %s foi alterada.' % materia)
+            corpo_email = ('A tramitação da matéria\
+                %s foi alterada.' % materia)
            destinatarios = AcompanharMateria.objects.values_list(
                'email', flat=True).filter(
                materia_cadastrada=materia)
@ -1496,7 +1497,7 @@ class TramitacaoView(FormMixin, GenericView):
                      corpo_email,
                      'sapl-test@interlegis.leg.br',
                      destinatarios,
-              fail_silently=True)
+                      fail_silently=True)
            return self.form_valid(form)
        else:
            return self.render_to_response({'form': form,
--- a/parlamentares/views.py
+++ b/parlamentares/views.py
@ -11,7 +11,6 @@ from django.utils.safestring import mark_safe
 from django.utils.translation import ugettext_lazy as _
 from django.views.generic.edit import FormMixin
 from vanilla import GenericView
-from django.core.exceptions import ObjectDoesNotExist

 import sapl
 from sapl.crud import build_crud
@ -781,7 +780,7 @@ class FiliacaoView(FormMixin, GenericView):
             'form': form,
             'legislatura_id': parlamentar.mandato_set.last().legislatura.id})

-    # Função usada para todos os caso de erro na filiação    
+    # Função usada para todos os caso de erro na filiação
    def error_message(self, parlamentar, form, mensagem):
        filiacoes = Filiacao.objects.filter(parlamentar=parlamentar)
        return self.render_to_response(
@ -805,11 +804,11 @@ class FiliacaoView(FormMixin, GenericView):
            parlamentar = Parlamentar.objects.get(id=pid)

            candidato_filiado = Filiacao.objects.filter(
-                                    parlamentar=parlamentar)
+                parlamentar=parlamentar)

            candidato_nao_desfiliou = Filiacao.objects.filter(
-                                      parlamentar=parlamentar,
-                                      data_desfiliacao=None)
+                parlamentar=parlamentar,
+                data_desfiliacao=None)

            # Vê se o candidato já se filiou alguma vez a algum partido
            if not candidato_filiado:
@ -844,8 +843,8 @@ class FiliacaoView(FormMixin, GenericView):
                        return self.error_message(parlamentar, form, mensagem)

                    if (data_desfiliacao and
-                       data_desfiliacao < data_fim and
-                       data_desfiliacao > data_init):
+                            data_desfiliacao < data_fim and
+                            data_desfiliacao > data_init):

                        mensagem = "A data de filiação e\
                        desfiliação não podem estar no intervalo\
@ -884,11 +883,11 @@ class FiliacaoEditView(FormMixin, GenericView):

    def error_message(self, parlamentar, form, mensagem):
        return self.render_to_response(
-                {'form': form,
-                 'parlamentar': parlamentar,
-                 'legislatura_id': parlamentar.mandato_set.last(
-                 ).legislatura_id,
-                 'mensagem_erro': mensagem})
+            {'form': form,
+             'parlamentar': parlamentar,
+             'legislatura_id': parlamentar.mandato_set.last(
+             ).legislatura_id,
+             'mensagem_erro': mensagem})

    def post(self, request, *args, **kwargs):
        filiacao = Filiacao.objects.get(id=kwargs['dk'])
@ -905,11 +904,11 @@ class FiliacaoEditView(FormMixin, GenericView):
            parlamentar = Parlamentar.objects.get(id=pid)

            candidato_filiado = Filiacao.objects.filter(
-                                    parlamentar=parlamentar)
+                parlamentar=parlamentar)

            candidato_nao_desfiliou = Filiacao.objects.filter(
-                                      parlamentar=parlamentar,
-                                      data_desfiliacao=None)
+                parlamentar=parlamentar,
+                data_desfiliacao=None)

            # Vê se o candidato já se filiou alguma vez a algum partido
            if not candidato_filiado:
@ -940,7 +939,7 @@ class FiliacaoEditView(FormMixin, GenericView):
                        data_init = todas_filiacoes[i].data
                        data_fim = todas_filiacoes[i].data_desfiliacao
                        if (data_filiacao >= data_init and
-                           data_filiacao < data_fim):
+                                data_filiacao < data_fim):

                            mensagem = "A data de filiação e\
                            desfiliação não podem estar no intervalo\
@ -950,8 +949,8 @@ class FiliacaoEditView(FormMixin, GenericView):
                                                      mensagem)

                        if (data_desfiliacao and
-                           data_desfiliacao < data_fim and
-                           data_desfiliacao > data_init):
+                                data_desfiliacao < data_fim and
+                                data_desfiliacao > data_init):

                            mensagem = "A data de filiação e\
                            desfiliação não podem estar no intervalo\
--- a/sapl/settings.py
+++ b/sapl/settings.py
@ -61,8 +61,9 @@ INSTALLED_APPS = (
    'crispy_forms_foundation',
    'sass_processor',
 )
+
 if DEBUG:
-   INSTALLED_APPS += ('debug_toolbar',)
+    INSTALLED_APPS += ('debug_toolbar',)

 MIDDLEWARE_CLASSES = (
    'django.contrib.sessions.middleware.SessionMiddleware',
--- a/sessao/models.py
+++ b/sessao/models.py
@ -5,7 +5,6 @@ from materia.models import MateriaLegislativa
 from parlamentares.models import (CargoMesa, Legislatura, Parlamentar,
                                  SessaoLegislativa)
 from sapl.utils import YES_NO_CHOICES, make_choices
-from django.contrib.auth.models import User


 class TipoSessaoPlenaria(models.Model):
--- a/sessao/urls.py
+++ b/sessao/urls.py
@ -1,13 +1,14 @@
 from django.conf.urls import include, url

 from sapl import settings
-from sessao.views import (EditExpedienteOrdemDiaView, EditMateriaOrdemDiaView,
-                          ExpedienteOrdemDiaView, ExpedienteView,
-                          ExplicacaoDelete, ExplicacaoEdit, ExplicacaoView,
-                          ListExpedienteOrdemDiaView, ListMateriaOrdemDiaView,
-                          MateriaOrdemDiaView, MesaView,
-                          OradorExpedienteDelete, OradorExpedienteEdit,
-                          OradorExpedienteView, PainelView,
+from sessao.views import (AcompanharMateriaView, EditExpedienteOrdemDiaView,
+                          EditMateriaOrdemDiaView, ExpedienteOrdemDiaView,
+                          ExpedienteView, ExplicacaoDelete, ExplicacaoEdit,
+                          ExplicacaoView, ListExpedienteOrdemDiaView,
+                          ListMateriaOrdemDiaView, MateriaOrdemDiaView,
+                          MesaView, OradorExpedienteDelete,
+                          OradorExpedienteEdit, OradorExpedienteView,
+                          PainelView, PautaExpedienteDetail, PautaOrdemDetail,
                          PautaSessaoDetailView, PautaSessaoListView,
                          PresencaOrdemDiaView, PresencaView, ResumoView,
                          SessaoCadastroView, SessaoListView,
@ -17,9 +18,7 @@ from sessao.views import (EditExpedienteOrdemDiaView, EditMateriaOrdemDiaView,
                          VotacaoNominalExpedienteEditView,
                          VotacaoNominalExpedienteView, VotacaoNominalView,
                          VotacaoView, sessao_crud, tipo_expediente_crud,
-                          tipo_resultado_votacao_crud, tipo_sessao_crud,
-                          PautaExpedienteDetail, PautaOrdemDetail,
-                          AcompanharMateriaView)
+                          tipo_resultado_votacao_crud, tipo_sessao_crud)

 urlpatterns_sessao = sessao_crud.urlpatterns + [
    url(r'^(?P<pk>\d+)/expediente$',
--- a/sessao/views.py
+++ b/sessao/views.py
@ -2,7 +2,7 @@ from datetime import datetime
 from re import sub

 from crispy_forms.helper import FormHelper
-from crispy_forms.layout import ButtonHolder, Fieldset, Layout, Submit, Column
+from crispy_forms.layout import ButtonHolder, Column, Fieldset, Layout, Submit
 from django import forms
 from django.core.exceptions import ObjectDoesNotExist
 from django.core.urlresolvers import reverse
@ -15,19 +15,19 @@ from django.views.generic.edit import FormMixin
 from rest_framework import generics

 import sapl
-from materia.models import (Autoria, TipoMateriaLegislativa,
-                            DocumentoAcessorio, Tramitacao)
+from materia.models import (Autoria, DocumentoAcessorio,
+                            TipoMateriaLegislativa, Tramitacao)
 from norma.models import NormaJuridica
 from parlamentares.models import Parlamentar
 from sapl.crud import build_crud
 from sessao.serializers import SessaoPlenariaSerializer

-from .models import (CargoMesa, ExpedienteMateria, ExpedienteSessao,
-                     IntegranteMesa, MateriaLegislativa, Orador,
-                     OradorExpediente, OrdemDia, PresencaOrdemDia,
+from .models import (AcompanharMateria, CargoMesa, ExpedienteMateria,
+                     ExpedienteSessao, IntegranteMesa, MateriaLegislativa,
+                     Orador, OradorExpediente, OrdemDia, PresencaOrdemDia,
                     RegistroVotacao, SessaoPlenaria, SessaoPlenariaPresenca,
                     TipoExpediente, TipoResultadoVotacao, TipoSessaoPlenaria,
-                     VotoParlamentar, AcompanharMateria)
+                     VotoParlamentar)

 tipo_sessao_crud = build_crud(
    TipoSessaoPlenaria, 'tipo_sessao_plenaria', [
@ -2546,7 +2546,8 @@ class AcompanharMateriaView(FormMixin, sessao_crud.CrudDetailView):
                return self.render_to_response(
                    {'form': form,
                     'materia': materia,
-                     'error': 'Essa matéria já está sendo acompanhada por este e-mail.'})
+                     'error': 'Essa matéria já está\
+                     sendo acompanhada por este e-mail.'})
            return self.form_valid(form)
        else:
            return self.render_to_response(