Fixes for wrong tewmplate names, new network added:

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

diff --git a/inc/modules/admin/what-refbanner.php b/inc/modules/admin/what-refbanner.php
index dae9146b221e9c8f382594b5edeba05ab567c728..92009735a150a9cb9b697c8b7902a7fc1dd544ad 100644 (file)
--- a/inc/modules/admin/what-refbanner.php
+++ b/inc/modules/admin/what-refbanner.php
@@ -46,9 +46,9 @@ if ((!defined('__SECURITY')) || (!isAdmin())) {
 addMenuDescription('admin', __FILE__);
 
 // Some sanity-check
-if ((!isPostRequestParameterSet('url')) || (!isPostRequestParameterSet(('alternate')))) {
+if ((!isPostRequestParameterSet('url')) || (!isPostRequestParameterSet('alternate'))) {
        unsetPostRequestParameter('ok');
-}
+} // END - if
 
 if (isFormSent()) {
        // Fix older calls from add-new-banner-form
@@ -57,17 +57,23 @@ if (isFormSent()) {
        switch (getRequestParameter('mode')) {
                case 'add':
                        // Check if banner is already added
-                       $result = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_refbanner` WHERE url='%s' LIMIT 1",
-                       array(postRequestParameter('url')), __FILE__, __LINE__);
+                       $result = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_refbanner` WHERE `url`='%s' LIMIT 1",
+                               array(postRequestParameter('url')), __FILE__, __LINE__);
+
+                       // Was the banner found?
                        if (SQL_HASZERONUMS($result)) {
                                // Add banner
-                               SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_refbanner` (url, alternate, visible)
+                               SQL_QUERY_ESC("INSERT INTO `{?_MYSQL_PREFIX?}_refbanner` (`url`, `alternate`, `visible`)
 VALUES ('%s','%s','%s')",
-                               array(postRequestParameter('url'), postRequestParameter('alternate'), postRequestParameter('visible')), __FILE__, __LINE__);
-                       } else {
-                               // Free memory
-                               SQL_FREERESULT($result);
-                       }
+                                       array(
+                                               postRequestParameter('url'),
+                                               postRequestParameter('alternate'),
+                                               postRequestParameter('visible')
+                                       ), __FILE__, __LINE__);
+                       } // END - if
+
+                       // Free memory
+                       SQL_FREERESULT($result);
                        break;
 
                case 'edit': // Update banner
@@ -76,7 +82,7 @@ VALUES ('%s','%s','%s')",
                                $id = bigintval($id);
 
                                // Update entry
-                               SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_refbanner` SET url='%s', alternate='%s', `visible`='%s' WHERE `id`=%s LIMIT 1",
+                               SQL_QUERY_ESC("UPDATE `{?_MYSQL_PREFIX?}_refbanner` SET `url`='%s', `alternate`='%s', `visible`='%s' WHERE `id`=%s LIMIT 1",
                                        array(
                                                postRequestParameter('url', $id),
                                                postRequestParameter('alternate', $id),
@@ -85,47 +91,40 @@ VALUES ('%s','%s','%s')",
                                        ), __FILE__, __LINE__);
                        }
                        break;
-       }
+       } // END - switch
 
        // Check if we have saved (updated)
        if (SQL_AFFECTEDROWS() == 1) {
                // Updated!
-               $content = getMessage('SETTINGS_SAVED');
+               $content = '{--SETTINGS_SAVED--}';
        } else {
                // Nothing has been updated
-               $content = '<span class="admin_failed">{--SETTINGS_NOT_SAVED--}</span>';
+               $content = '<span class="notice">{--SETTINGS_NOT_SAVED--}</span>';
        }
 
        // Display message
        loadTemplate('admin_settings_saved', false, $content);
-} elseif ((countPostSelection() > 0) && (isFormSent('edit'))) {
+} elseif ((ifPostContainsSelections()) && (isFormSent('edit'))) {
        // Edit banner
-       $SW = 2; $OUT = '';
+       $OUT = '';
        foreach (postRequestParameter('sel') as $id => $sel) {
                // Load data
-               $result = SQL_QUERY_ESC("SELECT url, alternate, visible FROM `{?_MYSQL_PREFIX?}_refbanner` WHERE `id`=%s LIMIT 1",
-               array(bigintval($id)), __FILE__, __LINE__);
-               list($url, $alt, $vis) = SQL_FETCHROW($result);
+               $result = SQL_QUERY_ESC("SELECT `id`, `url`, `alternate`, `visible` FROM `{?_MYSQL_PREFIX?}_refbanner` WHERE `id`=%s LIMIT 1",
+                       array(bigintval($id)), __FILE__, __LINE__);
+               $content = SQL_FETCHARRAY($result);
                SQL_FREERESULT($result);
 
                // Preapre data for the row
-               $content = array(
-                       'sw'  => $SW,
-                       'id'  => $id,
-                       'url' => $url,
-                       'alt' => $alt,
-                       'vis' => addSelectionBox('yn', $vis   , 'visible'),
-               );
+               $content['visible'] = addSelectionBox('yn', $content['visible'], 'visible');
 
                // Load row template and switch color
-               $OUT .= loadTemplate('admin_refbanner_edit_row', true, $content);
-               $SW = 3 - $SW;
-       }
+               $OUT .= loadTemplate('admin_edit_refbanner_row', true, $content);
+       } // END - foreach
 
        // Load main template
-       loadTemplate('admin_refbanner_edit', false, $OUT);
+       loadTemplate('admin_edit_refbanner', false, $OUT);
 } else {
-       if ((countPostSelection() > 0) && (isFormSent('del'))) {
+       if ((ifPostContainsSelections()) && (isFormSent('del'))) {
                // Delete banner
                foreach (postRequestParameter('sel') as $id => $sel) {
                        SQL_QUERY_ESC("DELETE LOW_PRIORITY FROM `{?_MYSQL_PREFIX?}_refbanner` WHERE `id`=%s LIMIT 1",
@@ -134,39 +133,26 @@ VALUES ('%s','%s','%s')",
        } // END - if
 
        // Referal levels
-       $result = SQL_QUERY("SELECT id, url, alternate, visible, counter, clicks FROM `{?_MYSQL_PREFIX?}_refbanner` ORDER BY url", __FILE__, __LINE__);
+       $result = SQL_QUERY('SELECT `id`, `url`, `alternate`, `visible`, `counter`, `clicks` FROM `{?_MYSQL_PREFIX?}_refbanner` ORDER BY `url` ASC', __FILE__, __LINE__);
 
        // Entries found?
-       if (SQL_NUMROWS($result) > 0) {
+       if (!SQL_HASZERONUMS($result)) {
                // Make referal banner editable and deletable
-               $OUT = ''; $SW = 2;
+               $OUT = '';
                while ($content = SQL_FETCHARRAY($result)) {
-                       // Preapre data for the row
-                       // @TODO Rewritings: alt->alternate,cnt->counter,clx->clicks in template
-                       $content = array(
-                               'sw'  => $SW,
-                               'id'  => $content['id'],
-                               'url' => $content['url'],
-                               'alt' => $content['alternate'],
-                               'vis' => translateYesNo($content['visible']),
-                               'cnt' => $content['counter'],
-                               'clx' => $content['clicks']
-                       );
-
                        // Load row template and switch color
-                       $OUT .= loadTemplate('admin_refbanner_row', true, $content);
-                       $SW = 3 - $SW;
-               }
+                       $OUT .= loadTemplate('admin_list_refbanner_row', true, $content);
+               } // END - while
 
                // Free memory
                SQL_FREERESULT($result);
 
                // Load main template
-               loadTemplate('admin_refbanner', false, $OUT);
-       }
+               loadTemplate('admin_list_refbanner', false, $OUT);
+       } // END - if
 
        // Form for adding new referal levels
-       loadTemplate('admin_add_banner');
+       loadTemplate('admin_add_refbanner');
 }
 
 // [EOF]