[prev in list] [next in list] [prev in thread] [next in thread] 

List:       squirrelmail-cvs
Subject:    [SM-CVS] CVS: squirrelmail/plugins/delete_move_next setup.php,1.27,1.28
From:       Tomas Kuliavas <tokul () users ! sourceforge ! net>
Date:       2003-11-27 15:53:44
[Download RAW message or body]

Update of /cvsroot/squirrelmail/squirrelmail/plugins/delete_move_next
In directory sc8-pr-cvs1:/tmp/cvs-serv24280

Modified Files:
	setup.php 
Log Message:
converting variable extraction to sqgetGlobalVar


Index: setup.php
===================================================================
RCS file: /cvsroot/squirrelmail/squirrelmail/plugins/delete_move_next/setup.php,v
retrieving revision 1.27
retrieving revision 1.28
diff -u -w -r1.27 -r1.28
--- setup.php	26 Nov 2003 20:45:28 -0000	1.27
+++ setup.php	27 Nov 2003 15:53:42 -0000	1.28
@@ -123,25 +123,13 @@
 
     global $PHP_SELF;
 
-    if ( !check_php_version(4,1) ) {
-        global $_GET, $_POST;
-    }
-
-    if (isset($_GET['delete_id'])) {
-        $delete_id = $_GET['delete_id'];
-    }
-    if (isset($_POST['move_id'])) {
-        $move_id = $_POST['move_id'];
-    }       
-
-    if (isset($delete_id)) {
+    if ( sqgetGlobalVar('delete_id', $delete_id, SQ_GET) ) {
         delete_move_next_delete();
         fix_sort_array();
-    } elseif (isset($move_id)) {
+    } elseif ( sqgetGlobalVar('move_id', $move_id, SQ_POST) ) {
         delete_move_next_move();
         fix_sort_array();
     }
-
 }
 
 function delete_move_next_read_t() {
@@ -287,12 +275,8 @@
 function delete_move_next_delete() {
     global $imapConnection, $auto_expunge;
 
-    if ( !check_php_version(4,1) ) {
-        global $_GET;
-    }
-
-    $delete_id = $_GET['delete_id'];
-    $mailbox = $_GET['mailbox'];
+    sqgetGlobalVar('delete_id', $delete_id, SQ_GET);
+    sqgetGlobalVar('mailbox', $mailbox, SQ_GET);
 
     sqimap_messages_delete($imapConnection, $delete_id, $delete_id, $mailbox);
     if ($auto_expunge) {
@@ -304,13 +288,9 @@
 function delete_move_next_move() {
     global $imapConnection, $mailbox, $auto_expunge, $lastTargetMailbox;
 
-    if ( !check_php_version(4,1) ) {
-        global $_POST;
-    }
-
-    $move_id = $_POST['move_id'];
+    sqgetGlobalVar('move_id', $move_id, SQ_POST);
     sqgetGlobalVar('mailbox', $mailbox, SQ_FORM);
-    $targetMailbox = $_POST['targetMailbox'];
+    sqgetGlobalVar('targetMailbox', $targetMailbox, SQ_POST);
 
     // Move message
     sqimap_messages_copy($imapConnection, $move_id, $move_id, $targetMailbox);
@@ -364,48 +344,30 @@
 
     global $username,$data_dir;
 
-    if ( !check_php_version(4,1) ) {
-        global $_POST;
-    }
-
-    if (isset($_POST['delete_move_next_ti'])) {
-        $delete_move_next_ti = $_POST['delete_move_next_ti'];
-    }
-    if (isset($_POST['delete_move_next_bi'])) {
-        $delete_move_next_bi = $_POST['delete_move_next_bi'];
-    }
-    if (isset($_POST['delete_move_next_formATtopi'])) {
-        $delete_move_next_formATtopi = $_POST['delete_move_next_formATtopi'];
-    }
-    if (isset($_POST['delete_move_next_formATbottomi'])) {
-        $delete_move_next_formATbottomi = $_POST['delete_move_next_formATbottomi'];
-    }
-
-    if (isset($delete_move_next_ti)) {
+    if ( sqgetGlobalVar('delete_move_next_ti', $delete_move_next_ti, SQ_POST) ) {
         setPref($data_dir, $username, 'delete_move_next_t', 'on');
     } else {
         setPref($data_dir, $username, 'delete_move_next_t', "off");
     }
 
-    if (isset($delete_move_next_formATtopi)) {
+    if ( sqgetGlobalVar('delete_move_next_formATtopi', $delete_move_next_formATtopi, SQ_POST) ) {
         setPref($data_dir, $username, 'delete_move_next_formATtop', 'on');
     } else {
         setPref($data_dir, $username, 'delete_move_next_formATtop', "off");
     }
 
 
-    if (isset($delete_move_next_bi)) {
+    if ( sqgetGlobalVar('delete_move_next_bi', $delete_move_next_bi, SQ_POST) ) {
         setPref($data_dir, $username, 'delete_move_next_b', 'on');
     } else {
         setPref($data_dir, $username, 'delete_move_next_b', "off");
     }
 
-    if (isset($delete_move_next_formATbottomi)) {
+    if ( sqgetGlobalVar('delete_move_next_formATbottomi', $delete_move_next_formATbottomi, SQ_POST) ) {
         setPref($data_dir, $username, 'delete_move_next_formATbottom', 'on');
     } else {
         setPref($data_dir, $username, 'delete_move_next_formATbottom', "off");
     }
-
 }
 
 



-------------------------------------------------------
This SF.net email is sponsored by: SF.net Giveback Program.
Does SourceForge.net help you be more productive?  Does it
help you create better code?  SHARE THE LOVE, and help us help
YOU!  Click Here: http://sourceforge.net/donate/
--
squirrelmail-cvs mailing list
List Address: squirrelmail-cvs@lists.sourceforge.net
List Info: https://lists.sourceforge.net/lists/listinfo/squirrelmail-cvs
http://squirrelmail.org/cvs
[prev in list] [next in list] [prev in thread] [next in thread]