MDL-8765 user preference usage cleanup

diff --git a/blocks/admin_bookmarks/block_admin_bookmarks.php b/blocks/admin_bookmarks/block_admin_bookmarks.php
index e177b7c1dd7320e852787124897006d1b5b51f69..ee427de847b8daf4cf72f35a1a2b72e64b46e2fd 100644 (file)
--- a/blocks/admin_bookmarks/block_admin_bookmarks.php
+++ b/blocks/admin_bookmarks/block_admin_bookmarks.php
@@ -36,8 +36,8 @@ class block_admin_bookmarks extends block_base {
 
         $this->content = new stdClass;
         $this->content->text = '';
-        if (isset($USER->preference['admin_bookmarks'])) {
-            $bookmarks = explode(',',$USER->preference['admin_bookmarks']);
+        if (get_user_preferences('admin_bookmarks')) {
+            $bookmarks = explode(',',get_user_preferences('admin_bookmarks'));
             // hmm... just a liiitle (potentially) processor-intensive
             // (recall that $adminroot->locate is a huge recursive call... and we're calling it repeatedly here
 

diff --git a/blocks/admin_bookmarks/create.php b/blocks/admin_bookmarks/create.php
index 1f365cea43e2f0311af67ad74b49f25adf76ab29..cf44ab0f89cab577eef727b277907b94fb85b80e 100644 (file)
--- a/blocks/admin_bookmarks/create.php
+++ b/blocks/admin_bookmarks/create.php
@@ -8,8 +8,8 @@ require_login();
 
 if ($section = optional_param('section', '', PARAM_ALPHAEXT) and confirm_sesskey()) {
 
-    if (isset($USER->preference['admin_bookmarks'])) {
-        $bookmarks = explode(',',$USER->preference['admin_bookmarks']);
+    if (get_user_preferences('admin_bookmarks')) {
+        $bookmarks = explode(',',get_user_preferences('admin_bookmarks'));
 
         if (in_array($section, $bookmarks)) {
             error(get_string('bookmarkalreadyexists','admin'));
@@ -39,11 +39,11 @@ if ($section = optional_param('section', '', PARAM_ALPHAEXT) and confirm_sesskey
 
     if (is_a($temp, 'admin_settingpage')) {
 
-        redirect($CFG->wwwroot . '/' . $CFG->admin . '/settings.php?section=' . $section, get_string('bookmarkadded','admin'), 1);
+        redirect($CFG->wwwroot . '/' . $CFG->admin . '/settings.php?section=' . $section);
 
     } elseif (is_a($temp, 'admin_externalpage')) {
 
-        redirect($temp->url, get_string('bookmarkadded','admin'), 1);
+        redirect($temp->url);
 
     }
 

diff --git a/blocks/admin_bookmarks/delete.php b/blocks/admin_bookmarks/delete.php
index 63875bff0f7e21ca0e9c7bdfb9fd437f70d78cb9..a73a39aee1a99d1227932a9bbf1eb7d000d45989 100644 (file)
--- a/blocks/admin_bookmarks/delete.php
+++ b/blocks/admin_bookmarks/delete.php
@@ -8,9 +8,9 @@ require_login();
 
 if ($section = optional_param('section', '', PARAM_ALPHAEXT) and confirm_sesskey()) {
 
-    if (isset($USER->preference['admin_bookmarks'])) {
+    if (get_user_preferences('admin_bookmarks')) {
 
-        $bookmarks = explode(',', $USER->preference['admin_bookmarks']);
+        $bookmarks = explode(',', get_user_preferences('admin_bookmarks'));
 
         $key = array_search($section, $bookmarks);
 
@@ -28,9 +28,9 @@ if ($section = optional_param('section', '', PARAM_ALPHAEXT) and confirm_sesskey
         if (is_a($temp, 'admin_externalpage')) {
             redirect($temp->url, get_string('bookmarkdeleted','admin'));
         } elseif (is_a($temp, 'admin_settingpage')) {
-            redirect($CFG->wwwroot . '/' . $CFG->admin . '/settings.php?section=' . $section, get_string('bookmarkdeleted','admin'), 1);
+            redirect($CFG->wwwroot . '/' . $CFG->admin . '/settings.php?section=' . $section);
         } else {
-            redirect($CFG->wwwroot, get_string('bookmarkdeleted', 'admin'),1);
+            redirect($CFG->wwwroot);
         }
         die;
 

diff --git a/login/change_password.php b/login/change_password.php
index e7f78e6a58669ec8dab7f5d9e5e4627b78c7e2f5..7bf7c6b5974393b6dae9fdc340cb1468eedd99c2 100644 (file)
--- a/login/change_password.php
+++ b/login/change_password.php
@@ -110,7 +110,7 @@
 
 
     print_header($strchangepassword, $strchangepassword, $navstr);
-    if (!empty($USER->preference['auth_forcepasswordchange'])) {
+    if (get_user_preferences('auth_forcepasswordchange')) {
         notify(get_string('forcepasswordchangenotice'));
     }
     $mform->display();

diff --git a/login/change_password_form.php b/login/change_password_form.php
index db04a154f254f69ce8edb4f6d3acdfb7d05bd526..92952f113686a479eccea3f40aa0489fda03ef72 100644 (file)
--- a/login/change_password_form.php
+++ b/login/change_password_form.php
@@ -45,10 +45,10 @@ class login_change_password_form extends moodleform {
         $mform->setType('id', PARAM_INT);
 
         // buttons
-        if (empty($USER->preference['auth_forcepasswordchange'])) {
-            $this->add_action_buttons(true);
-        } else {
+        if (get_user_preferences('auth_forcepasswordchange')) {
             $this->add_action_buttons(false);
+        } else {
+            $this->add_action_buttons(true);
         }
     }
 

diff --git a/mod/forum/discuss.php b/mod/forum/discuss.php
index 209c41735ee368b834d36fe350c43e355795ceba..369a5c0b92e6049f4525abd0d57ed620400ae5dc 100644 (file)
--- a/mod/forum/discuss.php
+++ b/mod/forum/discuss.php
@@ -98,11 +98,7 @@
     unset($SESSION->fromdiscussion);
 
     if ($mode) {
-        if (isguest()) {
-            $USER->preference['forum_displaymode'] = $mode;  // don't save it in database
-        } else {
-            set_user_preference('forum_displaymode', $mode);
-        }
+        set_user_preference('forum_displaymode', $mode);
     }
 
     $displaymode = get_user_preferences('forum_displaymode', $CFG->forum_displaymode);