From 949064b976d231f1ad5ff47bcccfb9fe3db885a5 Mon Sep 17 00:00:00 2001 From: LeandroRoberto Date: Thu, 15 Feb 2018 08:32:12 -0200 Subject: [PATCH] fixa tamanho de objeto de listagem passado para o template MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - as colunas passadas para a listagem de parlamentares são constituidas de tuplas (value, href, obj). Essa tupla é relevante para a primeira coluna, sendo que para as seguintes apenas os dois primeiros itens eram passados. No django<1.10 essa diferença de tamanhos não gerava erros no "for" do template, porém gerará no django>=1.11. - um pequena correção na tag de construção de menus --- sapl/base/templatetags/menus.py | 7 ++++--- sapl/parlamentares/views.py | 11 +++++++---- sapl/templates/parlamentares/parlamentares_list.html | 2 +- 3 files changed, 12 insertions(+), 8 deletions(-) diff --git a/sapl/base/templatetags/menus.py b/sapl/base/templatetags/menus.py index fdfbbf57a..34294fbd8 100644 --- a/sapl/base/templatetags/menus.py +++ b/sapl/base/templatetags/menus.py @@ -1,10 +1,11 @@ -import yaml - from django import template from django.core.urlresolvers import reverse from django.utils.translation import ugettext_lazy as _ +import yaml + from sapl.utils import sapl_logger + register = template.Library() @@ -76,7 +77,7 @@ def nav_run(context, path=None): return try: - rendered = yaml_template.render(context, request) + rendered = yaml_template.template.render(context) menu = yaml.load(rendered) resolve_urls_inplace(menu, root_pk, rm, context) except Exception as e: diff --git a/sapl/parlamentares/views.py b/sapl/parlamentares/views.py index dda454de7..7654dde6c 100644 --- a/sapl/parlamentares/views.py +++ b/sapl/parlamentares/views.py @@ -1,5 +1,5 @@ -import json from datetime import datetime +import json from django.contrib import messages from django.contrib.contenttypes.models import ContentType @@ -16,6 +16,7 @@ from django.utils.translation import ugettext_lazy as _ from django.views.decorators.clickjacking import xframe_options_exempt from django.views.generic import FormView from django.views.generic.edit import UpdateView + from sapl.base.models import Autor from sapl.comissoes.models import Participacao from sapl.crud.base import (RP_CHANGE, RP_DETAIL, RP_LIST, Crud, CrudAux, @@ -32,6 +33,7 @@ from .models import (CargoMesa, Coligacao, ComposicaoColigacao, ComposicaoMesa, NivelInstrucao, Parlamentar, Partido, SessaoLegislativa, SituacaoMilitar, TipoAfastamento, TipoDependente, Votante) + CargoMesaCrud = CrudAux.build(CargoMesa, 'cargo_mesa') PartidoCrud = CrudAux.build(Partido, 'partidos') SessaoLegislativaCrud = CrudAux.build(SessaoLegislativa, 'sessao_legislativa') @@ -496,7 +498,8 @@ class ParlamentarCrud(Crud): parlamentar = Parlamentar.objects.get( id=(row[0][1].split('/')[-1])) - row[0] += (parlamentar, ) + for index, value in enumerate(row): + row[index] += (None if index else parlamentar,) # Pega a Legislatura legislatura = Legislatura.objects.get( @@ -516,7 +519,7 @@ class ParlamentarCrud(Crud): # Caso não exista filiação com essas condições except ObjectDoesNotExist: - row[1] = ('Não possui filiação', None) + row[1] = ('Não possui filiação', None, None) # Caso exista mais de uma filiação nesse intervalo # Entretanto, NÃO DEVE OCORRER @@ -527,7 +530,7 @@ class ParlamentarCrud(Crud): # Caso encontre UMA filiação nessas condições else: - row[1] = (filiacao.partido.sigla, None) + row[1] = (filiacao.partido.sigla, None, None) return context diff --git a/sapl/templates/parlamentares/parlamentares_list.html b/sapl/templates/parlamentares/parlamentares_list.html index 0afaceae4..bc3955b06 100644 --- a/sapl/templates/parlamentares/parlamentares_list.html +++ b/sapl/templates/parlamentares/parlamentares_list.html @@ -42,7 +42,7 @@ {% for value, href, obj in value_list %} {% if forloop.first %} - + {% endif %}