X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmodules%2Fadmin%2Fwhat-add_points.php;h=08019d932c4d8cf670308cfff4afb3da7041be54;hp=3232be3b56cd07499354588eff721aa88ca1444c;hb=d8148e3f1f3a6762b2e786dbe99ada269dcf2ea0;hpb=56156f6c4392510cdbe0eb4f2ccefc23b43e2672 diff --git a/inc/modules/admin/what-add_points.php b/inc/modules/admin/what-add_points.php index 3232be3b56..08019d932c 100644 --- a/inc/modules/admin/what-add_points.php +++ b/inc/modules/admin/what-add_points.php @@ -41,13 +41,13 @@ if ((!defined('__SECURITY')) || (!IS_ADMIN())) { ADD_DESCR("admin", __FILE__); // Fix a notice -if (!isset($_GET['u_id'])) $_GET['u_id'] = ""; +if (!REQUEST_ISSET_GET(('uid'))) REQUEST_SET_GET('uid', ""); -if ($_GET['u_id'] == "all") { +if (REQUEST_GET('uid') == "all") { // Add points to all accounts - if ((isset($_POST['ok'])) && ($_POST['points'] > 0)) { - define('__POINTS_VALUE', $_POST['points']); - $result_main = SQL_QUERY("SELECT userid FROM `"._MYSQL_PREFIX."_user_data` WHERE status='CONFIRMED' ORDER BY userid", __FILE__, __LINE__); + if ((IS_FORM_SENT()) && (REQUEST_POST('points') > 0)) { + define('__POINTS_VALUE', REQUEST_POST('points')); + $result_main = SQL_QUERY("SELECT userid FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `status`='CONFIRMED' ORDER BY userid", __FILE__, __LINE__); while (list($uid) = SQL_FETCHROW($result_main)) { // Remove depth to prevent booking errors. This is a bad coding // practice, thats also why we need to write this project from @@ -55,66 +55,66 @@ if ($_GET['u_id'] == "all") { $GLOBALS['ref_level'] = -1; // Ok, add points and send an email to him... - ADD_POINTS_REFSYSTEM("admin_all", $uid, bigintval($_POST['points']), false, "0", false, "direct"); + ADD_POINTS_REFSYSTEM("admin_all", $uid, bigintval(REQUEST_POST('points')), false, "0", false, "direct"); // Prepare content $content = array( - 'text' => SQL_ESCAPE($_POST['reason']), - 'points' => bigintval($_POST['points']) + 'text' => SQL_ESCAPE(REQUEST_POST('reason')), + 'points' => bigintval(REQUEST_POST('points')) ); // Load email template and send email away $msg = LOAD_EMAIL_TEMPLATE("add-points", $content, bigintval($uid)); - SEND_EMAIL(bigintval($uid), ADMIN_ADD_SUBJ, $msg); + SEND_EMAIL(bigintval($uid), getMessage('ADMIN_ADD_SUBJ'), $msg); } // END - while // Free memory SQL_FREERESULT($result_main); // Output message - LOAD_TEMPLATE("admin_settings_saved", false, ADMIN_ALL_POINTS_ADDED); + LOAD_TEMPLATE("admin_settings_saved", false, getMessage('ADMIN_ALL_POINTS_ADDED')); } else { // Display form add points LOAD_TEMPLATE("admin_add_points_all"); } -} elseif (!empty($_GET['u_id'])) { +} elseif (REQUEST_ISSET_GET(('uid'))) { // User ID found in URL so we use this give him some credits - $result = SQL_QUERY_ESC("SELECT surname, family, email FROM `"._MYSQL_PREFIX."_user_data` WHERE userid=%s AND status='CONFIRMED' LIMIT 1", - array(bigintval($_GET['u_id'])), __FILE__, __LINE__); + $result = SQL_QUERY_ESC("SELECT surname, family, email FROM `{!_MYSQL_PREFIX!}_user_data` WHERE userid=%s AND `status`='CONFIRMED' LIMIT 1", + array(bigintval(REQUEST_GET('uid'))), __FILE__, __LINE__); if (SQL_NUMROWS($result) == 1) { // Selected user does exist list($sname, $fname, $email) = SQL_FETCHROW($result); SQL_FREERESULT($result); - if ((isset($_POST['ok'])) && (!empty($_POST['points']))) { + if ((IS_FORM_SENT()) && (REQUEST_ISSET_POST(('points')))) { // Remove depth to prevent booking errors. This is a bad coding // practice, thats also why we need to write this project from // scratch... unset($GLOBALS['ref_level']); // Ok, add points and send an email to him... - ADD_POINTS_REFSYSTEM("admin_single", bigintval($_GET['u_id']), bigintval($_POST['points']), false, "0", false, "direct"); + ADD_POINTS_REFSYSTEM("admin_single", bigintval(REQUEST_GET('uid')), bigintval(REQUEST_POST('points')), false, "0", false, "direct"); // Prepare content $content = array( - 'text' => SQL_ESCAPE($_POST['reason']), - 'points' => bigintval($_POST['points']) + 'text' => SQL_ESCAPE(REQUEST_POST('reason')), + 'points' => bigintval(REQUEST_POST('points')) ); // Message laden - $msg = LOAD_EMAIL_TEMPLATE("add-points", $content, bigintval($_GET['u_id'])); + $msg = LOAD_EMAIL_TEMPLATE("add-points", $content, bigintval(REQUEST_GET('uid'))); - SEND_EMAIL(bigintval($_GET['u_id']), ADMIN_ADD_SUBJ, $msg); - LOAD_TEMPLATE("admin_settings_saved", false, ADMIN_POINTS_ADDED); + SEND_EMAIL(bigintval(REQUEST_GET('uid')), getMessage('ADMIN_ADD_SUBJ'), $msg); + LOAD_TEMPLATE("admin_settings_saved", false, getMessage('ADMIN_POINTS_ADDED')); } else { // Opps, missing form here define('__USER_VALUE', "".$sname." ".$fname.""); - define('__UID' , bigintval($_GET['u_id'])); + define('__UID' , bigintval(REQUEST_GET('uid'))); LOAD_TEMPLATE("admin_add_points"); } } else { // User not found! - LOAD_TEMPLATE("admin_settings_saved", false, "
".ADMIN_MEMBER_404_1.$_GET['u_id'].ADMIN_MEMBER_404_2."
"); + LOAD_TEMPLATE("admin_settings_saved", false, "
".sprintf(getMessage('ADMIN_MEMBER_404'), REQUEST_GET('uid'))."
"); } } else { // Output selection form with all confirmed user accounts listed