X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=beg.php;h=e9e559138e952b77c901dd99c1283b930f6da3b9;hp=67e9855844707ace61455ef319cf8ee1004623ad;hb=544aa3fb48613160f812515673e64c46d18d2e1d;hpb=9afd6ec5878544a7982c50ed9c0dd7de37606d5b diff --git a/beg.php b/beg.php index 67e9855844..e9e559138e 100644 --- a/beg.php +++ b/beg.php @@ -1,7 +1,7 @@ 0) && (getConfig('beg_userid') != getUserData('userid'))) { + if ((isValidUserId(getUserData('userid'))) && (getBegUserid() != 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__); @@ -107,18 +108,23 @@ if (isGetRequestElementSet('userid')) { array(getUserData('userid'), detectRemoteAddr(), session_id()), __FILE__, __LINE__); // Entry not found, points set and not logged in? - if (((SQL_NUMROWS($result) == '0') || (isAdmin())) && ($points > 0) && (!isMember()) && (getConfig('beg_pay_mode') == 'NONE')) { - + if (((SQL_HASZERONUMS($result)) || (isAdmin())) && ($points > 0) && (!isMember()) && (getBegPayMode() == '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; @@ -129,28 +135,21 @@ if (isGetRequestElementSet('userid')) { // Add points to user or begging rallye account if (addPointsBeg(getUserData('userid'), $points)) { // Set 'done' message - $content['message'] = loadTemplate('beg_done', true); + $content['message'] = loadTemplate('beg_done', true, $content); } else { // Error! - $content['message'] = loadTemplate('beg_failed', true); + $content['message'] = loadTemplate('beg_failed', true, $content); } } else { // Error! - $content['message'] = loadTemplate('beg_failed', true); + $content['message'] = loadTemplate('beg_failed', true, $content); } } elseif (isMember()) { - // Logged in user found! - $content['message'] = loadTemplate('beg_login', true); - } 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'] - ); - + // Logged in user found + $content['message'] = loadTemplate('beg_login', true, $content); + } elseif (getBegPayMode() != 'NONE') { // Other pay-mode active! // Load message template depending on pay-mode - $content['message'] = loadTemplate('beg_pay_mode_'.strtolower(getConfig('beg_pay_mode')), true, $content); + $content['message'] = loadTemplate('beg_pay_mode_'.strtolower(getBegPayMode()), true, $content); $pay = true; } else { // Clicked received while reload lock is active @@ -167,14 +166,14 @@ if (isGetRequestElementSet('userid')) { loadTemplate('beg_link', false, $content); // Tracker code enabled? (We don't track users here! - if ((getConfig('beg_pay_mode') != 'NONE') && ($pay === true)) { + if ((getBegPayMode() != 'NONE') && ($pay === true)) { // Prepare content for template // @TODO Opps, what is missing here??? $content = array( ); // Include config-depending template - loadTemplate('beg_pay_code_'.strtolower(getConfig('beg_pay_mode')), false, $content); + loadTemplate('beg_pay_code_'.strtolower(getBegPayMode()), false, $content); } elseif (($pay === false) && (!isset($content['message']))) { // Cannot pay! :-( $content['message'] = loadTemplate('beg_failed', true); @@ -187,20 +186,22 @@ if (isGetRequestElementSet('userid')) { $errorCode = generateErrorCodeFromUserStatus(); } elseif ((getUserData('userid') == '0') || (getUserData('status') == 'failed')) { // Inalid or locked account, so let's find out - if (fetchUserData(getRequestElement('userid'), 'nickname')) { + if (fetchUserData(getRequestParameter('userid'), 'nickname')) { // Locked account $errorCode = getCode('ACCOUNT_LOCKED'); } else { // Invalid nickname! (404) $errorCode = getCode('USER_404'); } - } elseif (getUserData('userid') == getConfig('beg_userid')) { + } elseif (getUserData('userid') == getBegUserid()) { // Webmaster's id cannot beg for points! $errorCode = getCode('BEG_SAME_AS_OWN'); } - // Reload to index module - if ((!empty($errorCode)) && (!empty($errorCode))) redirectToUrl('modules.php?module=index&code=' . $errorCode . '&ext=beg'); + // Reload to index module if an error happens + if (!empty($errorCode)) { + redirectToUrl('modules.php?module=index&code=' . $errorCode . '&ext=beg'); + } // END - if } else { // No userid entered redirectToUrl('modules.php?module=index');