From bf20e232d99e8e847050473ddfcb3ab9939efc4a Mon Sep 17 00:00:00 2001
From: Robert Einsle <robert@einsle.de>
Date: Sun, 5 Feb 2017 12:58:06 +0100
Subject: [PATCH] fix view logic while no location

---
 billard/static/billard/css/billard.css | 2 +-
 billard/templates/billard/base.html    | 6 +++---
 billard/views.py                       | 7 +++++--
 3 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/billard/static/billard/css/billard.css b/billard/static/billard/css/billard.css
index a8fc8a1..a11a486 100644
--- a/billard/static/billard/css/billard.css
+++ b/billard/static/billard/css/billard.css
@@ -1,3 +1,3 @@
 body {
-    padding-top: 50px;
+    padding-top: 55px;
 }
diff --git a/billard/templates/billard/base.html b/billard/templates/billard/base.html
index bebcc74..ff03fa4 100644
--- a/billard/templates/billard/base.html
+++ b/billard/templates/billard/base.html
@@ -27,11 +27,11 @@
             </div>
             <div id="navbar" class="collapse navbar-collapse">
                 <ul class="nav navbar-nav">
-                    <li class="active"><a href="/billard/">Tische</a></li>
+                    <li class="{% ifequal request.path '/billard/' %}active{% endifequal %}"><a href="{% url "carom_index" %}">Tische</a></li>
                     {% if user.is_authenticated %}
-                    <li><a href="{% url 'logout' %}">Abmelden</a></li>
+                    <li class="{% ifequal request.path '/logout/' %}active{% endifequal %}"><a href="{% url 'logout' %}">Abmelden</a></li>
                     {% else %}
-                    <li><a href="{% url 'login' %}">Anmelden</a></li>
+                    <li class="{% ifequal request.path '/login/' %}active{% endifequal %}"><a href="{% url 'login' %}">Anmelden</a></li>
                     {% endif %}
                 </ul>
             </div><!--/.nav-collapse -->
diff --git a/billard/views.py b/billard/views.py
index 0bc8228..e4c5a75 100644
--- a/billard/views.py
+++ b/billard/views.py
@@ -45,8 +45,11 @@ def index(request):
             loc = request.GET['loc']
         if not Location.objects.filter(users__id=request.user.id).filter(id=loc).exists():
             resp = redirect('carom_index')
-            resp['Location'] += '?loc={}'.format(str(min_loc))
-            return resp
+            if min_loc is not None:
+                resp['Location'] += '?loc={}'.format(str(min_loc))
+                return resp
+            else:
+                return render(request, 'billard/index.html')
         if loc is None:
             loc = min_loc
         locations = Location.objects.filter(users__id=request.user.id).order_by('code')