Browse Source

Remove unnecessary legislatura id from parlamentares views

pull/202/head
Marcio Mazza 9 years ago
parent
commit
ac14ab3985
  1. 2
      parlamentares/urls.py
  2. 11
      parlamentares/views.py
  3. 2
      templates/parlamentares/parlamentares_detail.html
  4. 8
      templates/parlamentares/parlamentares_list.html

2
parlamentares/urls.py

@ -54,7 +54,7 @@ urlpatterns = [
MandatoEditView.as_view(),
name='parlamentares_mandato_edit'),
url(r'^parlamentares/(?P<pk>\d+)/(?P<pid>\d+)/editar$',
url(r'^parlamentares/(?P<pk>\d+)/editar$',
ParlamentaresEditarView.as_view(), name='parlamentares_editar'),
url(r'^mesa-diretora/$',

11
parlamentares/views.py

@ -447,16 +447,14 @@ class ParlamentaresEditarView(FormMixin, GenericView):
def get(self, request, *args, **kwargs):
pk = kwargs['pk']
pid = kwargs['pid']
parlamentar = Parlamentar.objects.get(id=pid)
parlamentar = Parlamentar.objects.get(pk=pk)
form = ParlamentaresEditForm(instance=parlamentar)
return self.render_to_response(
{'form': form, 'legislatura_id': pk, 'parlamentar': parlamentar})
{'form': form, 'parlamentar': parlamentar})
def post(self, request, *args, **kwargs):
pk = kwargs['pk']
pid = kwargs['pid']
parlamentar = Parlamentar.objects.get(id=pid)
parlamentar = Parlamentar.objects.get(pk=pk)
form = ParlamentaresEditForm(request.POST, instance=parlamentar)
if form.is_valid():
@ -482,8 +480,7 @@ class ParlamentaresEditarView(FormMixin, GenericView):
return self.form_valid(form)
else:
return self.render_to_response(
{'form': form, 'legislatura_id': pk})
return self.render_to_response({'form': form})
class DependenteForm(ModelForm):

2
templates/parlamentares/parlamentares_detail.html

@ -10,7 +10,7 @@
{% block sections_nav %}
<dl class="sub-nav left">
{% if parlamentar %}
<dd><a href="{% url 'parlamentares_editar' legislatura_id parlamentar.id %}" id="incluir-parlamentar" class="button secondary">{% trans 'Inicio' %}</a></dd>
<dd><a href="{% url 'parlamentares_editar' parlamentar.id %}" id="incluir-parlamentar" class="button secondary">{% trans 'Inicio' %}</a></dd>
<dd><a href="{% url 'parlamentares_mandato' parlamentar.id %}" id="incluir-parlamentar" class="button secondary">{% trans 'Mandatos' %}</a></dd>
<dd><a href="{% url 'parlamentares_filiacao' parlamentar.id %}" id="incluir-parlamentar" class="button secondary">{% trans 'Filiações Partidárias' %}</a></dd>
<dd><a href="{% url 'parlamentares_dependentes' parlamentar.id %}" id="incluir-parlamentar" class="button secondary">{% trans 'Dependentes' %}</a></dd>

8
templates/parlamentares/parlamentares_list.html

@ -19,7 +19,7 @@
<fieldset>
<legend>Parlamentares</legend>
<table>
<tr>
<tr>
<th>Nome do Parlamentar</th>
<th>Partido</th>
<th>Ativo?</th>
@ -27,10 +27,10 @@
{% for p in parlamentares %}
<tr>
<td><a href="{% url 'parlamentares_editar' legislatura_id p.id %}">
<td><a href="{% url 'parlamentares_editar' p.id %}">
{{p.nome}}
</a></td>
<td>{{p.partido}}</td>
</a></td>
<td>{{p.partido}}</td>
<td>{{p.ativo|yesno:"Sim,Não"}}</td>
</tr>
{% endfor %}

Loading…
Cancel
Save