diff --git a/parlamentares/views.py b/parlamentares/views.py
index 1a90a8e44..8cf1d4dda 100644
--- a/parlamentares/views.py
+++ b/parlamentares/views.py
@@ -1,6 +1,7 @@
import os
from re import sub
+import sapl
from crispy_forms.helper import FormHelper
from crispy_forms.layout import HTML, ButtonHolder, Fieldset, Layout, Submit
from django import forms
@@ -11,10 +12,8 @@ from django.utils.html import strip_tags
from django.utils.safestring import mark_safe
from django.utils.translation import ugettext_lazy as _
from django.views.generic.edit import FormMixin
-from vanilla import GenericView
-
-import sapl
from sapl.crud import build_crud
+from vanilla import GenericView
from .models import (CargoMesa, Coligacao, ComposicaoMesa, Dependente,
Filiacao, Legislatura, Mandato, NivelInstrucao,
@@ -809,15 +808,15 @@ class FiliacaoView(FormMixin, GenericView):
'legislatura_id': parlamentar.mandato_set.last().legislatura.id})
# Função usada para todos os caso de erro na filiação
- def error_message(self, parlamentar, form, mensagem):
+ def error_message(self, parlamentar, form, mensagem, request):
filiacoes = Filiacao.objects.filter(parlamentar=parlamentar)
+ messages.add_message(request, messages.INFO, mensagem)
return self.render_to_response(
{'parlamentar': parlamentar,
'filiacoes': filiacoes,
'form': form,
'legislatura_id': parlamentar.mandato_set.last(
- ).legislatura.id,
- 'mensagem_erro': mensagem})
+ ).legislatura.id})
def post(self, request, *args, **kwargs):
form = FiliacaoForm(request.POST)
@@ -849,14 +848,16 @@ class FiliacaoView(FormMixin, GenericView):
if candidato_nao_desfiliou:
mensagem = "Você não pode se filiar a algum partido\
sem antes se desfiliar do partido anterior"
- return self.error_message(parlamentar, form, mensagem)
+ return self.error_message(
+ parlamentar, form, mensagem, request)
# Dá erro caso a data de desfiliação seja anterior a de
# filiação
if data_desfiliacao and data_desfiliacao < data_filiacao:
- mensagem = "A data de filiação não pode\
+ mensagem = "A data de filiação não pode ser\
anterior à data de desfiliação"
- return self.error_message(parlamentar, form, mensagem)
+ return self.error_message(
+ parlamentar, form, mensagem, request)
# Esse bloco garante que não haverá intersecção entre os
# períodos de filiação
@@ -868,7 +869,8 @@ class FiliacaoView(FormMixin, GenericView):
mensagem = "A data de filiação e\
desfiliação não podem estar no intervalo\
de outro período de filiação"
- return self.error_message(parlamentar, form, mensagem)
+ return self.error_message(
+ parlamentar, form, mensagem, request)
if (data_desfiliacao and
data_desfiliacao < data_fim and
@@ -877,7 +879,8 @@ class FiliacaoView(FormMixin, GenericView):
mensagem = "A data de filiação e\
desfiliação não podem estar no intervalo\
de outro período de filiação"
- return self.error_message(parlamentar, form, mensagem)
+ return self.error_message(
+ parlamentar, form, mensagem, request)
if (data_desfiliacao and
data_filiacao <= data_init and
@@ -885,7 +888,8 @@ class FiliacaoView(FormMixin, GenericView):
mensagem = "A data de filiação e\
desfiliação não podem estar no intervalo\
de outro período de filiação"
- return self.error_message(parlamentar, form, mensagem)
+ return self.error_message(
+ parlamentar, form, mensagem, request)
# Salva a nova filiação caso tudo esteja correto
else:
@@ -897,7 +901,8 @@ class FiliacaoView(FormMixin, GenericView):
pid = kwargs['pk']
parlamentar = Parlamentar.objects.get(id=pid)
mensagem = ""
- return self.error_message(parlamentar, form, mensagem)
+ return self.error_message(
+ parlamentar, form, mensagem, request)
class FiliacaoEditView(FormMixin, GenericView):
@@ -917,13 +922,13 @@ class FiliacaoEditView(FormMixin, GenericView):
'legislatura_id': parlamentar.mandato_set.last(
).legislatura_id})
- def error_message(self, parlamentar, form, mensagem):
+ def error_message(self, parlamentar, form, mensagem, request):
+ messages.add_message(request, messages.INFO, mensagem)
return self.render_to_response(
{'form': form,
'parlamentar': parlamentar,
'legislatura_id': parlamentar.mandato_set.last(
- ).legislatura_id,
- 'mensagem_erro': mensagem})
+ ).legislatura_id})
def post(self, request, *args, **kwargs):
filiacao = Filiacao.objects.get(id=kwargs['dk'])
@@ -961,14 +966,16 @@ class FiliacaoEditView(FormMixin, GenericView):
if candidato_nao_desfiliou:
mensagem = "Você não pode se filiar a algum partido\
sem antes se desfiliar do partido anterior"
- return self.error_message(parlamentar, form, mensagem)
+ return self.error_message(
+ parlamentar, form, mensagem, request)
# Dá erro caso a data de desfiliação seja anterior a de
# filiação
if data_desfiliacao and data_desfiliacao < data_filiacao:
mensagem = "A data de filiação não pode\
anterior à data de desfiliação"
- return self.error_message(parlamentar, form, mensagem)
+ return self.error_message(
+ parlamentar, form, mensagem, request)
# Esse bloco garante que não haverá intersecção entre os
# períodos de filiação
@@ -986,7 +993,8 @@ class FiliacaoEditView(FormMixin, GenericView):
de outro período de filiação"
return self.error_message(parlamentar,
form,
- mensagem)
+ mensagem,
+ request)
if (data_desfiliacao and
data_desfiliacao < data_fim and
@@ -997,7 +1005,8 @@ class FiliacaoEditView(FormMixin, GenericView):
de outro período de filiação"
return self.error_message(parlamentar,
form,
- mensagem)
+ mensagem,
+ request)
if (data_desfiliacao and
data_filiacao <= data_init and
data_desfiliacao >= data_fim):
@@ -1006,7 +1015,8 @@ class FiliacaoEditView(FormMixin, GenericView):
de outro período de filiação"
return self.error_message(parlamentar,
form,
- mensagem)
+ mensagem,
+ request)
if 'Salvar' in request.POST:
filiacao.save()
diff --git a/templates/parlamentares/parlamentares_filiacao.html b/templates/parlamentares/parlamentares_filiacao.html
index 253470f98..d70ffb46c 100644
--- a/templates/parlamentares/parlamentares_filiacao.html
+++ b/templates/parlamentares/parlamentares_filiacao.html
@@ -24,8 +24,5 @@
{% endfor %}
-
-{% if mensagem_erro %} {{ mensagem_erro }}
{% endif %}
{% crispy form %}
-
{% endblock detail_content %}
diff --git a/templates/parlamentares/parlamentares_filiacao_edit.html b/templates/parlamentares/parlamentares_filiacao_edit.html
index 1c63e6179..6f7121b7d 100644
--- a/templates/parlamentares/parlamentares_filiacao_edit.html
+++ b/templates/parlamentares/parlamentares_filiacao_edit.html
@@ -3,6 +3,5 @@
{% load crispy_forms_tags %}
{% block actions %}{% endblock %}
{% block detail_content %}
-{% if mensagem_erro %} {{ mensagem_erro }}
{% endif %}
{% crispy form %}
{% endblock detail_content %}