Fix bug on schedule when the table is empty / added some comms

Jeremy
1 parent 11137fb4
Showing 3 changed files with 26 additions and 16 deletions Show diff stats
src/dashboard/views.py
src/scheduler/templates/scheduler/current_schedule.html
src/scheduler/views.py
@@ -35,12 +35,12 @@ def devices(request):
 @login_required
 def system(request):
     return render(request, 'dashboard/system.html')
-
+    
  
 @login_required
 def system_retrieve_logs(request):
     '''
-        Called by the dashboard system page with ajax request every 1s, to get the logs and print them
+        Called by the dashboard system page with ajax request every seconds, to get the logs and print them
     '''
     if request.is_ajax():
         alert_logs = Log.objects.filter(agent='Alert manager')
@@ -25,16 +25,22 @@
  
 			<div class="row">
  
-				<h3>Schedule info</h3>
+				<h2>Schedule info</h2>
+
+			{% if schedule = None %}
+
+			<h3>There is no schedule in the database</h3>
+
+			{% else %}
  
          		<div class="well">
                 	Schedule Night Start: {{schedule.plan_night_start|jdtodate}}
                 </div>
-         		
+
          		<div class="well">
                 	Schedule Start: {{schedule.plan_start|jdtodate}}
                 </div>
-             
+
              	<div class="well">
                     Schedule End: {{schedule.plan_end|jdtodate}}
          		</div>
@@ -53,10 +59,9 @@
  
 			</div>
  
-
 			<div class="row">
 				<h3>List of Sequences - <span class="label label-primary">{{sequences|length}}</span></h3>
-                
+
                 <div class="table-responsive">
                 	<table
                     	class="table table-bordered table-hover table-striped tablesorter">
@@ -92,8 +97,8 @@
                     </table>
                 </div>
  			</div>
+			{% endif %}
  
       	</div><!-- /#page-wrapper -->
  
  	{% endblock %}
-      	
 \ No newline at end of file
@@ -8,13 +8,19 @@ SIMULATION_FILE = &#39;file:./scheduler/sequences_cador.html&#39;
  
 @login_required
 def current_schedule(request):
-    schedule = Schedule.objects.order_by("-created")[0]
-    shs_list = schedule.shs.all()
-    sequences = [(shs.sequence, shs) for shs in shs_list]
-
-    nb_scheduled_sequences = len(shs_list)
-    executed_sequences = len([shs for shs in shs_list if shs.status == Sequence.EXECUTED])
-
+    if (len(Schedule.objects.all()) > 0):                       # checking if the schedule is empty
+        schedule = Schedule.objects.order_by("-created")[0]     # Sorting Schedule
+        shs_list = schedule.shs.all()                           # getting all the schedule has sequences references
+        sequences = [(shs.sequence, shs) for shs in shs_list]   # getting all sequences
+
+        nb_scheduled_sequences = len(shs_list)
+        executed_sequences = len([shs for shs in shs_list if shs.status == Sequence.EXECUTED])
+    else: # if empty set everything to 0 / None (variables are checked in src/templates/scheduler/current_schedule.html)
+        schedule = None
+        sequences = None
+        shs_list = None
+        nb_scheduled_sequences = 0
+        executed_sequences = 0
  
     return render(request, 'scheduler/current_schedule.html', {'sequences' : sequences, 'schedule' : schedule, 'nb_scheduled_sequences' : nb_scheduled_sequences, 'executed_sequences' : executed_sequences})
  
@@ -31,4 +37,3 @@ def schedule_simulation(request):
  
     sequences.sort(key=lambda seq: seq[1].tsp)
     return render(request, 'scheduler/current_schedule.html', {'sequences' : sequences, 'schedule' : schedule, 'nb_scheduled_sequences' : nb_scheduled_sequences, 'executed_sequences' : executed_sequences})
-