From 2c2179b5ce7dc98adf12e0de17c66f35bd5f72ef Mon Sep 17 00:00:00 2001
From: Piotr Gawron <piotr.gawron@uni.lu>
Date: Thu, 30 Nov 2017 13:57:05 +0100
Subject: [PATCH] redundant logging removed

---
 smash/web/api_views/subject.py | 14 +++++---------
 1 file changed, 5 insertions(+), 9 deletions(-)

diff --git a/smash/web/api_views/subject.py b/smash/web/api_views/subject.py
index 6c864c10..e26ae352 100644
--- a/smash/web/api_views/subject.py
+++ b/smash/web/api_views/subject.py
@@ -5,7 +5,7 @@ from django.db.models import Count, Case, When, Min
 from django.db.models import Q
 from django.http import JsonResponse
 
-from web.models import Subject, StudySubject, Visit, Appointment
+from web.models import StudySubject, Visit, Appointment, Subject
 from web.models.constants import SUBJECT_TYPE_CHOICES
 from web.views import e500_error
 from web.views.notifications import get_subjects_with_no_visit, get_subjects_with_reminder, get_today_midnight_date
@@ -16,14 +16,10 @@ logger = logging.getLogger(__name__)
 
 @login_required
 def cities(request):
-    try:
-        result_subjects = Subject.objects.filter(city__isnull=False).values_list('city').distinct()
-        return JsonResponse({
-            "cities": [x[0] for x in result_subjects]
-        })
-    except Exception as e:
-        logger.error(e, exc_info=True)
-        return e500_error(request)
+    result_subjects = Subject.objects.filter(city__isnull=False).values_list('city').distinct()
+    return JsonResponse({
+        "cities": [x[0] for x in result_subjects]
+    })
 
 
 @login_required
-- 
GitLab