diff options
author | Bjoern Schiessle <schiessle@owncloud.com> | 2015-03-26 23:56:44 +0300 |
---|---|---|
committer | Bjoern Schiessle <schiessle@owncloud.com> | 2015-03-27 00:42:29 +0300 |
commit | 5964cd3751684c9f26bc4e8635e247ae0c0b7223 (patch) | |
tree | 15220603e32d8116bb989009a705ef47f579e026 /apps | |
parent | 93ae742ad84bb296dd09871b70a7aaaf2d8aa710 (diff) |
always fall back to default share folder
Diffstat (limited to 'apps')
-rw-r--r-- | apps/files_sharing/lib/helper.php | 15 | ||||
-rw-r--r-- | apps/files_sharing/lib/sharedmount.php | 4 | ||||
-rw-r--r-- | apps/files_sharing/tests/helper.php | 6 |
3 files changed, 20 insertions, 5 deletions
diff --git a/apps/files_sharing/lib/helper.php b/apps/files_sharing/lib/helper.php index 8fabd8d42d7..1e17a524427 100644 --- a/apps/files_sharing/lib/helper.php +++ b/apps/files_sharing/lib/helper.php @@ -257,8 +257,21 @@ class Helper { */ public static function getShareFolder() { $shareFolder = \OC::$server->getConfig()->getSystemValue('share_folder', '/'); + $shareFolder = \OC\Files\Filesystem::normalizePath($shareFolder); + + if (!\OC\Files\Filesystem::file_exists($shareFolder)) { + $dir = ''; + $subdirs = explode('/', $shareFolder); + foreach ($subdirs as $subdir) { + $dir = $dir . '/' . $subdir; + if (!\OC\Files\Filesystem::is_dir($dir)) { + \OC\Files\Filesystem::mkdir($dir); + } + } + } + + return $shareFolder; - return \OC\Files\Filesystem::normalizePath($shareFolder); } /** diff --git a/apps/files_sharing/lib/sharedmount.php b/apps/files_sharing/lib/sharedmount.php index d16dbf89ccf..be8650af722 100644 --- a/apps/files_sharing/lib/sharedmount.php +++ b/apps/files_sharing/lib/sharedmount.php @@ -35,8 +35,8 @@ class SharedMount extends MountPoint implements MoveableMount { $mountPoint = basename($share['file_target']); $parent = dirname($share['file_target']); - while (!\OC\Files\Filesystem::is_dir($parent)) { - $parent = dirname($parent); + if (!\OC\Files\Filesystem::is_dir($parent)) { + $parent = Helper::getShareFolder(); } $newMountPoint = \OCA\Files_Sharing\Helper::generateUniqueTarget( diff --git a/apps/files_sharing/tests/helper.php b/apps/files_sharing/tests/helper.php index a9245ddafe5..5c196d4c8e3 100644 --- a/apps/files_sharing/tests/helper.php +++ b/apps/files_sharing/tests/helper.php @@ -30,9 +30,11 @@ class Test_Files_Sharing_Helper extends TestCase { function testSetGetShareFolder() { $this->assertSame('/', \OCA\Files_Sharing\Helper::getShareFolder()); - \OCA\Files_Sharing\Helper::setShareFolder('/Shared'); + \OCA\Files_Sharing\Helper::setShareFolder('/Shared/Folder'); - $this->assertSame('/Shared', \OCA\Files_Sharing\Helper::getShareFolder()); + $sharedFolder = \OCA\Files_Sharing\Helper::getShareFolder(); + $this->assertSame('/Shared/Folder', \OCA\Files_Sharing\Helper::getShareFolder()); + $this->assertTrue(\OC\Files\Filesystem::is_dir($sharedFolder)); // cleanup \OC::$server->getConfig()->deleteSystemValue('share_folder'); |