diff options
| author | Vincent Petry <pvince81@owncloud.com> | 2014-07-16 12:07:02 +0200 |
|---|---|---|
| committer | Vincent Petry <pvince81@owncloud.com> | 2014-07-16 12:07:02 +0200 |
| commit | 658828867ca9379abdb411ec188a9d7a45794dd1 (patch) | |
| tree | a93d317609c5031741c2049137d33e1e21eed808 /apps | |
| parent | 7d8a8e78fa7afad6a004341b67cab19f7a4eccf8 (diff) | |
| download | nextcloud-server-658828867ca9379abdb411ec188a9d7a45794dd1.tar.gz nextcloud-server-658828867ca9379abdb411ec188a9d7a45794dd1.zip | |
Correctly return the owner display name for children of shares
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/files_sharing/lib/cache.php | 2 | ||||
| -rw-r--r-- | apps/files_sharing/tests/cache.php | 19 |
2 files changed, 20 insertions, 1 deletions
diff --git a/apps/files_sharing/lib/cache.php b/apps/files_sharing/lib/cache.php index 6bc9e131949..209aa49dfa4 100644 --- a/apps/files_sharing/lib/cache.php +++ b/apps/files_sharing/lib/cache.php @@ -162,7 +162,7 @@ class Shared_Cache extends Cache { foreach ($sourceFolderContent as $key => $c) { $sourceFolderContent[$key]['path'] = $dir . $c['name']; $sourceFolderContent[$key]['uid_owner'] = $parent['uid_owner']; - $sourceFolderContent[$key]['displayname_owner'] = $parent['uid_owner']; + $sourceFolderContent[$key]['displayname_owner'] = \OC_User::getDisplayName($parent['uid_owner']); $sourceFolderContent[$key]['permissions'] = $sourceFolderContent[$key]['permissions'] & $this->storage->getPermissions($dir . $c['name']); } diff --git a/apps/files_sharing/tests/cache.php b/apps/files_sharing/tests/cache.php index c5408ba55e7..8472c7f1ab0 100644 --- a/apps/files_sharing/tests/cache.php +++ b/apps/files_sharing/tests/cache.php @@ -32,6 +32,9 @@ class Test_Files_Sharing_Cache extends Test_Files_Sharing_Base { function setUp() { parent::setUp(); + \OC_User::setDisplayName(self::TEST_FILES_SHARING_API_USER1, 'User One'); + \OC_User::setDisplayName(self::TEST_FILES_SHARING_API_USER2, 'User Two'); + self::loginHelper(self::TEST_FILES_SHARING_API_USER1); $this->user2View = new \OC\Files\View('/'. self::TEST_FILES_SHARING_API_USER2 . '/files'); @@ -133,11 +136,15 @@ class Test_Files_Sharing_Cache extends Test_Files_Sharing_Base { 'name' => 'shareddir', 'path' => 'files/shareddir', 'mimetype' => 'httpd/unix-directory', + 'uid_owner' => self::TEST_FILES_SHARING_API_USER1, + 'displayname_owner' => 'User One', ), array( 'name' => 'shared single file.txt', 'path' => 'files/shared single file.txt', 'mimetype' => 'text/plain', + 'uid_owner' => self::TEST_FILES_SHARING_API_USER1, + 'displayname_owner' => 'User One', ), ), $results @@ -153,16 +160,22 @@ class Test_Files_Sharing_Cache extends Test_Files_Sharing_Base { 'name' => 'bar.txt', 'path' => 'bar.txt', 'mimetype' => 'text/plain', + 'uid_owner' => self::TEST_FILES_SHARING_API_USER1, + 'displayname_owner' => 'User One', ), array( 'name' => 'emptydir', 'path' => 'emptydir', 'mimetype' => 'httpd/unix-directory', + 'uid_owner' => self::TEST_FILES_SHARING_API_USER1, + 'displayname_owner' => 'User One', ), array( 'name' => 'subdir', 'path' => 'subdir', 'mimetype' => 'httpd/unix-directory', + 'uid_owner' => self::TEST_FILES_SHARING_API_USER1, + 'displayname_owner' => 'User One', ), ), $results @@ -187,16 +200,22 @@ class Test_Files_Sharing_Cache extends Test_Files_Sharing_Base { 'name' => 'another too.txt', 'path' => 'another too.txt', 'mimetype' => 'text/plain', + 'uid_owner' => self::TEST_FILES_SHARING_API_USER1, + 'displayname_owner' => 'User One', ), array( 'name' => 'another.txt', 'path' => 'another.txt', 'mimetype' => 'text/plain', + 'uid_owner' => self::TEST_FILES_SHARING_API_USER1, + 'displayname_owner' => 'User One', ), array( 'name' => 'not a text file.xml', 'path' => 'not a text file.xml', 'mimetype' => 'application/xml', + 'uid_owner' => self::TEST_FILES_SHARING_API_USER1, + 'displayname_owner' => 'User One', ), ), $results |