From: Gunnar Wrobel <p@rdus.de>
Date: Mon, 15 Mar 2010 08:35:46 +0000 (+0100)
Subject: Test and fix domain handling.
X-Git-Url: https://git.internetallee.de/?a=commitdiff_plain;h=a856b1c2057809b0e3d4cdd5a7676c0a91594d61;p=horde.git

Test and fix domain handling.
---

diff --git a/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Folder.php b/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Folder.php
index 92cbc0f28..d06be64a7 100644
--- a/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Folder.php
+++ b/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Folder.php
@@ -501,7 +501,7 @@ class Horde_Kolab_Storage_Folder
                 if (strpos($user, '@') === false) {
                     $domain = strstr(Horde_Auth::getAuth(), '@');
                     if (!empty($domain)) {
-                        $user .= '@' . $domain;
+                        $user .= $domain;
                     }
                 }
                 $this->_owner = $user;
diff --git a/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Namespace.php b/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Namespace.php
index 56ee6ca24..8442e4e68 100644
--- a/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Namespace.php
+++ b/framework/Kolab_Storage/lib/Horde/Kolab/Storage/Namespace.php
@@ -155,7 +155,7 @@ class Horde_Kolab_Storage_Namespace
             $user = $path[1];
             $domain = strstr(array_pop($path), '@');
             if (!empty($domain)) {
-                $user .= '@' . $domain;
+                $user .= $domain;
             }
             return self::OTHER . ':' . $user;
         }
diff --git a/framework/Kolab_Storage/test/Horde/Kolab/Storage/NamespaceTest.php b/framework/Kolab_Storage/test/Horde/Kolab/Storage/NamespaceTest.php
index 9922c2e00..5a24eedb4 100644
--- a/framework/Kolab_Storage/test/Horde/Kolab/Storage/NamespaceTest.php
+++ b/framework/Kolab_Storage/test/Horde/Kolab/Storage/NamespaceTest.php
@@ -130,6 +130,21 @@ class Horde_Kolab_Storage_NamespaceTest extends PHPUnit_Framework_TestCase
         $this->assertEquals('test', $folder->getOwner());
     }
 
+    public function testFolderOwnerHasDomainFromFolderDomain()
+    {
+        $_SESSION['horde_auth']['userId'] = 'test@example.com';
+        $folder = new Horde_Kolab_Storage_Folder('user/test/mine');
+        $folder->restore($this->_storage, $this->_connection, new Horde_Kolab_Storage_Namespace());
+        $this->assertEquals('test@example.com', $folder->getOwner());
+    }
+
+    public function testFolderOwnerHasDomainFromCurrentUserIfNoFolderDomain()
+    {
+        $folder = new Horde_Kolab_Storage_Folder('user/test/mine@example.com');
+        $folder->restore($this->_storage, $this->_connection, new Horde_Kolab_Storage_Namespace());
+        $this->assertEquals('test@example.com', $folder->getOwner());
+    }
+
     public function testSetnameDoesAddDefaultPersonalNamespace()
     {
         $folder = new Horde_Kolab_Storage_Folder(null);