diff --git a/classes/local/table/interaction_remaining_table.php b/classes/local/table/interaction_remaining_table.php index d182632932928c136144f2fcd7f76ec86cbdbfd6..409c888971d13b57cbc67b9310ffcf08e92d0802 100644 --- a/classes/local/table/interaction_remaining_table.php +++ b/classes/local/table/interaction_remaining_table.php @@ -164,7 +164,7 @@ class interaction_remaining_table extends interaction_table { } // Otherwise, show latest action commited by user. global $CFG; - if($row->userid === -1) { + if ($row->userid === -1) { $userlink = get_string("anonymous_user"); } else { $userlink = \html_writer::link($CFG->wwwroot . '/user/profile.php?id=' . $row->userid, fullname($row)); diff --git a/classes/privacy/provider.php b/classes/privacy/provider.php index d2905a645ac5e81609038cd4dae7158d3de0dfef..350cea42f3d4c14fbe9939485e67490df5fd542c 100644 --- a/classes/privacy/provider.php +++ b/classes/privacy/provider.php @@ -148,7 +148,7 @@ class provider implements public static function get_users_in_context(userlist $userlist) { $context = $userlist->get_context(); if ($context instanceof \context_system) { - $sql = "SELECT userid + $sql = "SELECT userid FROM {tool_lifecycle_action_log}"; $userlist->add_from_sql('userid', $sql, array()); } diff --git a/tests/privacy_test.php b/tests/privacy_test.php index cce99eccca04c05d40eb579a421f96510b4cbdfd..f9046e05f61c4862d546cba0a70e28edf855fd6b 100644 --- a/tests/privacy_test.php +++ b/tests/privacy_test.php @@ -174,7 +174,6 @@ class tool_lifecycle_privacy_test extends provider_testcase { $u1 = $this->getDataGenerator()->create_user(); $u2 = $this->getDataGenerator()->create_user(); - $p1 = $this->generator->create_process($c1->id, $this->workflow->id); $p2 = $this->generator->create_process($c2->id, $this->workflow->id); @@ -187,7 +186,6 @@ class tool_lifecycle_privacy_test extends provider_testcase { $this->setUser($u2); interaction_manager::handle_interaction($this->emailstep->id, $p2->id, self::ACTION_KEEP); - $contextlist = new approved_contextlist($u1, 'tool_lifecycle', [1]); provider::delete_data_for_user($contextlist); $this->assertEquals(0, $DB->count_records_select('tool_lifecycle_action_log', "userid = $u1->id"));