From: Gunnar Wrobel <p@rdus.de>
Date: Wed, 4 Nov 2009 15:17:51 +0000 (+0100)
Subject: Fix constructor. Add todo. Use array_shift instead of array_pop.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=80c9f36981d57d06917e35575d2e9baa125c67f7;p=horde.git

Fix constructor. Add todo. Use array_shift instead of array_pop.
---

diff --git a/framework/Kolab_Server/lib/Horde/Kolab/Server/Object/Hash.php b/framework/Kolab_Server/lib/Horde/Kolab/Server/Object/Hash.php
index 2eef350f8..5569e51fc 100644
--- a/framework/Kolab_Server/lib/Horde/Kolab/Server/Object/Hash.php
+++ b/framework/Kolab_Server/lib/Horde/Kolab/Server/Object/Hash.php
@@ -42,7 +42,7 @@ implements Horde_Kolab_Server_Object_Interface
      * @param Horde_Kolab_Server_Object $object The represented object.
      */
     public function __construct(
-        Horde_Kolab_Server_Object $object
+        Horde_Kolab_Server_Object_Interface $object
     ) {
         $this->_object = $object;
     }
@@ -134,8 +134,9 @@ implements Horde_Kolab_Server_Object_Interface
     public function getSingle($attr)
     {
         $value = $this->getExternal($attr);
+        //@todo: Check if that can actually be something other than an array.
         if (is_array($value)) {
-            return array_pop($value);
+            return array_shift($value);
         } else {
             return $value;
         }