X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=mailid_top.php;h=0c5cee191397d832666c8daf1ede6c6d22443d61;hp=fa10da86cb92a5d280d8f62dbc8438d128904dad;hb=2cdcee9f025bbea379e2da13f87dc2f71940f3e5;hpb=64c8349613addc3da2242c5cd6b99d64e3fb5f8e diff --git a/mailid_top.php b/mailid_top.php index fa10da86cb..0c5cee1913 100644 --- a/mailid_top.php +++ b/mailid_top.php @@ -1,7 +1,7 @@ 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErrors() == 0)) { +if (isGetRequestElementSet('userid')) $url_userid = bigintval(getRequestElement('userid')); +if (isGetRequestElementSet('mailid')) $url_mid = bigintval(getRequestElement('mailid')); +if (isGetRequestElementSet('bonusid')) $url_bid = bigintval(getRequestElement('bonusid')); +if (isGetRequestElementSet('code')) $code = bigintval(getRequestElement('code')); +if (isGetRequestElementSet('mode')) $mode = getRequestElement('mode'); + +// 01 1 12 2 2 21 1 22 10 +if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErrors() == '0')) { // No image? Then output header if ($mode != 'img') loadIncludeOnce('inc/header.php'); // Maybe he wants to confirm an email? if ($url_mid > 0) { $result = SQL_QUERY_ESC("SELECT `id`, `link_type` FROM `{?_MYSQL_PREFIX?}_user_links` WHERE `stats_id`=%s AND `userid`=%s LIMIT 1", - array($url_mid, $url_userid), __FILE__, __LINE__); + array($url_mid, $url_userid), __FILE__, __LINE__); $type = 'mailid'; $urlId = $url_mid; } elseif ($url_bid > 0) { - $result = SQL_QUERY_ESC("SELECT `id`, link_type` FROM `{?_MYSQL_PREFIX?}_user_links` WHERE `bonus_id`=%s AND `userid`=%s LIMIT 1", + $result = SQL_QUERY_ESC("SELECT `id`, `link_type` FROM `{?_MYSQL_PREFIX?}_user_links` WHERE `bonus_id`=%s AND `userid`=%s LIMIT 1", array($url_bid, $url_userid), __FILE__, __LINE__); $type = 'bonusid'; $urlId = $url_bid; } if (SQL_NUMROWS($result) == 1) { - // Is the stats ID valid? + // Is the stats id valid? list($lid, $ltype) = SQL_FETCHROW($result); SQL_FREERESULT($result); + // Init result here with invalid to avoid possible missing variable + $result_mailid = false; + // @TODO Rewrite this to a filter switch ($ltype) { case 'NORMAL': @@ -92,7 +104,7 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr break; case 'BONUS': - $result_mailid = SQL_QUERY_ESC("SELECT `id, `id`, `is_notify` FROM `{?_MYSQL_PREFIX?}_bonus` WHERE `id`=%s LIMIT 1", + $result_mailid = SQL_QUERY_ESC("SELECT `id`, `id`, `is_notify` FROM `{?_MYSQL_PREFIX?}_bonus` WHERE `id`=%s LIMIT 1", array($url_bid), __FILE__, __LINE__); break; @@ -113,27 +125,15 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr SQL_FREERESULT($result_mailid); // Set sender to 0 when we have a bonus mail - if ($ltype == 'BONUS') $sender = 0; - - // Is the user's ID unlocked? - $result = SQL_QUERY_ESC("SELECT status, gender, surname, family, ref_payout FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `userid`=%s LIMIT 1", - array($url_userid), __FILE__, __LINE__); - if (SQL_NUMROWS($result) == 1) { - // Load data - list($status, $gender, $surname, $family, $ref_pay) = SQL_FETCHROW($result); - - // Free some memory - SQL_FREERESULT($result); - - if ($status == 'CONFIRMED') { - // Update last activity - SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET `last_online`=UNIX_TIMESTAMP(), `last_module`='mailid_top' WHERE `userid`=%s LIMIT 1", - array($url_userid), __FILE__, __LINE__); + if ($ltype == 'BONUS') $sender = '0'; + // Is the user id valid? + if (fetchUserData($url_userid) === true) { + // Is the user status CONFIRMED? + if (getUserData('status') == 'CONFIRMED') { // User has confirmed his account so we can procede... // @TODO Rewrite this to a filter - switch ($ltype) - { + switch ($ltype) { case 'NORMAL': $result = SQL_QUERY_ESC("SELECT `payment_id` FROM `{?_MYSQL_PREFIX?}_user_stats` WHERE `pool_id`=%s LIMIT 1", array(bigintval($pool)), __FILE__, __LINE__); @@ -170,9 +170,10 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr } // Is this entry valid? - if ($isValid) { - if (($time == '0') && ($payment > 0)) $time = '1'; + if ($isValid === true) { + if (($time == '0') && ($payment > 0)) $time = 1; if (($time > 0) && ($payment > 0)) { + $img_code = '0'; if (!empty($code)) { // Generate code $img_code = generateRandomCode(getConfig('code_length'), $code, $url_userid, $urlId); @@ -182,7 +183,7 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr switch ($mode) { case 'add': // Init stats data - $stats_data = 0; + $stats_data = '0'; // Count clicks // @TODO Rewrite this to a filter @@ -225,13 +226,13 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr if (getExtensionVersion('user') >= '0.1.2') { // Update counter SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET mails_confirmed=mails_confirmed + 1 WHERE `userid`=%s LIMIT 1", - array($url_userid), __FILE__, __LINE__); + array($url_userid), __FILE__, __LINE__); // Update random confirmed as well? if (getExtensionVersion('user') >= '0.3.4') { // Update second counter SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET rand_confirmed=rand_confirmed + 1 WHERE `userid`=%s LIMIT 1", - array($url_userid), __FILE__, __LINE__); + array($url_userid), __FILE__, __LINE__); } // END - if } // END - if @@ -245,7 +246,7 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr $template = 'mailid_points_done'; // Right code entered add points and remove entry - if (($ref_pay > 0) && (getConfig('allow_direct_pay') != 'Y')) { + if ((getUserData('ref_payout') > 0) && (getConfig('allow_direct_pay') != 'Y')) { // Don't add points over the referal system $locked = true; $template = 'mailid_points_locked'; @@ -258,7 +259,7 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr // Add points // @TODO Try to rewrite the following unset() unset($GLOBALS['ref_level']); - addPointsThroughReferalSystem('mailid_okay', $url_userid, $payment, false, '0', $locked); + addPointsThroughReferalSystem('mailid_okay', $url_userid, $payment, false, 0, $locked); // Shall I add bonus points for "turbo clickers" ? if (getExtensionVersion('bonus') >= '0.2.2') { @@ -267,14 +268,14 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr // Shall I exclude the webmaster's own userid from the active-rallye? if ((((getConfig('bonus_userid') == $url_userid) && (getConfig('bonus_include_own') == 'Y')) || (getConfig('bonus_userid') != $url_userid)) && (getConfig('def_refid') != $url_userid)) { // Add points and remember ranking are done in this function.... - addTurboBonus($payment, $url_userid, $type); + addTurboBonus($urlId, $url_userid, $type); // Set template to mailid_points_done2 which contains a link to the ranking list $template = 'mailid_points_done2'; if ($locked) $template = 'mailid_points_locked2'; $content['userid'] = $url_userid; - $content['type'] = $type; - $content['data'] = translateComma($payment); + $content['type'] = $type; + $content['data'] = $urlId; } // END - if } // END - if } // END - if @@ -287,7 +288,7 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr // Load template loadTemplate($template, false, $content); - } else { + } elseif ($sender > 0) { // Wrong image code! So add points to sender's account addPointsDirectly('mailid_payback', $sender, $payment); @@ -307,9 +308,9 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr case 'confirm': if ($code > 0) { // Export data into constants for the template - $content['code'] = $code; - $content['userid'] = $url_userid; - $content['type'] = $type; + $content['code'] = $code; + $content['userid'] = $url_userid; + $content['type'] = $type; $content['data'] = $urlId; $content['banner'] = loadTemplate('mailid_banner', true); if (getConfig('code_length') > 0) { @@ -324,6 +325,9 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr // Load template loadTemplate($templ, false, $content); + } else { + // Cannot confirm! + debug_report_bug('No code given.'); } break;