workflowoverview: show triggered 0 as well

Author: bluetom

classes/processor.php

@@ -376,6 +376,7 @@ public function get_count_of_courses_to_trigger_for_workflow($workflow) {
             } else {
                 $obj->automatic = true;
                 $obj->triggered = 0;
+                $obj->excluded = false;
                 // Only use triggers with true sql to display the real amounts for the others (instead of always 0).
                 $obj->sql = trigger_manager::get_trigger_sqlresult($trigger);
                 $obj->response = $lib->check_course(null, null);

templates/overview_trigger.mustache

@@ -60,6 +60,15 @@
                             </span>
                             </a>
                         {{/triggeredcourses}}
+                        {{^triggeredcourses}}
+                            {{^excludedcourses}}
+                                {{^exclude}}
+                                    <span class="btn btn-outline-success">
+                                    0
+                                </span>
+                                {{/exclude}}
+                            {{/excludedcourses}}
+                        {{/triggeredcourses}}
                         {{#excludedcourses}}
                             <a href="{{{popuplink}}}&excluded={{id}}&showdetails={{showdetails}}" data-toggle="tooltip" title="{{tooltip}}">
                             <span class="btn btn-outline-danger">

workflowoverview.php

@@ -277,7 +277,7 @@
                         $trigger->classfires = "border-danger";
                     }
                     $trigger->tooltip = "";
-                    if (isset($amounts[$trigger->sortindex]->excluded)) {
+                    if ($amounts[$trigger->sortindex]->excluded !== false) {
                         $trigger->excludedcourses = $amounts[$trigger->sortindex]->excluded;
                         $trigger->tooltip = get_string('courses_will_be_excluded',
                             'tool_lifecycle', $trigger->excludedcourses);
@@ -482,6 +482,9 @@
     $workflowprocesseslink = "<br>".$workflowprocesseslink;
 }
 
+if (!($isactive || $isdeactivated)) {
+    $lastrun = 0;
+}
 $data = [
     'editsettingslink' => (new moodle_url(urls::EDIT_WORKFLOW, ['wf' => $workflow->id]))->out(false),
     'title' => $workflow->title,
@@ -507,7 +510,8 @@
     'showdetailsicon' => $showdetails == 0,
     'isactive' => $isactive || $isdeactivated,
     'nextrun' => $nextrunout,
-    'lastrun' => userdate($lastrun, get_string('strftimedatetimeshort', 'langconfig')),
+    'lastrun' => $lastrun ?
+        userdate($lastrun, get_string('strftimedatetimeshort', 'langconfig')) : '-',
     'nomanualtriggerinvolved' => $nomanualtriggerinvolved,
     'disableworkflowlink' => $disableworkflowlink,
     'abortdisableworkflowlink' => $abortdisableworkflowlink,