Skip to content

Commit 385bcbe

Browse files
committed
exclude trigger: show excluded 0 as well - part 2
1 parent 70bc14a commit 385bcbe

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

classes/processor.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ public function get_count_of_courses_to_trigger_for_workflow($workflow) {
368368
foreach ($triggers as $trigger) {
369369
$trigger = (object)(array) $trigger; // Cast to normal object to be able to set dynamic properties.
370370
$settings = settings_manager::get_settings($trigger->id, settings_type::TRIGGER);
371-
$trigger->exclude = $settings['exclude'] ?? ($settings['invert'] ?? false);
371+
$trigger->exclude = $settings['exclude'] ?? false;
372372
$obj = new \stdClass();
373373
$lib = lib_manager::get_trigger_lib($trigger->subpluginname);
374374
if ($lib->is_manual_trigger()) {

workflowoverview.php

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -277,7 +277,8 @@
277277
$trigger->classfires = "border-danger";
278278
}
279279
$trigger->tooltip = "";
280-
if ($trigger->excludedcourses = $amounts[$trigger->sortindex]->excluded || $trigger->exclude) {
280+
if (isset($amounts[$trigger->sortindex]->excluded)) {
281+
$trigger->excludedcourses = $amounts[$trigger->sortindex]->excluded;
281282
$trigger->tooltip = get_string('courses_will_be_excluded',
282283
'tool_lifecycle', $trigger->excludedcourses);
283284
} else {

0 commit comments

Comments
 (0)