More misc fixes and rewrites (sorry, lame description)

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

diff --git a/inc/modules/admin/what-add_surfbar_url.php b/inc/modules/admin/what-add_surfbar_url.php
index 7a432fa319c164d6ddb9596daf400aa1299b922a..e0cd01575d8921c9ef1bf1835801e54f7b942507 100644 (file)
--- a/inc/modules/admin/what-add_surfbar_url.php
+++ b/inc/modules/admin/what-add_surfbar_url.php
@@ -46,15 +46,19 @@ if ((!defined('__SECURITY')) || (!IS_ADMIN())) {
 ADD_DESCR('admin', __FILE__);
 
 // Was an URL added?
-if ((REQUEST_ISSET_POST(('add'))) && (REQUEST_ISSET_POST(('url')))) {
-       // Dummy variables
-       $DATA = array(); $id = "reload_ye"; $skip = false;
+if ((REQUEST_ISSET_POST('add')) && (REQUEST_ISSET_POST('url'))) {
+       // Initialize variables
+       $DATA = array();
+       $id = 'reload_ye';
+       $skip = false;
+       $POST = REQUEST_POST_ARRAY();
 
        // Convert the "reload selections"
-       convertSelectionsToTimestamp(REQUEST_POST_ARRAY(), $DATA, $id, $skip);
+       // @TODO Find all convertSelectionsToTimestamp() calls and rewrite REQUEST_POST() calls to $POST
+       convertSelectionsToTimestamp($POST, $DATA, $id, $skip);
 
        // Then add this URL
-       if (SURFBAR_ADMIN_ADD_URL(REQUEST_POST('url'), REQUEST_POST('limit'), REQUEST_POST('reload'))) {
+       if (SURFBAR_ADMIN_ADD_URL($POST['url'], $POST['limit'], $POST['reload'])) {
                // URL was added
                LOAD_TEMPLATE('admin_settings_saved', false, getMessage('ADMIN_SURFBAR_URL_ADDED'));
        } else {
@@ -65,11 +69,11 @@ if ((REQUEST_ISSET_POST(('add'))) && (REQUEST_ISSET_POST(('url')))) {
 
 // Prepare content for template
 $content = array(
-       'reload' => createTimeSelections(0, "reload", "WDhms")
+       'reload' => createTimeSelections(0, 'reload', 'WDhms')
 );
 
 // Load template
-LOAD_TEMPLATE("admin_add_surfbar_url", false, $content);
+LOAD_TEMPLATE('admin_add_surfbar_url', false, $content);
 
 //
 ?>