X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmodules%2Fadmin%2Fwhat-refbanner.php;h=92009735a150a9cb9b697c8b7902a7fc1dd544ad;hp=174f837e97e8060a0eb26204ae6bd4fea60ebef0;hb=a4dd1eb45ae25ed017a862a4edac1e2828acbee6;hpb=0a69b4e17157eda9b6e1c70ff494292ccf8cbbd0

diff --git a/inc/modules/admin/what-refbanner.php b/inc/modules/admin/what-refbanner.php
index 174f837e97..92009735a1 100644
--- a/inc/modules/admin/what-refbanner.php
+++ b/inc/modules/admin/what-refbanner.php
@@ -18,6 +18,7 @@
  * 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                  *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
@@ -45,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
@@ -56,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__);
-			if (SQL_NUMROWS($result) == '0') {
+			$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
@@ -75,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),
@@ -84,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) && (isPostRequestParameterSet('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) && (isPostRequestParameterSet('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",
@@ -133,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]