diff --git a/sapl/base/forms.py b/sapl/base/forms.py index 7dae693bd..9e7103c2a 100644 --- a/sapl/base/forms.py +++ b/sapl/base/forms.py @@ -1,9 +1,9 @@ import logging import os -from crispy_forms.bootstrap import FieldWithButtons, InlineRadios, StrictButton +from crispy_forms.bootstrap import FieldWithButtons, InlineRadios, StrictButton, FormActions from sapl.crispy_layout_mixin import SaplFormHelper -from crispy_forms.layout import HTML, Button, Div, Field, Fieldset, Layout, Row +from crispy_forms.layout import HTML, Button, Div, Field, Fieldset, Layout, Row, Submit from django import forms from django.conf import settings from django.contrib.auth import get_user_model @@ -1464,10 +1464,18 @@ class PartidoUpdateForm(PartidoForm): row3 = to_row([('observacao', 12)]) row4 = to_row([('logo_partido', 12)]) + buttons = FormActions( + *[ + HTML('''%s''' % _('Cancelar')) + ], + Submit('salvar', _('Salvar'), css_class='float-right', + onclick='return true;'), + css_class='form-group row justify-content-between' + ) + self.helper = SaplFormHelper() self.helper.layout = Layout( - row1, row2, row3, row4, - form_actions(label='Salvar') + row1, row2, row3, row4, to_row([(buttons, 12)]), ) def clean(self): @@ -1479,7 +1487,7 @@ class PartidoUpdateForm(PartidoForm): is_historico = cleaned_data['historico'] == 'True' if is_historico: - if not cleaned_data['data_criacao'] or not cleaned_data['data_extincao']: + if not (cleaned_data['data_criacao'] and cleaned_data['data_extincao']): raise ValidationError("Certifique-se de que a data de inicio e fim de historico estão preenchidas") if self.instance.data_criacao and self.instance.data_criacao > cleaned_data['data_criacao']: raise ValidationError("Data de inicio de historico deve ser posterior a data de criação do partido.") diff --git a/sapl/parlamentares/models.py b/sapl/parlamentares/models.py index ba036bc8f..e897129d5 100644 --- a/sapl/parlamentares/models.py +++ b/sapl/parlamentares/models.py @@ -442,7 +442,7 @@ class Filiacao(models.Model): # parlamentares e tela de Filiações do Parlamentar ordering = ('parlamentar', '-data', '-data_desfiliacao') - def get_nome_partido_ano(self, ano): + def nome_partido_ano(self, ano): historico = HistoricoPartido.objects.filter(partido=self.partido) for h in historico: if h.inicio_historico.year <= ano <= h.fim_historico.year: diff --git a/sapl/parlamentares/views.py b/sapl/parlamentares/views.py index 325ba61d2..42bac2e3f 100644 --- a/sapl/parlamentares/views.py +++ b/sapl/parlamentares/views.py @@ -1078,7 +1078,7 @@ def partido_parlamentar_sessao_legislativa(sessao, parlamentar): logger.info("Filiação do parlamentar com (data<={} e data_desfiliacao>={}) " "ou (data<={} e data_desfiliacao=Null encontrada com sucesso." .format(sessao.data_fim, sessao.data_fim, sessao.data_fim)) - return filiacao.get_nome_partido_ano(sessao.data_fim.year).sigla + return filiacao.nome_partido_ano(sessao.data_fim.year).sigla def altera_field_mesa_public_view(request): diff --git a/sapl/templates/parlamentares/partido_update.html b/sapl/templates/parlamentares/partido_update.html index 8165bccb4..ed68db4b2 100644 --- a/sapl/templates/parlamentares/partido_update.html +++ b/sapl/templates/parlamentares/partido_update.html @@ -13,15 +13,15 @@ $("#id_data_modificacao").attr("disabled", true); check_change = () => { - if($("#id_historico").val() == 'nao'){ + if($("#id_historico").val() == 'False'){ $("#div_id_data_criacao").find("label").text("Data Criação") $("#div_id_data_extincao").find("label").text("Data Extinção") $( "#id_observacao" ).prop( "disabled", false ); } else { - $("#div_id_data_criacao").find("label").text("Inicio do Histórico") - $("#div_id_data_extincao").find("label").text("Final do Histórico") + $("#div_id_data_criacao").find("label").text("Inicio do Histórico*") + $("#div_id_data_extincao").find("label").text("Final do Histórico*") $( "#id_observacao" ).prop( "disabled", true ); $( "#id_observacao" ).val(""); } diff --git a/sapl/utils.py b/sapl/utils.py index c6413f32e..e6e2dabc3 100644 --- a/sapl/utils.py +++ b/sapl/utils.py @@ -820,7 +820,7 @@ def filiacao_data(parlamentar, data_inicio, data_fim=None): data_referencia = data_fim if data_fim else data_inicio - return ' | '.join([f.get_nome_partido_ano(data_referencia.year).sigla for f in filiacoes]) + return ' | '.join([f.nome_partido_ano(data_referencia.year).sigla for f in filiacoes]) def parlamentares_ativos(data_inicio, data_fim=None):