From 1337a1bac1e92060b581fca3fda33a64993d622c Mon Sep 17 00:00:00 2001
From: Edward Ribeiro <eribeiro@interlegis.leg.br>
Date: Wed, 11 Mar 2020 15:28:15 -0300
Subject: [PATCH] HOT-FIX: corrige bug fix introduzido em 28670b5bc98

---
 sapl/protocoloadm/forms.py | 25 +++++++++++++++----------
 1 file changed, 15 insertions(+), 10 deletions(-)

diff --git a/sapl/protocoloadm/forms.py b/sapl/protocoloadm/forms.py
index 4c1b2957e..240d61891 100644
--- a/sapl/protocoloadm/forms.py
+++ b/sapl/protocoloadm/forms.py
@@ -1127,18 +1127,13 @@ class DocumentoAdministrativoForm(FileFieldCheckMixin, ModelForm):
         tipo_documento = int(self.data['tipo'])
         ano_documento = int(self.data['ano'])
 
-        
-        equal_docs = DocumentoAdministrativo.objects.filter(numero=numero_documento,
-                                                                ano=ano_documento,
-                                                                complemento=complemento)
-        if equal_docs.exists() and equal_docs.first().pk != self.instance.pk:
-            raise ValidationError("Um documento administrativo com esse numero, complemento e ano já existe.")
 
         # não permite atualizar para numero/ano/tipo existente
         if self.instance.pk:
             mudanca_doc = numero_documento != self.instance.numero \
                 or ano_documento != self.instance.ano \
-                or tipo_documento != self.instance.tipo.pk
+                or tipo_documento != self.instance.tipo.pk \
+                or complemento != self.instance.complemento
 
         if not self.instance.pk or mudanca_doc:
             doc_exists = DocumentoAdministrativo.objects.filter(numero=numero_documento,
@@ -1147,9 +1142,19 @@ class DocumentoAdministrativoForm(FileFieldCheckMixin, ModelForm):
                                                                 complemento=complemento).exists()
 
             if doc_exists:
-                self.logger.error("DocumentoAdministrativo (numero={}, tipo={} e ano={}) já existe."
-                                  .format(numero_documento, tipo_documento, ano_documento))
-                raise ValidationError(_('Documento já existente'))
+                self.logger.error("DocumentoAdministrativo "
+                                  "(numero={}, tipo={}, ano={}, "
+                                  "complemento={}) já existe."
+                                  .format(numero_documento,
+                                          tipo_documento,
+                                          ano_documento,
+                                          complemento))
+                tipo = TipoDocumentoAdministrativo.objects.get(
+                    id=tipo_documento)
+                raise ValidationError(
+                    _('{}/{} ({}) já existente!'.format(numero_documento,
+                                                        ano_documento,
+                                                        tipo)))
 
         # campos opcionais, mas que se informados devem ser válidos
         if numero_protocolo and ano_protocolo: