From 927b420b54d2c89a3432e96b8788950156b33a80 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jo=C3=A3o=20Rodrigues?= Date: Mon, 16 Dec 2019 20:23:38 -0300 Subject: [PATCH] Fix #3064 --- sapl/materia/forms.py | 48 +++++++++++++++++++++---------------------- 1 file changed, 24 insertions(+), 24 deletions(-) diff --git a/sapl/materia/forms.py b/sapl/materia/forms.py index e3f46d595..c3c6a1e43 100644 --- a/sapl/materia/forms.py +++ b/sapl/materia/forms.py @@ -206,39 +206,39 @@ class MateriaLegislativaForm(FileFieldCheckMixin, ModelForm): data_apresentacao = cleaned_data['data_apresentacao'] ano = cleaned_data['ano'] - protocolo = cleaned_data['numero_protocolo'] - protocolo_antigo = self.instance.numero_protocolo + protocolo_novo_numero = cleaned_data['numero_protocolo'] + protocolo_antigo_numero = self.instance.numero_protocolo - if protocolo: - if not Protocolo.objects.filter(numero=protocolo, ano=ano).exists(): - self.logger.error("Protocolo %s/%s não" - " existe" % (protocolo, ano)) - raise ValidationError(_('Protocolo %s/%s não' - ' existe' % (protocolo, ano))) + if protocolo_novo_numero: + if not Protocolo.objects.filter(numero=protocolo_novo_numero, ano=ano).exists(): + self.logger.error("O Protocolo %s/%s não" + " existe!" % (protocolo_novo_numero, ano)) + raise ValidationError(_('O Protocolo %s/%s não' + ' existe!' % (protocolo_novo_numero, ano))) - if protocolo_antigo != protocolo: + if protocolo_antigo_numero != protocolo_novo_numero: exist_materia = MateriaLegislativa.objects.filter( - numero_protocolo=protocolo, + numero_protocolo=protocolo_novo_numero, ano=ano).exists() - + + protocolo_novo = Protocolo.objects.get(numero=protocolo_novo_numero, ano=ano) exist_doc = DocumentoAdministrativo.objects.filter( - protocolo_id=protocolo, + protocolo_id=protocolo_novo.id, ano=ano).exists() if exist_materia or exist_doc: - self.logger.error("Protocolo %s/%s ja possui" - " documento vinculado" - % (protocolo, ano)) - raise ValidationError(_('Protocolo %s/%s ja possui' - ' documento vinculado' - % (protocolo, ano))) - - p = Protocolo.objects.get(numero=protocolo, ano=ano) - if p.tipo_materia != cleaned_data['tipo']: - self.logger.error("Tipo do Protocolo ({}) deve ser o mesmo do Tipo Matéria ({})." - .format(cleaned_data['tipo'], p.tipo_materia)) + self.logger.error("O Protocolo %s/%s já possui" + " um documento vinculado." + % (protocolo_novo_numero, ano)) + raise ValidationError(_('O Protocolo %s/%s já possui' + ' um documento vinculado.' + % (protocolo_novo_numero, ano))) + + if protocolo_novo.tipo_materia != cleaned_data['tipo']: + self.logger.error("O Tipo de Matéria do Protocolo ({}) deve ser o mesmo Tipo da Matéria ({})." + .format(cleaned_data['tipo'], protocolo_novo.tipo_materia)) raise ValidationError( - _('Tipo do Protocolo deve ser o mesmo do Tipo Matéria')) + _('O Tipo de Matéria do Protocolo deve ser o mesmo Tipo da Matéria.')) ano_origem_externa = cleaned_data['ano_origem_externa'] data_origem_externa = cleaned_data['data_origem_externa']