MDL-34458 Librairies: Replaced deprecated get_context_instance()

NoelDeMartin · Jul 24, 2012 · 21c08c6 · 21c08c6
1 parent 90310b1
commit 21c08c6
Show file tree

Hide file tree

Showing 29 changed files with 52 additions and 52 deletions.
diff --git a/question/addquestion.php b/question/addquestion.php
@@ -47,11 +47,11 @@
 if ($cmid) {
     list($module, $cm) = get_module_from_cmid($cmid);
     require_login($cm->course, false, $cm);
-    $thiscontext = get_context_instance(CONTEXT_MODULE, $cmid);
+    $thiscontext = context_module::instance($cmid);
     $hiddenparams['cmid'] = $cmid;
 } else if ($courseid) {
     require_login($courseid, false);
-    $thiscontext = get_context_instance(CONTEXT_COURSE, $courseid);
+    $thiscontext = context_course::instance($courseid);
     $module = null;
     $cm = null;
     $hiddenparams['courseid'] = $courseid;

diff --git a/question/editlib.php b/question/editlib.php
@@ -1586,7 +1586,7 @@ function question_edit_setup($edittab, $baseurl, $requirecmid = false, $requirec
         $courseid = $cm->course;
         $thispageurl->params(compact('cmid'));
         require_login($courseid, false, $cm);
-        $thiscontext = get_context_instance(CONTEXT_MODULE, $cmid);
+        $thiscontext = context_module::instance($cmid);
     } else {
         $module = null;
         $cm = null;
@@ -1598,7 +1598,7 @@ function question_edit_setup($edittab, $baseurl, $requirecmid = false, $requirec
         if ($courseid){
             $thispageurl->params(compact('courseid'));
             require_login($courseid, false);
-            $thiscontext = get_context_instance(CONTEXT_COURSE, $courseid);
+            $thiscontext = context_course::instance($courseid);
         } else {
             $thiscontext = null;
         }

diff --git a/question/engine/datalib.php b/question/engine/datalib.php
@@ -1240,7 +1240,7 @@ protected function compute_value($draftitemid, $text) {
         global $USER;
 
         $fs = get_file_storage();
-        $usercontext = get_context_instance(CONTEXT_USER, $USER->id);
+        $usercontext = context_user::instance($USER->id);
 
         $files = $fs->get_area_files($usercontext->id, 'user', 'draft',
                 $draftitemid, 'sortorder, filepath, filename', false);

diff --git a/question/engine/tests/helpers.php b/question/engine/tests/helpers.php
@@ -628,7 +628,7 @@ protected function setUp() {
 
         $this->displayoptions = new question_display_options();
         $this->quba = question_engine::make_questions_usage_by_activity('unit_test',
-            get_context_instance(CONTEXT_SYSTEM));
+            context_system::instance());
     }
 
     protected function tearDown() {

diff --git a/question/engine/tests/questionattemptiterator_test.php b/question/engine/tests/questionattemptiterator_test.php
@@ -44,7 +44,7 @@ class question_attempt_iterator_test extends advanced_testcase {
 
     protected function setUp() {
         $this->quba = question_engine::make_questions_usage_by_activity('unit_test',
-                get_context_instance(CONTEXT_SYSTEM));
+                context_system::instance());
         $this->quba->set_preferred_behaviour('deferredfeedback');
 
         $slot = $this->quba->add_question(test_question_maker::make_question('description'));

diff --git a/question/engine/tests/questionusagebyactivity_test.php b/question/engine/tests/questionusagebyactivity_test.php
@@ -42,7 +42,7 @@ class question_usage_by_activity_test extends advanced_testcase {
     public function test_set_get_preferred_model() {
         // Set up
         $quba = question_engine::make_questions_usage_by_activity('unit_test',
-                get_context_instance(CONTEXT_SYSTEM));
+                context_system::instance());
 
         // Exercise SUT and verify.
         $quba->set_preferred_behaviour('deferredfeedback');
@@ -52,7 +52,7 @@ public function test_set_get_preferred_model() {
     public function test_set_get_id() {
         // Set up
         $quba = question_engine::make_questions_usage_by_activity('unit_test',
-                get_context_instance(CONTEXT_SYSTEM));
+                context_system::instance());
 
         // Exercise SUT and verify
         $quba->set_id_from_database(123);
@@ -62,15 +62,15 @@ public function test_set_get_id() {
     public function test_fake_id() {
         // Set up
         $quba = question_engine::make_questions_usage_by_activity('unit_test',
-                get_context_instance(CONTEXT_SYSTEM));
+                context_system::instance());
 
         // Exercise SUT and verify
         $this->assertNotEmpty($quba->get_id());
     }
 
     public function test_create_usage_and_add_question() {
         // Exercise SUT
-        $context = get_context_instance(CONTEXT_SYSTEM);
+        $context = context_system::instance();
         $quba = question_engine::make_questions_usage_by_activity('unit_test', $context);
         $quba->set_preferred_behaviour('deferredfeedback');
         $tf = test_question_maker::make_question('truefalse', 'true');
@@ -87,7 +87,7 @@ public function test_create_usage_and_add_question() {
     public function test_get_question() {
         // Set up.
         $quba = question_engine::make_questions_usage_by_activity('unit_test',
-                get_context_instance(CONTEXT_SYSTEM));
+                context_system::instance());
         $quba->set_preferred_behaviour('deferredfeedback');
         $tf = test_question_maker::make_question('truefalse', 'true');
         $slot = $quba->add_question($tf);
@@ -103,7 +103,7 @@ public function test_extract_responses() {
         // Start a deferred feedback attempt with CBM and add the question to it.
         $tf = test_question_maker::make_question('truefalse', 'true');
         $quba = question_engine::make_questions_usage_by_activity('unit_test',
-                get_context_instance(CONTEXT_SYSTEM));
+                context_system::instance());
         $quba->set_preferred_behaviour('deferredcbm');
         $slot = $quba->add_question($tf);
         $quba->start_all_questions();
@@ -129,7 +129,7 @@ public function test_access_out_of_sequence_throws_exception() {
         // Start a deferred feedback attempt with CBM and add the question to it.
         $tf = test_question_maker::make_question('truefalse', 'true');
         $quba = question_engine::make_questions_usage_by_activity('unit_test',
-                get_context_instance(CONTEXT_SYSTEM));
+                context_system::instance());
         $quba->set_preferred_behaviour('deferredcbm');
         $slot = $quba->add_question($tf);
         $quba->start_all_questions();

diff --git a/question/preview.php b/question/preview.php
@@ -50,11 +50,11 @@
 if ($cmid = optional_param('cmid', 0, PARAM_INT)) {
     $cm = get_coursemodule_from_id(false, $cmid);
     require_login($cm->course, false, $cm);
-    $context = get_context_instance(CONTEXT_MODULE, $cmid);
+    $context = context_module::instance($cmid);
 
 } else if ($courseid = optional_param('courseid', 0, PARAM_INT)) {
     require_login($courseid);
-    $context = get_context_instance(CONTEXT_COURSE, $courseid);
+    $context = context_course::instance($courseid);
 
 } else {
     require_login();

diff --git a/question/question.php b/question/question.php
@@ -100,10 +100,10 @@
 if ($cmid){
     list($module, $cm) = get_module_from_cmid($cmid);
     require_login($cm->course, false, $cm);
-    $thiscontext = get_context_instance(CONTEXT_MODULE, $cmid);
+    $thiscontext = context_module::instance($cmid);
 } elseif ($courseid) {
     require_login($courseid, false);
-    $thiscontext = get_context_instance(CONTEXT_COURSE, $courseid);
+    $thiscontext = context_course::instance($courseid);
     $module = null;
     $cm = null;
 } else {

diff --git a/question/type/questiontypebase.php b/question/type/questiontypebase.php
@@ -1005,7 +1005,7 @@ public function generate_test($name, $courseid=null) {
         $form->penalty = 0.3333333;
         $form->generalfeedback = "Well done";
 
-        $context = get_context_instance(CONTEXT_COURSE, $courseid);
+        $context = context_course::instance($courseid);
         $newcategory = question_make_default_categories(array($context));
         $form->category = $newcategory->id . ',1';
 

diff --git a/report/backups/index.php b/report/backups/index.php
@@ -76,7 +76,7 @@
 
     // Create the row and add it to the table
     $cells = array(
-        format_string($backuprow->fullname, true, array('context' => get_context_instance(CONTEXT_COURSE, $backuprow->courseid))),
+        format_string($backuprow->fullname, true, array('context' => context_course::instance($backuprow->courseid))),
         userdate($backuprow->laststarttime, $strftimedatetime),
         '-',
         userdate($backuprow->lastendtime, $strftimedatetime),

diff --git a/report/completion/index.php b/report/completion/index.php
@@ -507,8 +507,8 @@ function csv_quote($value) {
 
                 // Display icon
                 $iconlink = $CFG->wwwroot.'/course/view.php?id='.$criterion->courseinstance;
-                $icontitle = format_string($crs->fullname, true, array('context' => get_context_instance(CONTEXT_COURSE, $crs->id, MUST_EXIST)));
-                $iconalt = format_string($crs->shortname, true, array('context' => get_context_instance(CONTEXT_COURSE, $crs->id)));
+                $icontitle = format_string($crs->fullname, true, array('context' => context_course::instance($crs->id, MUST_EXIST)));
+                $iconalt = format_string($crs->shortname, true, array('context' => context_course::instance($crs->id)));
                 break;
 
             case COMPLETION_CRITERIA_TYPE_ROLE:

diff --git a/report/completion/user.php b/report/completion/user.php
@@ -175,7 +175,7 @@
 
             // Get course info
             $c_course = $DB->get_record('course', array('id' => $c_info->course_id));
-            $course_context = get_context_instance(CONTEXT_COURSE, $c_course->id, MUST_EXIST);
+            $course_context = context_course::instance($c_course->id, MUST_EXIST);
             $course_name = format_string($c_course->fullname, true, array('context' => $course_context));
 
             // Get completions

diff --git a/report/courseoverview/index.php b/report/courseoverview/index.php
@@ -32,7 +32,7 @@
 $numcourses = optional_param('numcourses', 20, PARAM_INT);
 
 if (empty($CFG->enablestats)) {
-    if (has_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM))) {
+    if (has_capability('moodle/site:config', context_system::instance())) {
         redirect("$CFG->wwwroot/$CFG->admin/settings.php?section=stats", get_string('mustenablestats', 'admin'), 3);
     } else {
         print_error('statsdisable');

diff --git a/report/courseoverview/reportsgraph.php b/report/courseoverview/reportsgraph.php
@@ -33,7 +33,7 @@
 
 require_login();
 
-require_capability('report/courseoverview:view', get_context_instance(CONTEXT_SYSTEM));
+require_capability('report/courseoverview:view', context_system::instance());
 
 stats_check_uptodate();
 

diff --git a/report/log/index.php b/report/log/index.php
@@ -111,7 +111,7 @@
 
 require_login($course);
 
-$context = get_context_instance(CONTEXT_COURSE, $course->id);
+$context = context_course::instance($course->id);
 
 require_capability('report/log:view', $context);
 

diff --git a/report/log/locallib.php b/report/log/locallib.php
@@ -97,12 +97,12 @@ function report_log_print_mnet_selector_form($hostid, $course, $selecteduser=0,
         $showcourses = 1;
     }
 
-    $sitecontext = get_context_instance(CONTEXT_SYSTEM);
+    $sitecontext = context_system::instance();
 
     // Context for remote data is always SITE
     // Groups for remote data are always OFF
     if ($hostid == $CFG->mnet_localhost_id) {
-        $context = get_context_instance(CONTEXT_COURSE, $course->id);
+        $context = context_course::instance($course->id);
 
         /// Setup for group handling.
         if ($course->groupmode == SEPARATEGROUPS and !has_capability('moodle/site:accessallgroups', $context)) {
@@ -397,8 +397,8 @@ function report_log_print_selector_form($course, $selecteduser=0, $selecteddate=
         $showcourses = 1;
     }
 
-    $sitecontext = get_context_instance(CONTEXT_SYSTEM);
-    $context = get_context_instance(CONTEXT_COURSE, $course->id);
+    $sitecontext = context_system::instance();
+    $context = context_course::instance($course->id);
 
     /// Setup for group handling.
     if ($course->groupmode == SEPARATEGROUPS and !has_capability('moodle/site:accessallgroups', $context)) {

diff --git a/report/outline/index.php b/report/outline/index.php
@@ -34,7 +34,7 @@
 $PAGE->set_pagelayout('report');
 
 require_login($course);
-$context = get_context_instance(CONTEXT_COURSE, $course->id);
+$context = context_course::instance($course->id);
 require_capability('report/outline:view', $context);
 
 add_to_log($course->id, 'course', 'report outline', "report/outline/index.php?id=$course->id", $course->id);

diff --git a/report/participation/index.php b/report/participation/index.php
@@ -60,7 +60,7 @@
 }
 
 require_login($course);
-$context = get_context_instance(CONTEXT_COURSE, $course->id);
+$context = context_course::instance($course->id);
 require_capability('report/participation:view', $context);
 
 add_to_log($course->id, "course", "report participation", "report/participation/index.php?id=$course->id", $course->id);

diff --git a/report/security/locallib.php b/report/security/locallib.php
@@ -781,7 +781,7 @@ function report_security_check_riskbackup($detailed=false) {
     $result->status  = null;
     $result->link    = null;
 
-    $syscontext = get_context_instance(CONTEXT_SYSTEM);
+    $syscontext = context_system::instance();
 
     $params = array('capability'=>'moodle/backup:userinfo', 'permission'=>CAP_ALLOW, 'contextid'=>$syscontext->id);
     $sql = "SELECT DISTINCT r.id, r.name, r.shortname, r.sortorder, r.archetype

diff --git a/report/stats/index.php b/report/stats/index.php
@@ -58,7 +58,7 @@
 }
 
 require_login($course);
-$context = get_context_instance(CONTEXT_COURSE, $course->id);
+$context = context_course::instance($course->id);
 require_capability('report/stats:view', $context);
 
 $PAGE->set_url(new moodle_url('/report/stats/index.php', array('course' => $course->id,
@@ -87,7 +87,7 @@
 report_stats_report($course, $report, $mode, $user, $roleid, $time);
 
 if (empty($CFG->enablestats)) {
-    if (has_capability('moodle/site:config', get_context_instance(CONTEXT_SYSTEM))) {
+    if (has_capability('moodle/site:config', context_system::instance())) {
         redirect("$CFG->wwwroot/$CFG->admin/settings.php?section=stats", get_string('mustenablestats', 'admin'), 3);
     } else {
         print_error('statsdisable');

diff --git a/report/stats/locallib.php b/report/stats/locallib.php
@@ -41,7 +41,7 @@ function report_stats_mode_menu($course, $mode, $time, $url) {
     $options = array();
     $options[STATS_MODE_GENERAL] = get_string('statsmodegeneral');
     $options[STATS_MODE_DETAILED] = get_string('statsmodedetailed');
-    if (has_capability('report/stats:view', get_context_instance(CONTEXT_SYSTEM))) {
+    if (has_capability('report/stats:view', context_system::instance())) {
         $options[STATS_MODE_RANKED] = get_string('reports');
     }
     $popupurl = $url."?course=$course->id&time=$time";
@@ -88,7 +88,7 @@ function report_stats_report($course, $report, $mode, $user, $roleid, $time) {
     $courseoptions = array();
 
     foreach ($courses as $c) {
-        $context = get_context_instance(CONTEXT_COURSE, $c->id);
+        $context = context_course::instance($c->id);
 
         if (has_capability('report/stats:view', $context)) {
             $courseoptions[$c->id] = format_string($c->shortname, true, array('context' => $context));
@@ -228,7 +228,7 @@ function report_stats_report($course, $report, $mode, $user, $roleid, $time) {
                         $a[] = $stat->line2;
                     }
                     if (empty($CFG->loglifetime) || ($stat->timeend-(60*60*24)) >= (time()-60*60*24*$CFG->loglifetime)) {
-                        if (has_capability('report/log:view', get_context_instance(CONTEXT_COURSE, $course->id))) {
+                        if (has_capability('report/log:view', context_course::instance($course->id))) {
                             $a[] = '<a href="'.$CFG->wwwroot.'/report/log/index.php?id='.
                                 $course->id.'&amp;chooselog=1&amp;showusers=1&amp;showcourses=1&amp;user='
                                 .$userid.'&amp;date='.usergetmidnight($stat->timeend-(60*60*24)).'">'
@@ -282,7 +282,7 @@ function report_stats_report($course, $report, $mode, $user, $roleid, $time) {
                     krsort($rolesdata);
                     $row = array_merge(array($times[$time]),$rolesdata);
                     if (empty($CFG->loglifetime) || ($stat->timeend-(60*60*24)) >= (time()-60*60*24*$CFG->loglifetime)) {
-                        if (has_capability('report/log:view', get_context_instance(CONTEXT_COURSE, $course->id))) {
+                        if (has_capability('report/log:view', context_course::instance($course->id))) {
                             $row[] = '<a href="'.$CFG->wwwroot.'/report/log/index.php?id='
                                 .$course->id.'&amp;chooselog=1&amp;showusers=1&amp;showcourses=1&amp;user='.$userid
                                 .'&amp;date='.usergetmidnight($time-(60*60*24)).'">'

diff --git a/repository/coursefiles/lib.php b/repository/coursefiles/lib.php
@@ -74,7 +74,7 @@ public function get_listing($encodedpath = '', $page = '') {
             $filepath = null;
             list($context, $course, $cm) = get_context_info_array($this->context->id);
             $courseid = is_object($course) ? $course->id : SITEID;
-            $context = get_context_instance(CONTEXT_COURSE, $courseid);
+            $context = context_course::instance($courseid);
         }
 
         if ($fileinfo = $browser->get_file_info($context, $component, $filearea, $itemid, $filepath, $filename)) {

diff --git a/repository/draftfiles_ajax.php b/repository/draftfiles_ajax.php
@@ -41,7 +41,7 @@
 $draftid = required_param('itemid', PARAM_INT);
 $filepath = optional_param('filepath', '/', PARAM_PATH);
 
-$user_context = get_context_instance(CONTEXT_USER, $USER->id);
+$user_context = context_user::instance($USER->id);
 
 echo $OUTPUT->header(); // send headers
 

diff --git a/repository/draftfiles_manager.php b/repository/draftfiles_manager.php
@@ -63,7 +63,7 @@
 $draftpath   = optional_param('draftpath', '/',    PARAM_PATH);
 
 // user context
-$user_context = get_context_instance(CONTEXT_USER, $USER->id);
+$user_context = context_user::instance($USER->id);
 
 
 $PAGE->set_context($user_context);

diff --git a/repository/lib.php b/repository/lib.php
@@ -642,7 +642,7 @@ public final function check_capability() {
     public static function draftfile_exists($itemid, $filepath, $filename) {
         global $USER;
         $fs = get_file_storage();
-        $usercontext = get_context_instance(CONTEXT_USER, $USER->id);
+        $usercontext = context_user::instance($USER->id);
         if ($fs->get_file($usercontext->id, 'user', 'draft', $itemid, $filepath, $filename)) {
             return true;
         } else {
@@ -2200,7 +2200,7 @@ public function get_short_filename($str, $maxlength) {
     public static function overwrite_existing_draftfile($itemid, $filepath, $filename, $newfilepath, $newfilename) {
         global $USER;
         $fs = get_file_storage();
-        $user_context = get_context_instance(CONTEXT_USER, $USER->id);
+        $user_context = context_user::instance($USER->id);
         if ($file = $fs->get_file($user_context->id, 'user', 'draft', $itemid, $filepath, $filename)) {
             if ($tempfile = $fs->get_file($user_context->id, 'user', 'draft', $itemid, $newfilepath, $newfilename)) {
                 // delete existing file to release filename
@@ -2226,7 +2226,7 @@ public static function overwrite_existing_draftfile($itemid, $filepath, $filenam
     public static function delete_tempfile_from_draft($draftitemid, $filepath, $filename) {
         global $USER;
         $fs = get_file_storage();
-        $user_context = get_context_instance(CONTEXT_USER, $USER->id);
+        $user_context = context_user::instance($USER->id);
         if ($file = $fs->get_file($user_context->id, 'user', 'draft', $draftitemid, $filepath, $filename)) {
             $file->delete();
             return true;
@@ -2643,13 +2643,13 @@ function initialise_filepicker($args) {
         $disable_types = $args->disable_types;
     }
 
-    $user_context = get_context_instance(CONTEXT_USER, $USER->id);
+    $user_context = context_user::instance($USER->id);
 
     list($context, $course, $cm) = get_context_info_array($context->id);
     $contexts = array($user_context, get_system_context());
     if (!empty($course)) {
         // adding course context
-        $contexts[] = get_context_instance(CONTEXT_COURSE, $course->id);
+        $contexts[] = context_course::instance($course->id);
     }
     $externallink = (int)get_config(null, 'repositoryallowexternallinks');
     $repositories = repository::get_instances(array(