From 9b3cd99736ee9f8082af4398ae64dd0a132d7347 Mon Sep 17 00:00:00 2001
From: Dries Buytaert <dries@buytaert.net>
Date: Sun, 18 Feb 2001 17:05:57 +0000
Subject: [PATCH] - 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)
---
modules/account.module | 14 ++++++++++----
1 file changed, 10 insertions(+), 4 deletions(-)
diff --git a/modules/account.module b/modules/account.module
index c92cbfbd55be..88a2684269db 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'");
}
--
GitLab