Skip to content

Commit

Permalink
Merge branch 'MDL-63138-35' of git://github.com/rezaies/moodle into M…
Browse files Browse the repository at this point in the history
…OODLE_35_STABLE
  • Loading branch information
David Monllao committed Sep 18, 2018
2 parents 4e6ec59 + a027e6d commit 7c0ecc1
Show file tree
Hide file tree
Showing 2 changed files with 140 additions and 3 deletions.
19 changes: 16 additions & 3 deletions mod/folder/lib.php
Original file line number Diff line number Diff line change
Expand Up @@ -788,15 +788,28 @@ function folder_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_folder_core_calendar_provide_event_action(calendar_event $event,
\core_calendar\action_factory $factory) {
$cm = get_fast_modinfo($event->courseid)->instances['folder'][$event->instance];
\core_calendar\action_factory $factory,
int $userid = 0) {
global $USER;

if (!$userid) {
$userid = $USER->id;
}

$cm = get_fast_modinfo($event->courseid, $userid)->instances['folder'][$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);
$completiondata = $completion->get_data($cm, false, $userid);

if ($completiondata->completionstate != COMPLETION_INCOMPLETE) {
return null;
Expand Down
124 changes: 124 additions & 0 deletions mod/folder/tests/lib_test.php
Original file line number Diff line number Diff line change
Expand Up @@ -118,6 +118,91 @@ public function test_folder_core_calendar_provide_event_action() {
$this->assertTrue($actionevent->is_actionable());
}

public function test_folder_core_calendar_provide_event_action_for_non_user() {
global $CFG;

// Create a course.
$course = $this->getDataGenerator()->create_course();

// Create the activity.
$folder = $this->getDataGenerator()->create_module('folder', array('course' => $course->id));

// Create a calendar event.
$event = $this->create_action_event($course->id, $folder->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_folder_core_calendar_provide_event_action($event, $factory);

// Confirm the event is not shown at all.
$this->assertNull($actionevent);
}

public function test_folder_core_calendar_provide_event_action_in_hidden_section() {
// Create a course.
$course = $this->getDataGenerator()->create_course();

// Create a student.
$student = $this->getDataGenerator()->create_and_enrol($course, 'student');

// Create the activity.
$folder = $this->getDataGenerator()->create_module('folder', array('course' => $course->id));

// Create a calendar event.
$event = $this->create_action_event($course->id, $folder->id,
\core_completion\api::COMPLETION_EVENT_TYPE_DATE_COMPLETION_EXPECTED);

// Set sections 0 as hidden.
set_section_visible($course->id, 0, 0);

// Create an action factory.
$factory = new \core_calendar\action_factory();

// Decorate action event.
$actionevent = mod_folder_core_calendar_provide_event_action($event, $factory, $student->id);

// Confirm the event is not shown at all.
$this->assertNull($actionevent);
}

public function test_folder_core_calendar_provide_event_action_for_user() {
// Create a course.
$course = $this->getDataGenerator()->create_course();

// Create a student.
$student = $this->getDataGenerator()->create_and_enrol($course, 'student');

// Create the activity.
$folder = $this->getDataGenerator()->create_module('folder', array('course' => $course->id));

// Create a calendar event.
$event = $this->create_action_event($course->id, $folder->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_folder_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_folder_core_calendar_provide_event_action_already_completed() {
global $CFG;

Expand Down Expand Up @@ -149,6 +234,45 @@ public function test_folder_core_calendar_provide_event_action_already_completed
$this->assertNull($actionevent);
}

public function test_folder_core_calendar_provide_event_action_already_completed_for_user() {
global $CFG;

$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.
$folder = $this->getDataGenerator()->create_module('folder', array('course' => $course->id),
array('completion' => 2, 'completionview' => 1, 'completionexpected' => time() + DAYSECS));

// Get some additional data.
$cm = get_coursemodule_from_instance('folder', $folder->id);

// Create a calendar event.
$event = $this->create_action_event($course->id, $folder->id,
\core_completion\api::COMPLETION_EVENT_TYPE_DATE_COMPLETION_EXPECTED);

// Mark the activity as completed for the student.
$completion = new completion_info($course);
$completion->set_module_viewed($cm, $student->id);

// Now, log out.
$this->setUser();

// Create an action factory.
$factory = new \core_calendar\action_factory();

// Decorate action event for the student.
$actionevent = mod_folder_core_calendar_provide_event_action($event, $factory, $student->id);

// Ensure result was null.
$this->assertNull($actionevent);
}

/**
* Creates an action event.
*
Expand Down

0 comments on commit 7c0ecc1

Please sign in to comment.