More HTML tags ported to XHTML (all lower-case), bug #33 resolved

[mailer.git] / inc / modules / admin / what-add_points.php

diff --git a/inc/modules/admin/what-add_points.php b/inc/modules/admin/what-add_points.php
index f514b885cdee17f65e6ffebc4523d5000e16493f..3232be3b56cd07499354588eff721aa88ca1444c 100644 (file)
--- a/inc/modules/admin/what-add_points.php
+++ b/inc/modules/admin/what-add_points.php
@@ -47,38 +47,26 @@ if ($_GET['u_id'] == "all") {
        // Add points to all accounts
        if ((isset($_POST['ok'])) && ($_POST['points'] > 0)) {
                define('__POINTS_VALUE', $_POST['points']);
-               $result_main = SQL_QUERY("SELECT userid FROM "._MYSQL_PREFIX."_user_data WHERE status='CONFIRMED' ORDER BY userid", __FILE__, __LINE__);
+               $result_main = SQL_QUERY("SELECT userid FROM `"._MYSQL_PREFIX."_user_data` WHERE status='CONFIRMED' ORDER BY userid", __FILE__, __LINE__);
                while (list($uid) = SQL_FETCHROW($result_main)) {
-                       // User ID found in URL so we use this give him some credits
-                       $result = SQL_QUERY_ESC("SELECT surname, family, email FROM "._MYSQL_PREFIX."_user_data WHERE userid=%s AND status='CONFIRMED' LIMIT 1",
-                        array(bigintval($uid)), __FILE__, __LINE__);
-                       if (SQL_NUMROWS($result) == 1) {
-                               // Selected user does exist
-                               list($sname, $fname, $email) = SQL_FETCHROW($result);
-                               SQL_FREERESULT($result);
-
-                               if ((isset($_POST['ok'])) && (!empty($_POST['points']))) {
-                                       global $DEPTH;
-                                       // Remove depth to prevent booking errors. This is a bad coding
-                                       // practice, thats also why we need to write this project from
-                                       // scratch...
-                                       unset($DEPTH);
+                       // Remove depth to prevent booking errors. This is a bad coding
+                       // practice, thats also why we need to write this project from
+                       // scratch...
+                       $GLOBALS['ref_level'] = -1;
 
-                                       // Ok, add points and send an email to him...
-                                       ADD_POINTS_REFSYSTEM("admin_all", $uid, bigintval($_POST['points']), false, "0", false, "direct");
+                       // Ok, add points and send an email to him...
+                       ADD_POINTS_REFSYSTEM("admin_all", $uid, bigintval($_POST['points']), false, "0", false, "direct");
 
-                                       // Prepare content
-                                       $content = array(
-                                               'text'   => SQL_ESCAPE($_POST['reason']),
-                                               'points' => bigintval($_POST['points'])
-                                       );
+                       // Prepare content
+                       $content = array(
+                               'text'   => SQL_ESCAPE($_POST['reason']),
+                               'points' => bigintval($_POST['points'])
+                       );
 
-                                       // Load email template and send email away
-                                       $msg = LOAD_EMAIL_TEMPLATE("add-points", $content, bigintval($uid));
-                                       SEND_EMAIL(bigintval($uid), ADMIN_ADD_SUBJ, $msg);
-                               }
-                       }
-               }
+                       // Load email template and send email away
+                       $msg = LOAD_EMAIL_TEMPLATE("add-points", $content, bigintval($uid));
+                       SEND_EMAIL(bigintval($uid), ADMIN_ADD_SUBJ, $msg);
+               } // END - while
 
                // Free memory
                SQL_FREERESULT($result_main);
@@ -91,19 +79,18 @@ if ($_GET['u_id'] == "all") {
        }
 } elseif (!empty($_GET['u_id'])) {
        // User ID found in URL so we use this give him some credits
-       $result = SQL_QUERY_ESC("SELECT surname, family, email FROM "._MYSQL_PREFIX."_user_data WHERE userid=%s AND status='CONFIRMED' LIMIT 1",
-        array(bigintval($_GET['u_id'])), __FILE__, __LINE__);
+       $result = SQL_QUERY_ESC("SELECT surname, family, email FROM `"._MYSQL_PREFIX."_user_data` WHERE userid=%s AND status='CONFIRMED' LIMIT 1",
+               array(bigintval($_GET['u_id'])), __FILE__, __LINE__);
        if (SQL_NUMROWS($result) == 1) {
                // Selected user does exist
                list($sname, $fname, $email) = SQL_FETCHROW($result);
                SQL_FREERESULT($result);
 
                if ((isset($_POST['ok'])) && (!empty($_POST['points']))) {
-                       global $DEPTH;
                        // Remove depth to prevent booking errors. This is a bad coding
                        // practice, thats also why we need to write this project from
                        // scratch...
-                       unset($DEPTH);
+                       unset($GLOBALS['ref_level']);
 
                        // Ok, add points and send an email to him...
                        ADD_POINTS_REFSYSTEM("admin_single", bigintval($_GET['u_id']), bigintval($_POST['points']), false, "0", false, "direct");
@@ -121,13 +108,13 @@ if ($_GET['u_id'] == "all") {
                        LOAD_TEMPLATE("admin_settings_saved", false, ADMIN_POINTS_ADDED);
                } else {
                        // Opps, missing form here
-                       define('__USER_VALUE', "<A href=\"".CREATE_EMAIL_LINK($email, "user_data")."\">".$sname." ".$fname."</A>");
+                       define('__USER_VALUE', "<a href=\"".CREATE_EMAIL_LINK($email, "user_data")."\">".$sname." ".$fname."</a>");
                        define('__UID'       , bigintval($_GET['u_id']));
                        LOAD_TEMPLATE("admin_add_points");
                }
        } else {
                // User not found!
-               OUTPUT_HTML("<STRONG class=\"admin_failed\">".ADMIN_MEMBER_404_1.$_GET['u_id'].ADMIN_MEMBER_404_2."</STRONG>");
+               LOAD_TEMPLATE("admin_settings_saved", false, "<div class=\"admin_failed\">".ADMIN_MEMBER_404_1.$_GET['u_id'].ADMIN_MEMBER_404_2."</div>");
        }
 } else {
        // Output selection form with all confirmed user accounts listed