From: Michael Gapczynski <mtgap@owncloud.com>
Date: Sun, 19 May 2013 19:04:41 +0000 (-0400)
Subject: Fix undefined variable for copying empty folders
X-Git-Tag: v6.0.0alpha2~745^2~4
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=fc5bce1f76a3a67a4ac9095d15441e363dfd03d1;p=nextcloud-server.git

Fix undefined variable for copying empty folders
---

diff --git a/lib/files/view.php b/lib/files/view.php
index 875a6c1a1f8..8a37a0bcc62 100644
--- a/lib/files/view.php
+++ b/lib/files/view.php
@@ -467,7 +467,7 @@ class View {
 					}
 				} else {
 					if ($this->is_dir($path1) && ($dh = $this->opendir($path1))) {
-						$this->mkdir($path2);
+						$result = $this->mkdir($path2);
 						while ($file = readdir($dh)) {
 							if (!Filesystem::isIgnoredDir($file)) {
 								$result = $this->copy($path1 . '/' . $file, $path2 . '/' . $file);
diff --git a/tests/lib/files/view.php b/tests/lib/files/view.php
index 554bc7291ad..a51d99e793d 100644
--- a/tests/lib/files/view.php
+++ b/tests/lib/files/view.php
@@ -235,9 +235,12 @@ class View extends \PHPUnit_Framework_TestCase {
 		\OC\Files\Filesystem::mount($storage2, array(), '/substorage');
 
 		$rootView = new \OC\Files\View('');
+		$rootView->mkdir('substorage/emptyfolder');
 		$rootView->copy('substorage', 'anotherfolder');
 		$this->assertTrue($rootView->is_dir('/anotherfolder'));
 		$this->assertTrue($rootView->is_dir('/substorage'));
+		$this->assertTrue($rootView->is_dir('/anotherfolder/emptyfolder'));
+		$this->assertTrue($rootView->is_dir('/substorage/emptyfolder'));
 		$this->assertTrue($rootView->file_exists('/anotherfolder/foo.txt'));
 		$this->assertTrue($rootView->file_exists('/anotherfolder/foo.png'));
 		$this->assertTrue($rootView->file_exists('/anotherfolder/folder/bar.txt'));