diff options
author | Dan McGee <dan@archlinux.org> | 2011-03-01 12:24:34 -0600 |
---|---|---|
committer | Lukas Fleischer <archlinux@cryptocrack.de> | 2011-03-04 10:29:17 +0100 |
commit | cdc01130cf03ceac98626e9bedc369da3f0fd2c7 (patch) | |
tree | 516c0b5e2daff6e43a6ac9706a162c0ac239d9fe | |
parent | ddbe6d0d335ba7bc7170ad234667ca62f638bd3b (diff) |
Fix PHP notices in account pages
Signed-off-by: Dan McGee <dan@archlinux.org>
Signed-off-by: Lukas Fleischer <archlinux@cryptocrack.de>
-rw-r--r-- | web/html/account.php | 38 | ||||
-rw-r--r-- | web/lib/acctfuncs.inc | 8 |
2 files changed, 28 insertions, 18 deletions
diff --git a/web/html/account.php b/web/html/account.php index e8a3218..47b7417 100644 --- a/web/html/account.php +++ b/web/html/account.php @@ -16,22 +16,24 @@ echo "<div class=\"pgbox\">\n"; echo " <div class=\"pgboxtitle\"><span class=\"f3\">".__("Accounts")."</span></div>\n"; echo " <div class=\"pgboxbody\">\n"; +$action = in_request("Action"); + if (isset($_COOKIE["AURSID"])) { # visitor is logged in # $dbh = db_connect(); $atype = account_from_sid($_COOKIE["AURSID"]); - if ($_REQUEST["Action"] == "SearchAccounts") { + if ($action == "SearchAccounts") { # security check # if ($atype == "Trusted User" || $atype == "Developer") { # the user has entered search criteria, find any matching accounts # - search_results_page($atype, $_REQUEST["O"], $_REQUEST["SB"], - $_REQUEST["U"], $_REQUEST["T"], $_REQUEST["S"], - $_REQUEST["E"], $_REQUEST["R"], $_REQUEST["I"]); + search_results_page($atype, in_request("O"), in_request("SB"), + in_request("U"), in_request("T"), in_request("S"), + in_request("E"), in_request("R"), in_request("I")); } else { # a non-privileged user is trying to access the search page @@ -39,13 +41,13 @@ if (isset($_COOKIE["AURSID"])) { print __("You are not allowed to access this area.")."<br />\n"; } - } elseif ($_REQUEST["Action"] == "DisplayAccount") { + } elseif ($action == "DisplayAccount") { # the user has clicked 'edit', display the account details in a form # $q = "SELECT Users.*, AccountTypes.AccountType "; $q.= "FROM Users, AccountTypes "; $q.= "WHERE AccountTypes.ID = Users.AccountTypeID "; - $q.= "AND Users.ID = ".intval($_REQUEST["ID"]); + $q.= "AND Users.ID = ".intval(in_request("ID")); $result = db_query($q, $dbh); if (!mysql_num_rows($result)) { print __("Could not retrieve information for the specified user."); @@ -66,13 +68,13 @@ if (isset($_COOKIE["AURSID"])) { } } - } elseif ($_REQUEST["Action"] == "AccountInfo") { + } elseif ($action == "AccountInfo") { # no editing, just looking up user info # $q = "SELECT Users.*, AccountTypes.AccountType "; $q.= "FROM Users, AccountTypes "; $q.= "WHERE AccountTypes.ID = Users.AccountTypeID "; - $q.= "AND Users.ID = ".intval($_REQUEST["ID"]); + $q.= "AND Users.ID = ".intval(in_request("ID")); $result = db_query($q, $dbh); if (!mysql_num_rows($result)) { print __("Could not retrieve information for the specified user."); @@ -83,14 +85,14 @@ if (isset($_COOKIE["AURSID"])) { $row["IRCNick"]); } - } elseif ($_REQUEST["Action"] == "UpdateAccount") { + } elseif ($action == "UpdateAccount") { # user is submitting their modifications to an existing account # process_account_form($atype, "edit", "UpdateAccount", - $_REQUEST["U"], $_REQUEST["T"], $_REQUEST["S"], - $_REQUEST["E"], $_REQUEST["P"], $_REQUEST["C"], - $_REQUEST["R"], $_REQUEST["L"], $_REQUEST["I"], - $_REQUEST["N"], $_REQUEST["ID"]); + in_request("U"), in_request("T"), in_request("S"), + in_request("E"), in_request("P"), in_request("C"), + in_request("R"), in_request("L"), in_request("I"), + in_request("N"), in_request("ID")); } else { @@ -133,15 +135,15 @@ if (isset($_COOKIE["AURSID"])) { } else { # visitor is not logged in # - if ($_REQUEST["Action"] == "AccountInfo") { + if ($action == "AccountInfo") { print __("You must log in to view user information."); - } elseif ($_REQUEST["Action"] == "NewAccount") { + } elseif ($action == "NewAccount") { # process the form input for creating a new account # process_account_form("","new", "NewAccount", - $_REQUEST["U"], 1, 0, $_REQUEST["E"], - $_REQUEST["P"], $_REQUEST["C"], $_REQUEST["R"], - $_REQUEST["L"], $_REQUEST["I"], $_REQUEST["N"]); + in_request("U"), 1, 0, in_request("E"), + in_request("P"), in_request("C"), in_request("R"), + in_request("L"), in_request("I"), in_request("N")); } else { # display the account request form diff --git a/web/lib/acctfuncs.inc b/web/lib/acctfuncs.inc index 5dfe50e..d5137be 100644 --- a/web/lib/acctfuncs.inc +++ b/web/lib/acctfuncs.inc @@ -1,5 +1,13 @@ <?php +# Helper function- retrieve request param if available, "" otherwise +function in_request($name) { + if (isset($_REQUEST[$name])) { + return $_REQUEST[$name]; + } + return ""; +} + # Display the standard Account form, pass in default values if any function display_account_form($UTYPE,$A,$U="",$T="",$S="", |