X-Git-Url: https://git.mxchange.org/?p=mailer.git;a=blobdiff_plain;f=inc%2Fmodules%2Fmember%2Fwhat-categories.php;h=45e79d7492528bf8cf62b26c3dc4871debd1d0ce;hp=bc0a4213626dafb222e25f354a47ec72f8cf395b;hb=db0c6702086eea2c44d0aae1702dc2e77a0afc4e;hpb=57227d33e870ec5cd271209c4a978a52b45c2dd6

diff --git a/inc/modules/member/what-categories.php b/inc/modules/member/what-categories.php
index bc0a421362..45e79d7492 100644
--- a/inc/modules/member/what-categories.php
+++ b/inc/modules/member/what-categories.php
@@ -14,10 +14,9 @@
  * $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 - 2011 by Mailer Developer Team                   *
  * For more information visit: http://www.mxchange.org                  *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
@@ -44,23 +43,24 @@ if (!defined('__SECURITY')) {
 }
 
 // Add description as navigation point
-addMenuDescription('member', __FILE__);
+addYouAreHereLink('member', __FILE__);
 
 $whereStatement = " WHERE `visible`='Y'";
 if (isAdmin()) $whereStatement = '';
 
-$result = SQL_QUERY("SELECT id, cat FROM `{?_MYSQL_PREFIX?}_cats`".$whereStatement." ORDER BY `sort`", __FILE__, __LINE__);
-$cats = SQL_NUMROWS($result);
+// Get all categories
+$result = SQL_QUERY("SELECT `id`, `cat` FROM `{?_MYSQL_PREFIX?}_cats`".$whereStatement." ORDER BY `sort` ASC", __FILE__, __LINE__);
 
-if ($cats > 0) {
+// Do we have entries?
+if (!SQL_HASZERONUMS($result)) {
 	$LEAST = false;
 	if (isFormSent()) {
-		$cnt = '0';
+		$count = '0';
 		foreach (postRequestParameter('cat') as $cat => $joined) {
-			if ($joined != 'Y') $cnt++;
+			if ($joined != 'Y') $count++;
 		} // END - foreach
 
-		if (($cats - $cnt) < getConfig('least_cats')) {
+		if ((SQL_NUMROWS($result) - $count) < getLeastCats()) {
 			unsetPostRequestParameter('ok');
 			$LEAST = true;
 		} // END - if
@@ -69,7 +69,7 @@ if ($cats > 0) {
 	// Is the form sent?
 	if (isFormSent()) {
 		// Start counting all
-		$cnt = '0';
+		$count = '0';
 
 		// Go through all entries
 		foreach (postRequestParameter('cat') as $cat => $joined) {
@@ -82,8 +82,11 @@ if ($cats > 0) {
 					$result_user = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_user_cats` WHERE `userid`=%s AND `cat_id`=%s LIMIT 1",
 						array(getMemberId(), bigintval($cat)), __FILE__, __LINE__);
 
-					if (SQL_NUMROWS($result_user) == '0') {
-						$sql = "INSERT INTO `{?_MYSQL_PREFIX?}_user_cats` (userid, cat_id) VALUES ('%s','%s')";
+					// Debug message
+					//* DEBUG: */ logDebugMessage(__FILE__, __LINE__, 'cat='.$cat.',joined='.$joined.',getMemberId()='.getMemberId().',SQL_HASZERONUMS()='.intval(SQL_HASZERONUMS($result_user)));
+
+					if (SQL_HASZERONUMS($result_user)) {
+						$sql = "INSERT INTO `{?_MYSQL_PREFIX?}_user_cats` (`userid`, `cat_id`) VALUES (%s,%s)";
 					} // END - if
 
 					// Free memory
@@ -100,71 +103,67 @@ if ($cats > 0) {
 				SQL_QUERY_ESC(trim($sql), array(getMemberId(), bigintval($cat)), __FILE__, __LINE__);
 
 				// Count this row
-				$cnt += SQL_AFFECTEDROWS();
+				$count += SQL_AFFECTEDROWS();
 			} // END - if
 		} // END - foreach
 
 		// Categories saved?
-		if ($cnt > 0) {
+		if ($count > 0) {
 			// Output message
-			loadTemplate('admin_settings_saved', false, getMessage('MEMBER_CATS_SAVED'));
+			displayMessage('{--MEMBER_CATEGORIES_SAVED--}');
 		} else {
 			// None save
-			loadTemplate('admin_settings_saved', false, getMessage('MEMBER_CATS_NOT_SAVED'));
+			displayMessage('{--MEMBER_CATEGORIES_NOT_SAVED--}');
 		}
 	} else {
 		if ($LEAST === true) {
 			// Also here we have to secure it... :(
-			loadTemplate('admin_settings_saved', false, getMessage('CATS_LEAST'));
+			displayMessage('{--CHOOSE_MORE_CATEGORIES--}');
 		} // END - if
 
 		// Put some data into constants for the template
-		$content['rows'] = ($cats*2+4);
+		$content['rows'] = (SQL_NUMROWS($result) * 2 + 4);
 
 		// Load header template
 		loadTemplate('member_cats_header', false, $content);
 
 		// Start switching colors and load all visible categories
 		// @TODO Rewrite this to use $OUT .= ...
-		$OUT = ''; $SW = 2;
+		$OUT = '';
 		while ($content = SQL_FETCHARRAY($result)) {
 			// Default he has not joined
 			$content['jn'] = ' checked="checked"';
 			$content['jy'] = '';
 
 			// When we found an entry don't read it, just change the jx elements
-			if (isPostRequestParameterSet(('cat'))) {
+			if ((isFormSent()) && (isPostRequestParameterSet('cat'))) {
 				// Form sent?
 				if (postRequestParameter('cat', $content['id']) =='Y') {
 					$content['jy'] = ' checked="checked"';
 					$content['jn'] = '';
-				}
+				} // END - if
 			} else {
 				// Check if he has an entry
 				if (countSumTotalData(getMemberId(), 'user_cats', 'id', 'userid', true, sprintf(" AND `cat_id`=%s", bigintval($content['id']))) == 1) {
 					$content['jn'] = '';
 					$content['jy'] = ' checked="checked"';
-				}
+				} // END - if
 			}
 
-			// Add some more
-			$content['sw'] = $SW;
-
 			// Load row template and switch colors
 			loadTemplate('member_cat_row', false, $content);
-			$SW = 3 - $SW;
-		}
+		} // END - while
 
 		// Load footer template
 		loadTemplate('member_cats_footer');
 	}
 } else {
 	// No cateogries are defined yet
-	loadTemplate('admin_settings_saved', true, getMessage('MEMBER_NO_CATS'));
+	displayMessage('{--MEMBER_NO_CATEGORIES--}');
 }
 
 // Free result
 SQL_FREERESULT($result);
 
-//
+// [EOF]
 ?>