From 2e77366b94a543fd69531a699a4446dd706c8d8c Mon Sep 17 00:00:00 2001
From: Sander Roosendaal <roosendaalsander@gmail.com>
Date: Mon, 25 Mar 2019 20:44:13 +0100
Subject: [PATCH] fixes #447

---
 rowers/forms.py                     |  4 ++--
 rowers/templates/list_workouts.html | 21 +++++++--------------
 rowers/views/workoutviews.py        |  6 ++++--
 3 files changed, 13 insertions(+), 18 deletions(-)

diff --git a/rowers/forms.py b/rowers/forms.py
index cf130391..d0608bb1 100644
--- a/rowers/forms.py
+++ b/rowers/forms.py
@@ -63,8 +63,8 @@ class LoginForm(forms.Form):
 class SearchForm(forms.Form):
     q = forms.CharField(max_length=255,required=False,
                         widget=forms.TextInput(
-                            attrs={'placeholder': 'Search'}),
-                        label='Search')
+                            attrs={'placeholder': 'keyword or leave empty'}),
+                        label='Filter by Keyword')
 
     
     
diff --git a/rowers/templates/list_workouts.html b/rowers/templates/list_workouts.html
index f576429f..7a7db9b9 100644
--- a/rowers/templates/list_workouts.html
+++ b/rowers/templates/list_workouts.html
@@ -69,27 +69,20 @@
     {{ the_div |safe }}
   </li>
   <li class="grid_2">
-    <p>
+    <p>Filter on date
       <form enctype="multipart/form-data"  method="post">
         <table>
           {{ dateform.as_table }}
         </table>
         {% csrf_token %}
-        <input name='daterange' type="submit" value="Submit">
-      </form>
     </p>
-    {% if team %}
+    <p>and keyword</p>
     <p>
-      <form id="searchform" action="/rowers/list-workouts/team/{{ team.id }}/{{ startdate|date:"Y-m-d" }}/{{ enddate|date:"Y-m-d" }}/"
-            method="get" accept-charset="utf-8">
-        {% else %}
-        <form id="searchform" action="/rowers/list-workouts/{{ startdate|date:"Y-m-d" }}/{{ enddate|date:"Y-m-d" }}/"
-              method="get" accept-charset="utf-8">
-          {% endif %}
-          {{ searchform }}
-          <input type="submit" value="GO">
-        </input>
-        </form>
+      {{ searchform }}
+    </p>
+    <p>
+      <input name='daterange' type="submit" value="Submit">
+    </form>
     </p>
     <p>
     {% if rankingonly and not team %}
diff --git a/rowers/views/workoutviews.py b/rowers/views/workoutviews.py
index a2dd3f36..c976e6f7 100644
--- a/rowers/views/workoutviews.py
+++ b/rowers/views/workoutviews.py
@@ -1178,12 +1178,15 @@ def workouts_view(request,message='',successmessage='',
     startdate = datetime.datetime.combine(startdate,datetime.time())
     enddate = datetime.datetime.combine(enddate,datetime.time(23,59,59))
 
-    
+    query = None
     if request.method == 'POST':
         dateform = DateRangeForm(request.POST)
+        searchform = SearchForm(request.POST)
         if dateform.is_valid():
             startdate = dateform.cleaned_data['startdate']
             enddate = dateform.cleaned_data['enddate']
+        if searchform.is_valid():
+            query = searchform.cleaned_data['q']
     else:
         dateform = DateRangeForm(initial={
             'startdate':startdate,
@@ -1293,7 +1296,6 @@ def workouts_view(request,message='',successmessage='',
     for w in workoutsnohr:
         res = dataprep.workout_trimp(w)
         
-    query = request.GET.get('q')
     if query:
         query_list = query.split()
         workouts = workouts.filter(