Bugfixes for my recent checkin

diff --git a/lib/moodlelib.php b/lib/moodlelib.php
index 686265a3e1bb163d0d87821df61d9646c0596df9..5dc48ab11a0796136abdd98482c65f3c38e58faa 100644 (file)
--- a/lib/moodlelib.php
+++ b/lib/moodlelib.php
@@ -152,7 +152,7 @@ function reload_user_preferences() {
  * @todo Add a better description and include usage examples.
  * @param string $name The key to set as preference for the specified user
  * @param string $value The value to set forthe $name key in the specified user's record
- * @param object $user A moodle user object
+ * @param integer $userid A moodle user ID
  * @todo Add inline links to $USER and user functions in above line.
  * @return boolean
  */
@@ -189,9 +189,30 @@ function set_user_preference($name, $value, $userid=NULL) {
     }
 }
 
+/**
+ * Unsets a preference completely by deleting it from the database
+ * Optionally, can set a preference for a different user id
+ * @uses $USER
+ * @param string  $name The key to unset as preference for the specified user
+ * @param integer $userid A moodle user ID
+ * @return boolean
+ */
+function unset_user_preference($name, $userid=NULL) {
+
+    global $USER;
+
+    if (empty($userid)){ 
+        $userid = $USER->id;
+    }
+
+    return delete_records('user_preferences', 'userid', $userid, 'name', $name);
+}
+
+
 /**
  * Sets a whole array of preferences for the current user
  * @param array $prefarray An array of key/value pairs to be set
+ * @param integer $userid A moodle user ID
  * @return boolean
  */
 function set_user_preferences($prefarray, $userid=NULL) {
@@ -225,6 +246,7 @@ function set_user_preferences($prefarray, $userid=NULL) {
  * otherwise NULL.
  * @param string $name Name of the key to use in finding a preference value
  * @param string $default Value to be returned if the $name key is not set in the user preferences
+ * @param integer $userid A moodle user ID
  * @uses $USER
  * @return string
  */

diff --git a/login/change_password.php b/login/change_password.php
index 8d6c10ec061dcc39a40cc10b8aa7739831ddb4a6..1559afd660a7eab48fc9f0ae017a4558e61047b5 100644 (file)
--- a/login/change_password.php
+++ b/login/change_password.php
@@ -55,7 +55,7 @@
             }
             
             // register success changing password
-            set_user_preference('auth_forcepasswordchange', false);
+            unset_user_preference('auth_forcepasswordchange');
             
             $USER = $user;
             $USER->loggedin = true;

diff --git a/user/edit.html b/user/edit.html
index 32c9b6e165d881596b65a3fb175f8998046cbccd..5c88cbb9234ce66fa4735561faaf903bdd107f8f 100644 (file)
--- a/user/edit.html
+++ b/user/edit.html
@@ -66,7 +66,7 @@ if (isadmin()) {
              }
              echo "<tr>";
              echo '<td align=right><p>'.get_string('forcepasswordchange').':</td>';
-             echo '<td><input type="checkbox" name="forcepasswordchange "'.$checked.' /> '.get_string('forcepasswordchangehelp').'</td>';            
+             echo '<td><input type="checkbox" name="forcepasswordchange" '.$checked.' /> '.get_string('forcepasswordchangehelp').'</td>';            
              echo "</tr>";
         }
                  

diff --git a/user/edit.php b/user/edit.php
index d2b66b0027b49fd0d791534240bf3cd9560650f2..9a302b276e825e5955154595e6639006980fa25a 100644 (file)
--- a/user/edit.php
+++ b/user/edit.php
@@ -90,7 +90,6 @@
             $user = $usernew;
 
         } else {
-        $db->debug = true;
             $timenow = time();
 
             if (!$usernew->picture = save_profile_image($user->id,$um,'users')) {
@@ -120,10 +119,10 @@
                     }
                 }
                 // store forcepasswordchange in user's preferences
-                if (isset($usernew->forcepasswordchange)){
+                if (!empty($usernew->forcepasswordchange)){
                     set_user_preference('auth_forcepasswordchange', 1, $user->id);
                 } else {
-                    set_user_preference('auth_forcepasswordchange', 0, $user->id);
+                    unset_user_preference('auth_forcepasswordchange', $user->id);
                 }
             } else {
                 if (isset($usernew->newpassword)) {