From 614f53909e643fefaaf58da1a0e9dc2631dcebd6 Mon Sep 17 00:00:00 2001
From: Patrick <patrick@fet.at>
Date: Tue, 8 Dec 2020 14:30:35 +0000
Subject: [PATCH] add exception and logging infos

---
 fet2020/members/views.py | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/fet2020/members/views.py b/fet2020/members/views.py
index 3be37b71..ae028d21 100644
--- a/fet2020/members/views.py
+++ b/fet2020/members/views.py
@@ -10,6 +10,9 @@ from .serializers import MemberSerializer, JobSerializer, JobGroupSerializer, Jo
 from rest_framework import viewsets
 #from rest_framework import permissions
 
+import logging
+logger = logging.getLogger(__name__)
+
 
 def __remove_if_zero_active_mem(job_group):
     for elem in job_group:
@@ -54,7 +57,12 @@ def index(request):
 def jobs_view(request, slug=None):
     pinned_job_groups, unpinned_job_groups = __get_job_groups()
 
-    description = JobGroup.all_jobgroups.filter(slug=slug).values().first()['description']
+    try:
+        description = JobGroup.all_jobgroups.filter(slug=slug).values().first()['description']
+    except Exception as e:
+        logger.info("Wrong job '{}'".format(slug))
+        raise Http404("wrong job")
+
     job_members = JobMember.active_member.get_all_by_slug(slug=slug)
 
     context = {
@@ -73,6 +81,7 @@ def members_view(request, filter=None):
     if filter in Member.MemberRole:
         members = deque(Member.all_members.filter(role=filter))
     else:
+        logger.info("Wrong member role '{}'".format(filter))
         raise Http404("no member role")
 
     context = {
@@ -89,6 +98,7 @@ def profile_view(request, member_name=None):
 
     member = Member.all_members.filter(nickname=member_name).first()
     if not member:
+        logger.info("Wrong member name '{}'".format(member_name))
         raise Http404("no member")
 
     jobs = deque(JobMember.members.filter(member__nickname=member_name))