MDL-34467 lti: Repalce all instances of get_context_instanc() with respective context_XXX::instance()

mod/lti/grade.php

@@ -68,7 +68,7 @@ if ($l) {  // Two ways to specify the module
 $course = $DB->get_record('course', array('id' => $cm->course), '*', MUST_EXIST);
 
 require_login($course, false, $cm);
-$context = get_context_instance(CONTEXT_MODULE, $cm->id);
+$context = context_module::instance($cm->id);
 require_capability('mod/lti:grade', $context);
 
 $url = new moodle_url('/mod/lti/grade.php', array('id' => $cm->id));

mod/lti/instructor_edit_tool_type.php

@@ -37,7 +37,7 @@ $PAGE->set_pagelayout('popup');
 $action = optional_param('action', null, PARAM_TEXT);
 $typeid = optional_param('typeid', null, PARAM_INT);
 
-require_capability('mod/lti:addcoursetool', get_context_instance(CONTEXT_COURSE, $courseid));
+require_capability('mod/lti:addcoursetool', context_course::instance($courseid));
 
 if (!empty($typeid)) {
     $type = lti_get_type($typeid);

mod/lti/lib.php

@@ -395,7 +395,7 @@ function lti_grade_item_delete($basiclti) {
 function lti_extend_settings_navigation($settings, $parentnode) {
     global $PAGE;
 
-    if (has_capability('mod/lti:grade', get_context_instance(CONTEXT_MODULE, $PAGE->cm->id))) {
+    if (has_capability('mod/lti:grade', context_module::instance($PAGE->cm->id))) {
         $keys = $parentnode->get_children_key_list();
 
         $node = navigation_node::create('Submissions',

mod/lti/locallib.php

@@ -462,7 +462,7 @@ function lti_get_ims_role($user, $cmid, $courseid) {
         //If no cmid is passed, check if the user is a teacher in the course
         //This allows other modules to programmatically "fake" a launch without
         //a real LTI instance
-        $coursecontext = get_context_instance(CONTEXT_COURSE, $courseid);
+        $coursecontext = context_course::instance($courseid);
 
         if (has_capability('moodle/course:manageactivities', $coursecontext)) {
             array_push($roles, 'Instructor');
@@ -470,7 +470,7 @@ function lti_get_ims_role($user, $cmid, $courseid) {
             array_push($roles, 'Learner');
         }
     } else {
-        $context = get_context_instance(CONTEXT_MODULE, $cmid);
+        $context = context_module::instance($cmid);
 
         if (has_capability('mod/lti:manage', $context)) {
             array_push($roles, 'Instructor');

mod/lti/request_tool.php

@@ -35,7 +35,7 @@ $course = $DB->get_record('course', array('id' => $lti->course));
 
 require_login($course);
 
-require_capability('mod/lti:requesttooladd', get_context_instance(CONTEXT_COURSE, $lti->course));
+require_capability('mod/lti:requesttooladd', context_course::instance($lti->course));
 
 $baseurl = lti_get_domain_from_url($lti->toolurl);
 

mod/lti/return.php

@@ -61,7 +61,7 @@ if (!empty($errormsg)) {
 
     echo htmlspecialchars($errormsg);
 
-    $canaddtools = has_capability('mod/lti:addcoursetool', get_context_instance(CONTEXT_COURSE, $courseid));
+    $canaddtools = has_capability('mod/lti:addcoursetool', context_course::instance($courseid));
 
     if ($unsigned == 1 && $canaddtools) {
         echo '<br /><br />';

mod/lti/service.php

@@ -98,7 +98,7 @@ switch ($messagetype) {
         $ltiinstance = $DB->get_record('lti', array('id' => $parsed->instanceid));
 
         //Getting the grade requires the context is set
-        $context = get_context_instance(CONTEXT_COURSE, $ltiinstance->course);
+        $context = context_course::instance($ltiinstance->course);
         $PAGE->set_context($context);
 
         lti_verify_sourcedid($ltiinstance, $parsed);

mod/lti/view.php

@@ -73,7 +73,7 @@ if ($tool) {
 }
 
 $PAGE->set_cm($cm, $course); // set's up global $COURSE
-$context = get_context_instance(CONTEXT_MODULE, $cm->id);
+$context = context_module::instance($cm->id);
 $PAGE->set_context($context);
 
 $url = new moodle_url('/mod/lti/view.php', array('id'=>$cm->id));