From ae83fafdd040e237fc694ed3992baf0e86a63070 Mon Sep 17 00:00:00 2001
From: Sander Roosendaal <roosendaalsander@gmail.com>
Date: Mon, 10 Sep 2018 21:11:57 +0200
Subject: [PATCH] some more logic around fillers and start dates

---
 rowers/models.py                          | 56 +++++++++++++++++++++++
 rowers/templates/trainingplan.html        | 19 ++++++--
 rowers/templates/trainingplan_create.html |  5 +-
 rowers/templatetags/rowerfilters.py       |  7 +++
 4 files changed, 81 insertions(+), 6 deletions(-)

diff --git a/rowers/models.py b/rowers/models.py
index 924c4a4a..00fd6da7 100644
--- a/rowers/models.py
+++ b/rowers/models.py
@@ -1095,7 +1095,20 @@ def createmacrofillers(plan):
         thedate = cycles[0].startdate-datetime.timedelta(days=1)
         cycles = cycles[1:]
                 
+    cycles = TrainingMacroCycle.objects.filter(
+        plan = plan
+        ).order_by("startdate")
 
+    if cycles[0].startdate > plan.startdate:
+        macr = TrainingMacroCycle(
+            plan=plan,
+            startdate = plan.startdate,
+            enddate = cycles[0].startdate-datetime.timedelta(days=1),
+            type='filler',
+            name='Filler'
+        )
+        macr.save()
+    
 def createmesofillers(plan):
     fillers = TrainingMesoCycle.objects.filter(
         plan = plan, type = 'filler'
@@ -1131,6 +1144,20 @@ def createmesofillers(plan):
             macr.save()
         thedate = cycles[0].startdate-datetime.timedelta(days=1)
         cycles = cycles[1:]
+
+    cycles = TrainingMesoCycle.objects.filter(
+        plan = plan
+        ).order_by("startdate")
+
+    if cycles[0].startdate > plan.startdate:
+        macr = TrainingMesoCycle(
+            plan=plan,
+            startdate = plan.startdate,
+            enddate = cycles[0].startdate-datetime.timedelta(days=1),
+            type='filler',
+            name='Filler'
+        )
+        macr.save()
                 
 
 def createmicrofillers(plan):
@@ -1169,6 +1196,20 @@ def createmicrofillers(plan):
         thedate = cycles[0].startdate-datetime.timedelta(days=1)
         cycles = cycles[1:]
 
+    cycles = TrainingMicroCycle.objects.filter(
+        plan = plan
+        ).order_by("startdate")
+
+    if cycles[0].startdate > plan.startdate:
+        macr = TrainingMicroCycle(
+            plan=plan,
+            startdate = plan.startdate,
+            enddate = cycles[0].startdate-datetime.timedelta(days=1),
+            type='filler',
+            name='Filler'
+        )
+        macr.save()
+
 def microcyclecheckdates(plan):
     cycles = TrainingMicroCycle.objects.filter(
         plan=plan
@@ -1203,6 +1244,21 @@ def mesocyclecheckdates(plan):
         thedate = cycles[0].startdate-datetime.timedelta(days=1)
         cycles = cycles[1:]
 
+    cycles = TrainingMesoCycle.objects.filter(
+        plan=plan
+        ).order_by("startdate")
+
+    thedate = plan.startdate
+    while cycles:
+        if cycles[0].startdate < thedate:
+            cycles[0].startdate = thedate
+            cycles[0].save()
+        try:
+            thedate = cycles[1].startdate-datetime.timedelta(days=1)
+        except IndexError:
+            pass
+        cycles = cycles[1:]
+
 def macrocyclecheckdates(plan):
     cycles = TrainingMacroCycle.objects.filter(
         plan=plan
diff --git a/rowers/templates/trainingplan.html b/rowers/templates/trainingplan.html
index 62dbb83c..cb063a44 100644
--- a/rowers/templates/trainingplan.html
+++ b/rowers/templates/trainingplan.html
@@ -29,7 +29,7 @@
       are typically used to address specific phases of preparation and to indicate the racing
       season and may span several months.
       Meso cycles can be used to group sequences of three to five light, medium and
-      hard weeks. </p>
+      hard weeks. It is recommended to work from left to right, starting with the macro cycles.</p>
   </div>
   <div class="grid_4 alpha">
     <h2>Macro Cycles</h2>
@@ -42,6 +42,7 @@
   </div>
 </div>
 <div class="grid_12 alpha">
+  {% now "Y-m-d" as todays_date %}
   {% for key,macrocycle in cycles.items %}
   <div class="grid_12 alpha">
     {% if macrocycle.0.type == 'filler' %}
@@ -51,9 +52,11 @@
         {% endif %}
       <div class="padded">
         <h3>{{ macrocycle.0.name }} ({{ macrocycle.0.startdate }} - {{ macrocycle.0.enddate }})</h3>
+        {% if todays_date <= macrocycle.0.enddate|date:"Y-m-d" %}
         <p><a href="/rowers/macrocycle/{{ macrocycle.0.id }}">edit</a>
-              /
-              <a href="/rowers/deletemacrocycle/{{ macrocycle.0.id }}">delete</a></p>
+          /
+          <a href="/rowers/deletemacrocycle/{{ macrocycle.0.id }}">delete</a></p>
+        {% endif %}
       </div>
     </div>
     <div class="grid_8 alpha">
@@ -65,9 +68,13 @@
         {% endif %}
         <div class="padded">
           <h3>{{ mesocycle.0.name }} ({{ mesocycle.0.startdate }} - {{ mesocycle.0.enddate }})</h3>
+          {% if mesocycle.0.plan.type == 'userdefined' %}
+          {% if todays_date <= mesocycle.0.enddate|date:"Y-m-d" %}
           <p><a href="/rowers/mesocycle/{{ mesocycle.0.id }}">edit</a>
               /
-              <a href="/rowers/deletemesocycle/{{ mesocycle.0.id }}">delete</a></p>
+            <a href="/rowers/deletemesocycle/{{ mesocycle.0.id }}">delete</a></p>
+          {% endif %}
+          {% endif %}
         </div>
       </div>
       <div class="grid_4 omega">
@@ -79,10 +86,14 @@
             {% endif %}
           <div class="padded">
             <h3>{{ microcycle.name }} ({{ microcycle.startdate }} - {{ microcycle.enddate }})</h3>
+            {% if microcycle.plan.type == 'userdefined' %}
+            {% if todays_date <= microcycle.enddate|date:"Y-m-d" %}
             <p>
               <a href="/rowers/microcycle/{{ microcycle.id }}">edit</a>
               /
               <a href="/rowers/deletemicrocycle/{{ microcycle.id }}">delete</a></p>
+            {% endif %}
+            {% endif %}
           </div>
         </div>
         {% endfor %}
diff --git a/rowers/templates/trainingplan_create.html b/rowers/templates/trainingplan_create.html
index 83a7654b..350e2119 100644
--- a/rowers/templates/trainingplan_create.html
+++ b/rowers/templates/trainingplan_create.html
@@ -100,8 +100,9 @@
           <td> {{ plan.startdate }} </td>
           <td> {{ plan.enddate }}</td>
           <td><a href="/rowers/plan/{{ plan.id }}">{{ plan.name }}</a></td>
-          <td> <a href="/rowers/editplan/{{ plan.id }}">Edit</a>
-          <td> <a href="/rowers/deleteplan/{{ plan.id }}">Delete</a>
+          <td> <a href="/rowers/editplan/{{ plan.id }}">Edit</a></td>
+          <td> <a href="/rowers/plan/{{ plan.id }}">Plan</a></td>
+          <td> <a href="/rowers/deleteplan/{{ plan.id }}">Delete</a></td>
         </tr>
         {% endfor %}
       </tbody>
diff --git a/rowers/templatetags/rowerfilters.py b/rowers/templatetags/rowerfilters.py
index 80e888a7..dede3b70 100644
--- a/rowers/templatetags/rowerfilters.py
+++ b/rowers/templatetags/rowerfilters.py
@@ -275,3 +275,10 @@ def future_registered(race,r):
     is_complete, has_registered = race_rower_status(r,race)
     is_open = race.evaluation_closure > timezone.now()
     return has_registered and not is_complete and is_open
+
+@property
+def is_past_due(self):
+    return datetime.date.today() > self.date
+@property
+def is_not_past_due(self):
+    return datetime.date.today() <= self.date