X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fmember%2Fwhat-categories.php;h=bc0a4213626dafb222e25f354a47ec72f8cf395b;hb=de3147a2ad17532517bb3e67d8dee67d68cdcec6;hp=24b550ea4edd6f96e7041030e32e43a86ff734d4;hpb=64c8349613addc3da2242c5cd6b99d64e3fb5f8e;p=mailer.git

diff --git a/inc/modules/member/what-categories.php b/inc/modules/member/what-categories.php
index 24b550ea4e..bc0a421362 100644
--- a/inc/modules/member/what-categories.php
+++ b/inc/modules/member/what-categories.php
@@ -1,7 +1,7 @@
 <?php
 /************************************************************************
- * MXChange v0.2.1                                    Start: 10/19/2003 *
- * ===============                              Last change: 06/30/2004 *
+ * Mailer v0.2.1-FINAL                                Start: 10/19/2003 *
+ * ===================                          Last change: 06/30/2004 *
  *                                                                      *
  * -------------------------------------------------------------------- *
  * File              : what-categories.php                              *
@@ -46,7 +46,6 @@ if (!defined('__SECURITY')) {
 // Add description as navigation point
 addMenuDescription('member', __FILE__);
 
-$UID = getUserId();
 $whereStatement = " WHERE `visible`='Y'";
 if (isAdmin()) $whereStatement = '';
 
@@ -56,51 +55,68 @@ $cats = SQL_NUMROWS($result);
 if ($cats > 0) {
 	$LEAST = false;
 	if (isFormSent()) {
-		$cnt = 0;
-		foreach (postRequestElement('cat') as $cat => $joined) {
+		$cnt = '0';
+		foreach (postRequestParameter('cat') as $cat => $joined) {
 			if ($joined != 'Y') $cnt++;
-		}
+		} // END - foreach
 
 		if (($cats - $cnt) < getConfig('least_cats')) {
-			unsetPostRequestElement('ok');
+			unsetPostRequestParameter('ok');
 			$LEAST = true;
-		}
-	}
+		} // END - if
+	} // END - if
 
+	// Is the form sent?
 	if (isFormSent()) {
-		foreach (postRequestElement('cat') as $cat => $joined) {
+		// Start counting all
+		$cnt = '0';
+
+		// Go through all entries
+		foreach (postRequestParameter('cat') as $cat => $joined) {
+			// Ini sql here
+			$sql = '';
+
+			// Has the user joined on this category?
 			switch ($joined) {
 				case 'Y':
-					$sql = '';
-					$result_user = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_user_cats` WHERE `userid`=%s AND cat_id=%s LIMIT 1",
-					array($UID, bigintval($cat)), __FILE__, __LINE__);
+					$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) {
+					if (SQL_NUMROWS($result_user) == '0') {
 						$sql = "INSERT INTO `{?_MYSQL_PREFIX?}_user_cats` (userid, cat_id) VALUES ('%s','%s')";
-					} else {
-						// Free memory
-						SQL_FREERESULT($result_user);
-					}
+					} // END - if
+
+					// Free memory
+					SQL_FREERESULT($result_user);
 					break;
 
 				case 'N':
-					$sql = "DELETE LOW_PRIORITY FROM `{?_MYSQL_PREFIX?}_user_cats` WHERE `userid`=%s AND cat_id=%s LIMIT 1";
+					$sql = "DELETE LOW_PRIORITY FROM `{?_MYSQL_PREFIX?}_user_cats` WHERE `userid`=%s AND `cat_id`=%s LIMIT 1";
 					break;
-			}
+			} // END - switch
 
 			if (!empty($sql)) {
 				// Run SQL command
-				$result = SQL_QUERY_ESC(trim($sql), array($UID, bigintval($cat)), __FILE__, __LINE__);
-			}
+				SQL_QUERY_ESC(trim($sql), array(getMemberId(), bigintval($cat)), __FILE__, __LINE__);
+
+				// Count this row
+				$cnt += SQL_AFFECTEDROWS();
+			} // END - if
+		} // END - foreach
+
+		// Categories saved?
+		if ($cnt > 0) {
+			// Output message
+			loadTemplate('admin_settings_saved', false, getMessage('MEMBER_CATS_SAVED'));
+		} else {
+			// None save
+			loadTemplate('admin_settings_saved', false, getMessage('MEMBER_CATS_NOT_SAVED'));
 		}
-
-		// Categories saved...
-		loadTemplate('admin_settings_saved', true, getMessage('MEMBER_CATS_SAVED'));
 	} else {
-		if ($LEAST) {
+		if ($LEAST === true) {
 			// Also here we have to secure it... :(
-			loadTemplate('admin_settings_saved', false, sprintf(getMessage('CATS_LEAST'), getConfig('least_cats')));
-		}
+			loadTemplate('admin_settings_saved', false, getMessage('CATS_LEAST'));
+		} // END - if
 
 		// Put some data into constants for the template
 		$content['rows'] = ($cats*2+4);
@@ -117,15 +133,15 @@ if ($cats > 0) {
 			$content['jy'] = '';
 
 			// When we found an entry don't read it, just change the jx elements
-			if (isPostRequestElementSet(('cat'))) {
+			if (isPostRequestParameterSet(('cat'))) {
 				// Form sent?
-				if (postRequestElement('cat', $content['id']) =='Y') {
+				if (postRequestParameter('cat', $content['id']) =='Y') {
 					$content['jy'] = ' checked="checked"';
 					$content['jn'] = '';
 				}
 			} else {
 				// Check if he has an entry
-				if (countSumTotalData($UID, 'user_cats', 'id', 'userid', true, sprintf(" AND `cat_id`=%s", bigintval($content['id']))) == 1) {
+				if (countSumTotalData(getMemberId(), 'user_cats', 'id', 'userid', true, sprintf(" AND `cat_id`=%s", bigintval($content['id']))) == 1) {
 					$content['jn'] = '';
 					$content['jy'] = ' checked="checked"';
 				}