Our convention says if(-else) blocks should not be put in one line

[mailer.git] / inc / modules / admin / what-email_archiv.php

diff --git a/inc/modules/admin/what-email_archiv.php b/inc/modules/admin/what-email_archiv.php
index eba3a148f015dde1651526f3206f5435f38b6b00..e5f66afc1197cd734ce1e08ef2dce39521eef342 100644 (file)
--- a/inc/modules/admin/what-email_archiv.php
+++ b/inc/modules/admin/what-email_archiv.php
@@ -59,7 +59,7 @@ ORDER BY
        `timestamp` DESC";
 
 // Check for maximum pages
-$result_maximum = SQL_QUERY(, __FILE__, __LINE__);
+$result_maximum = SQL_QUERY($sql, __FILE__, __LINE__);
 
 // Set offset an current page to default values
 if (!isGetRequestParameterSet('page'))   setGetRequestParameter('page'  , 1);
@@ -77,7 +77,7 @@ $numPages = round(SQL_NUMROWS($result_maximum) / getConfig('mails_page') + 0.5);
 // Free the result which we don't need
 SQL_FREERESULT($result_maximum);
 
-if (SQL_NUMROWS($result) > 0) {
+if (!SQL_HASZERONUMS($result)) {
        // Mail orders are in pool so we can display them
 
        // Add navigation table rows
@@ -85,7 +85,7 @@ if (SQL_NUMROWS($result) > 0) {
        $content['bottom_email_nav'] = addEmailNavigation($numPages, getConfig('mails_page'), false, 3, true);
 
        // List all entries
-       $OUT = ''; $SW = 2;
+       $OUT = '';
        while ($pool = SQL_FETCHARRAY($result)) {
                // Check sent mails and clicks
                $result_mails = SQL_QUERY_ESC("SELECT `max_rec`, `clicks` FROM `{?_MYSQL_PREFIX?}_user_stats` WHERE `pool_id`=%s LIMIT 1",
@@ -101,7 +101,6 @@ if (SQL_NUMROWS($result) > 0) {
                $pool['unconfirmed'] = generateUnconfirmedAdminLink($pool['id'],  ($pool['max_rec'] - $pool['clicks']), 'mid');
 
                // Prepare data for the row template
-               $content['sw'] = $SW;
                $content['timestamp'] = generateDateTime($pool['timestamp'], 0);
 
                // Merge pool data with content
@@ -109,7 +108,6 @@ if (SQL_NUMROWS($result) > 0) {
 
                // Load row template and switch colors
                $OUT .= loadTemplate('admin_email_archiv_row', true, $content);
-               $SW = 3 - $SW;
        } // END - while
 
        // Remmber generated rows in array