X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=beg.php;h=387c9e6accc6915b216beaceaa66e9835433c3ea;hb=626d46112349bc87ad5f64b8c16d9954144bff07;hp=57b930c8d9fc4869b1b720f163def020db07383d;hpb=d9f35786166902b9bc3f402d4f2abeed5ae0528d;p=mailer.git

diff --git a/beg.php b/beg.php
index 57b930c8d9..387c9e6acc 100644
--- a/beg.php
+++ b/beg.php
@@ -14,8 +14,6 @@
  * $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 - 2009 by Roland Haeder                           *
  * Copyright (c) 2009, 2010 by Mailer Developer Team                    *
@@ -85,20 +83,22 @@ if (isGetRequestParameterSet('userid')) {
 
 	// Check if account was found
 	if ((isUserDataValid()) && (getUserData('status') == 'CONFIRMED')) {
-		// Multiply configured values with 100000 and divide with 100000 so we can also handle small values
-		// If we need more number behind the decimal dot then we just need to increase all these three
-		// numbers matching to the numbers behind the decimal dot. Simple! ;-)
+		/*
+		 * Multiply configured values with 100000 and divide with 100000 so we can also handle small values
+		 * If we need more number behind the decimal dot then we just need to increase all these three
+		 * numbers matching to the numbers behind the decimal dot. Simple! ;-)
+		 */
 		$points = mt_rand((getConfig('beg_points') * 100000), (getConfig('beg_points_max') * 100000)) / 100000;
 
 		// Set nickname / userid for template
-		$content['userid']      = getRequestParameter('userid');
-		$content['clicks']      = (getUserData('beg_clicks') + 1);
-		$content['banner']      = loadTemplate('beg_banner', true);
-		$content['points']      = translateComma($points);
+		$content['userid'] = getRequestParameter('userid');
+		$content['clicks'] = (getUserData('beg_clicks') + 1);
+		$content['banner'] = loadTemplate('beg_banner', true);
+		$content['points'] = $points;
 	} // END - if
 
 	// User id valid and not webmaster's id?
-	if ((getUserData('userid') > 0) && (getConfig('beg_userid') != getUserData('userid'))) {
+	if ((isValidUserId(getUserData('userid'))) && (getConfig('beg_userid') != getUserData('userid'))) {
 		// Update counter
 		SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET `beg_clicks`=`beg_clicks`+1 WHERE `userid`=%s LIMIT 1",
 			array(getUserData('userid')), __FILE__, __LINE__);
@@ -109,17 +109,22 @@ if (isGetRequestParameterSet('userid')) {
 
 		// Entry not found, points set and not logged in?
 		if (((SQL_HASZERONUMS($result)) || (isAdmin())) && ($points > 0) && (!isMember()) && (getConfig('beg_pay_mode') == 'NONE')) {
-
 			// Admin is testing?
 			if (!isAdmin()) {
-				// Remember remote address, userid and timestamp for next click
-				// but only when there is no admin begging.
-				// Admins shall be able to test it!
+				/*
+				 * Remember remote address, userid and timestamp for next click
+				 * but only when there is no admin begging.
+				 * Admins shall be able to test it!
+				 */
 				SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_beg_ips` (`userid`, `remote_ip`,`sid`, `timeout`) VALUES ('%s','%s','%s', UNIX_TIMESTAMP())",
-					array(getUserData('userid'), detectRemoteAddr(), session_id()), __FILE__, __LINE__);
+					array(
+						getUserData('userid'),
+						detectRemoteAddr(),
+						session_id()
+					), __FILE__, __LINE__);
 
 				// Was is successfull?
-				$pay (SQL_AFFECTEDROWS() == 1);
+				$pay (!SQL_HASZEROAFFECTED());
 			} else {
 				// Is admin!
 				$pay = true;
@@ -143,13 +148,6 @@ if (isGetRequestParameterSet('userid')) {
 			// Logged in user found!
 			$content['message'] = loadTemplate('beg_login', true, $content);
 		} elseif (getConfig('beg_pay_mode') != 'NONE') { // Other pay-mode active!
-			// Prepare content for template
-			$content = array(
-				'clicks' => $content['clicks'],
-				'points' => $content['points'],
-				'userid' => $content['userid']
-			);
-
 			// Load message template depending on pay-mode
 			$content['message'] = loadTemplate('beg_pay_mode_'.strtolower(getConfig('beg_pay_mode')), true, $content);
 			$pay = true;