Comments fixed, ext-network continued, fix for mod stats:

[mailer.git] / inc / modules / member / what-categories.php

diff --git a/inc/modules/member/what-categories.php b/inc/modules/member/what-categories.php
index 24b550ea4edd6f96e7041030e32e43a86ff734d4..74e03ce2fbe8dab21a228a3d77848fd5a765c9aa 100644 (file)
--- 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,7 @@ if (!defined('__SECURITY')) {
 // Add description as navigation point
 addMenuDescription('member', __FILE__);
 
-$UID = getUserId();
+$UID = getMemberId();
 $whereStatement = " WHERE `visible`='Y'";
 if (isAdmin()) $whereStatement = '';
 
@@ -56,51 +56,68 @@ $cats = SQL_NUMROWS($result);
 if ($cats > 0) {
        $LEAST = false;
        if (isFormSent()) {
-               $cnt = 0;
+               $cnt = '0';
                foreach (postRequestElement('cat') as $cat => $joined) {
                        if ($joined != 'Y') $cnt++;
-               }
+               } // END - foreach
 
                if (($cats - $cnt) < getConfig('least_cats')) {
                        unsetPostRequestElement('ok');
                        $LEAST = true;
-               }
-       }
+               } // END - if
+       } // END - if
 
+       // Is the form sent?
        if (isFormSent()) {
+               // Start counting all
+               $cnt = '0';
+
+               // Go through all entries
                foreach (postRequestElement('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($UID, 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($UID, 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) {
                        // Also here we have to secure it... :(
                        loadTemplate('admin_settings_saved', false, sprintf(getMessage('CATS_LEAST'), getConfig('least_cats')));
-               }
+               } // END - if
 
                // Put some data into constants for the template
                $content['rows'] = ($cats*2+4);