From 891861eaf512438d04119e63538fdf656f1efdf8 Mon Sep 17 00:00:00 2001 From: Roberto Rosario Date: Fri, 14 Jun 2019 00:20:14 -0400 Subject: [PATCH] Raise errors when DEBUG is True Signed-off-by: Roberto Rosario --- mayan/apps/common/generics.py | 62 ++++++++++++++++++++--------------- 1 file changed, 35 insertions(+), 27 deletions(-) diff --git a/mayan/apps/common/generics.py b/mayan/apps/common/generics.py index 3ed2df6a60..37f5c268ea 100644 --- a/mayan/apps/common/generics.py +++ b/mayan/apps/common/generics.py @@ -1,5 +1,6 @@ from __future__ import absolute_import, unicode_literals +from django.conf import settings from django.contrib import messages from django.core.exceptions import ImproperlyConfigured, ValidationError from django.db import transaction @@ -513,6 +514,7 @@ class SingleObjectCreateView( try: self.object.validate_unique() except ValidationError as exception: + raise context = self.get_context_data() error_message = self.get_error_message_duplicate() or _( @@ -531,17 +533,21 @@ class SingleObjectCreateView( try: self.object.save(**save_extra_data) except Exception as exception: - context = self.get_context_data() + raise + if settings.DEBUG: + raise + else: + context = self.get_context_data() - messages.error( - message=_('%(object)s not created, error: %(error)s') % { - 'object': self.get_object_name(context=context), - 'error': exception - }, request=self.request - ) - return super( - SingleObjectCreateView, self - ).form_invalid(form=form) + messages.error( + message=_('%(object)s not created, error: %(error)s') % { + 'object': self.get_object_name(context=context), + 'error': exception + }, request=self.request + ) + return super( + SingleObjectCreateView, self + ).form_invalid(form=form) else: context = self.get_context_data() @@ -558,12 +564,6 @@ class SingleObjectCreateView( return self.error_message_duplicate -class SingleObjectDynamicFormCreateView( - DynamicFormViewMixin, SingleObjectCreateView -): - pass - - class SingleObjectDeleteView( ObjectNameMixin, DeleteExtraDataMixin, ViewPermissionCheckMixin, RestrictedQuerysetMixin, ExtraContextMixin, RedirectionMixin, DeleteView @@ -597,8 +597,7 @@ class SingleObjectDeleteView( 'error': exception }, request=self.request ) - - raise exception + raise else: messages.success( message=_( @@ -659,6 +658,12 @@ class SingleObjectDownloadView(ViewPermissionCheckMixin, ObjectPermissionCheckMi VirtualFile = VirtualFile +class SingleObjectDynamicFormCreateView( + DynamicFormViewMixin, SingleObjectCreateView +): + pass + + class SingleObjectEditView( ObjectNameMixin, ViewPermissionCheckMixin, RestrictedQuerysetMixin, ExtraContextMixin, FormExtraKwargsMixin, RedirectionMixin, UpdateView @@ -685,15 +690,18 @@ class SingleObjectEditView( try: self.object.save(**save_extra_data) except Exception as exception: - messages.error( - message=_('%(object)s not updated, error: %(error)s.') % { - 'object': object_name, - 'error': exception - }, request=self.request - ) - return super( - SingleObjectEditView, self - ).form_invalid(form=form) + if settings.DEBUG: + raise + else: + messages.error( + message=_('%(object)s not updated, error: %(error)s.') % { + 'object': object_name, + 'error': exception + }, request=self.request + ) + return super( + SingleObjectEditView, self + ).form_invalid(form=form) else: messages.success( message=_(