Reverting to Erin's original design; sqgetGlobalVar() should not touch vars that...

git-svn-id: https://svn.code.sf.net/p/squirrelmail/code/trunk/squirrelmail@9265 7612ce4b-ef26-0410-bec9-ea0150e637f0

diff --git a/functions/global.php b/functions/global.php
index 634d9d38f1a9521a0884c7bd96507aac2c5b0d8e..35b9fcc02f7b12af754bee3f14c576e2881c5245 100644 (file)
--- a/functions/global.php
+++ b/functions/global.php
@@ -185,9 +185,6 @@ define('SQ_FORM',6);
  *    sqgetGlobalVar('username',$username,SQ_SESSION);
  *  -- no quotes around last param!
  *
- * WARNING: Since 1.4.5 and 1.5.1 function nulls $value, 
- * if variable is not available.
- *
  * @param string name the name of the var to search
  * @param mixed value the variable to return
  * @param int search constant defining where to look
@@ -240,8 +237,7 @@ function sqgetGlobalVar($name, &$value, $search = SQ_INORDER) {
         }
         break;
     }
-    /* Nothing found, reset $value and return FALSE */
-    $value = null;
+    /* Nothing found, return FALSE */
     return FALSE;
 }