From cd13434f60cffb0585b1a9ef55889b67fac94ada Mon Sep 17 00:00:00 2001 From: Eduardo Calil Date: Thu, 17 Mar 2016 20:36:40 -0300 Subject: [PATCH] Arruma namespace em varios lugares --- comissoes/views.py | 6 ++--- materia/test_materia_urls.py | 22 +++++++++++++++++++ materia/views.py | 5 +++-- parlamentares/test_parlamentares_urls.py | 7 ++++++ parlamentares/views.py | 6 ++--- templates/base.html | 2 +- templates/comissoes/comissao_list.html | 4 ++-- templates/comissoes/composicao.html | 4 ++-- .../materia/proposicao/proposicao_list.html | 6 ++--- templates/materia/tramitacao.html | 2 +- .../parlamentares/parlamentares_detail.html | 2 +- .../parlamentares/parlamentares_list.html | 2 +- 12 files changed, 49 insertions(+), 19 deletions(-) create mode 100644 materia/test_materia_urls.py create mode 100644 parlamentares/test_parlamentares_urls.py diff --git a/comissoes/views.py b/comissoes/views.py index 0dd5a4812..afc10fe15 100644 --- a/comissoes/views.py +++ b/comissoes/views.py @@ -21,7 +21,7 @@ ComissaoCrud = Crud.build(Comissao, 'modulo_comissoes') class CadastrarComissaoView(CreateView): template_name = "comissoes/cadastrar_comissao.html" form_class = CadastrarComissaoForm - success_url = reverse_lazy('comissao:list') + success_url = reverse_lazy('comissoes') class ComposicaoView(FormView): @@ -124,7 +124,7 @@ class ComissaoParlamentarIncluirView(FormView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('comissao:composicao', kwargs={'pk': pk}) + return reverse('comissoes:composicao', kwargs={'pk': pk}) class ComissaoParlamentarEditView(FormView): @@ -166,7 +166,7 @@ class ComissaoParlamentarEditView(FormView): def get_success_url(self): pk = self.kwargs['pk'] - return reverse('comissao:composicao', kwargs={'pk': pk}) + return reverse('comissoes:composicao', kwargs={'pk': pk}) class MateriasTramitacaoListView(ListView): diff --git a/materia/test_materia_urls.py b/materia/test_materia_urls.py new file mode 100644 index 000000000..9851788bf --- /dev/null +++ b/materia/test_materia_urls.py @@ -0,0 +1,22 @@ +# import pytest +from django.core.urlresolvers import reverse + + +def test_urls_materia(): + st = reverse('materia:pesquisar_materia_list') + assert st == '/materia/pesquisar-materia-list' + + st = reverse('materia:relatoria_edit', kwargs={'pk': '11', 'id': '99'}) + assert st == '/materia/11/relatoria/99/edit' + + st = reverse('materia:tramitacao_edit', kwargs={'pk': '3', 'id': '8'}) + assert st == '/materia/3/tramitacao/8/edit' + + st = reverse('materia:adicionar_proposicao') + assert st == '/materia/proposicao' + + st = reverse('materia:editar_proposicao', kwargs={'pk': '3'}) + assert st == '/materia/proposicao/3/edit' + + st = reverse('materia:list_proposicao') + assert st == '/materia/proposicao_list' diff --git a/materia/views.py b/materia/views.py index df3e31c0a..97272335e 100644 --- a/materia/views.py +++ b/materia/views.py @@ -1259,7 +1259,8 @@ class MateriaLegislativaPesquisaView(FormView): kwargs['situacao'] = form.data['situacao'] request.session['kwargs'] = kwargs - return redirect('pesquisar_materia_list') + + return redirect('materia:pesquisar_materia_list') class PesquisaMateriaListView(ListView): @@ -1325,7 +1326,7 @@ class ProposicaoView(CreateView): form_class = ProposicaoForm def get_success_url(self): - return reverse('list_proposicao') + return reverse('materia:list_proposicao') def get(self, request, *args, **kwargs): return self.render_to_response({'form': self.get_form()}) diff --git a/parlamentares/test_parlamentares_urls.py b/parlamentares/test_parlamentares_urls.py new file mode 100644 index 000000000..634563aa9 --- /dev/null +++ b/parlamentares/test_parlamentares_urls.py @@ -0,0 +1,7 @@ +# import pytest +from django.core.urlresolvers import reverse + + +def test_urls_materia(): + st = reverse('materia:pesquisar_materia_list') + assert st == '/materia/pesquisar-materia-list' diff --git a/parlamentares/views.py b/parlamentares/views.py index deacb89b2..46ccf6128 100644 --- a/parlamentares/views.py +++ b/parlamentares/views.py @@ -197,7 +197,7 @@ class ParlamentaresEditarView(UpdateView): template_name = "parlamentares/parlamentares_cadastro.html" form_class = ParlamentaresEditForm model = Parlamentar - success_url = reverse_lazy('parlamentares') + success_url = reverse_lazy('parlamentares:parlamentares') def form_valid(self, form): parlamentar = form.instance @@ -344,7 +344,7 @@ class MesaDiretoraView(FormView): id=int(request.POST['cargo'])) composicao.save() - return self.form_valid(form=None) + return redirect('parlamentares:mesa_diretora') elif 'Excluir' in request.POST: @@ -360,7 +360,7 @@ class MesaDiretoraView(FormView): cargo_id=int(ids[1]) ) composicao.delete() - return self.form_valid(form=None) + return redirect('parlamentares:mesa_diretora') else: mesa = ComposicaoMesa.objects.filter( sessao_legislativa=request.POST['sessao']) diff --git a/templates/base.html b/templates/base.html index f629e8bd8..649c63126 100644 --- a/templates/base.html +++ b/templates/base.html @@ -149,7 +149,7 @@ {% block help %} {% if view.help_path %} - {% trans 'Ajuda' %} + {% trans 'Ajuda' %} {% endif %} {% endblock %} diff --git a/templates/comissoes/comissao_list.html b/templates/comissoes/comissao_list.html index a2eefb4ff..b2c5c0197 100644 --- a/templates/comissoes/comissao_list.html +++ b/templates/comissoes/comissao_list.html @@ -4,7 +4,7 @@ {% block base_content %}
- + {% blocktrans with verbose_name=view.verbose_name %} Adicionar Comissão {% endblocktrans %}
@@ -22,7 +22,7 @@ {% for comissao in page_obj %} - {{comissao}} + {{comissao}} {{comissao.sigla}} {{comissao.tipo}} diff --git a/templates/comissoes/composicao.html b/templates/comissoes/composicao.html index ffc69a81d..ff6569780 100644 --- a/templates/comissoes/composicao.html +++ b/templates/comissoes/composicao.html @@ -33,7 +33,7 @@ {% for participacao in participacoes %} {% if participacao.composicao_id == composicao_id %} - {{participacao.parlamentar.nome_parlamentar}} + {{participacao.parlamentar.nome_parlamentar}} {{participacao.cargo}} {{participacao.titular|yesno:"Sim,Não"}} {{participacao.data_designacao|date:"d/m/Y"}} @@ -46,7 +46,7 @@ - Incluir Parlamentar + Incluir Parlamentar {% endif %} {% endblock detail_content %} diff --git a/templates/materia/proposicao/proposicao_list.html b/templates/materia/proposicao/proposicao_list.html index d666badbe..7971890ad 100644 --- a/templates/materia/proposicao/proposicao_list.html +++ b/templates/materia/proposicao/proposicao_list.html @@ -8,7 +8,7 @@

Proposições


- Nova Proposição + Nova Proposição
{% endblock %} @@ -26,9 +26,9 @@ {% for proposicao in page_obj %} {% if proposicao.data_envio %} - {{ proposicao.data_envio|date:'d/m/Y H:i' }} + {{ proposicao.data_envio|date:'d/m/Y H:i' }} {% else %} - Em elaboração... + Em elaboração... {% endif %} {{ proposicao.tipo }} {{ proposicao.descricao|safe }} diff --git a/templates/materia/tramitacao.html b/templates/materia/tramitacao.html index 2576e2342..a879dc5a9 100644 --- a/templates/materia/tramitacao.html +++ b/templates/materia/tramitacao.html @@ -20,7 +20,7 @@ {% for t in tramitacoes %} - {{t.data_tramitacao|date:'d/m/Y'}} + {{t.data_tramitacao|date:'d/m/Y'}} {{t.unidade_tramitacao_local}} {{t.unidade_tramitacao_destino}} {{t.status.descricao}} diff --git a/templates/parlamentares/parlamentares_detail.html b/templates/parlamentares/parlamentares_detail.html index 1cec6e5d7..3eed3c3dd 100644 --- a/templates/parlamentares/parlamentares_detail.html +++ b/templates/parlamentares/parlamentares_detail.html @@ -3,7 +3,7 @@ {% block actions %}
{% if legislatura_id != 0 %} - {% trans 'Incluir Parlamentar' %} + {% trans 'Incluir Parlamentar' %} {% endif %}
{% endblock actions %} diff --git a/templates/parlamentares/parlamentares_list.html b/templates/parlamentares/parlamentares_list.html index ab9f40abf..3d00cce02 100644 --- a/templates/parlamentares/parlamentares_list.html +++ b/templates/parlamentares/parlamentares_list.html @@ -30,7 +30,7 @@ {% for p in parlamentares %} - {{p.nome}} + {{p.nome}} {{p.partido}} {{p.ativo|yesno:"Sim,Não"}}