A lot CSS classes rewritten, please update all your themes.

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

diff --git a/inc/modules/admin/what-list_payouts.php b/inc/modules/admin/what-list_payouts.php
index 0d9ad579c4414b1708263f4c31124f70d912edd1..d53474bca9cb97364480c757f01aa6e19aa3b8c5 100644 (file)
--- a/inc/modules/admin/what-list_payouts.php
+++ b/inc/modules/admin/what-list_payouts.php
@@ -53,22 +53,22 @@ if (isGetRequestParameterSet('pid')) {
        SQL_FREERESULT($result);
 
        // Obtain some data
-       if (!isGetRequestParameterSet('task') && (!empty($userid)) && ($userid > 0)) {
+       if (!isGetRequestParameterSet('task') && (!empty($userid)) && (isValidUserId($userid))) {
                // Get task id from database
                $result = SQL_QUERY_ESC("SELECT `id` FROM `{?_MYSQL_PREFIX?}_task_system` WHERE `userid`=%s AND `task_type`='PAYOUT_REQUEST' AND `task_created`=%s LIMIT 1",
                        array(bigintval($userid), bigintval($tstamp)), __FILE__, __LINE__);
-               list($task) = SQL_FETCHROW($result);
+               list($taskId) = SQL_FETCHROW($result);
                SQL_FREERESULT($result);
-               if (empty($task)) $task = '0';
+               if (empty($taskId)) $taskId = '0';
        } elseif ((empty($userid)) || ($userid == '0')) {
                // Cannot obtain member id!
                loadTemplate('admin_settings_saved', false, '{--ADMIN_PAYOUT_FAILED_OBTAIN_USERID--}');
        } else {
                // Get task id from URL
-               $task = getRequestParameter('task');
+               $taskId = getRequestParameter('task');
        }
 
-       if ((!empty($task)) && (!empty($userid)) && ($userid > 0)) {
+       if ((!empty($taskId)) && (!empty($userid)) && (isValidUserId($userid))) {
                // Load user's data
                if (!fetchUserData($userid)) {
                        // Abort here because it is not valid!
@@ -117,7 +117,7 @@ LIMIT 1",
                                                                $data['from_pass'] = base64_encode($data['from_pass']);
                                                                $tpass = base64_encode($tpass);
                                                                break;
-                                               }
+                                               } // END - switch
 
                                                // Transfer variables...
                                                $reason = encodeString(getMessage('ADMIN_PAYOUT_REASON'), false);
@@ -131,8 +131,8 @@ LIMIT 1",
 
                                        if ($ret[0] == $data['engine_ret_ok']) {
                                                // Clear task
-                                               if ($task > 0) {
-                                                       runFilterChain('solve_task', $task);
+                                               if ($taskId > 0) {
+                                                       runFilterChain('solve_task', $taskId);
                                                }
 
                                                // Clear payout request
@@ -160,12 +160,12 @@ LIMIT 1",
                                        }
                                } else {
                                        // Cannot load payout id
-                                       loadTemplate('admin_settings_saved', false, '<div class="admin_failed">{--ADMIN_PAYOUT_FAILED_OBTAIN_PAYOUT_ID--}</div>');
+                                       loadTemplate('admin_settings_unsaved', false, '{--ADMIN_PAYOUT_FAILED_OBTAIN_PAYOUT_ID--}');
                                }
                        } else {
                                // Prepare content
                                $content = array(
-                                       'task' => $task,
+                                       'task' => $taskId,
                                        'pid'  => bigintval(getRequestParameter('pid')),
                                        'user' => '<a href="' . generateEmailLink(getUserData('email'), 'user_data') . '">' . translateGender(getUserData('gender')) . ' ' . getUserData('surname') . ' ' . getUserData('family') . '</a>',
                                );
@@ -176,9 +176,9 @@ LIMIT 1",
                } elseif ((getRequestParameter('do') == 'reject') && (getUserData('email') != '')) {
                        // Ok, now we can output the form or execute rejecting
                        if (isFormSent()) {
-                               if ($task > 0) {
+                               if ($taskId > 0) {
                                        // Clear task
-                                       runFilterChain('solve_task', $task);
+                                       runFilterChain('solve_task', $taskId);
                                } // END - if
 
                                // Clear payout request
@@ -196,7 +196,7 @@ LIMIT 1",
                        } else {
                                // Prepare content
                                $content = array(
-                                       'task' => $task,
+                                       'task' => $taskId,
                                        'pid'  => bigintval(getRequestParameter('pid')),
                                        'user' => '<a href="' . generateEmailLink(getUserData('email'), 'user_data') . '">' . translateGender(getUserData('gender')) . ' ' . getUserData('surname') . ' ' . getUserData('family') . '</a>',
                                );
@@ -208,7 +208,7 @@ LIMIT 1",
                        // Cannot load user data
                        loadTemplate('admin_settings_saved', false, '{--ADMIN_PAYOUT_FAILED_OBTAIN_USERDATA--}');
                }
-       } elseif ((empty($task)) || ($task == '0')) {
+       } elseif ((empty($taskId)) || ($taskId == '0')) {
                // Failed loading task id
                loadTemplate('admin_settings_saved', false, '{--ADMIN_PAYOUT_FAILED_OBTAIN_TASK_ID--}');
        }