X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fadmin%2Fwhat-sub_points.php;h=94f4f9ded805bb57164edfe8f720aa0cb8ef9fd5;hb=1355d2c0b29510dbd407f9cde9f41a5ac02d01fa;hp=4a2f0269567bffa248738897bbb61c5fb0524845;hpb=5bdeaf8b452206598b6c6cd4f941145b11a0eccc;p=mailer.git

diff --git a/inc/modules/admin/what-sub_points.php b/inc/modules/admin/what-sub_points.php
index 4a2f026956..94f4f9ded8 100644
--- a/inc/modules/admin/what-sub_points.php
+++ b/inc/modules/admin/what-sub_points.php
@@ -10,7 +10,12 @@
  * -------------------------------------------------------------------- *
  * Kurzbeschreibung  : Manuell einem Mitglied Punkte gutschreiben       *
  * -------------------------------------------------------------------- *
- *                                                                      *
+ * $Revision::                                                        $ *
+ * $Date::                                                            $ *
+ * $Tag:: 0.2.1-FINAL                                                 $ *
+ * $Author::                                                          $ *
+ * Needs to be in all Files and every File needs "svn propset           *
+ * svn:keywords Date Revision" (autoprobset!) at least!!!!!!            *
  * -------------------------------------------------------------------- *
  * Copyright (c) 2003 - 2008 by Roland Haeder                           *
  * For more information visit: http://www.mxchange.org                  *
@@ -33,97 +38,87 @@
 
 // Some security stuff...
 if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
-	$INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), "/inc") + 4) . "/security.php";
+	$INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), '/inc') + 4) . '/security.php';
 	require($INC);
 }
 
 // Add description as navigation point
-ADD_DESCR("admin", __FILE__);
+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
-	define('__POINTS_VALUE', $_POST['points']);
-	if ((isset($_POST['ok'])) && ($_POST['points'] > 0)) {
-		$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)) {
-			// 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($uid)), __FILE__, __LINE__);
-
-			// Entry found?
-			if (SQL_NUMROWS($result) == 1) {
-				// Selected user does exist
-				list($sname, $fname, $email) = SQL_FETCHROW($result);
-
-				// Free result
-				SQL_FREERESULT($result);
-
-				if ((isset($_POST['ok'])) && (!empty($_POST['points']))) {
-					// Ok, add points to used points and send an email to him...
-					SUB_POINTS("admin_all", $uid, $_POST['points']);
-
-					// Prepare content
-					$content = array(
-						'text'   => SQL_ESCAPE($_POST['reason']),
-						'points' => bigintval($_POST['points'])
-					);
-
-					// Load message and send it away
-					$msg = LOAD_EMAIL_TEMPLATE("sub-points", $content, bigintval($uid));
-					SEND_EMAIL(bigintval($uid), getMessage('ADMIN_SUB_SUBJ'), $msg);
-				}
-			}
-		}
+	// @TODO Rewrite this constant
+	define('__POINTS_VALUE', REQUEST_POST('points'));
+
+	// Is the form sent?
+	if ((isFormSent()) && (REQUEST_POST('points') > 0)) {
+		$result_main = SQL_QUERY("SELECT userid, email FROM `{!_MYSQL_PREFIX!}_user_data` WHERE `status`='CONFIRMED' ORDER BY `userid` ASC",
+		__FILE__, __LINE__);
+		while ($content = SQL_FETCHARRAY($result_main)) {
+			// Ok, add points to used points and send an email to him...
+			SUB_POINTS("admin_all", $content['userid'], REQUEST_POST('points'));
+
+			// Add more content
+			$content['text']   = SQL_ESCAPE(REQUEST_POST('reason'));
+			$content['points'] = bigintval(REQUEST_POST('points'));
+
+			// Load message and send it away
+			$msg = LOAD_EMAIL_TEMPLATE("sub-points", $content, bigintval($content['userid']));
+			sendEmail($content['email'], getMessage('ADMIN_SUB_SUBJ'), $msg);
+		} // END - while
+
 		// Free memory
 		SQL_FREERESULT($result_main);
 
 		// Output message
-		LOAD_TEMPLATE("admin_settings_saved", false, getMessage('ADMIN_ALL_POINTS_SUBTRACTED'));
+		LOAD_TEMPLATE('admin_settings_saved', false, getMessage('ADMIN_ALL_POINTS_SUBTRACTED'));
 	} else {
 		// Display form add points
 		LOAD_TEMPLATE("admin_sub_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);
+		$content = SQL_FETCHARRAY($result);
 
-		if ((isset($_POST['ok'])) && (!empty($_POST['points']))) {
+		if ((isFormSent()) && (REQUEST_ISSET_POST(('points')))) {
 			// Ok, add to used points and send an email to him...
-			SUB_POINTS("admin_single", bigintval($_GET['u_id']), $_POST['points']);
+			SUB_POINTS("admin_single", bigintval(REQUEST_GET('uid')), REQUEST_POST('points'));
 
-			// Prepare content
-			$content = array(
-				'text'   => SQL_ESCAPE($_POST['reason']),
-				'points' => bigintval($_POST['points'])
-			);
+			// Add more content
+			$content['text']   = SQL_ESCAPE(REQUEST_POST('reason'));
+			$content['points'] = bigintval(REQUEST_POST('points'));
 
 			// Load email and send it away
-			$msg = LOAD_EMAIL_TEMPLATE("sub-points", $content, bigintval($_GET['u_id']));
-			SEND_EMAIL(bigintval($_GET['u_id']), getMessage('ADMIN_SUB_SUBJ'), $msg);
+			$msg = LOAD_EMAIL_TEMPLATE("sub-points", $content, bigintval(REQUEST_GET('uid')));
+			sendEmail($content['email'], getMessage('ADMIN_SUB_SUBJ'), $msg);
 
 			// Output message
-			LOAD_TEMPLATE("admin_settings_saved", false, getMessage('ADMIN_POINTS_SUBTRACTED'));
+			LOAD_TEMPLATE('admin_settings_saved', false, getMessage('ADMIN_POINTS_SUBTRACTED'));
 		} else {
-			// Opps, missing form here
-			define('__USER_VALUE', "<a href=\"".CREATE_EMAIL_LINK($email, "user_data")."\">".$sname." ".$fname."</a>");
-			define('__UID', bigintval($_GET['u_id']));
+			// @TODO Rewrite these constants
+			define('__USER_VALUE', "<a href=\"".generateMemberEmailLink($content['email'], "user_data")."\">".$content['surname']." ".$content['family']."</a>");
+			define('__UID', bigintval(REQUEST_GET('uid')));
+
+			// Load form
 			LOAD_TEMPLATE("admin_sub_points");
 		}
 	} else {
 		// User not found!
-		LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"admin_failed\">".ADMIN_MEMBER_404_1.$_GET['u_id'].ADMIN_MEMBER_404_2."</strong>");
+		LOAD_TEMPLATE('admin_settings_saved', false, "<div class=\"admin_failed\">".sprintf(getMessage('ADMIN_MEMBER_404'), REQUEST_GET('uid'))."</div>");
 	}
+
+	// Free result
+	SQL_FREERESULT($result);
 } else {
 	// Output selection form with all confirmed user accounts listed
-	ADD_MEMBER_SELECTION_BOX("0", true);
+	ADD_MEMBER_SELECTION_BOX('0', true);
 }
 
 //