Better feedback for saving categories
[mailer.git] / inc / modules / member / what-categories.php
index 9f2f814446bd8e5722794bf15e0cfdf5b6d362a5..c6674511b9ebb259c71f5de9deac71333594a4ef 100644 (file)
@@ -17,7 +17,7 @@
  * Needs to be in all Files and every File needs "svn propset           *
  * svn:keywords Date Revision" (autoprobset!) at least!!!!!!            *
  * -------------------------------------------------------------------- *
- * Copyright (c) 2003 - 2008 by Roland Haeder                           *
+ * Copyright (c) 2003 - 2009 by Roland Haeder                           *
  * For more information visit: http://www.mxchange.org                  *
  *                                                                      *
  * This program is free software; you can redistribute it and/or modify *
 
 // Some security stuff...
 if (!defined('__SECURITY')) {
-       $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), '/inc') + 4) . '/security.php';
-       require($INC);
-} elseif (!IS_MEMBER()) {
-       redirectToUrl('modules.php?module=index');
+       die();
+} elseif (!isMember()) {
+       redirectToIndexMemberOnlyModule();
 }
 
 // Add description as navigation point
-ADD_DESCR('member', __FILE__);
+addMenuDescription('member', __FILE__);
 
 $UID = getUserId();
 $whereStatement = " WHERE `visible`='Y'";
-if (IS_ADMIN()) $whereStatement = '';
+if (isAdmin()) $whereStatement = '';
 
-$result = SQL_QUERY("SELECT id, cat FROM `{!_MYSQL_PREFIX!}_cats`".$whereStatement." ORDER BY `sort`", __FILE__, __LINE__);
+$result = SQL_QUERY("SELECT id, cat FROM `{?_MYSQL_PREFIX?}_cats`".$whereStatement." ORDER BY `sort`", __FILE__, __LINE__);
 $cats = SQL_NUMROWS($result);
 
 if ($cats > 0) {
        $LEAST = false;
-       if (IS_FORM_SENT()) {
+       if (isFormSent()) {
                $cnt = 0;
-               foreach (REQUEST_POST('cat') as $cat => $joined) {
-                       if ($joined == 'N') $cnt++;
-               }
+               foreach (postRequestElement('cat') as $cat => $joined) {
+                       if ($joined != 'Y') $cnt++;
+               } // END - foreach
 
                if (($cats - $cnt) < getConfig('least_cats')) {
-                       REQUEST_UNSET_POST('ok');
+                       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 = '';
 
-       if (IS_FORM_SENT()) {
-               foreach (REQUEST_POST('cat') as $cat => $joined) {
+                       // 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) {
-                                               $sql = "INSERT INTO `{!_MYSQL_PREFIX!}_user_cats` (userid, cat_id) VALUES ('%s','%s')";
-                                       } else {
-                                               // Free memory
-                                               SQL_FREERESULT($result_user);
-                                       }
+                                               $sql = "INSERT INTO `{?_MYSQL_PREFIX?}_user_cats` (userid, cat_id) VALUES ('%s','%s')";
+                                       } // 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...
-               LOAD_TEMPLATE('admin_settings_saved', true, getMessage('MEMBER_CATS_SAVED'));
        } else {
                if ($LEAST) {
                        // Also here we have to secure it... :(
-                       LOAD_TEMPLATE('admin_settings_saved', false, sprintf(getMessage('CATS_LEAST'), getConfig('least_cats')));
-               }
+                       loadTemplate('admin_settings_saved', false, sprintf(getMessage('CATS_LEAST'), getConfig('least_cats')));
+               } // END - if
+
                // Put some data into constants for the template
-               define('__ROWS', ($cats*2+4));
+               $content['rows'] = ($cats*2+4);
 
                // Load header template
-               LOAD_TEMPLATE("member_cats_header");
+               loadTemplate('member_cats_header', false, $content);
 
                // Start switching colors and load all visible categories
                // @TODO Rewrite this to use $OUT .= ...
@@ -117,15 +134,15 @@ if ($cats > 0) {
                        $content['jy'] = '';
 
                        // When we found an entry don't read it, just change the jx elements
-                       if (REQUEST_ISSET_POST(('cat'))) {
+                       if (isPostRequestElementSet(('cat'))) {
                                // Form sent?
-                               if (REQUEST_POST('cat', $content['id']) =='Y') {
+                               if (postRequestElement('cat', $content['id']) =='Y') {
                                        $content['jy'] = ' checked="checked"';
                                        $content['jn'] = '';
                                }
                        } else {
                                // Check if he has an entry
-                               if (GET_TOTAL_DATA($UID, "user_cats", "id", 'userid', true, sprintf(" AND cat_id=%s", bigintval($content['id']))) == 1) {
+                               if (countSumTotalData($UID, 'user_cats', 'id', 'userid', true, sprintf(" AND `cat_id`=%s", bigintval($content['id']))) == 1) {
                                        $content['jn'] = '';
                                        $content['jy'] = ' checked="checked"';
                                }
@@ -135,16 +152,16 @@ if ($cats > 0) {
                        $content['sw'] = $SW;
 
                        // Load row template and switch colors
-                       LOAD_TEMPLATE("member_cat_row", false, $content);
+                       loadTemplate('member_cat_row', false, $content);
                        $SW = 3 - $SW;
                }
 
                // Load footer template
-               LOAD_TEMPLATE("member_cats_footer");
+               loadTemplate('member_cats_footer');
        }
 } else {
        // No cateogries are defined yet
-       LOAD_TEMPLATE('admin_settings_saved', true, getMessage('MEMBER_NO_CATS'));
+       loadTemplate('admin_settings_saved', true, getMessage('MEMBER_NO_CATS'));
 }
 
 // Free result