From de52162131547a7ccde4a64d91e203ee82c0136b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Vin=C3=ADcius=20Cantu=C3=A1ria?= Date: Wed, 17 Jun 2020 11:34:35 -0300 Subject: [PATCH] =?UTF-8?q?Corrige=20classifica=C3=A7=C3=A3o=20dos=20logs?= =?UTF-8?q?=20de=20erros=20para=20logs=20de=20avisos=20(#3197)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- sapl/materia/forms.py | 31 +++++++++++-------------------- sapl/parlamentares/views.py | 3 +-- sapl/protocoloadm/views.py | 6 +++--- sapl/sessao/views.py | 8 ++++---- 4 files changed, 19 insertions(+), 29 deletions(-) diff --git a/sapl/materia/forms.py b/sapl/materia/forms.py index c0a3b3777..65d227894 100644 --- a/sapl/materia/forms.py +++ b/sapl/materia/forms.py @@ -211,10 +211,8 @@ class MateriaLegislativaForm(FileFieldCheckMixin, ModelForm): 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))) + self.logger.warning("Protocolo %s/%s não existe" % (protocolo, ano)) + raise ValidationError(_('Protocolo %s/%s não existe' % (protocolo, ano))) if protocolo_antigo != protocolo: exist_materia = MateriaLegislativa.objects.filter( @@ -517,11 +515,9 @@ class TramitacaoForm(ModelForm): raise ValidationError(msg) if cleaned_data['data_tramitacao'] > timezone.now().date(): - self.logger.error('A data de tramitação informada ({}) não é ' + - 'menor ou igual a data de hoje!'.format(cleaned_data['data_tramitacao'])) - msg = _( - 'A data de tramitação deve ser ' + - 'menor ou igual a data de hoje!') + self.logger.warning('A data de tramitação informada ({}) não é menor ou igual a data de hoje!' + .format(cleaned_data['data_tramitacao'])) + msg = _('A data de tramitação deve ser menor ou igual a data de hoje!') raise ValidationError(msg) if (ultima_tramitacao and @@ -535,10 +531,8 @@ class TramitacaoForm(ModelForm): if data_enc_form: if data_enc_form < data_tram_form: - msg = _('A data de encaminhamento deve ser ' + - 'maior que a data de tramitação!') - self.logger.error("A data de encaminhamento ({}) deve ser " - "maior que a data de tramitação! ({})" + msg = _('A data de encaminhamento deve ser maior que a data de tramitação!') + self.logger.warning("A data de encaminhamento ({}) deve ser maior que a data de tramitação! ({})" .format(data_enc_form, data_tram_form)) raise ValidationError(msg) @@ -904,8 +898,7 @@ class AnexadaForm(ModelForm): except ObjectDoesNotExist: msg = _('A {} {}/{} não existe no cadastro de matérias legislativas.' .format(cleaned_data['tipo'], cleaned_data['numero'], cleaned_data['ano'])) - self.logger.error("A matéria a ser anexada não existe no cadastro" - " de matérias legislativas.") + self.logger.warning("A matéria a ser anexada não existe no cadastro de matérias legislativas.") raise ValidationError(msg) materia_principal = self.instance.materia_principal @@ -1742,11 +1735,9 @@ class TramitacaoEmLoteForm(ModelForm): if data_enc_form: if data_enc_form < data_tram_form: - self.logger.error('A data de encaminhamento ({}) deve ser ' - 'maior que a data de tramitação ({})!' - .format(data_enc_form, data_tram_form)) - msg = _('A data de encaminhamento deve ser ' + - 'maior que a data de tramitação!') + self.logger.warning('A data de encaminhamento ({}) deve ser maior que a data de tramitação ({})!' + .format(data_enc_form, data_tram_form)) + msg = _('A data de encaminhamento deve ser maior que a data de tramitação!') raise ValidationError(msg) if data_prazo_form: diff --git a/sapl/parlamentares/views.py b/sapl/parlamentares/views.py index 7e1c63025..50f042dec 100644 --- a/sapl/parlamentares/views.py +++ b/sapl/parlamentares/views.py @@ -585,8 +585,7 @@ class ParlamentarCrud(Crud): ". Tentando obter id da legislatura.") return int(self.request.GET['pk']) except: - self.logger.error( - "user=" + username + ". Legislatura não possui ID. Buscando em todas as entradas.") + self.logger.warning("User=" + username + ". Legislatura não possui ID. Buscando em todas as entradas.") legislaturas = Legislatura.objects.all() for l in legislaturas: if l.atual(): diff --git a/sapl/protocoloadm/views.py b/sapl/protocoloadm/views.py index b61b91ff8..83091c6b9 100755 --- a/sapl/protocoloadm/views.py +++ b/sapl/protocoloadm/views.py @@ -1368,9 +1368,9 @@ class TramitacaoAdmCrud(MasterDetailCrud): if tramitacao.pk != ultima_tramitacao.pk: username = request.user.username - self.logger.error("user=" + username + ". Não é possível deletar a tramitação de pk={}. " - "Somente a última tramitação (pk={}) pode ser deletada!." - .format(tramitacao.pk, ultima_tramitacao.pk)) + self.logger.warning("User={}. Não é possível deletar a tramitação de pk={}. " + "Somente a última tramitação (pk={}) pode ser deletada!." + .format(username, tramitacao.pk, ultima_tramitacao.pk)) msg = _('Somente a última tramitação pode ser deletada!') messages.add_message(request, messages.ERROR, msg) return HttpResponseRedirect(url) diff --git a/sapl/sessao/views.py b/sapl/sessao/views.py index 25cde87d1..26da9d23a 100755 --- a/sapl/sessao/views.py +++ b/sapl/sessao/views.py @@ -2801,8 +2801,8 @@ class VotacaoNominalAbstract(SessaoPermissionMixin): parlamentar=parlamentar) except ObjectDoesNotExist: username = self.request.user.username - self.logger.error('user=' + username + '. Objeto voto_parlamentar do ' + - 'parlamentar de id={} não existe.'.format(parlamentar.pk)) + self.logger.warning('User={}. Objeto voto_parlamentar do parlamentar de id={} não existe.' + .format(username, parlamentar.pk)) yield [parlamentar, None] else: yield [parlamentar, voto.voto] @@ -4398,8 +4398,8 @@ class VotacaoEmBlocoNominalView(PermissionRequiredForAppCrudMixin, TemplateView) parlamentar=parlamentar) except ObjectDoesNotExist: username = self.request.user.username - self.logger.error('user=' + username + '. Objeto voto_parlamentar do ' + - 'parlamentar de id={} não existe.'.format(parlamentar.pk)) + self.logger.warning('User={}. Objeto voto_parlamentar do parlamentar de id={} não existe.' + .format(username, parlamentar.pk)) yield [parlamentar, None] else: yield [parlamentar, voto.voto]