From 1c55f28cb7cfa16e89a2425f4f96c3ef07cc7244 Mon Sep 17 00:00:00 2001 From: Marcio Mazza Date: Fri, 4 Mar 2016 11:21:03 -0300 Subject: [PATCH] Ajusta pep8 e outros detalhes --- comissoes/forms.py | 1 + compilacao/templatetags/compilacao_filters.py | 1 + compilacao/utils.py | 2 +- lexml/views.py | 2 -- materia/views.py | 32 ++++++++++--------- parlamentares/views.py | 3 -- sapl/context_processors.py | 10 +++--- scripts/redbaron.py | 14 ++++---- sessao/views.py | 23 +++++++------ 9 files changed, 46 insertions(+), 42 deletions(-) diff --git a/comissoes/forms.py b/comissoes/forms.py index feb4b1fd3..55ca1d2d9 100644 --- a/comissoes/forms.py +++ b/comissoes/forms.py @@ -2,6 +2,7 @@ from crispy_forms.helper import FormHelper from crispy_forms.layout import Fieldset, Layout from django import forms from django.forms import ModelForm +from django.utils.translation import ugettext_lazy as _ import crispy_layout_mixin from crispy_layout_mixin import form_actions diff --git a/compilacao/templatetags/compilacao_filters.py b/compilacao/templatetags/compilacao_filters.py index f5be36dac..ac2d103dc 100644 --- a/compilacao/templatetags/compilacao_filters.py +++ b/compilacao/templatetags/compilacao_filters.py @@ -1,6 +1,7 @@ from django import template from django.core.signing import Signer from django.db.models import Q +from django.utils.translation import ugettext_lazy as _ from compilacao.models import Dispositivo, TipoDispositivo diff --git a/compilacao/utils.py b/compilacao/utils.py index 022d1389e..ca00f66b4 100644 --- a/compilacao/utils.py +++ b/compilacao/utils.py @@ -4,7 +4,7 @@ def int_to_roman(int_value): # if isinstance(int_value, type(1)): # raise TypeError("expected integer, got %s" % type(int_value)) if not 0 < int_value < 4000: - raise ValueError(_("Argument must be between 1 and 3999")) + raise ValueError("Argument must be between 1 and 3999") ints = (1000, 900, 500, 400, 100, 90, 50, 40, 10, 9, 5, 4, 1) nums = ('M', 'CM', 'D', 'CD', 'C', 'XC', 'L', 'XL', 'X', 'IX', 'V', 'IV', 'I') diff --git a/lexml/views.py b/lexml/views.py index d8daec7a2..d6957d1ef 100644 --- a/lexml/views.py +++ b/lexml/views.py @@ -1,5 +1,3 @@ -from django.utils.translation import ugettext_lazy as _ - from crud import Crud from .models import LexmlProvedor, LexmlPublicador diff --git a/materia/views.py b/materia/views.py index de30173ee..83e5fd9ae 100644 --- a/materia/views.py +++ b/materia/views.py @@ -735,7 +735,8 @@ class RelatoriaView(FormMixin, GenericView): materia = MateriaLegislativa.objects.get(id=kwargs['pk']) if not materia.tramitacao_set.all(): - msg = _('Adicione alguma Tramitação antes de adicionar uma Comissão!') + msg = _( + 'Adicione alguma Tramitação antes de adicionar uma Comissão!') messages.add_message(request, messages.INFO, msg) return self.render_to_response( {'object': materia, @@ -790,7 +791,8 @@ class RelatoriaView(FormMixin, GenericView): materia=materia).last() if not materia.tramitacao_set.all(): - msg = _('Adicione alguma Tramitação antes de adicionar uma Comissão!') + msg = _( + 'Adicione alguma Tramitação antes de adicionar uma Comissão!') messages.add_message(request, messages.INFO, msg) return self.render_to_response( {'object': materia, @@ -906,15 +908,15 @@ def criar_email_tramitacao(request, casa_legislativa, materia, hash_txt=''): 'email/tramitacao.html'], {"casa_legislativa": casa_nome, "data_registro": datetime.now().strftime( - "%d/%m/%Y"), + "%d/%m/%Y"), "cod_materia": materia.id, "logotipo": casa_legislativa.logotipo, "descricao_materia": materia.ementa, "autoria": autores, "data": materia.tramitacao_set.last( - ).data_tramitacao, + ).data_tramitacao, "status": materia.tramitacao_set.last( - ).status, + ).status, "texto_acao": materia.tramitacao_set.last().texto, "hash_txt": hash_txt, @@ -985,11 +987,11 @@ def do_envia_email_confirmacao(request, materia, email): destinatario.hash,) recipients.append(destinatario.email) messages.append({ - 'recipient': destinatario.email, - 'subject': subject, - 'txt_message': email_texts[0], - 'html_message': email_texts[1] - }) + 'recipient': destinatario.email, + 'subject': subject, + 'txt_message': email_texts[0], + 'html_message': email_texts[1] + }) enviar_emails(sender, recipients, messages) return None @@ -1016,11 +1018,11 @@ def do_envia_email_tramitacao(request, materia): destinatario.hash,) recipients.append(destinatario.email) messages.append({ - 'recipient': destinatario.email, - 'subject': subject, - 'txt_message': email_texts[0], - 'html_message': email_texts[1] - }) + 'recipient': destinatario.email, + 'subject': subject, + 'txt_message': email_texts[0], + 'html_message': email_texts[1] + }) enviar_emails(sender, recipients, messages) return None diff --git a/parlamentares/views.py b/parlamentares/views.py index 8eeb9e474..af25f05d4 100644 --- a/parlamentares/views.py +++ b/parlamentares/views.py @@ -1,11 +1,8 @@ import os -from io import BytesIO from re import sub from django.contrib import messages -from django.core.exceptions import ValidationError from django.core.urlresolvers import reverse -from django.forms.fields import ImageField from django.utils.html import strip_tags from django.utils.translation import ugettext_lazy as _ from django.views.generic.edit import FormMixin diff --git a/sapl/context_processors.py b/sapl/context_processors.py index ee0fe992d..d21d67d50 100644 --- a/sapl/context_processors.py +++ b/sapl/context_processors.py @@ -5,11 +5,11 @@ from base.views import get_casalegislativa def parliament_info(request): context = { - 'state': _('Estado'), - 'state_abbr': _('UF'), - 'city': _('Cidade'), - 'parliament_type': _('Câmara Municipal'), - 'address': _('Rua Lorem Ipsum de Amet, Casa X'), + 'state': 'Estado', + 'state_abbr': 'UF', + 'city': 'Cidade', + 'parliament_type': 'Câmara Municipal', + 'address': 'Rua Lorem Ipsum de Amet, Casa X', 'postal_code': '70000-000', 'phone_number': '00 0000-0000', 'url_portal': '#', diff --git a/scripts/redbaron.py b/scripts/redbaron.py index ba6f8c14d..250e92c14 100644 --- a/scripts/redbaron.py +++ b/scripts/redbaron.py @@ -63,15 +63,15 @@ def frase(n): return ' ' in n.value -def get(r): - return [s for s in r('string') - if not inter(s) - # and frase(s) - and not s.value in ok] +def mark(n): + mark.ok |= {n.value} +mark.ok = set() -def mark(n): - ok |= {n.value} +def get(r): + return [s for s in r('string') + if not inter(s) and s.value not in mark.ok] + # and frase(s) def fix(n): diff --git a/sessao/views.py b/sessao/views.py index 8d3051ff9..86f81b182 100644 --- a/sessao/views.py +++ b/sessao/views.py @@ -661,7 +661,8 @@ class EditExpedienteOrdemDiaView(sessao_crud.CrudDetailView): return self.form_valid(form) else: context.update( - {'error_message': _("Não foi possível salvar formulário!")}) + {'error_message': _( + "Não foi possível salvar formulário!")}) return self.form_invalid(form) elif 'delete-ordemdia' in request.POST: ordemdia.delete() @@ -934,10 +935,12 @@ class ResumoView(sessao_crud.CrudDetailView): abertura = self.object.data_inicio.strftime('%d/%m/%Y') encerramento = self.object.data_fim.strftime('%d/%m/%Y') - context.update({'basica': [_('Tipo de Sessão: ') + str(self.object.tipo), - _('Abertura: ') + abertura, - _('Encerramento: ') + encerramento, - ]}) + context.update({'basica': [ + _('Tipo de Sessão: %(tipo)s') % {'tipo': self.object.tipo}, + _('Abertura: %(abertura)') % {'abertura': abertura}, + _('Encerramento: %(encerramento)') % { + 'encerramento': encerramento}, + ]}) # ===================================================================== # Conteúdo Multimídia if self.object.url_audio: @@ -2142,10 +2145,12 @@ class PautaSessaoDetailView(sessao_crud.CrudDetailView): else: encerramento = "" - context.update({'basica': [_('Tipo de Sessão: ') + str(self.object.tipo), - _('Abertura: ') + abertura, - _('Encerramento: ') + encerramento, - ]}) + context.update({'basica': [ + _('Tipo de Sessão: %(tipo)s') % {'tipo': self.object.tipo}, + _('Abertura: %(abertura)') % {'abertura': abertura}, + _('Encerramento: %(encerramento)') % { + 'encerramento': encerramento}, + ]}) # ===================================================================== # Matérias Expediente materias = ExpedienteMateria.objects.filter(