From 14615b647d28474423a11cc3c708fa2a320823d8 Mon Sep 17 00:00:00 2001 From: Marcio Mazza Date: Wed, 6 Sep 2017 16:19:54 -0300 Subject: [PATCH] Corrige problemas de PEP 8 --- sapl/base/tests/teststub_urls.py | 6 +-- .../tests/test_tipo_texto_articulado_form.py | 3 +- sapl/decorators.py | 2 +- sapl/materia/forms.py | 11 +++-- sapl/materia/models.py | 2 +- sapl/materia/urls.py | 2 +- sapl/materia/views.py | 17 ++++---- sapl/norma/tests/test_norma.py | 4 +- sapl/norma/views.py | 2 +- sapl/painel/views.py | 14 +++---- .../parlamentares/tests/test_parlamentares.py | 2 +- sapl/parlamentares/views.py | 2 +- sapl/protocoloadm/forms.py | 5 +-- sapl/protocoloadm/tests/test_protocoloadm.py | 3 +- sapl/protocoloadm/views.py | 5 +-- sapl/test_urls.py | 40 +++++++++---------- 16 files changed, 55 insertions(+), 65 deletions(-) diff --git a/sapl/base/tests/teststub_urls.py b/sapl/base/tests/teststub_urls.py index 4ad6c6d9e..fbdcbb443 100644 --- a/sapl/base/tests/teststub_urls.py +++ b/sapl/base/tests/teststub_urls.py @@ -4,6 +4,6 @@ from django.views.generic.base import TemplateView from sapl.urls import urlpatterns as original_patterns urlpatterns = original_patterns + patterns('', url(r'^zzzz$', - TemplateView.as_view( - template_name='index.html'), - name='zzzz')) + TemplateView.as_view( + template_name='index.html'), + name='zzzz')) diff --git a/sapl/compilacao/tests/test_tipo_texto_articulado_form.py b/sapl/compilacao/tests/test_tipo_texto_articulado_form.py index e121d771f..d9db055e3 100644 --- a/sapl/compilacao/tests/test_tipo_texto_articulado_form.py +++ b/sapl/compilacao/tests/test_tipo_texto_articulado_form.py @@ -3,8 +3,7 @@ from django.utils.translation import ugettext as _ from model_mommy import mommy from sapl.compilacao import forms -from sapl.compilacao.models import (PerfilEstruturalTextoArticulado, - TipoNota) +from sapl.compilacao.models import PerfilEstruturalTextoArticulado, TipoNota from sapl.compilacao.views import choice_models_in_extenal_views diff --git a/sapl/decorators.py b/sapl/decorators.py index 97f9af7a6..0e301565c 100644 --- a/sapl/decorators.py +++ b/sapl/decorators.py @@ -34,7 +34,7 @@ def vigencia_atual(decorated_method): print(_('{} {}'.format( _(instancia_sem_atributo), _(mensagem_decorator) - ) + ) ) ) diff --git a/sapl/materia/forms.py b/sapl/materia/forms.py index e2ff7f535..678264194 100644 --- a/sapl/materia/forms.py +++ b/sapl/materia/forms.py @@ -1653,7 +1653,7 @@ class EtiquetaPesquisaForm(forms.Form): # Então verifica se o usuário preencheu o Incial e mas não # preencheu o Final, ou vice-versa if (not cleaned_data['data_inicial'] or - not cleaned_data['data_final']): + not cleaned_data['data_final']): raise ValidationError(_( 'Caso pesquise por data, os campos de Data Incial e ' + 'Data Final devem ser preenchidos obrigatoriamente')) @@ -1665,17 +1665,16 @@ class EtiquetaPesquisaForm(forms.Form): # O mesmo processo anterior é feito com o processo if (cleaned_data['processo_inicial'] or - cleaned_data['processo_final']): + cleaned_data['processo_final']): if (not cleaned_data['processo_inicial'] or - not cleaned_data['processo_final']): + not cleaned_data['processo_final']): raise ValidationError(_( 'Caso pesquise por número de processo, os campos de ' + 'Processo Inicial e Processo Final ' + 'devem ser preenchidos obrigatoriamente')) elif (cleaned_data['processo_final'] < cleaned_data['processo_inicial']): - raise ValidationError(_( - 'O processo final não pode ser menor que o inicial')) + raise ValidationError(_( + 'O processo final não pode ser menor que o inicial')) return cleaned_data - diff --git a/sapl/materia/models.py b/sapl/materia/models.py index e244fde60..43027bea9 100644 --- a/sapl/materia/models.py +++ b/sapl/materia/models.py @@ -803,7 +803,7 @@ class UnidadeTramitacao(models.Model): class Meta: verbose_name = _('Unidade de Tramitação') verbose_name_plural = _('Unidades de Tramitação') - ordering = ['orgao','comissao','parlamentar'] + ordering = ['orgao', 'comissao', 'parlamentar'] def __str__(self): if self.orgao and self.comissao and self.parlamentar: diff --git a/sapl/materia/urls.py b/sapl/materia/urls.py index e803f18e0..e0a056011 100644 --- a/sapl/materia/urls.py +++ b/sapl/materia/urls.py @@ -8,7 +8,7 @@ from sapl.materia.views import (AcompanhamentoConfirmarView, CriarProtocoloMateriaView, DespachoInicialCrud, DocumentoAcessorioCrud, DocumentoAcessorioEmLoteView, - ImpressosView, EtiquetaPesquisaView, + EtiquetaPesquisaView, ImpressosView, LegislacaoCitadaCrud, MateriaAssuntoCrud, MateriaLegislativaCrud, MateriaLegislativaPesquisaView, MateriaTaView, diff --git a/sapl/materia/views.py b/sapl/materia/views.py index 2d11ecf67..80e5e9a1d 100644 --- a/sapl/materia/views.py +++ b/sapl/materia/views.py @@ -12,7 +12,7 @@ from django.core.urlresolvers import reverse from django.http import HttpResponse, JsonResponse from django.http.response import Http404, HttpResponseRedirect from django.shortcuts import get_object_or_404, redirect -from django.template import Context, loader, RequestContext +from django.template import Context, RequestContext, loader from django.utils import formats from django.utils.translation import ugettext_lazy as _ from django.views.generic import CreateView, ListView, TemplateView, UpdateView @@ -21,6 +21,7 @@ from django.views.generic.edit import FormView from django_filters.views import FilterView import sapl +import weasyprint from sapl.base.models import Autor, CasaLegislativa from sapl.comissoes.models import Comissao, Participacao from sapl.compilacao.models import (STATUS_TA_IMMUTABLE_RESTRICT, @@ -46,11 +47,10 @@ from .email_utils import do_envia_email_confirmacao from .forms import (AcessorioEmLoteFilterSet, AcompanhamentoMateriaForm, AdicionarVariasAutoriasFilterSet, DespachoInicialForm, DocumentoAcessorioForm, EtiquetaPesquisaForm, - MateriaAssuntoForm, - MateriaLegislativaFilterSet, MateriaSimplificadaForm, - PrimeiraTramitacaoEmLoteFilterSet, ReceberProposicaoForm, - RelatoriaForm, TramitacaoEmLoteFilterSet, - filtra_tramitacao_destino, + MateriaAssuntoForm, MateriaLegislativaFilterSet, + MateriaSimplificadaForm, PrimeiraTramitacaoEmLoteFilterSet, + ReceberProposicaoForm, RelatoriaForm, + TramitacaoEmLoteFilterSet, filtra_tramitacao_destino, filtra_tramitacao_destino_and_status, filtra_tramitacao_status) from .models import (AcompanhamentoMateria, Anexada, AssuntoMateria, Autoria, @@ -61,8 +61,6 @@ from .models import (AcompanhamentoMateria, Anexada, AssuntoMateria, Autoria, TipoProposicao, Tramitacao, UnidadeTramitacao) from .signals import tramitacao_signal -import weasyprint - AssuntoMateriaCrud = Crud.build(AssuntoMateria, 'assunto_materia') OrigemCrud = Crud.build(Origem, '') @@ -1751,6 +1749,7 @@ class ImpressosView(PermissionRequiredMixin, TemplateView): template_name = 'materia/impressos/impressos.html' permission_required = ('materia.can_access_impressos', ) + def gerar_pdf_impressos(request, context): template = loader.get_template('materia/impressos/pdf.html') html = template.render(RequestContext(request, context)) @@ -1797,5 +1796,3 @@ class EtiquetaPesquisaView(PermissionRequiredMixin, FormView): context['materias'] = materias return gerar_pdf_impressos(self.request, context) - - diff --git a/sapl/norma/tests/test_norma.py b/sapl/norma/tests/test_norma.py index fa8214e58..72c27b22a 100644 --- a/sapl/norma/tests/test_norma.py +++ b/sapl/norma/tests/test_norma.py @@ -1,13 +1,11 @@ import pytest - from django.core.urlresolvers import reverse from django.utils.translation import ugettext_lazy as _ - from model_mommy import mommy from sapl.materia.models import MateriaLegislativa, TipoMateriaLegislativa from sapl.norma.forms import NormaJuridicaForm, NormaRelacionadaForm -from sapl.norma.models import (NormaJuridica, TipoNormaJuridica) +from sapl.norma.models import NormaJuridica, TipoNormaJuridica @pytest.mark.django_db(transaction=False) diff --git a/sapl/norma/views.py b/sapl/norma/views.py index 28835a2a0..8e70bd8e1 100644 --- a/sapl/norma/views.py +++ b/sapl/norma/views.py @@ -1,4 +1,5 @@ from datetime import datetime + from django.core.exceptions import ObjectDoesNotExist from django.core.urlresolvers import reverse from django.http import JsonResponse @@ -146,7 +147,6 @@ class NormaCrud(Crud): def layout_key(self): return 'NormaJuridicaCreate' - class ListView(Crud.ListView, RedirectView): def get_redirect_url(self, *args, **kwargs): diff --git a/sapl/painel/views.py b/sapl/painel/views.py index 1b1678833..64deab910 100644 --- a/sapl/painel/views.py +++ b/sapl/painel/views.py @@ -261,7 +261,7 @@ def get_materia_aberta(pk): def get_presentes(pk, response, materia): if type(materia) == OrdemDia: presentes = PresencaOrdemDia.objects.filter( - sessao_plenaria_id=pk) + sessao_plenaria_id=pk) elif type(materia) == ExpedienteMateria: presentes = SessaoPlenariaPresenca.objects.filter( sessao_plenaria_id=pk) @@ -384,12 +384,12 @@ def get_dados_painel(request, pk): # com prioridade para Ordem do Dia. if ordem_dia: return JsonResponse(get_votos( - get_presentes(pk, response, ordem_dia), - ordem_dia)) + get_presentes(pk, response, ordem_dia), + ordem_dia)) elif expediente: return JsonResponse(get_votos( - get_presentes(pk, response, expediente), - expediente)) + get_presentes(pk, response, expediente), + expediente)) # Caso não tenha nenhuma aberta, a matéria a ser mostrada no Painel deve ser # a última votada @@ -407,8 +407,8 @@ def get_dados_painel(request, pk): # Se alguma ordem E algum expediente já tiver sido votado... if last_ordem_voto and last_expediente_voto: materia = ultima_ordem_votada\ - if last_ordem_voto.pk >= last_expediente_voto.pk\ - else ultimo_expediente_votado + if last_ordem_voto.pk >= last_expediente_voto.pk\ + else ultimo_expediente_votado # Caso somente um deles tenha resultado, prioriza a Ordem do Dia elif last_ordem_voto: diff --git a/sapl/parlamentares/tests/test_parlamentares.py b/sapl/parlamentares/tests/test_parlamentares.py index a10d235e5..3793d400d 100644 --- a/sapl/parlamentares/tests/test_parlamentares.py +++ b/sapl/parlamentares/tests/test_parlamentares.py @@ -3,7 +3,7 @@ from django.core.urlresolvers import reverse from django.utils.translation import ugettext_lazy as _ from model_mommy import mommy -from sapl.parlamentares.forms import (FrenteForm, LegislaturaForm, MandatoForm) +from sapl.parlamentares.forms import FrenteForm, LegislaturaForm, MandatoForm from sapl.parlamentares.models import (Dependente, Filiacao, Legislatura, Mandato, Parlamentar, Partido, TipoDependente) diff --git a/sapl/parlamentares/views.py b/sapl/parlamentares/views.py index 75e1494c0..1879f5f5d 100644 --- a/sapl/parlamentares/views.py +++ b/sapl/parlamentares/views.py @@ -1,5 +1,5 @@ -from datetime import datetime import json +from datetime import datetime from django.contrib import messages from django.contrib.contenttypes.models import ContentType diff --git a/sapl/protocoloadm/forms.py b/sapl/protocoloadm/forms.py index 626c25186..aa40f9d71 100644 --- a/sapl/protocoloadm/forms.py +++ b/sapl/protocoloadm/forms.py @@ -1,5 +1,6 @@ from datetime import datetime +import django_filters from crispy_forms.bootstrap import InlineRadios from crispy_forms.helper import FormHelper from crispy_forms.layout import HTML, Button, Fieldset, Layout, Submit @@ -8,10 +9,9 @@ from django.core.exceptions import ObjectDoesNotExist, ValidationError from django.db import models from django.forms import ModelForm from django.utils.translation import ugettext_lazy as _ -import django_filters from sapl.base.models import Autor, TipoAutor -from sapl.crispy_layout_mixin import form_actions, to_row, SaplFormLayout +from sapl.crispy_layout_mixin import SaplFormLayout, form_actions, to_row from sapl.materia.models import (MateriaLegislativa, TipoMateriaLegislativa, UnidadeTramitacao) from sapl.utils import (RANGE_ANOS, AnoNumeroOrderingFilter, @@ -21,7 +21,6 @@ from .models import (DocumentoAcessorioAdministrativo, DocumentoAdministrativo, Protocolo, TipoDocumentoAdministrativo, TramitacaoAdministrativo) - TIPOS_PROTOCOLO = [('0', 'Recebido'), ('1', 'Enviado'), ('', 'Ambos')] TIPOS_PROTOCOLO_CREATE = [('0', 'Recebido'), ('1', 'Enviado')] diff --git a/sapl/protocoloadm/tests/test_protocoloadm.py b/sapl/protocoloadm/tests/test_protocoloadm.py index 0ffbb5e4b..aa0da3b52 100644 --- a/sapl/protocoloadm/tests/test_protocoloadm.py +++ b/sapl/protocoloadm/tests/test_protocoloadm.py @@ -9,8 +9,7 @@ from model_mommy import mommy from sapl.materia.models import UnidadeTramitacao from sapl.protocoloadm.forms import (AnularProcoloAdmForm, DocumentoAdministrativoForm, - MateriaLegislativa, - ProtocoloDocumentForm, + MateriaLegislativa, ProtocoloDocumentForm, ProtocoloMateriaForm) from sapl.protocoloadm.models import (DocumentoAdministrativo, Protocolo, StatusTramitacaoAdministrativo, diff --git a/sapl/protocoloadm/views.py b/sapl/protocoloadm/views.py index 043333323..82de1d73d 100644 --- a/sapl/protocoloadm/views.py +++ b/sapl/protocoloadm/views.py @@ -11,9 +11,10 @@ from django.http import Http404, HttpResponse, JsonResponse from django.shortcuts import redirect from django.utils.translation import ugettext_lazy as _ from django.views.generic import CreateView, ListView -from django.views.generic.base import TemplateView, RedirectView +from django.views.generic.base import RedirectView, TemplateView from django_filters.views import FilterView +import sapl from sapl.base.models import Autor from sapl.comissoes.models import Comissao from sapl.crud.base import Crud, CrudAux, MasterDetailCrud, make_pagination @@ -21,7 +22,6 @@ from sapl.materia.models import MateriaLegislativa, TipoMateriaLegislativa from sapl.parlamentares.models import Legislatura, Parlamentar from sapl.protocoloadm.models import Protocolo from sapl.utils import create_barcode, get_client_ip -import sapl from .forms import (AnularProcoloAdmForm, DocumentoAcessorioAdministrativoForm, DocumentoAdministrativoFilterSet, @@ -32,7 +32,6 @@ from .models import (DocumentoAcessorioAdministrativo, DocumentoAdministrativo, StatusTramitacaoAdministrativo, TipoDocumentoAdministrativo, TramitacaoAdministrativo) - TipoDocumentoAdministrativoCrud = CrudAux.build( TipoDocumentoAdministrativo, '') diff --git a/sapl/test_urls.py b/sapl/test_urls.py index bfccb3672..8b5207276 100644 --- a/sapl/test_urls.py +++ b/sapl/test_urls.py @@ -173,26 +173,26 @@ apps_url_patterns_prefixs_and_users = { '/ta', ]}, 'redireciona_urls': { - 'prefixs': [ - '/default_index_html', - '/consultas/parlamentar/parlamentar_', - '/consultas/comissao/comissao_', - '/consultas/pauta_sessao/pauta_sessao_', - '/consultas/sessao_plenaria/', - '/relatorios_administrativos/relatorios_administrativos_index_html', - '/tramitacaoMaterias/tramitacaoMaterias', - '/tramitacaoMaterias/materia_mostrar_proc', - '/generico/materia_pesquisar_', - '/consultas/mesa_diretora/mesa_diretora_index_html', - '/consultas/mesa_diretora/parlamentar/parlamentar_', - '/generico/norma_juridica_pesquisar_', - '/consultas/norma_juridica/norma_juridica_mostrar_proc', - '/historicoTramitacoes/historicoTramitacoes', - '/atasSessao', - '/presencaSessao', - '/resumoPropositurasAutor', - '/propositurasAnoAutorTipo', - ]}, + 'prefixs': [ + '/default_index_html', + '/consultas/parlamentar/parlamentar_', + '/consultas/comissao/comissao_', + '/consultas/pauta_sessao/pauta_sessao_', + '/consultas/sessao_plenaria/', + '/relatorios_administrativos/relatorios_administrativos_index_html', + '/tramitacaoMaterias/tramitacaoMaterias', + '/tramitacaoMaterias/materia_mostrar_proc', + '/generico/materia_pesquisar_', + '/consultas/mesa_diretora/mesa_diretora_index_html', + '/consultas/mesa_diretora/parlamentar/parlamentar_', + '/generico/norma_juridica_pesquisar_', + '/consultas/norma_juridica/norma_juridica_mostrar_proc', + '/historicoTramitacoes/historicoTramitacoes', + '/atasSessao', + '/presencaSessao', + '/resumoPropositurasAutor', + '/propositurasAnoAutorTipo', + ]}, 'lexml': { 'prefixs': [ '/lexml',