X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmodules%2Fadmin%2Fwhat-add_points.php;h=5d31043b18b6b9e079c97bedc325903a37138a22;hp=442720021aac27d5c4bca25476b6e6a37fee5956;hb=0715fa7aa8e5e70bcf1d957fb09ae655c3896c4e;hpb=5071030af40e69ca4284642f44758964e18f5be8 diff --git a/inc/modules/admin/what-add_points.php b/inc/modules/admin/what-add_points.php index 442720021a..5d31043b18 100644 --- a/inc/modules/admin/what-add_points.php +++ b/inc/modules/admin/what-add_points.php @@ -1,7 +1,7 @@ 0)) { - // @TODO Rewrite this constant - define('__POINTS_VALUE', REQUEST_POST('points')); + if ((isFormSent()) && (postRequestParameter('points') > 0)) { + // Points + $content['points'] = bigintval(postRequestParameter('points')); // Select all users - $result_main = SQL_QUERY("SELECT userid FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `status`='CONFIRMED' ORDER BY `userid` ASC", - __FILE__, __LINE__); + $result_main = SQL_QUERY("SELECT `userid` FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `status`='CONFIRMED' ORDER BY `userid` ASC", + __FILE__, __LINE__); // Process all entries - while ($content = SQL_FETCHARRAY($result_main)) { + while ($row = SQL_FETCHARRAY($result_main)) { + // Merge both arrays + $content = merge_array($content, $row); + // Remove depth to prevent booking errors. This is a bad coding // practice, thats also why we need to write this project from // scratch... $GLOBALS['ref_level'] = -1; // Ok, add points and send an email to him... - ADD_POINTS_REFSYSTEM_DIRECT('admin_all', $content['userid'], bigintval(REQUEST_POST('points'))); + addPointsDirectly('admin_all', $content['userid'], bigintval(postRequestParameter('points'))); // Prepare content - $content['text'] = SQL_ESCAPE(REQUEST_POST('reason')); - $content['points'] = bigintval(REQUEST_POST('points')); + $content['text'] = SQL_ESCAPE(postRequestParameter('reason')); + $content['points'] = bigintval(postRequestParameter('points')); // Load email template and send email away - $msg = LOAD_EMAIL_TEMPLATE("add-points", $content, bigintval($content['userid'])); - sendEmail(bigintval($content['userid']), getMessage('ADMIN_ADD_SUBJ'), $msg); + $message = loadEmailTemplate('add-points', $content, bigintval($content['userid'])); + sendEmail(bigintval($content['userid']), getMessage('ADMIN_ADD_SUBJ'), $message); } // END - while // Free memory SQL_FREERESULT($result_main); // Output message - LOAD_TEMPLATE('admin_settings_saved', false, getMessage('ADMIN_ALL_POINTS_ADDED')); + loadTemplate('admin_settings_saved', false, getMessage('ADMIN_ALL_POINTS_ADDED')); } else { // Display form add points - LOAD_TEMPLATE("admin_add_points_all"); + loadTemplate('admin_add_points_all'); } -} 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(REQUEST_GET('uid'))), __FILE__, __LINE__); - if (SQL_NUMROWS($result) == 1) { +} elseif (isGetRequestParameterSet('userid')) { + // User id found in URL so we use this give him some credits + if (fetchUserData(getRequestParameter('userid'))) { // Selected user does exist - list($sname, $fname, $email) = SQL_FETCHROW($result); - SQL_FREERESULT($result); - - 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']); + $content = getUserDataArray(); - // Ok, add points and send an email to him... - ADD_POINTS_REFSYSTEM_DIRECT('admin_single', bigintval(REQUEST_GET('uid')), bigintval(REQUEST_POST('points'))); + if ((isFormSent()) && (isPostRequestParameterSet('points'))) { + // Add points and send an email to him... + addPointsDirectly('admin_single', bigintval(getRequestParameter('userid')), bigintval(postRequestParameter('points'))); // Prepare content $content = array( - 'text' => SQL_ESCAPE(REQUEST_POST('reason')), - 'points' => bigintval(REQUEST_POST('points')) + 'text' => SQL_ESCAPE(postRequestParameter('reason')), + 'points' => bigintval(postRequestParameter('points')) ); // Message laden - $msg = LOAD_EMAIL_TEMPLATE("add-points", $content, bigintval(REQUEST_GET('uid'))); + $message = loadEmailTemplate('add-points', $content, bigintval(getRequestParameter('userid'))); - sendEmail(bigintval(REQUEST_GET('uid')), getMessage('ADMIN_ADD_SUBJ'), $msg); - LOAD_TEMPLATE('admin_settings_saved', false, getMessage('ADMIN_POINTS_ADDED')); + sendEmail(bigintval(getRequestParameter('userid')), getMessage('ADMIN_ADD_SUBJ'), $message); + loadTemplate('admin_settings_saved', false, getMessage('ADMIN_POINTS_ADDED')); } else { - // Opps, missing form here - // @TODO Rewrite these both constants - define('__USER_VALUE', "".$sname." ".$fname.""); - define('__UID' , bigintval(REQUEST_GET('uid'))); - LOAD_TEMPLATE("admin_add_points"); + // Prepare content + $content['user'] = '' . $content['surname'] . ' ' . $content['family'] . ''; + $content['userid'] = bigintval(getRequestParameter('userid')); + + // Output template + loadTemplate('admin_add_points', false, $content); } } else { // User not found! - LOAD_TEMPLATE('admin_settings_saved', false, "
".sprintf(getMessage('ADMIN_MEMBER_404'), REQUEST_GET('uid'))."
"); + loadTemplate('admin_settings_saved', false, '
' . getMaskedMessage('ADMIN_MEMBER_404', bigintval(getRequestParameter('userid'))) . '
'); } } else { // Output selection form with all confirmed user accounts listed - ADD_MEMBER_SELECTION_BOX('0', true); + addMemberSelectionBox(0, true); } -// +// [EOF] ?>