- fixed 2 bugs and 1 confusing issue in modules/account.module:

    * last access field should not be updated when admin edits stuff
    * saving empty access list caused warning message
    * clicking the access links was confusing (no more links)

1 files changed, 10 insertions, 4 deletions

diff --git a/modules/account.module b/modules/account.module
index c92cbfbd5..88a268426 100644
--- a/modules/account.module
+++ b/modules/account.module
@@ -143,7 +143,7 @@ function account_access($account) {
   $data = explode(";", $account->access);
   foreach ($data as $array) {
     $access = explode(":", $array);
-    if ($access[0]) $output .= " <A HREF=\"admin.php?mod=$access[0]\">$access[0]</A>";
+    if ($access[0]) $output .= " $access[0]";
   }
   return $output;
 }
@@ -173,9 +173,15 @@ function account_comments($id) {
 }
 
 function account_edit_save($name, $edit) {
-  foreach ($edit as $key=>$value) if ($key != "access") $query .= "$key = '". addslashes($value) ."', ";
-  db_query("UPDATE users SET $query last_access = '". time() ."' WHERE userid = '$name'");
-  foreach ($edit[access] as $key=>$value) user_set(user_load($name), "access", $value, 1);
+  foreach ($edit as $key=>$value) {
+    if ($key != "access") $query .= "$key = '". addslashes($value) ."', ";
+  }
+
+  db_query("UPDATE users SET $query access = '' WHERE userid = '$name'");
+
+  if ($edit["access"]) {
+    foreach ($edit["access"] as $key=>$value) user_set(user_load($name), "access", $value, 1);
+  }
 
   watchdog("message", "account: modified user '$name'");
 }