fixed bug where a started course could show standardtimecontrols

Author: rtschu

block_evasys_sync.php

@@ -102,6 +102,7 @@ public function get_content() {
         $enddisabled = false;
         $emailsentnotice = false;
         $periodsetnotice = false;
+        $wasstarted = false;
 
         // Set start to today and end to a week from now.
         $start = time();
@@ -128,6 +129,8 @@ public function get_content() {
                 // If the persistenceclass exists and the state is automatic and not opened
                 // the period must have been set.
                 $periodsetnotice = true;
+            } else {
+                $wasstarted = true;
             }
             if ($state >= course_evaluation_allocation::STATE_AUTO_OPENED || $nostudents) {
                 // If the course was already opened, disable the start date. If there are no students disable all controls.
@@ -230,7 +233,7 @@ public function get_content() {
             * In case of the automated workflow, we require surveys
             * in order to be able to automatically trigger the evaluation. */
             'showcontrols' => ($hassurveys || !$ismodeautomated) && count($evasyscourses) > 0 && !$invalidcourses,
-            'usestandardtimelayout' => (!$ismodeautomated && $hasstandardtime),
+            'usestandardtimelayout' => (!$ismodeautomated && $hasstandardtime && !$wasstarted),
             // Choose mode.
             'direct' => $ismodeautomated,
             'startdisabled' => $startdisabled || $standardttimemode,