config.php partly solved, see #117

[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 2db438ae4c4a4eabe42dd0605372f9a335d2e098..9f2f814446bd8e5722794bf15e0cfdf5b6d362a5 100644 (file)
--- a/inc/modules/member/what-categories.php
+++ b/inc/modules/member/what-categories.php
@@ -10,7 +10,7 @@
  * -------------------------------------------------------------------- *
  * Kurzbeschreibung  : Kategorien ab- bzw. anwaehlen                    *
  * -------------------------------------------------------------------- *
- * $Revision:: 856                                                    $ *
+ * $Revision::                                                        $ *
  * $Date::                                                            $ *
  * $Tag:: 0.2.1-FINAL                                                 $ *
  * $Author::                                                          $ *
@@ -38,18 +38,18 @@
 
 // Some security stuff...
 if (!defined('__SECURITY')) {
-       $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), "/inc") + 4) . "/security.php";
+       $INC = substr(dirname(__FILE__), 0, strpos(dirname(__FILE__), '/inc') + 4) . '/security.php';
        require($INC);
 } elseif (!IS_MEMBER()) {
-       LOAD_URL("modules.php[13~?module=index");
+       redirectToUrl('modules.php?module=index');
 }
 
 // Add description as navigation point
-ADD_DESCR("member", __FILE__);
+ADD_DESCR('member', __FILE__);
 
 $UID = getUserId();
 $whereStatement = " WHERE `visible`='Y'";
-if (IS_ADMIN()) $whereStatement = "";
+if (IS_ADMIN()) $whereStatement = '';
 
 $result = SQL_QUERY("SELECT id, cat FROM `{!_MYSQL_PREFIX!}_cats`".$whereStatement." ORDER BY `sort`", __FILE__, __LINE__);
 $cats = SQL_NUMROWS($result);
@@ -59,7 +59,7 @@ if ($cats > 0) {
        if (IS_FORM_SENT()) {
                $cnt = 0;
                foreach (REQUEST_POST('cat') as $cat => $joined) {
-                       if ($joined == "N") $cnt++;
+                       if ($joined == 'N') $cnt++;
                }
 
                if (($cats - $cnt) < getConfig('least_cats')) {
@@ -71,9 +71,9 @@ if ($cats > 0) {
        if (IS_FORM_SENT()) {
                foreach (REQUEST_POST('cat') as $cat => $joined) {
                        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",
+                               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__);
 
                                        if (SQL_NUMROWS($result_user) == 0) {
@@ -84,7 +84,7 @@ if ($cats > 0) {
                                        }
                                        break;
 
-                               case "N":
+                               case 'N':
                                        $sql = "DELETE LOW_PRIORITY FROM `{!_MYSQL_PREFIX!}_user_cats` WHERE userid=%s AND cat_id=%s LIMIT 1";
                                        break;
                        }
@@ -96,11 +96,11 @@ if ($cats > 0) {
                }
 
                // Categories saved...
-               LOAD_TEMPLATE("admin_settings_saved", true, getMessage('MEMBER_CATS_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')));
+                       LOAD_TEMPLATE('admin_settings_saved', false, sprintf(getMessage('CATS_LEAST'), getConfig('least_cats')));
                }
                // Put some data into constants for the template
                define('__ROWS', ($cats*2+4));
@@ -110,24 +110,24 @@ if ($cats > 0) {
 
                // Start switching colors and load all visible categories
                // @TODO Rewrite this to use $OUT .= ...
-               $OUT = ""; $SW = 2;
+               $OUT = ''; $SW = 2;
                while ($content = SQL_FETCHARRAY($result)) {
                        // Default he has not joined
-                       $content['jn'] = " checked=\"checked\"";
-                       $content['jy'] = "";
+                       $content['jn'] = ' checked="checked"';
+                       $content['jy'] = '';
 
                        // When we found an entry don't read it, just change the jx elements
                        if (REQUEST_ISSET_POST(('cat'))) {
                                // Form sent?
                                if (REQUEST_POST('cat', $content['id']) =='Y') {
-                                       $content['jy'] = " checked=\"checked\"";
-                                       $content['jn'] = "";
+                                       $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) {
-                                       $content['jn'] = "";
-                                       $content['jy'] = " checked=\"checked\"";
+                               if (GET_TOTAL_DATA($UID, "user_cats", "id", 'userid', true, sprintf(" AND cat_id=%s", bigintval($content['id']))) == 1) {
+                                       $content['jn'] = '';
+                                       $content['jy'] = ' checked="checked"';
                                }
                        }
 
@@ -144,7 +144,7 @@ if ($cats > 0) {
        }
 } else {
        // No cateogries are defined yet
-       LOAD_TEMPLATE("admin_settings_saved", true, getMessage('MEMBER_NO_CATS'));
+       LOAD_TEMPLATE('admin_settings_saved', true, getMessage('MEMBER_NO_CATS'));
 }
 
 // Free result