From ac8ead52c8040ac4152e582c9e3b655f7913781e Mon Sep 17 00:00:00 2001 From: Darren Riedlinger Date: Sun, 6 Mar 2016 12:46:36 -0800 Subject: [PATCH] getattr on usersettings.setting dict always returns False. Use settings.get() instead --- helpdesk/views/public.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helpdesk/views/public.py b/helpdesk/views/public.py index b020cfb2..0d952485 100644 --- a/helpdesk/views/public.py +++ b/helpdesk/views/public.py @@ -25,7 +25,7 @@ def homepage(request): if (request.user.is_staff or (request.user.is_authenticated() and helpdesk_settings.HELPDESK_ALLOW_NON_STAFF_TICKET_UPDATE)): try: - if getattr(request.user.usersettings.settings, 'login_view_ticketlist', False): + if request.user.usersettings.settings.get('login_view_ticketlist', False): return HttpResponseRedirect(reverse('helpdesk_list')) else: return HttpResponseRedirect(reverse('helpdesk_dashboard'))