Fixes and cleanups
[mailer.git] / inc / modules / admin / what-list_unconfirmed.php
index 9ef88e563553176ffc2be97e1b7978411ed88b5b..0b9172531c70229e65399944e60d3ffb454373a4 100644 (file)
  * $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                    *
- * For more information visit: http://www.mxchange.org                  *
+ * Copyright (c) 2009 - 2011 by Mailer Developer Team                   *
+ * For more information visit: http://mxchange.org                      *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
  * it under the terms of the GNU General Public License as published by *
@@ -43,21 +41,24 @@ if ((!defined('__SECURITY')) || (!isAdmin())) {
 } // END - if
 
 // Add description as navigation point
-addMenuDescription('admin', __FILE__);
+addYouAreHereLink('admin', __FILE__);
 
 if (!isExtensionActive('mailid')) {
-       loadTemplate('admin_settings_saved', false, generateExtensionInactiveNotInstalledMessage('mailid'));
+       displayMessage('{%pipe,generateExtensionInactiveNotInstalledMessage=mailid%}');
        return;
 } // END - if
 
 // Don't load the admin_list_unconfirmed template by default
 $listed = false;
 
+// Init query
+$sql = '';
+
 // List confirmation links from normal or bonus mails
-if (isGetRequestParameterSet('mid')) {
+if (isGetRequestElementSet('mid')) {
        // SQL query for mail data
-       $sql = "SELECT
-       s.id, p.sender, p.subject, p.text, p.url, p.timestamp, s.max_rec
+       $sql = sprintf("SELECT
+       s.`id`, p.`sender`, p.`subject`, p.`text`, p.`url`, p.`timestamp`, s.`max_rec`
 FROM
        `{?_MYSQL_PREFIX?}_pool` AS p
 LEFT JOIN
@@ -65,140 +66,152 @@ LEFT JOIN
 ON
        p.id=s.pool_id
 WHERE
-       p.id='".getRequestParameter('mid')."'
-LIMIT 1";
+       p.`id`=%s
+LIMIT 1",
+               bigintval(getRequestElement('mid'))
+       );
 
        // Column, type and id for member's mail
-       $col = 'stats_id'; $type = 'NORMAL'; $ID = '-1';
+       $col = 'stats_id';
+       $type = 'NORMAL';
+       $ID = -1;
 
        // Load admin_list_unconfirmed template
-       $listed = true; $DATA = getRequestParameter('mid'); $LINK = 'mailid';
-} elseif ((isGetRequestParameterSet('bid')) && (isExtensionActive('bonus'))) {
-       // @TODO This constant might be unused? define('__LIST_UNCON_TITLE', '{--LIST_UNCONFIRMED_BONUS_LINKS--}');
+       $listed = true;
+       $DATA = getRequestElement('mid');
+       $mailType = 'mailid';
+} elseif ((isGetRequestElementSet('bid')) && (isExtensionActive('bonus'))) {
+       // @TODO This constant might be unused? define('__LIST_UNCON_TITLE', '{--ADMIN_LIST_UNCONFIRMED_BONUS_LINKS--}');
 
        // SQL query for mail data (both ids are required for compatiblity to above normal mail
-       $sql = "SELECT `id`, `id`, `subject`, `text`, `url`, `timestamp`, `mails_sent` FROM `{?_MYSQL_PREFIX?}_bonus` WHERE `id`='".getRequestParameter('bid')."' LIMIT 1";
+       $sql = sprintf("SELECT
+       `id`,`id` AS `sender`,`subject`,`text`,`url`,`timestamp`,`mails_sent` AS `max_rec`
+FROM
+       `{?_MYSQL_PREFIX?}_bonus`
+WHERE
+       `id`=%s
+LIMIT 1",
+               bigintval(getRequestElement('bid'))
+       );
 
        // Column, type and id for member's mail
-       $col = 'bonus_id'; $type = 'BONUS'; $ID = getRequestParameter('bid');
+       $col = 'bonus_id';
+       $type = 'BONUS';
+       $ID = getRequestElement('bid');
 
        // Load admin_list_unconfirmed template
-       $listed = true; $DATA = $ID; $LINK = 'bonusid';
+       $listed = true;
+       $DATA = $ID;
+       $mailType = 'bonusid';
 } else {
        // @TODO "Please do not call me directly." Should be rewritten to a nice selection depending on ext-bonus
-       loadTemplate('admin_settings_saved', false, '{--ADMIN_CALL_NOT_DIRECTLY--}');
+       displayMessage('{--ADMIN_CALL_NOT_DIRECTLY--}');
 }
 
 // Shall I display links or not?
-if ($listed === true) {
+if (($listed === true) && (!empty($sql))) {
        // Load mail data
        $result_master = SQL_QUERY($sql, __FILE__, __LINE__);
 
        // Do we have an entry?
        if (SQL_NUMROWS($result_master) == 1) {
-               // Mail order / bonus mail found!
-               // @TODO Rewrite this to $content = SQL_FETCHARRAY(), look some lines above for two differrent queries
-               list($stats_id, $sender, $subject, $text, $url, $stamp, $max) = SQL_FETCHROW($result_master);
+               // Mail order / bonus mail found
+               $poolData = SQL_FETCHARRAY($result_master);
 
-               // Transfer data to constants for the template
-               if (($stats_id > 0) && ($ID == '-1')) $ID = $stats_id;
-               if ($col == 'bonus_id') $sender = '0';
+               // If there is a pool id and $ID is not set, we take it
+               if (($poolData['id'] > 0) && ($ID == '-1')) {
+                       $ID = $poolData['id'];
+               } // END - if
+
+               // Bonus mails do always have a sender 'NULL'
+               if ($col == 'bonus_id') {
+                       $poolData['sender'] = NULL;
+               } // END - if
 
                // Load unconfirmed mail links. Hmmm, this select query is pretty cool
                // but it does only show unconfirmed mail links from existing user
                // accounts. So if you have delete one you did not see those links
                $result = SQL_QUERY_ESC("SELECT
-       l.userid, u.status, u.surname, u.family, u.gender, u.email
+       l.`userid`,
+       u.`status`,
+       u.`surname`,
+       u.`family`,
+       u.`gender`,
+       u.`email`
 FROM
-       `{?_MYSQL_PREFIX?}_user_links` AS l
+       `{?_MYSQL_PREFIX?}_user_links` AS `l`
 LEFT JOIN
-       `{?_MYSQL_PREFIX?}_user_data` AS u
+       `{?_MYSQL_PREFIX?}_user_data` AS `u`
 ON
-       l.userid=u.userid
+       l.`userid`=u.`userid`
 WHERE
-       l.%s='%s'
+       l.`%s`=%s
 ORDER BY
-       l.userid ASC
+       l.`userid` ASC
 LIMIT %s",
                        array(
                                $col,
                                $ID,
-                               bigintval($max)
+                               bigintval($poolData['max_rec'])
                        ),__FILE__, __LINE__);
 
-               // Total number of unconfirmed mails
-               $unconfirmed = SQL_NUMROWS($result);
-
-               if ($unconfirmed > 0) {
+               // Do we have entries?
+               if (!SQL_HASZERONUMS($result)) {
                        // At least one link left to confirm
                        $OUT = '';
-                       while ($content = SQL_FETCHARRAY($result)) {
+                       while ($row = SQL_FETCHARRAY($result)) {
                                // User data found? We can take any field of u.
-                               if (!is_null($content['status'])) {
+                               if (!is_null($row['status'])) {
                                        // Prepare data for the row template
-                                       $content = array(
-                                               'userid' => $content['userid'],
-                                               'link'   => $LINK,
+                                       $row = array(
+                                               'userid' => $row['userid'],
+                                               'link'   => $mailType,
                                                'id'     => $ID,
-                                               'email'  => '<a href="' . generateEmailLink($content['email'], 'user_data') . '">' . translateGender($content['gender']) . ' ' . $content['surname'] . ' ' . $content['family'] . '</a>',
-                                               'status' => $content['status'],
+                                               'email'  => '<a href="' . generateEmailLink($row['email'], 'user_data') . '">{%pipe,translateGender=' . $row['gender'] . '%} ' . $row['surname'] . ' ' . $row['family'] . '</a>',
+                                               'status' => $row['status'],
                                        );
 
                                        // Load row template and switch colors
-                                       $OUT .= loadTemplate('admin_list_unconfirmed_row', true, $content);
+                                       $OUT .= loadTemplate('admin_list_unconfirmed_row', true, $row);
                                } else {
                                        // No user data found
-                                       $OUT .= loadTemplate('admin_list_unconfirmed_row_404', true, $content);
+                                       $OUT .= loadTemplate('admin_list_unconfirmed_row_404', true, $row);
                                }
                        } // END - while
 
                        // Render it in our new listing
                        $OUT = loadTemplate('admin_list_unconfirmed_list', true, $OUT);
-
-                       // Free memory
-                       SQL_FREERESULT($result);
                } else {
                        // All links are confirmed... strange, you shall normally not get a link to this place in this scenario... hmmm.
-                       $OUT = loadTemplate('admin_settings_saved', true, '{--ADMIN_UNCONFIRMED_NO_LINK_LEFT--}');
+                       $OUT = displayMessage('{--ADMIN_UNCONFIRMED_NO_LINK_LEFT--}', true);
                }
 
                // Prepare content
-               $content = array(
-                       'sender'      => $sender,
-                       'subject'     => $subject,
-                       'text'        => $text,
-                       'url'         => $url,
-                       'unconfirmed' => $unconfirmed,
-                       'stamp'       => generateDateTime($stamp, 2),
-                       'rows'        => $OUT
-               );
+               $content                = $poolData;
+               $content['unconfirmed'] = SQL_NUMROWS($result);
+               $content['timestamp']   = generateDateTime($poolData['timestamp'], 2);
+               $content['rows']        = $OUT;
+
+               // Free memory
+               SQL_FREERESULT($result);
 
                // Load final template
                loadTemplate('admin_list_unconfirmed', false, $content);
-       } elseif (getRequestParameter('mid') > 0) {
+       } elseif (getRequestElement('mid') > 0) {
                // Data in pool or in user_stats not found, so let's find out where data is missing
-               $result1 = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_pool` WHERE `id`=%s LIMIT 1",
-                       array(bigintval($ID)), __FILE__, __LINE__);
-               $result2 = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_user_stats` WHERE `pool_id`=%s LIMIT 1",
-                       array(bigintval($ID)), __FILE__, __LINE__);
-
-               if (SQL_NUMROWS($result1) == 1) {
+               if (countSumTotalData(bigintval($ID), 'pool', 'id', 'id', true) == 1) {
                        // pool table
-                       loadTemplate('admin_settings_saved', false, getMaskedMessage('ADMIN_UNCONFIRMED_POOL_MISSING', $ID));
-               } elseif (SQL_NUMROWS($result2) == 1) {
+                       displayMessage('{%message,ADMIN_UNCONFIRMED_POOL_MISSING=' . $ID . '%}');
+               } elseif (countSumTotalData(bigintval($ID), 'user_stats', 'id', 'pool_id', true) == 1) {
                        // user_stats table
-                       loadTemplate('admin_settings_saved', false, getMaskedMessage('ADMIN_UNCONFIRMED_STATS_MISSING', $ID));
+                       displayMessage('{%message,ADMIN_UNCONFIRMED_STATS_MISSING=' . $ID . '%}');
                } else {
                        // both or link is invalid
-                       loadTemplate('admin_settings_saved', false, '{--ADMIN_UNCONFIRMED_INVALID_LINK--}');
+                       displayMessage('{--ADMIN_UNCONFIRMED_INVALID_LINK--}');
                }
-
-               // Free memory
-               SQL_FREERESULT($result1);
-               SQL_FREERESULT($result2);
-       } elseif (isGetRequestParameterSet('bid')) {
+       } elseif (isGetRequestElementSet('bid')) {
                // Data in bonus table not found
-               loadTemplate('admin_settings_saved', false, '{--ADMIN_UNCONFIRMED_INVALID_LINK--}');
+               displayMessage('{--ADMIN_UNCONFIRMED_INVALID_LINK--}');
        }
 
        // Free result