From 2e552dd8a9e293ea28c08e07d1dd5995af35e399 Mon Sep 17 00:00:00 2001 From: Eduardo Calil Date: Fri, 11 Aug 2017 15:44:13 -0300 Subject: [PATCH] =?UTF-8?q?Resolve=20detalhes=20ap=C3=B3s=20code=20review?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- sapl/base/views.py | 7 ++++--- sapl/utils.py | 10 +--------- 2 files changed, 5 insertions(+), 12 deletions(-) diff --git a/sapl/base/views.py b/sapl/base/views.py index 336a90af2..1450bc16b 100644 --- a/sapl/base/views.py +++ b/sapl/base/views.py @@ -230,7 +230,8 @@ class RelatorioPresencaSessaoView(FilterView): # Parlamentares com Mandato no intervalo de tempo (Ativos) parlamentares_qs = parlamentares_ativos( _range[0], _range[1]).order_by('nome_parlamentar') - parlamentares_id = [p.id for p in parlamentares_qs] + parlamentares_id = parlamentares_qs.values_list( + 'id', flat=True) # Presenças de cada Parlamentar em Sessões presenca_sessao = SessaoPlenariaPresenca.objects.filter( @@ -277,11 +278,11 @@ class RelatorioPresencaSessaoView(FilterView): if total_sessao != 0: parlamentares_presencas[i].update( {'sessao_porc': round( - sessao_count * 100 / total_sessao, 1)}) + sessao_count * 100 / total_sessao, 2)}) if total_ordemdia != 0: parlamentares_presencas[i].update( {'ordemdia_porc': round( - ordemdia_count * 100 / total_ordemdia, 1)}) + ordemdia_count * 100 / total_ordemdia, 2)}) context['date_range'] = _range context['total_ordemdia'] = total_ordemdia diff --git a/sapl/utils.py b/sapl/utils.py index e32eec329..f707a2c91 100644 --- a/sapl/utils.py +++ b/sapl/utils.py @@ -615,15 +615,7 @@ def filiacao_data(parlamentar, data_inicio, data_fim=None): data__gte=data_inicio, data__lte=data_fim) - siglas = '' - - for f in filiacoes: - if not siglas: - siglas = f.partido.sigla - else: - siglas = siglas + ' | ' + f.partido.sigla - - return siglas + return ' | '.join([f.partido.sigla for f in filiacoes]) def parlamentares_ativos(data_inicio, data_fim=None):