From da107fb730078d8aaa16441b99524391dce24c2c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Rodrigues?= Date: Mon, 17 Jun 2019 14:56:05 -0300 Subject: [PATCH] Fix #2836 (#2839) * Fix #2836 * Fix #2836 * Fix #2836 --- sapl/sessao/forms.py | 12 +- sapl/sessao/views.py | 160 ++++++++++++++---- sapl/templates/sessao/votacao/votacao.html | 14 +- .../votacao/votacao_simbolica_bloco.html | 7 +- 4 files changed, 146 insertions(+), 47 deletions(-) diff --git a/sapl/sessao/forms.py b/sapl/sessao/forms.py index 523e1cede..9b3fc1f90 100644 --- a/sapl/sessao/forms.py +++ b/sapl/sessao/forms.py @@ -464,6 +464,9 @@ class VotacaoForm(forms.Form): abstencoes = forms.IntegerField(label='Abstenções') total_presentes = forms.IntegerField( required=False, widget=forms.HiddenInput()) + total_votantes = forms.IntegerField( + required=False, widget=forms.HiddenInput() + ) voto_presidente = forms.IntegerField( label='A totalização inclui o voto do Presidente?') total_votos = forms.IntegerField(required=False, label='total') @@ -479,15 +482,16 @@ class VotacaoForm(forms.Form): votos_nao = cleaned_data['votos_nao'] abstencoes = cleaned_data['abstencoes'] qtde_presentes = cleaned_data['total_presentes'] + qtde_votantes = cleaned_data['total_votantes'] qtde_votos = votos_sim + votos_nao + abstencoes voto_presidente = cleaned_data['voto_presidente'] - if qtde_presentes and not voto_presidente: - qtde_presentes -= 1 + if qtde_votantes and not voto_presidente: + qtde_votantes -= 1 - if qtde_presentes and qtde_votos != qtde_presentes: + if qtde_votantes and qtde_votos != qtde_votantes: raise ValidationError( - 'O total de votos não corresponde com a quantidade de presentes!') + 'O total de votos não corresponde com a quantidade de votantes!') return cleaned_data diff --git a/sapl/sessao/views.py b/sapl/sessao/views.py index 0c4d3a029..295f22543 100755 --- a/sapl/sessao/views.py +++ b/sapl/sessao/views.py @@ -2084,13 +2084,24 @@ class VotacaoView(SessaoPermissionMixin): ordem_id = kwargs['oid'] ordem = OrdemDia.objects.get(id=ordem_id) - qtde_presentes = PresencaOrdemDia.objects.filter( - sessao_plenaria_id=self.object.id).count() + + presentes_id = [ + presente.parlamentar.id for presente in PresencaOrdemDia.objects.filter( + sessao_plenaria_id=self.kwargs['pk'] + ) + ] + qtde_presentes = len(presentes_id) + + presenca_ativos = Parlamentar.objects.filter( + id__in=presentes_id, ativo=True + ) + qtde_ativos = len(presenca_ativos) materia = {'materia': ordem.materia, 'ementa': ordem.materia.ementa} context.update({'votacao_titulo': titulo, 'materia': materia, - 'total_presentes': qtde_presentes}) + 'total_presentes': qtde_presentes, + 'total_votantes': qtde_ativos}) return self.render_to_response(context) @@ -2110,13 +2121,24 @@ class VotacaoView(SessaoPermissionMixin): ordem_id = kwargs['oid'] ordem = OrdemDia.objects.get(id=ordem_id) - qtde_presentes = PresencaOrdemDia.objects.filter( - sessao_plenaria_id=self.object.id).count() + + presentes_id = [ + presente.parlamentar.id for presente in PresencaOrdemDia.objects.filter( + sessao_plenaria_id=self.kwargs['pk'] + ) + ] + qtde_presentes = len(presentes_id) + + presenca_ativos = Parlamentar.objects.filter( + id__in=presentes_id, ativo=True + ) + qtde_ativos = len(presenca_ativos) materia = {'materia': ordem.materia, 'ementa': ordem.materia.ementa} context.update({'votacao_titulo': titulo, 'materia': materia, - 'total_presentes': qtde_presentes}) + 'total_presentes': qtde_presentes, + 'total_votantes': qtde_ativos}) context.update({'form': form}) # ==================================================== @@ -2129,21 +2151,19 @@ class VotacaoView(SessaoPermissionMixin): materia_id = kwargs['mid'] ordem_id = kwargs['oid'] - qtde_presentes = PresencaOrdemDia.objects.filter( - sessao_plenaria_id=self.object.id).count() qtde_votos = (int(request.POST['votos_sim']) + int(request.POST['votos_nao']) + int(request.POST['abstencoes'])) if (int(request.POST['voto_presidente']) == 0): - qtde_presentes -= 1 + qtde_ativos -= 1 - if (qtde_votos > qtde_presentes or qtde_votos < qtde_presentes): + if qtde_votos != qtde_ativos: msg = _( - 'O total de votos não corresponde com a quantidade de presentes!') + 'O total de votos não corresponde com a quantidade de votantes!') messages.add_message(request, messages.ERROR, msg) return self.render_to_response(context) - elif (qtde_presentes == qtde_votos): + else: try: votacao = RegistroVotacao() votacao.numero_votos_sim = int(request.POST['votos_sim']) @@ -2788,14 +2808,25 @@ class VotacaoExpedienteView(SessaoPermissionMixin): expediente_id = kwargs['oid'] expediente = ExpedienteMateria.objects.get(id=expediente_id) - qtde_presentes = SessaoPlenariaPresenca.objects.filter( - sessao_plenaria_id=self.object.id).count() + + presentes_id = [ + presente.parlamentar.id for presente in SessaoPlenariaPresenca.objects.filter( + sessao_plenaria_id=self.kwargs['pk'] + ) + ] + qtde_presentes = len(presentes_id) + + presentes_ativos = Parlamentar.objects.filter( + id__in=presentes_id, ativo=True + ) + qtde_ativos = len(presentes_ativos) materia = {'materia': expediente.materia, 'ementa': expediente.materia.ementa} context.update({'votacao_titulo': titulo, 'materia': materia, - 'total_presentes': qtde_presentes}) + 'total_presentes': qtde_presentes, + 'total_votantes': qtde_ativos}) return self.render_to_response(context) @@ -2815,14 +2846,25 @@ class VotacaoExpedienteView(SessaoPermissionMixin): expediente_id = kwargs['oid'] expediente = ExpedienteMateria.objects.get(id=expediente_id) - qtde_presentes = SessaoPlenariaPresenca.objects.filter( - sessao_plenaria_id=self.object.id).count() + + presentes_id = [ + presente.parlamentar.id for presente in SessaoPlenariaPresenca.objects.filter( + sessao_plenaria_id=self.kwargs['pk'] + ) + ] + qtde_presentes = len(presentes_id) + + presentes_ativos = Parlamentar.objects.filter( + id__in=presentes_id, ativo=True + ) + qtde_ativos = len(presentes_ativos) materia = {'materia': expediente.materia, 'ementa': expediente.materia.ementa} context.update({'votacao_titulo': titulo, 'materia': materia, - 'total_presentes': qtde_presentes}) + 'total_presentes': qtde_presentes, + 'total_votantes': qtde_ativos}) context.update({'form': form}) # ==================================================== @@ -2835,17 +2877,17 @@ class VotacaoExpedienteView(SessaoPermissionMixin): materia_id = kwargs['mid'] expediente_id = kwargs['oid'] - qtde_presentes = SessaoPlenariaPresenca.objects.filter( - sessao_plenaria_id=self.object.id).count() qtde_votos = (int(request.POST['votos_sim']) + int(request.POST['votos_nao']) + int(request.POST['abstencoes'])) if (int(request.POST['voto_presidente']) == 0): - qtde_presentes -= 1 + qtde_ativos -= 1 - if qtde_votos != qtde_presentes: - form._errors["total_votos"] = ErrorList([u""]) + if qtde_votos != qtde_ativos: + msg = _( + 'O total de votos não corresponde com a quantidade de votantes!') + messages.add_message(request, messages.ERROR, msg) return self.render_to_response(context) else: try: @@ -3594,17 +3636,41 @@ class VotacaoEmBlocoSimbolicaView(PermissionRequiredForAppCrudMixin, TemplateVie if request.POST['origem'] == 'ordem': ordens = OrdemDia.objects.filter( id__in=request.POST.getlist('marcadas_1')) - qtde_presentes = PresencaOrdemDia.objects.filter( - sessao_plenaria_id=self.kwargs['pk']).count() + + presentes_id = [ + presente.parlamentar.id for presente in PresencaOrdemDia.objects.filter( + sessao_plenaria_id=self.kwargs['pk'] + ) + ] + qtde_presentes = len(presentes_id) + + presenca_ativos = Parlamentar.objects.filter( + id__in=presentes_id, ativo=True + ) + qtde_ativos = len(presenca_ativos) + context.update({'ordens': ordens, - 'total_presentes': qtde_presentes}) + 'total_presentes': qtde_presentes, + 'total_votantes': qtde_ativos}) else: expedientes = ExpedienteMateria.objects.filter( id__in=request.POST.getlist('marcadas_1')) - qtde_presentes = SessaoPlenariaPresenca.objects.filter( - sessao_plenaria_id=self.kwargs['pk']).count() + + presentes_id = [ + presente.parlamentar.id for presente in SessaoPlenariaPresenca.objects.filter( + sessao_plenaria_id=self.kwargs['pk'] + ) + ] + qtde_presentes = len(presentes_id) + + presenca_ativos = Parlamentar.objects.filter( + id__in=presentes_id, ativo=True + ) + qtde_ativos = len(presenca_ativos) + context.update({'expedientes': expedientes, - 'total_presentes': qtde_presentes}) + 'total_presentes': qtde_presentes, + 'total_votantes': qtde_ativos}) if 'salvar-votacao' in request.POST: form = VotacaoForm(request.POST) @@ -3729,17 +3795,41 @@ class VotacaoEmBlocoSimbolicaView(PermissionRequiredForAppCrudMixin, TemplateVie if self.request.POST['origem'] == 'ordem': ordens = OrdemDia.objects.filter( id__in=self.request.POST.getlist('ordens')) - qtde_presentes = PresencaOrdemDia.objects.filter( - sessao_plenaria_id=self.kwargs['pk']).count() + + presentes_id = [ + presente.parlamentar.id for presente in PresencaOrdemDia.objects.filter( + sessao_plenaria_id=self.kwargs['pk'] + ) + ] + qtde_presentes = len(presentes_id) + + presenca_ativos = Parlamentar.objects.filter( + id__in=presentes_id, ativo=True + ) + qtde_ativos = len(presenca_ativos) + context.update({'ordens': ordens, - 'total_presentes': qtde_presentes}) + 'total_presentes': qtde_presentes, + 'total_votantes': qtde_ativos}) elif self.request.POST['origem'] == 'expediente': expedientes = ExpedienteMateria.objects.filter( id__in=self.request.POST.getlist('expedientes')) - qtde_presentes = SessaoPlenariaPresenca.objects.filter( - sessao_plenaria_id=self.kwargs['pk']).count() + + presentes_id = [ + presente.parlamentar.id for presente in SessaoPlenariaPresenca.objects.filter( + sessao_plenaria_id=self.kwargs['pk'] + ) + ] + qtde_presentes = len(presentes_id) + + presenca_ativos = Parlamentar.objects.filter( + id__in=presentes_id, ativo=True + ) + qtde_ativos = len(presenca_ativos) + context.update({'expedientes': expedientes, - 'total_presentes': qtde_presentes}) + 'total_presentes': qtde_presentes, + 'total_votantes': qtde_ativos}) context.update({'resultado_votacao': TipoResultadoVotacao.objects.all(), 'form': form, diff --git a/sapl/templates/sessao/votacao/votacao.html b/sapl/templates/sessao/votacao/votacao.html index 958c8b9a6..d7320c5b1 100644 --- a/sapl/templates/sessao/votacao/votacao.html +++ b/sapl/templates/sessao/votacao/votacao.html @@ -15,7 +15,7 @@ {% for field in form %} {% if field.errors %} {% if field.label == 'total'%} -
  • O total de votos não corresponde com a quantidade de presentes!
    • +
  • O total de votos não corresponde com a quantidade de votantes!
    • {% else %}
  • O campo {{field.label}} é obrigatório!
    • {% endif %} @@ -32,19 +32,21 @@

    Total presentes: {{total_presentes}} (com presidente) +
    + Total votantes: {{total_votantes}} (com presidente)
    -
    Sim:
    -
    Não:
    -
    Abstenções:
    +
    Sim*:
    +
    Não*:
    +
    Abstenções*:
    - A totalização inclui o voto do Presidente? + A totalização inclui o voto do Presidente?* {% for tipo in view.get_tipos_votacao %} diff --git a/sapl/templates/sessao/votacao/votacao_simbolica_bloco.html b/sapl/templates/sessao/votacao/votacao_simbolica_bloco.html index b3cfb9a62..d53a4c4d8 100644 --- a/sapl/templates/sessao/votacao/votacao_simbolica_bloco.html +++ b/sapl/templates/sessao/votacao/votacao_simbolica_bloco.html @@ -8,9 +8,9 @@
    Votação Simbólica {% if ordens or expedientes %} - {% if total_presentes == 0 %} + {% if total_votantes == 0 %} {% if origem == 'ordem' %} Voltar @@ -36,7 +36,10 @@ {% endfor %} {% endif %} Total presentes: {{total_presentes}} (com presidente) +
    + Total votantes: {{total_votantes}} (com presidente) +