From c0bb26aaf96607296831f25790e91bab8632ee3a Mon Sep 17 00:00:00 2001 From: Shamim Rezaie Date: Tue, 11 Sep 2018 14:25:45 +1000 Subject: [PATCH 1/2] MDL-63141 mod_imscp: Add userid param to mod_imscp calendar callbacks --- mod/imscp/lib.php | 8 ++-- mod/imscp/tests/lib_test.php | 73 ++++++++++++++++++++++++++++++++++++ 2 files changed, 78 insertions(+), 3 deletions(-) diff --git a/mod/imscp/lib.php b/mod/imscp/lib.php index 1a7eea2c440..a7d48263e89 100644 --- a/mod/imscp/lib.php +++ b/mod/imscp/lib.php @@ -485,15 +485,17 @@ function imscp_check_updates_since(cm_info $cm, $from, $filter = array()) { * * @param calendar_event $event * @param \core_calendar\action_factory $factory + * @param int $userid User id to use for all capability checks, etc. Set to 0 for current user (default). * @return \core_calendar\local\event\entities\action_interface|null */ function mod_imscp_core_calendar_provide_event_action(calendar_event $event, - \core_calendar\action_factory $factory) { - $cm = get_fast_modinfo($event->courseid)->instances['imscp'][$event->instance]; + \core_calendar\action_factory $factory, + int $userid = 0) { + $cm = get_fast_modinfo($event->courseid, $userid)->instances['imscp'][$event->instance]; $completion = new \completion_info($cm->get_course()); - $completiondata = $completion->get_data($cm, false); + $completiondata = $completion->get_data($cm, false, $userid); if ($completiondata->completionstate != COMPLETION_INCOMPLETE) { return null; diff --git a/mod/imscp/tests/lib_test.php b/mod/imscp/tests/lib_test.php index cd2e13e9718..721c78083e7 100644 --- a/mod/imscp/tests/lib_test.php +++ b/mod/imscp/tests/lib_test.php @@ -133,6 +133,40 @@ class mod_imscp_lib_testcase extends advanced_testcase { $this->assertTrue($actionevent->is_actionable()); } + public function test_imscp_core_calendar_provide_event_action_for_user() { + $this->resetAfterTest(); + $this->setAdminUser(); + + // Create a course. + $course = $this->getDataGenerator()->create_course(); + + // Create a student. + $student = $this->getDataGenerator()->create_and_enrol($course, 'student'); + + // Create the activity. + $imscp = $this->getDataGenerator()->create_module('imscp', array('course' => $course->id)); + + // Create a calendar event. + $event = $this->create_action_event($course->id, $imscp->id, + \core_completion\api::COMPLETION_EVENT_TYPE_DATE_COMPLETION_EXPECTED); + + // Now log out. + $this->setUser(); + + // Create an action factory. + $factory = new \core_calendar\action_factory(); + + // Decorate action event for the student. + $actionevent = mod_imscp_core_calendar_provide_event_action($event, $factory, $student->id); + + // Confirm the event was decorated. + $this->assertInstanceOf('\core_calendar\local\event\value_objects\action', $actionevent); + $this->assertEquals(get_string('view'), $actionevent->get_name()); + $this->assertInstanceOf('moodle_url', $actionevent->get_url()); + $this->assertEquals(1, $actionevent->get_item_count()); + $this->assertTrue($actionevent->is_actionable()); + } + public function test_imscp_core_calendar_provide_event_action_already_completed() { global $CFG; @@ -167,6 +201,45 @@ class mod_imscp_lib_testcase extends advanced_testcase { $this->assertNull($actionevent); } + public function test_imscp_core_calendar_provide_event_action_already_completed_for_user() { + global $CFG; + + $this->resetAfterTest(); + $this->setAdminUser(); + + $CFG->enablecompletion = 1; + + // Create a course. + $course = $this->getDataGenerator()->create_course(array('enablecompletion' => 1)); + + // Create a student. + $student = $this->getDataGenerator()->create_and_enrol($course, 'student'); + + // Create the activity. + $imscp = $this->getDataGenerator()->create_module('imscp', array('course' => $course->id), + array('completion' => 2, 'completionview' => 1, 'completionexpected' => time() + DAYSECS)); + + // Get some additional data. + $cm = get_coursemodule_from_instance('imscp', $imscp->id); + + // Create a calendar event. + $event = $this->create_action_event($course->id, $imscp->id, + \core_completion\api::COMPLETION_EVENT_TYPE_DATE_COMPLETION_EXPECTED); + + // Mark the activity as completed. + $completion = new completion_info($course); + $completion->set_module_viewed($cm, $student->id); + + // Create an action factory. + $factory = new \core_calendar\action_factory(); + + // Decorate action event for the student. + $actionevent = mod_imscp_core_calendar_provide_event_action($event, $factory, $student->id); + + // Ensure result was null. + $this->assertNull($actionevent); + } + /** * Creates an action event. * From 412697a70f08deb05b1e88897f289952742c068e Mon Sep 17 00:00:00 2001 From: Shamim Rezaie Date: Tue, 11 Sep 2018 15:35:21 +1000 Subject: [PATCH 2/2] MDL-63141 mod_imscp: Check if the module is visible to the user --- mod/imscp/lib.php | 5 +++ mod/imscp/tests/lib_test.php | 67 ++++++++++++++++++++++++++++++++++++ 2 files changed, 72 insertions(+) diff --git a/mod/imscp/lib.php b/mod/imscp/lib.php index a7d48263e89..b6378623a04 100644 --- a/mod/imscp/lib.php +++ b/mod/imscp/lib.php @@ -493,6 +493,11 @@ function mod_imscp_core_calendar_provide_event_action(calendar_event $event, int $userid = 0) { $cm = get_fast_modinfo($event->courseid, $userid)->instances['imscp'][$event->instance]; + if (!$cm->uservisible) { + // The module is not visible to the user for any reason. + return null; + } + $completion = new \completion_info($cm->get_course()); $completiondata = $completion->get_data($cm, false, $userid); diff --git a/mod/imscp/tests/lib_test.php b/mod/imscp/tests/lib_test.php index 721c78083e7..f54e03ce07e 100644 --- a/mod/imscp/tests/lib_test.php +++ b/mod/imscp/tests/lib_test.php @@ -134,6 +134,8 @@ class mod_imscp_lib_testcase extends advanced_testcase { } public function test_imscp_core_calendar_provide_event_action_for_user() { + global $CFG; + $this->resetAfterTest(); $this->setAdminUser(); @@ -151,6 +153,7 @@ class mod_imscp_lib_testcase extends advanced_testcase { \core_completion\api::COMPLETION_EVENT_TYPE_DATE_COMPLETION_EXPECTED); // Now log out. + $CFG->forcelogin = true; // We don't want to be logged in as guest, as guest users might still have some capabilities. $this->setUser(); // Create an action factory. @@ -167,6 +170,70 @@ class mod_imscp_lib_testcase extends advanced_testcase { $this->assertTrue($actionevent->is_actionable()); } + public function test_imscp_core_calendar_provide_event_action_as_non_user() { + global $CFG; + + $this->resetAfterTest(); + $this->setAdminUser(); + + // Create the activity. + $course = $this->getDataGenerator()->create_course(); + $imscp = $this->getDataGenerator()->create_module('imscp', array('course' => $course->id)); + + // Create a calendar event. + $event = $this->create_action_event($course->id, $imscp->id, + \core_completion\api::COMPLETION_EVENT_TYPE_DATE_COMPLETION_EXPECTED); + + // Now log out. + $CFG->forcelogin = true; // We don't want to be logged in as guest, as guest users might still have some capabilities. + $this->setUser(); + + // Create an action factory. + $factory = new \core_calendar\action_factory(); + + // Decorate action event. + $actionevent = mod_imscp_core_calendar_provide_event_action($event, $factory); + + // Confirm the event is not shown at all. + $this->assertNull($actionevent); + } + + public function test_imscp_core_calendar_provide_event_action_in_hidden_section() { + global $CFG; + + $this->resetAfterTest(); + $this->setAdminUser(); + + // Create a course. + $course = $this->getDataGenerator()->create_course(); + + // Create a student. + $student = $this->getDataGenerator()->create_and_enrol($course, 'student'); + + // Create the activity. + $imscp = $this->getDataGenerator()->create_module('imscp', array('course' => $course->id)); + + // Create a calendar event. + $event = $this->create_action_event($course->id, $imscp->id, + \core_completion\api::COMPLETION_EVENT_TYPE_DATE_COMPLETION_EXPECTED); + + // Set sections 0 as hidden. + set_section_visible($course->id, 0, 0); + + // Now log out. + $CFG->forcelogin = true; // We don't want to be logged in as guest, as guest users might still have some capabilities. + $this->setUser(); + + // Create an action factory. + $factory = new \core_calendar\action_factory(); + + // Decorate action event for the student. + $actionevent = mod_imscp_core_calendar_provide_event_action($event, $factory, $student->id); + + // Confirm the event is not shown at all. + $this->assertNull($actionevent); + } + public function test_imscp_core_calendar_provide_event_action_already_completed() { global $CFG;