Ensure we store serialized data in the cache.

diff --git a/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Cache/List.php b/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Cache/List.php
index 4e3d67e..0f35de0 100644 (file)
--- a/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Cache/List.php
+++ b/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Cache/List.php
@@ -97,7 +97,7 @@ class Horde_Kolab_Storage_Cache_List
     private function _load()
     {
         if ($this->_data === false) {
-            $this->_data = $this->_cache->loadListData($this->_list_id);
+            $this->_data = unserialize($this->_cache->loadListData($this->_list_id));
             if (!is_array($this->_data)) {
                 $this->_data = array();
             }
@@ -111,7 +111,7 @@ class Horde_Kolab_Storage_Cache_List
      */
     public function save()
     {
-        $this->_cache->storeListData($this->_list_id, $this->_data);
+        $this->_cache->storeListData($this->_list_id, serialize($this->_data));
     }
 
     /**

diff --git a/framework/Kolab_Storage/test/Horde/Kolab/Storage/Unit/List/Decorator/CacheTest.php b/framework/Kolab_Storage/test/Horde/Kolab/Storage/Unit/List/Decorator/CacheTest.php
index ec7e664..0ec23c1 100644 (file)
--- a/framework/Kolab_Storage/test/Horde/Kolab/Storage/Unit/List/Decorator/CacheTest.php
+++ b/framework/Kolab_Storage/test/Horde/Kolab/Storage/Unit/List/Decorator/CacheTest.php
@@ -282,7 +282,7 @@ extends Horde_Kolab_Storage_TestCase
                 $cache
             )
         );
-        $cache->storeListData($list->getConnectionId(), array('S' => time(), 'V' => '1'));
+        $cache->storeListData($list->getConnectionId(), serialize(array('S' => time(), 'V' => '1')));
         $this->mockDriver->expects($this->never())
             ->method('getMailboxes') 
             ->will($this->returnValue(array('INBOX')));
@@ -298,8 +298,7 @@ extends Horde_Kolab_Storage_TestCase
                 $cache
             )
         );
-        $cache->storeListData($list->getConnectionId(), 'S', time());
-        $cache->storeListData($list->getConnectionId(), 'V', '2');
+        $cache->storeListData($list->getConnectionId(), serialize(array('S' => time(), 'V' => '2')));
         $this->mockDriver->expects($this->once())
             ->method('getMailboxes') 
             ->will($this->returnValue(array('INBOX')));