X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=mailid_top.php;h=36a5c007856fbe55ff9fffaa52399a9f51bd51f9;hp=0209f2e52cdc6d4f56dcf5972d6d245aa3c2c56e;hb=1fd39b2564946ce7f19776abab8d65a31928fba1;hpb=3f230c917afccc24b270fa6393c9796b557d4ce1 diff --git a/mailid_top.php b/mailid_top.php index 0209f2e52c..36a5c00785 100644 --- a/mailid_top.php +++ b/mailid_top.php @@ -1,7 +1,7 @@ 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErrors() == 0)) { +if ((isValidUserId($url_userid)) && (($url_mid > 0) || ($url_bid > 0)) && (!ifFatalErrorsDetected())) { // 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", + $result_main = 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__); $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_main = 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? - list($lid, $ltype) = SQL_FETCHROW($result); - SQL_FREERESULT($result); + if (SQL_NUMROWS($result_main) == 1) { + // Is the stats id valid? + list($lid, $ltype) = SQL_FETCHROW($result_main); // Init result here with invalid to avoid possible missing variable $result_mailid = false; @@ -109,7 +109,7 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr break; default: // Unknown type - debug_report_bug('Unknown mail type ' . $ltype . ' detected.'); + debug_report_bug(__FILE__, __LINE__, 'Unknown mail type ' . $ltype . ' detected.'); break; } @@ -121,23 +121,13 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr // Correct notification switch in non-bonus mails if (($notify != 'Y') && ($notify != 'N')) $notify = 'N'; - // Free some memory - 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 ($ltype == 'BONUS') $sender = '0'; - if ($status == 'CONFIRMED') { + // 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) { @@ -172,15 +162,15 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr break; default: // Unknown type - debug_report_bug('Unknown mail type ' . $ltype . ' detected.'); + debug_report_bug(__FILE__, __LINE__, 'Unknown mail type ' . $ltype . ' detected.'); break; } // Is this entry valid? if ($isValid === true) { - if (($time == 0) && ($payment > 0)) $time = 1; + if (($time == '0') && ($payment > 0)) $time = 1; if (($time > 0) && ($payment > 0)) { - $img_code = 0; + $img_code = '0'; if (!empty($code)) { // Generate code $img_code = generateRandomCode(getConfig('code_length'), $code, $url_userid, $urlId); @@ -190,7 +180,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 @@ -200,7 +190,7 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr array($url_mid), __FILE__, __LINE__); // Update mediadata as well - if (getExtensionVersion('mediadata') >= '0.0.4') { + if (isExtensionInstalledAndNewer('mediadata', '0.0.4')) { // Update database updateMediadataEntry(array('total_clicks', 'normal_clicks'), 'add', 1); } // END - if @@ -212,7 +202,7 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr array($url_bid), __FILE__, __LINE__); // Update mediadata as well - if (getExtensionVersion('mediadata') >= '0.0.4') { + if (isExtensionInstalledAndNewer('mediadata', '0.0.4')) { // Update database updateMediadataEntry(array('total_clicks', 'bonus_clicks'), 'add', 1); } // END - if @@ -220,26 +210,25 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr break; default: // Unknown type - debug_report_bug('Unknown mail type ' . $ltype . ' detected.'); + debug_report_bug(__FILE__, __LINE__, 'Unknown mail type ' . $ltype . ' detected.'); break; } // END - switch // Export data into constants for the template - $content['points'] = translateComma($payment); $content['banner'] = loadTemplate('mailid_banner', true); // Only when user extension = v0.1.2: Update mails-confirmed counter // @TODO Rewrite these blocks to filter - if (getExtensionVersion('user') >= '0.1.2') { + if (isExtensionInstalledAndNewer('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') { + if (isExtensionInstalledAndNewer('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__); + SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_user_data` SET `rand_confirmed`=`rand_confirmed` + 1 WHERE `userid`=%s LIMIT 1", + array($url_userid), __FILE__, __LINE__); } // END - if } // END - if @@ -247,13 +236,13 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr insertUserStatsRecord($url_userid, $type, $stats_data); // Right code entered? - if (bigintval(postRequestElement('gfx_check')) == $img_code) { + if (bigintval(postRequestParameter('gfx_check')) == $img_code) { // Add points over referal system is the default $locked = false; $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'; @@ -269,11 +258,11 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr addPointsThroughReferalSystem('mailid_okay', $url_userid, $payment, false, 0, $locked); // Shall I add bonus points for "turbo clickers" ? - if (getExtensionVersion('bonus') >= '0.2.2') { + if (isExtensionInstalledAndNewer('bonus', '0.2.2')) { // Is an active-rallye running and this is not a notification mail? - if ((getConfig('bonus_active') == 'Y') && ($notify != 'Y')) { + if ((isBonusRallyeActive()) && ($notify != 'Y')) { // 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)) { + if ((((getBonusUserId() == $url_userid) && (getConfig('bonus_include_own') == 'Y')) || (getBonusUserId() != $url_userid)) && (getConfig('def_refid') != $url_userid)) { // Add points and remember ranking are done in this function.... addTurboBonus($urlId, $url_userid, $type); @@ -288,14 +277,11 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr } // END - if // Load total points - $content['total'] = translateComma( - countSumTotalData($url_userid, 'user_points', 'points') - - countSumTotalData($url_userid, 'user_data', 'used_points') - ); + $content['total'] = getTotalPoints($url_userid); // Load template loadTemplate($template, false, $content); - } else { + } elseif (isValidUserId($sender)) { // Wrong image code! So add points to sender's account addPointsDirectly('mailid_payback', $sender, $payment); @@ -332,6 +318,9 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr // Load template loadTemplate($templ, false, $content); + } else { + // Cannot confirm! + debug_report_bug(__FILE__, __LINE__, 'No code given.'); } break; @@ -351,37 +340,40 @@ if (($url_userid) > 0 && (($url_mid > 0) || ($url_bid > 0)) && (getTotalFatalErr break; default: // Unknown mode - debug_report_bug('Unknown mode ' . $mode . ' detected.'); + debug_report_bug(__FILE__, __LINE__, 'Unknown mode ' . $mode . ' detected.'); break; } // END - switch } else { - loadTemplate('admin_settings_saved', false, "
{--MAIL_ALREADY_CONFIRMED--} (6)
"); + loadTemplate('admin_settings_saved', false, '
{--MAIL_ALREADY_CONFIRMED--} (6)
'); $mode = 'failed'; } } else { - loadTemplate('admin_settings_saved', false, "
{--MAIL_ALREADY_CONFIRMED--} (5)
"); + loadTemplate('admin_settings_saved', false, '
{--MAIL_ALREADY_CONFIRMED--} (5)
'); $mode = 'failed'; } } else { - loadTemplate('admin_settings_saved', false, "
{--MAIL_ALREADY_CONFIRMED--} (4)
"); + loadTemplate('admin_settings_saved', false, '
{--MAIL_ALREADY_CONFIRMED--} (4)
'); $mode = 'failed'; } } else { - SQL_FREERESULT($result); - loadTemplate('admin_settings_saved', false, "
{--MAIL_ALREADY_CONFIRMED--} (3)
"); + loadTemplate('admin_settings_saved', false, '
{--MAIL_ALREADY_CONFIRMED--} (3)
'); $mode = 'failed'; } } else { - SQL_FREERESULT($result); - loadTemplate('admin_settings_saved', false, "
{--MAIL_ALREADY_CONFIRMED--} (2)
"); + loadTemplate('admin_settings_saved', false, '
{--MAIL_ALREADY_CONFIRMED--} (2)
'); $mode = 'failed'; } + + // Free result + SQL_FREERESULT($result_mailid); } else { - SQL_FREERESULT($result); - loadTemplate('admin_settings_saved', false, "
{--MAIL_ALREADY_CONFIRMED--} (1)
"); + loadTemplate('admin_settings_saved', false, '
{--MAIL_ALREADY_CONFIRMED--} (1)
'); $mode = 'failed'; } + // Free result + SQL_FREERESULT($result_main); + // Insert footer if no image if ($mode != 'img') { // Write footer