From: Michael J. Rubinsky <mrubinsk@horde.org>
Date: Thu, 5 Mar 2009 01:27:03 +0000 (-0500)
Subject: Don't statically cache these, make it an instance variable...it can lead to
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=9e7e654f726bdb271987e7e2ab1116c368e65c60;p=horde.git

Don't statically cache these, make it an instance variable...it can lead to
unexpected results in some cases when  a script instantiates two Horde_Servce_Facebook
instances.
---

diff --git a/framework/Service_Facebook/lib/Horde/Service/Facebook.php b/framework/Service_Facebook/lib/Horde/Service/Facebook.php
index 8deb0a1c5..2893eae40 100644
--- a/framework/Service_Facebook/lib/Horde/Service/Facebook.php
+++ b/framework/Service_Facebook/lib/Horde/Service/Facebook.php
@@ -121,7 +121,7 @@ class Horde_Service_Facebook
      *
      * @var hash of Horde_Service_Facebook_* objects
      */
-    static protected $_objCache = array();
+     protected $_objCache = array();
 
 
     const API_VALIDATION_ERROR = 1;
@@ -209,16 +209,16 @@ class Horde_Service_Facebook
 
         // If not, assume it's a method/action class...
         $class = 'Horde_Service_Facebook_' . ucfirst($value);
-        if (!empty(self::$_objCache[$class])) {
-            return self::$_objCache[$class];
+        if (!empty($this->_objCache[$class])) {
+            return $this->_objCache[$class];
         }
 
         if (!class_exists($class)) {
             throw new Horde_Service_Facebook_Exception(sprintf("%s class not found", $class));
         }
 
-        self::$_objCache[$class] = new $class($this, $this->_request);
-        return self::$_objCache[$class];
+        $this->_objCache[$class] = new $class($this, $this->_request);
+        return $this->_objCache[$class];
     }
 
     /**