From 88f5287ba722ba143d2f737211d49e813e9efc06 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vin=C3=ADcius=20Cantu=C3=A1ria?= Date: Thu, 12 Mar 2020 14:04:41 -0300 Subject: [PATCH] =?UTF-8?q?HOT-FIX=20Corrige=20ordem=20de=20inclus=C3=A3o?= =?UTF-8?q?=20de=20comiss=C3=A3o=20no=20cleaned=20data?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- sapl/comissoes/forms.py | 39 +++++++++++++++------------------------ 1 file changed, 15 insertions(+), 24 deletions(-) diff --git a/sapl/comissoes/forms.py b/sapl/comissoes/forms.py index 92d6695de..5aadad6b4 100644 --- a/sapl/comissoes/forms.py +++ b/sapl/comissoes/forms.py @@ -39,44 +39,35 @@ class ComposicaoForm(forms.ModelForm): self.fields['comissao'].widget.attrs['disabled'] = 'disabled' def clean(self): - cleaned_data = super(ComposicaoForm, self).clean() + data = super().clean() + data['comissao'] = self.initial['comissao'] + comissao_pk = self.initial['comissao'].id if not self.is_valid(): - return cleaned_data + return data + + periodo = data['periodo'] - periodo = cleaned_data['periodo'] - comissao_pk = self.initial['comissao'].id - cleaned_data['comissao'] = self.initial['comissao'] - if periodo.data_fim: intersecao_periodo = Composicao.objects.filter( - Q(periodo__data_inicio__lte=periodo.data_fim, - periodo__data_fim__gte=periodo.data_fim) | - Q(periodo__data_inicio__gte=periodo.data_inicio, - periodo__data_fim__lte=periodo.data_inicio), + Q(periodo__data_inicio__lte=periodo.data_fim, periodo__data_fim__gte=periodo.data_fim) | + Q(periodo__data_inicio__gte=periodo.data_inicio, periodo__data_fim__lte=periodo.data_inicio), comissao_id=comissao_pk) else: intersecao_periodo = Composicao.objects.filter( - Q(periodo__data_inicio__gte=periodo.data_inicio, - periodo__data_fim__lte=periodo.data_inicio), + Q(periodo__data_inicio__gte=periodo.data_inicio, periodo__data_fim__lte=periodo.data_inicio), comissao_id=comissao_pk) if intersecao_periodo: if periodo.data_fim: - self.logger.error('O período informado ({} a {})' - 'choca com períodos já ' - 'cadastrados para esta comissão' - .format(periodo.data_inicio, periodo.data_fim)) + self.logger.error('O período informado ({} a {}) choca com períodos já cadastrados para esta comissão' + .format(periodo.data_inicio, periodo.data_fim)) else: - self.logger.error('O período informado ({} - )' - 'choca com períodos já ' - 'cadastrados para esta comissão' - .format(periodo.data_inicio)) - raise ValidationError('O período informado ' - 'choca com períodos já ' - 'cadastrados para esta comissão') + self.logger.error('O período informado ({} - ) choca com períodos já cadastrados para esta comissão' + .format(periodo.data_inicio)) + raise ValidationError('O período informado choca com períodos já cadastrados para esta comissão') - return cleaned_data + return data class PeriodoForm(forms.ModelForm):