X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmodules%2Fmember%2Fwhat-unconfirmed.php;h=65abc76504b93fd9e585997d272411cde69c4d8f;hp=4ed7e8d9cecdcaedf9c2e4dd2b72f59fdb0c07f2;hb=03486c08011d4c233e2455c8e5335ecc0818333f;hpb=ab8f09f780b313fd4196ec6010ebc9e960e4da85 diff --git a/inc/modules/member/what-unconfirmed.php b/inc/modules/member/what-unconfirmed.php index 4ed7e8d9ce..65abc76504 100644 --- a/inc/modules/member/what-unconfirmed.php +++ b/inc/modules/member/what-unconfirmed.php @@ -1,23 +1,22 @@ 0) { +if (!SQL_HASZERONUMS($result)) { // Please confirm these mails! - $sum = 0; $OUT = ''; $SW = 2; + $sum = '0'; $OUT = ''; // Init content $content = array(); // @TODO Try to rewrite this to $content = SQL_FETCHARRAY() - while (list($id, $id2, $type) = SQL_FETCHROW($result)) { + while ($row = SQL_FETCHARRAY($result)) { // Load data from stats table... - $cat = ''; $result_data = false; - switch ($type) - { + $cat = ''; + $result_data = false; + $message = '{--MEMBER_GENERAL_MAIL_PROBLEM--}'; + $data = $row['stats_id'] . '/' . $row['bonus_id'] . '/' . $row['link_type']; + switch ($row['link_type']) { case 'NORMAL': $result_data = SQL_QUERY_ESC("SELECT s.subject, s.timestamp_ordered, s.cat_id, s.payment_id, p.sender @@ -159,55 +159,58 @@ ON WHERE s.id=%s LIMIT 1", - array(bigintval($id)), __FILE__, __LINE__); - $type = 'mailid'; $DATA = $id; $PROBLEM = getMessage('NORMAL_MAIL_PROBLEM'); + array(bigintval($row['stats_id'])), __FILE__, __LINE__); + $row['link_type'] = 'mailid'; + $data = $row['stats_id']; + $message = '{--NORMAL_MAIL_PROBLEM--}'; break; case 'BONUS': - $result_data = SQL_QUERY_ESC("SELECT subject, timestamp, cat_id, points, 0 FROM `{?_MYSQL_PREFIX?}_bonus` WHERE `id`=%s LIMIT 1", - array(bigintval($id2)), __FILE__, __LINE__); - $type = 'bonusid'; $DATA = $id2; $PROBLEM = getMessage('BONUS_MAIL_PROBLEM'); - break; + $result_data = SQL_QUERY_ESC("SELECT `subject`, `timestamp`, `cat_id`, `points`, 0 FROM `{?_MYSQL_PREFIX?}_bonus` WHERE `id`=%s LIMIT 1", + array(bigintval($row['bonus_id'])), __FILE__, __LINE__); + $row['link_type'] = 'bonusid'; + $data = $row['bonus_id']; + $message = '{--BONUS_MAIL_PROBLEM--}'; + break; default: // Unknown type detected! - logDebugMessage(__FILE__, __LINE__, sprintf("Unknown mail type %s detected.", $type)); + logDebugMessage(__FILE__, __LINE__, sprintf("Unknown mail type %s detected.", $row['link_type'])); break; } // Data found to this mail? - if ((SQL_NUMROWS($result_data) == 1) && (($type == 'mailid') || ($type == 'bonusid'))) { - // Mail was found! + if ((SQL_NUMROWS($result_data) == 1) && (($row['link_type'] == 'mailid') || ($row['link_type'] == 'bonusid'))) { + // Mail was found list($subject, $timestamp, $cat, $pay, $sender) = SQL_FETCHROW($result_data); // Subject line found? if (empty($subject)) { // No subject line! - $subject = getMessage('DEFAULT_SUBJECT_LINE'); + $subject = '{--DEFAULT_SUBJECT_LINE--}'; } // END - if // Prepare sender id - if (($sender > 0) && ($type == 'mailid')) { + if ((isValidUserId($sender)) && ($row['link_type'] == 'mailid')) { // Sender id $sender = bigintval($sender); - } elseif ($type == 'bonusid') { + } elseif ($row['link_type'] == 'bonusid') { // Is admin - $sender = getMessage('_ADMIN_SHORT'); + $sender = '{--USERNAME_ADMIN_SHORT--}'; } else { // Deleted - $sender = getMessage('EMAIL_STATUS_DELETED'); + $sender = '{--EMAIL_STATUS_DELETED--}'; } // Prepare data for template $content = array( - 'sw' => $SW, - 'userid' => getUserId(), - 'data' => bigintval($DATA), - 'type' => $type, - 'subject' => $subject, - 'sender' => $sender, - 'stamp' => generateDateTime($timestamp, 2), - 'cat' => getCategory($cat), - 'points' => translateComma($pay), + 'data' => bigintval($data), + // @TODO This 'userid' cannot be saved because of encapsulated EL code + 'userid' => getMemberId(), + 'link_type' => $row['link_type'], + 'subject' => $subject, + 'sender' => $sender, + 'timestamp' => generateDateTime($timestamp, 2), + 'points' => $pay, ); // Load row template @@ -222,9 +225,8 @@ LIMIT 1", } else { // Prepare data for template $content = array( - 'sw' => $SW, - 'data' => $content, - 'probl' => $PROBLEM, + 'data' => $data, + 'message' => $message, ); // Display points or not? @@ -237,16 +239,13 @@ LIMIT 1", // Free result SQL_FREERESULT($result_data); - - // Switch color - $SW = 3 - $SW; } // END - while // Free memory SQL_FREERESULT($result); // Remember total points - $content['total_points'] = translateComma($sum); + $content['total_points'] = $sum; // Remember all generated rows in constant for the template $content['rows'] = $OUT; @@ -259,7 +258,7 @@ LIMIT 1", } } else { // No mails left to confirm... :) - loadTemplate('admin_settings_saved', false, getMessage('MEMBER_NO_MAILS_TO_CONFIRM')); + displayMessage('{--MEMBER_NO_MAILS_TO_CONFIRM--}'); } // [EOF]