fixes issue #624

rowers/models.py

@@ -2222,6 +2222,7 @@ class TrainingMicroCycle(models.Model):
         return stri
 
     def save(self, *args, **kwargs):
+
         if self.enddate < self.startdate: # pragma: no cover
             startdate = self.startdate
             enddate = self.enddate
@@ -2249,9 +2250,7 @@ class TrainingMicroCycle(models.Model):
             if othercycle.enddate >= self.startdate and othercycle.enddate <= self.enddate:
                 self.startdate = othercycle.enddate+datetime.timedelta(days=1)
 
-
+        if not self.enddate < self.startdate:
-
-        if not self.enddate <= self.startdate:
             super(TrainingMicroCycle,self).save(*args, **kwargs)
 
 class TrainingMesoCycleForm(ModelForm):

rowers/plannedsessions.py

@@ -379,6 +379,10 @@ def get_todays_micro(plan,thedate=date.today()):
 
             if thismicro:
                 thismicro = thismicro[0]
+            else:
+                mms = TrainingMicroCycle.objects.all()
+                
+                return None
 
     return thismicro
 

rowers/templates/list_workouts.html

@@ -115,14 +115,14 @@
           <i class="fas fa-arrow-alt-to-left"></i>
         </a>
         <a
-          href="?page={{ workouts.previous_page_number }}&q={{ request.GET.q }}">
+          href="?page={{ workouts.previous_page_number }}&q={{ request.GET.q }}&when={{ timeperiod }}">
           <i class="fas fa-arrow-alt-left"></i>
         </a>
         {% else %}
-        <a href="?page=1">
+        <a href="?page=1&when={{ timeperiod }}">
           <i class="fas fa-arrow-alt-to-left"></i>
         </a>
-        <a href="?page={{ workouts.previous_page_number }}">
+        <a href="?page={{ workouts.previous_page_number }}&when={{ timeperiod }}">
           <i class="fas fa-arrow-alt-left"></i>
         </a>
         {% endif %}
@@ -138,14 +138,14 @@
           <i class="fas fa-arrow-alt-right"></i>
         </a>
         <a
-          href="?page={{ workouts.paginator.num_pages }}&q={{ request.GET.q }}">
+          href="?page={{ workouts.paginator.num_pages }}&q={{ request.GET.q }}&when={{ timeperiod }}">
           <i class="fas fa-arrow-alt-to-right"></i>
         </a>
         {% else %}
         <a href="{{ request.path }}?page={{ workouts.next_page_number }}&when={{ timeperiod }}">
           <i class="fas fa-arrow-alt-right"></i>
         </a>
-        <a href="?page={{ workouts.paginator.num_pages }}">
+        <a href="?page={{ workouts.paginator.num_pages }}&when={{ timeperiod }}">
           <i class="fas fa-arrow-alt-to-right"></i>
         </a>
         {% endif %}

rowers/tests/test_unit_tests.py

@@ -247,6 +247,7 @@ class PlannedSessionTests(TestCase):
 
                 nextsunday = sundays[i]
 
+
                 micro = TrainingMicroCycle(startdate=monday,
                                            enddate=nextsunday,
                                            plan=cycle,
@@ -255,6 +256,7 @@ class PlannedSessionTests(TestCase):
                 micro.save()
 
 
+
         # sessions
         startdatetime = self.w1.startdatetime
 
@@ -346,6 +348,8 @@ class PlannedSessionTests(TestCase):
     def test_todays_micro(self):
         micro1 = plannedsessions.get_todays_micro(self.plan)
         micro2 = plannedsessions.get_todays_micro(self.plan,thedate=datetime.date.today())
+        self.assertFalse(micro1 is None)
+        self.assertFalse(micro2 is None)
         self.assertEqual(micro1.id,micro2.id)
 
     def test_get_session_ids(self):

rowers/views/workoutviews.py

@@ -6170,9 +6170,12 @@ def workout_summary_edit_view(request,id,message="",successmessage=""
 
 
         df['time'] =  rowdata.df['TimeStamp (sec)'].values
-        timeprev = int(df['time'].values[0])
+        try:
-        timenext = int(df['time'].values[result[0]])
+            timeprev = int(df['time'].values[0])
-        s = '{delta}sec'.format(delta=timenext-timeprev)
+            timenext = int(df['time'].values[result[0]])
+            s = '{delta}sec'.format(delta=timenext-timeprev)
+        except IndexError:
+            s = '0sec'
 
 
         for i in range(len(result)-1):