o PHP 4.0.4 seemed to have problems with the usort function being defined inline...

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

diff --git a/functions/addressbook.php b/functions/addressbook.php
index 8e20224c1945df4a9817e88407fb8034891c7b38..8779c99e84564d5c4af0d26452c2b8d4e2c4a9d3 100644 (file)
--- a/functions/addressbook.php
+++ b/functions/addressbook.php
@@ -13,6 +13,7 @@
    include("../functions/abook_local_file.php");
    include("../functions/abook_ldap_server.php");
 
+
    // Create and initialize an addressbook object. 
    // Returns the created object
    function addressbook_init($showerr = true, $onlylocal = false) {
@@ -53,6 +54,16 @@
       return $abook;
    }
 
+   
+   // Had to move this function outside of the Addressbook Class
+   // PHP 4.0.4 Seemed to be having problems with inline functions.
+   function cmp($a,$b) {   
+      if($a["backend"] > $b["backend"]) 
+            return 1;
+         else if($a["backend"] < $b["backend"]) 
+            return -1;
+         return (strtolower($a["name"]) > strtolower($b["name"])) ? 1 : -1;
+   }
 
 
   /**
@@ -60,6 +71,7 @@
    ** backends and provide services to the functions above.
    **
    **/
+   
    class AddressBook { 
       var $backends    = array();
       var $numbackends = 0;
@@ -154,26 +166,15 @@
         return $ret;
       }
 
-
+      
       // Return a sorted search
       function s_search($expression, $bnum = -1) {
-
-        $ret = $this->search($expression, $bnum);
-        if(!is_array($ret))
-           return $ret;
-
-        // Inline function - Not nice, but still.. 
-        function cmp($a,$b) {   
-           if($a["backend"] > $b["backend"]) 
-              return 1;
-           else if($a["backend"] < $b["backend"]) 
-              return -1;
-        
-           return (strtolower($a["name"]) > strtolower($b["name"])) ? 1 : -1;
-        }
-
-        usort($ret, 'cmp');
-        return $ret;
+             
+        $ret = $this->search($expression, $bnum);
+        if(!is_array($ret))
+           return $ret;
+            usort($ret, 'cmp');
+            return $ret;
       }
 
 
@@ -363,8 +364,8 @@
         return false;  // Not reached
       } // end of modify()
 
-   } // End of class Addressbook
 
+   } // End of class Addressbook
 
   /**
    ** Generic backend that all other backends extend