More misc fixes and rewrites (sorry, lame description)

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

diff --git a/inc/modules/admin/what-lock_sponsor.php b/inc/modules/admin/what-lock_sponsor.php
index e43494f1053a876d471cd95d509523a294209d71..fcda54a4f562458d60e748dc41565af4707ee5ca 100644 (file)
--- a/inc/modules/admin/what-lock_sponsor.php
+++ b/inc/modules/admin/what-lock_sponsor.php
@@ -38,59 +38,58 @@
 
 // Some security stuff...
 if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
-       $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);
-}
+} // END - if
 
 // Add description as navigation point
 ADD_DESCR('admin', __FILE__);
 $message = '';
 
-if (REQUEST_ISSET_GET(('id'))) {
+if (REQUEST_ISSET_GET('id')) {
        // Check for selected sponsor
-       $result = SQL_QUERY_ESC("SELECT gender, surname, family, email, status FROM `{!_MYSQL_PREFIX!}_sponsor_data` WHERE id='%s' LIMIT 1",
-        array(bigintval(REQUEST_GET('id'))), __FILE__, __LINE__);
-       if (SQL_NUMROWS($result) == 1) {
+       $result_sponsor = SQL_QUERY_ESC("SELECT `gender`, `surname`, `family`, `email`, `status` FROM `{!_MYSQL_PREFIX!}_sponsor_data` WHERE `id`='%s' LIMIT 1",
+               array(bigintval(REQUEST_GET('id'))), __FILE__, __LINE__);
+       if (SQL_NUMROWS($result_sponsor) == 1) {
                // Get sponsor's current status and let only confirmed and locked status pass
-               list($gender, $sname, $fname, $email, $status) = SQL_FETCHROW($result);
-               SQL_FREERESULT($result);
+               $DATA = SQL_FETCHARRAY($result_sponsor);
 
-               if (($status == "CONFIRMED") || ($status == "LOCKED")) {
+               if (($DATA['status'] == 'CONFIRMED') || ($DATA['status'] == 'LOCKED')) {
                        // Transfer data to constants
-                       define('__GENDER'  , TRANSLATE_GENDER($gender));
-                       define('__SURNAME', $sname);
-                       define('__FAMILY' , $fname);
+                       define('__GENDER'  , translateGender($DATA['gender']));
+                       define('__SURNAME', $DATA['surname']);
+                       define('__FAMILY' , $DATA['family']);
                        define('__ID'     , bigintval(REQUEST_GET('id')));
 
-                       if (IS_FORM_SENT()) {
+                       if (isFormSent()) {
                                // Create messages
-                               if ($status == "CONFIRMED") {
+                               if ($DATA['status'] == 'CONFIRMED') {
                                        // Message when sponsor's account got lock
                                        define('__EMAIL_MSG', getMessage('SPONSOR_ACCOUNT_LOCKED'));
                                        $subject = getMessage('SPONSOR_SUBJECT_LOCKED');
-                                       $status = "LOCKED";
+                                       $DATA['status'] = 'LOCKED';
                                } else {
                                        // Message when sponsor's account got unlock
                                        define('__EMAIL_MSG', getMessage('SPONSOR_ACCOUNT_UNLOCKED'));
                                        $subject = getMessage('SPONSOR_SUBJECT_UNLOCKED');
-                                       $status = "CONFIRMED";
+                                       $DATA['status'] = 'CONFIRMED';
                                }
 
                                // Load email message
-                               $msg = LOAD_EMAIL_TEMPLATE("lock_sponsor", REQUEST_POST('reason'), bigintval(REQUEST_GET('id')));
+                               $message = LOAD_EMAIL_TEMPLATE('lock_sponsor', REQUEST_POST('reason'), bigintval(REQUEST_GET('id')));
 
                                // And send it away
-                               SEND_EMAIL($email, $subject, $msg);
+                               sendEmail($DATA['email'], $subject, $message);
 
                                // Update sponsor's account
-                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_sponsor_data` SET `status`='".$status."' WHERE id='%s' LIMIT 1",
+                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_sponsor_data` SET `status`='".$DATA['status']."' WHERE `id`='%s' LIMIT 1",
                                        array(bigintval(REQUEST_GET('id'))), __FILE__, __LINE__);
-                       } elseif (REQUEST_ISSET_POST(('no'))) {
+                       } elseif (REQUEST_ISSET_POST('no')) {
                                // No don't lock / unlock now!
-                               LOAD_URL('modules.php?module=admin&what=list_sponsor&id='.bigintval(REQUEST_GET('id')));
+                               redirectToUrl('modules.php?module=admin&what=list_sponsor&id='.bigintval(REQUEST_GET('id')));
                        } else {
                                // Create header and text messages
-                               if ($status == 'CONFIRMED') {
+                               if ($DATA['status'] == 'CONFIRMED') {
                                        // Messages when sponsor's account is confirmed
                                        define('__HEADER_MESSAGE', getMessage('SPONSOR_LOCK_SPONSOR_HEADER'));
                                        define('__TEXT_MESSAGE'  , getMessage('SPONSOR_LOCK_SPONSOR_TEXT'));
@@ -101,7 +100,7 @@ if (REQUEST_ISSET_GET(('id'))) {
                                }
 
                                // Create email link
-                               define('__EMAIL_LINK', '<a href="mailto:' . $email . '">{!__GENDER!} {!__SURNAME!} {!__FAMILY!}</a>');
+                               define('__EMAIL_LINK', '<a href="' . generateEmailLink($DATA['email'], 'sponsor_data') . '">{!__GENDER!} {!__SURNAME!} {!__FAMILY!}</a>');
 
                                // Display form
                                LOAD_TEMPLATE('admin_lock_sponsor');
@@ -114,6 +113,9 @@ if (REQUEST_ISSET_GET(('id'))) {
                // Sponsor not found!
                $message = sprintf(getMessage('ADMIN_SPONSOR_404'), bigintval(REQUEST_GET('id')));
        }
+
+       // Free result
+       SQL_FREERESULT($result_sponsor);
 } else {
        // Not called by what-list_sponsor.php
        $message = getMessage('ADMIN_CALL_NOT_DIRECTLY');
@@ -122,7 +124,7 @@ if (REQUEST_ISSET_GET(('id'))) {
 if (!empty($message)) {
        // Output message
        LOAD_TEMPLATE('admin_settings_saved', false, $message);
-}
+} // END - if
 
 //
 ?>