Fixes saving of settings, inconsistency fixed in config entries

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

diff --git a/inc/modules/admin/what-sub_points.php b/inc/modules/admin/what-sub_points.php
index 448167d1393fb84bede8df75c957afe39db6c24a..813cf06a9252ad8bbd0b139ad2cf1b0c0a18b906 100644 (file)
--- a/inc/modules/admin/what-sub_points.php
+++ b/inc/modules/admin/what-sub_points.php
@@ -1,7 +1,7 @@
 <?php
 /************************************************************************
- * MXChange v0.2.1                                    Start: 11/23/2003 *
- * ===============                              Last change: 09/23/2004 *
+ * Mailer v0.2.1-FINAL                                Start: 11/23/2003 *
+ * ===================                          Last change: 09/23/2004 *
  *                                                                      *
  * -------------------------------------------------------------------- *
  * File              : what-sub_points.php                              *
@@ -51,7 +51,7 @@ if (getRequestElement('userid') == 'all') {
        // Is the form sent?
        if ((isFormSent()) && (postRequestElement('points') > 0)) {
                $result_main = SQL_QUERY("SELECT userid, email FROM `{?_MYSQL_PREFIX?}_user_data` WHERE `status`='CONFIRMED' ORDER BY `userid` ASC",
-               __FILE__, __LINE__);
+                       __FILE__, __LINE__);
                while ($content = merge_array($content, SQL_FETCHARRAY($result_main))) {
                        // Ok, add points to used points and send an email to him...
                        subtractPoints('admin_all', $content['userid'], postRequestElement('points'));
@@ -75,14 +75,12 @@ if (getRequestElement('userid') == 'all') {
                loadTemplate('admin_sub_points_all');
        }
 } elseif (isGetRequestElementSet('userid')) {
-       // 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(getRequestElement('userid'))),__FILE__, __LINE__);
-       if (SQL_NUMROWS($result) == 1) {
+       // User id found in URL so we use this give him some credits
+       if (fetchUserData(getRequestElement('userid'))) {
                // Selected user does exist
-               $content = SQL_FETCHARRAY($result);
+               $content = getUserDataArray();
 
-               if ((isFormSent()) && (isPostRequestElementSet(('points')))) {
+               if ((isFormSent()) && (isPostRequestElementSet('points'))) {
                        // Ok, add to used points and send an email to him...
                        subtractPoints('admin_single', bigintval(getRequestElement('userid')), postRequestElement('points'));
 
@@ -98,7 +96,7 @@ if (getRequestElement('userid') == 'all') {
                        loadTemplate('admin_settings_saved', false, getMessage('ADMIN_POINTS_SUBTRACTED'));
                } else {
                        // Prepare content
-                       $content['user']   = "<a href=\"".generateEmailLink($content['email'], 'user_data')."\">".$content['surname']." ".$content['family']."</a>";
+                       $content['user']   = '<a href="' . generateEmailLink($content['email'], 'user_data') . '">' . $content['surname'] . ' ' . $content['family'] . '</a>';
                        $content['userid'] = bigintval(getRequestElement('userid'));
 
                        // Load form
@@ -106,11 +104,8 @@ if (getRequestElement('userid') == 'all') {
                }
        } else {
                // User not found!
-               loadTemplate('admin_settings_saved', false, "<div class=\"admin_failed\">".sprintf(getMessage('ADMIN_MEMBER_404'), getRequestElement('userid'))."</div>");
+               loadTemplate('admin_settings_saved', false, '<div class="admin_failed">' . getMaskedMessage('ADMIN_MEMBER_404', getRequestElement('userid')) . '</div>');
        }
-
-       // Free result
-       SQL_FREERESULT($result);
 } else {
        // Output selection form with all confirmed user accounts listed
        addMemberSelectionBox(0, true);