From d40dd47731aedc795b812c1f9b4d371538b72f10 Mon Sep 17 00:00:00 2001
From: Sander Roosendaal <roosendaalsander@gmail.com>
Date: Fri, 12 Oct 2018 17:57:58 +0200
Subject: [PATCH] session create

---
 .../templates/plannedsession_multicreate.html |   4 +-
 rowers/templates/plannedsessioncreate.html    |   4 +-
 .../plannedsessions_multiclone_select.html    | 145 ++++--------------
 rowers/urls.py                                |  61 ++------
 rowers/views.py                               |  35 +++--
 5 files changed, 73 insertions(+), 176 deletions(-)

diff --git a/rowers/templates/plannedsession_multicreate.html b/rowers/templates/plannedsession_multicreate.html
index 0a3ede15..3f8f8ffa 100644
--- a/rowers/templates/plannedsession_multicreate.html
+++ b/rowers/templates/plannedsession_multicreate.html
@@ -48,11 +48,11 @@
           {% endfor %}
         </tbody>
       </table>
-      <a href="/rowers/sessions/multicreate/{{ timeperiod }}/rower/{{ rower.id }}/extra/{{ extrasessions }}">
+      <a href="/rowers/sessions/multicreate/user/{{ rower.user.id }}/extra/{{ extrasessions }}/?when={{ timeperiod }}">
         Add More
       </a>
       or
-      <a href="/rowers/sessions/multiclone/{{ timeperiod }}/rower/{{ rower.id }}">
+      <a href="/rowers/sessions/multiclone/user/{{ rower.user.id }}/?when={{ timeperiod }}">
         Clone multiple sessions
       </a>
       <button class="button green small" type="submit">Submit</button>
diff --git a/rowers/templates/plannedsessioncreate.html b/rowers/templates/plannedsessioncreate.html
index 162646f2..e66d61d0 100644
--- a/rowers/templates/plannedsessioncreate.html
+++ b/rowers/templates/plannedsessioncreate.html
@@ -43,10 +43,10 @@
           <td> {{ ps.sessionvalue }} </td>
           <td> {{ ps.sessionunit }} </td>
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/edit/{{ timeperiod }}/rower/{{ rower.id }}">Edit</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/edit/user/{{ rower.user.id }}/?when={{ timeperiod }}">Edit</a>
           </td>
           <td>
-            <a class="small" href="/rowers/sessions/{{ ps.id }}/clone/{{ timeperiod }}/rower/{{ rower.id }}">Clone</a>
+            <a class="small" href="/rowers/sessions/{{ ps.id }}/clone/user/{{ rower.user.id }}/?when={{ timeperiod }}">Clone</a>
           </td>
 
           <td>
diff --git a/rowers/templates/plannedsessions_multiclone_select.html b/rowers/templates/plannedsessions_multiclone_select.html
index d17e200a..bbe7a459 100644
--- a/rowers/templates/plannedsessions_multiclone_select.html
+++ b/rowers/templates/plannedsessions_multiclone_select.html
@@ -1,10 +1,10 @@
-{% extends "base.html" %}
+{% extends "newbase.html" %}
 {% load staticfiles %}
 {% load rowerfilters %}
 
 {% block title %}Workouts{% endblock %}
 
-{% block content %}
+{% block main %}
 
 <script>
   function toggle(source) {
@@ -64,123 +64,46 @@
 
 </script>
 
-
-<div class="grid_12 alpha">
-  {% include "planningbuttons.html" %}
-</div>
-<div class="grid_6 alpha">
-  <h1>Clone Multiple Sessions</h1>
-</div>
-<div id="timeperiod" class="grid_2 dropdown">
-  <button class="grid_2 alpha button gray small dropbtn">Select Time Period ({{ timeperiod|verbosetimeperiod }})</button>
-  <div class="dropdown-content">
-    <a class="button gray small alpha"
-       href="/rowers/sessions/multiclone/today">
-      Today
-    </a>
-    <a class="button gray small alpha"
-       href="/rowers/sessions/multiclone/thisweek">
-      This Week
-    </a>
-    <a class="button gray small alpha"
-       href="/rowers/sessions/multiclone/thismonth">
-      This Month
-    </a>
-    <a class="button gray small alpha"
-       href="/rowers/sessions/multiclone/lastweek">
-      Last Week
-    </a>
-    <a class="button gray small alpha"
-       href="/rowers/sessions/multiclone/lastmonth">
-      Last Month
-    </a>
-    <a class="button gray small alpha"
-       href="/rowers/sessions/multiclone/nextweek">
-      Next Week
-    </a>
-    <a class="button gray small alpha"
-       href="/rowers/sessions/multiclone/nextmonth">
-      Next Month
-    </a>
-  </div>
-</div>
-{% if user.is_authenticated and user|is_manager %}
-<div class="grid_2 dropdown">
-  <button class="grid_2 alpha button green small dropbtn">
-    Select Rower 
-  </button>
-  <div class="dropdown-content">
-    {% for member in user|team_rowers %}
-    <a class="button green small" href="/rowers/sessions/multiclone/{{ timeperiod }}/rower/{{ member.id }}">{{ member.user.first_name }} {{ member.user.last_name }}</a>
-    {% endfor %}
-  </div>
-</div>
-<div class="grid_12 alpha">
-  <div class="grid_6 alpha">
-    <form enctype="multipart/form-data" method="post">
-      {% endif %}
-      <div class="grid_4 alpha">
-        <table>
-          {{ dateform.as_table }}
-        </table>
-        {% csrf_token %}
-      </div>
-      <div class="grid_2 omega">
-        <input name='daterange' class="button green" type="submit" value="Submit">	
-      </div>
-    </form>
-  </div>
-  <div class="grid_5 prefix_1 omega">
-    <form id="searchform" method="get" accept-charset="utf-8">
-      <div class="grid_3 prefix_1 alpha">
-        <input class="searchfield" id="searchbox" name="q" type="text" placeholder="Search">
-      </div>
-      <div class="grid_1 omega">
-        <button class="button blue small" type="submit">
-          Search
-        </button>
-      </div>
-    </form>
-  </div>
-
-</div>
+<h1>Clone Multiple Sessions</h1>
 
 
 <form enctype="multipart/form-data" method="post">
-  <div id="workouts_table" class="grid_8 alpha">
+  <ul class="main-content">
+    <li class="grid_2">
 
 
-  {% if plannedsessions %}
-  
-  <input type="checkbox" onClick="toggle(this)" /> Toggle All<br/>
-
-  <table width="100%" class="listtable">
-    {{ form.as_table }}
-  </table>
-
-{% else %}
-<p> No sessions found </p>
-{% endif %}
-</div>
-  <div id="form_settings" class="grid_4 alpha">
-    <p>Select one or more planned sessions on the left,
-      select the date when the new cycle starts below
-      and press submit</p>
-  {% csrf_token %}
-  <table>
-    {{ dateshiftform.as_table }}
-  </table>
-  <div class="grid_1 prefix_2 suffix_1">
-    <p>
-      <input name='workoutselectform' class="button green" type="submit" value="Submit">
+      {% if plannedsessions %}
+      
+      <input type="checkbox" onClick="toggle(this)" /> Toggle All<br/>
+      
+      <table width="100%" class="listtable">
+        {{ form.as_table }}
+      </table>
+      
+      {% else %}
+      <p> No sessions found </p>
+      {% endif %}
+    </li>
+    <li class="grid_2">
+      <p>Select one or more planned sessions on the left,
+        select the date when the new cycle starts below
+        and press submit</p>
+      {% csrf_token %}
+      <table>
+        {{ dateshiftform.as_table }}
+      </table>
+      <p>
+        <input name='workoutselectform' class="button green" type="submit" value="Submit">
       </p>
-  </div>
-  <div class="grid_4">
-    <p>You can use the date and search forms above to search through all
+      <p>You can use the date and search forms above to search through all
       sessions.</p>
-  </div>
-</div>
+    </li>
+  </ul>
 </form>
 
 
 {% endblock %}
+
+{% block sidebar %}
+{% include 'menu_plan.html' %}
+{% endblock %}
diff --git a/rowers/urls.py b/rowers/urls.py
index e2833eb2..53e87a49 100644
--- a/rowers/urls.py
+++ b/rowers/urls.py
@@ -446,46 +446,25 @@ urlpatterns = [
     url(r'^sessions/teamcreate/team/$',
         views.plannedsession_teamcreate_view),
     url(r'^sessions/teamedit/(?P<sessionid>\d+)$',views.plannedsession_teamedit_view),
-    url(r'^sessions/teamedit/(?P<sessionid>\d+)/(?P<timeperiod>[\w\ ]+.*)$',views.plannedsession_teamedit_view),
-
     url(r'^sessions/create/$',views.plannedsession_create_view),
     url(r'^sessions/create/user/(?P<userid>\d+)$',
         views.plannedsession_create_view),
-
-    url(r'^sessions/multiclone$',views.plannedsession_multiclone_view),
-    url(r'^sessions/multiclone/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)/extra/(?P<extrasessions>\d+)$',
+    url(r'^sessions/multiclone/$',views.plannedsession_multiclone_view),
+    url(r'^sessions/multiclone/user/(?P<userid>\d+)/extra/(?P<extrasessions>\d+)$/',
         views.plannedsession_multiclone_view),
-    url(r'^sessions/multiclone/user/(?P<userid>\d+)$',
+    url(r'^sessions/multiclone/user/(?P<userid>\d+)/$',
         views.plannedsession_multiclone_view),
-    url(
-        r'^sessions/multiclone/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)$',
-        views.plannedsession_multiclone_view),
-    url(r'^sessions/multiclone/(?P<timeperiod>[\w\ ]+.*)$',
-        views.plannedsession_multiclone_view),
-    
-    url(r'^sessions/multicreate$',views.plannedsession_multicreate_view),
-    url(r'^sessions/multicreate/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)/extra/(?P<extrasessions>\d+)$',
+    url(r'^sessions/multicreate/$',views.plannedsession_multicreate_view),
+    url(r'^sessions/multicreate/user/(?P<userid>\d+)/extra/(?P<extrasessions>\d+)/$',
         views.plannedsession_multicreate_view),
-    url(r'^sessions/multicreate/user/(?P<userid>\d+)$',
+    url(r'^sessions/multicreate/user/(?P<userid>\d+)/$',
         views.plannedsession_multicreate_view),
-    url(
-        r'^sessions/multicreate/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)$',
-        views.plannedsession_multicreate_view),
-    url(r'^sessions/multicreate/(?P<timeperiod>[\w\ ]+.*)$',
-        views.plannedsession_multicreate_view),
-
-    url(r'^sessions/(?P<id>\d+)/edit/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)$',views.plannedsession_edit_view),
-    url(r'^sessions/(?P<id>\d+)/edit/(?P<timeperiod>[\w\ ]+.*)$',views.plannedsession_edit_view),
-    url(r'^sessions/(?P<id>\d+)/edit$',views.plannedsession_edit_view),
-
-    url(r'^sessions/(?P<id>\d+)/clone$',views.plannedsession_clone_view),
-    url(r'^sessions/(?P<id>\d+)/clone/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)$',views.plannedsession_clone_view),
-    url(r'^sessions/(?P<id>\d+)/clone/(?P<timeperiod>[\w\ ]+.*)$',views.plannedsession_clone_view),
-
+    url(r'^sessions/(?P<id>\d+)/edit/$',views.plannedsession_edit_view),
+    url(r'^sessions/(?P<id>\d+)/edit/user/(?P<userid>/$',views.plannedsession_edit_view),
+    url(r'^sessions/(?P<id>\d+)/clone/user/(?P<userid>/$',views.plannedsession_clone_view),
+    url(r'^sessions/(?P<id>\d+)/clone/$',views.plannedsession_clone_view),
     url(r'^sessions/(?P<id>\d+)$',views.plannedsession_view,
         name='plannedsession_view'),
-    url(r'^sessions/(?P<id>\d+)/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)$',views.plannedsession_view,
-        name='plannedsession_view'),
     url(r'^sessions/(?P<id>\d+)/user/(?P<userid>\d+)$',views.plannedsession_view,
         name='plannedsession_view'),
     url(r'^sessions/(?P<id>\d+)/deleteconfirm$',views.plannedsession_deleteconfirm_view),
@@ -494,33 +473,15 @@ urlpatterns = [
         views.plannedsessions_manage_view),
     url(r'^sessions/manage/user/(?P<userid>\d+)/session/(?P<initialsession>\d+)$',
         views.plannedsessions_manage_view),
-    url(r'^sessions/manage/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)/session/(?P<initialsession>\d+)$',
-        views.plannedsessions_manage_view),
-    url(r'^sessions/manage/(?P<timeperiod>[\w\ ]+.*)/session/(?P<initialsession>\d+)$',
-        views.plannedsessions_manage_view),
-
-    
     url(r'^sessions/manage/?$',
         views.plannedsessions_manage_view),
     url(r'^sessions/manage/user/(?P<userid>\d+)$',
         views.plannedsessions_manage_view),
-    url(r'^sessions/manage/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)$',
-        views.plannedsessions_manage_view),
-    url(r'^sessions/manage/(?P<timeperiod>[\w\ ]+.*)$',
-        views.plannedsessions_manage_view),
     url(r'^sessions/coach$',views.plannedsessions_coach_view),
-    url(r'^sessions/coach/(?P<timeperiod>[\w\ ]+.*)/team/(?P<teamid>\d+)$',
-        views.plannedsessions_coach_view),
-    url(r'^sessions/coach/(?P<timeperiod>[\w\ ]+.*)$',
-        views.plannedsessions_coach_view),
     url(r'^sessions/print/?$',views.plannedsessions_print_view),
     url(r'^sessions/print/user/(?P<userid>\d+)$',views.plannedsessions_print_view),
-    url(r'^sessions/print/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)$',views.plannedsessions_print_view),
-    url(r'^sessions/print/(?P<timeperiod>[\w\ ]+.*)$',views.plannedsessions_print_view),
-    url(r'^sessions/?$',views.plannedsessions_view),
+    url(r'^sessions/$',views.plannedsessions_view),
     url(r'^sessions/user/(?P<userid>\d+)$',views.plannedsessions_view),
-    url(r'^sessions/(?P<timeperiod>[\w\ ]+.*)/user/(?P<userid>\d+)$',views.plannedsessions_view),
-    url(r'^sessions/(?P<timeperiod>[\w\ ]+.*)$',views.plannedsessions_view),
     url(r'^courses/(?P<id>\d+)/edit$',views.course_edit_view,
         name='course_edit_view'),
     url(r'^courses/(?P<id>\d+)/delete$',views.course_delete_view),
diff --git a/rowers/views.py b/rowers/views.py
index 0d59dc38..c604b8e6 100644
--- a/rowers/views.py
+++ b/rowers/views.py
@@ -13541,15 +13541,20 @@ def agegrouprecordview(request,sex='male',weightcategory='hwt',
 @user_passes_test(hasplannedsessions,login_url="/rowers/planmembership/",
                   redirect_field_name=None)
 def plannedsession_multiclone_view(
-        request,timeperiod='none',
-        rowerid=0,userid=0,
+        request,
+        userid=0,
         startdate=timezone.now()-datetime.timedelta(days=30),
         enddate=timezone.now()):
 
-    r = getrequestrower(request,rowerid=rowerid)
+    r = getrequestrower(request,userid=userid)
 
-    if timeperiod != 'none':
-        startdate,enddate = get_dates_timeperiod(timeperiod)
+    when = request.GET.get('when')
+    if when:
+        timeperiod = when
+    else:
+        timeperiod = 'thisweek'
+        
+    startdate,enddate = get_dates_timeperiod(timeperiod)
         
         
     if request.method == 'POST' and 'daterange' in request.POST:
@@ -13591,10 +13596,11 @@ def plannedsession_multiclone_view(
 
             url = reverse(plannedsession_multicreate_view,
                           kwargs = {
-                              'rowerid':r.id,
-                              'timeperiod':timeperiod,
+                              'userid':r.user.id,
                               })
 
+            url += '?when='+when
+            
             return HttpResponseRedirect(url)
 
     sps = PlannedSession.objects.filter(
@@ -13638,6 +13644,7 @@ def plannedsession_multiclone_view(
                    'form':form,
                    'dateshiftform':dateshiftform,
                    'rower':r,
+                   'active':'nav-plan',
                    'timeperiod':timeperiod,
                    }
                   )
@@ -13777,12 +13784,18 @@ def plannedsession_create_view(request,
 
 @user_passes_test(hasplannedsessions,login_url="/rowers/planmembership/",
                   redirect_field_name=None)
-def plannedsession_multicreate_view(request,timeperiod='thisweek',
-                                    teamid=0,rowerid=0,userid=0,extrasessions=0):
+def plannedsession_multicreate_view(request,
+                                    teamid=0,userid=0,extrasessions=0):
 
     extrasessions=int(extrasessions)
 
-    r = getrequestrower(request,rowerid=rowerid)
+    r = getrequestrower(request,userid=userid)
+
+    when = request.GET.get('when')
+    if when:
+        timeperiod = when
+    else:
+        timeperiod = 'thisweek'
 
     startdate,enddate = get_dates_timeperiod(timeperiod)
     try:
@@ -13834,6 +13847,7 @@ def plannedsession_multicreate_view(request,timeperiod='thisweek',
     
         
     initials = [initial for i in range(extrasessions)]
+
     PlannedSessionFormSet = modelformset_factory(
         PlannedSession,
         form=PlannedSessionFormSmall,
@@ -16081,7 +16095,6 @@ def rower_trainingplan_view(request,
     else:
         thismeso = None
 
-    print thismacro,thismeso,thismicro,'aap'
                 
     return render(request,'trainingplan.html',
                   {