Some fixes for surfbar extension (#123), double->single, misc cleanups/fixes

[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 d62a40bc111068640dc01c3766aaae5e7491f13e..384fd186c73b373ac47241a50310ccfd8deee978 100644 (file)
--- a/inc/modules/admin/what-lock_sponsor.php
+++ b/inc/modules/admin/what-lock_sponsor.php
@@ -38,59 +38,59 @@
 
 // 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__);
+       $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) {
                // 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);
 
-               if (($status == "CONFIRMED") || ($status == "LOCKED")) {
+               if (($status == 'CONFIRMED') || ($status == 'LOCKED')) {
                        // Transfer data to constants
-                       define('__GENDER'  , TRANSLATE_GENDER($gender));
+                       define('__GENDER'  , translateGender($gender));
                        define('__SURNAME', $sname);
                        define('__FAMILY' , $fname);
                        define('__ID'     , bigintval(REQUEST_GET('id')));
 
-                       if (IS_FORM_SENT()) {
+                       if (isFormSent()) {
                                // Create messages
-                               if ($status == "CONFIRMED") {
+                               if ($status == 'CONFIRMED') {
                                        // Message when sponsor's account got lock
                                        define('__EMAIL_MSG', getMessage('SPONSOR_ACCOUNT_LOCKED'));
                                        $subject = getMessage('SPONSOR_SUBJECT_LOCKED');
-                                       $status = "LOCKED";
+                                       $status = 'LOCKED';
                                } else {
                                        // Message when sponsor's account got unlock
                                        define('__EMAIL_MSG', getMessage('SPONSOR_ACCOUNT_UNLOCKED'));
                                        $subject = getMessage('SPONSOR_SUBJECT_UNLOCKED');
-                                       $status = "CONFIRMED";
+                                       $status = 'CONFIRMED';
                                }
 
                                // Load email message
                                $msg = LOAD_EMAIL_TEMPLATE("lock_sponsor", REQUEST_POST('reason'), bigintval(REQUEST_GET('id')));
 
                                // And send it away
-                               SEND_EMAIL($email, $subject, $msg);
+                               sendEmail($email, $subject, $msg);
 
                                // Update sponsor's account
-                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_sponsor_data` SET `status`='".$status."' WHERE id='%s' LIMIT 1",
-                                       array(bigintval(REQUEST_GET('id'))), __FILE__, __LINE__);
-                       } elseif (REQUEST_ISSET_POST(('no'))) {
+                               SQL_QUERY_ESC("UPDATE `{!_MYSQL_PREFIX!}_sponsor_data` SET `status`='".$status."' WHERE `id`='%s' LIMIT 1",
+                               array(bigintval(REQUEST_GET('id'))), __FILE__, __LINE__);
+                       } 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 ($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,10 +101,10 @@ if (REQUEST_ISSET_GET(('id'))) {
                                }
 
                                // Create email link
-                               define('__EMAIL_LINK', "<a href=\"mailto:".$email."\">{!__GENDER!} {!__SURNAME!} {!__FAMILY!}</a>");
+                               define('__EMAIL_LINK', '<a href="mailto:' . $email . '">{!__GENDER!} {!__SURNAME!} {!__FAMILY!}</a>');
 
                                // Display form
-                               LOAD_TEMPLATE("admin_lock_sponsor");
+                               LOAD_TEMPLATE('admin_lock_sponsor');
                        }
                } else {
                        // Cannot change status on unconfirmed or pending accounts!