mkdir() now checks isWriteable() to see if it has permission, rmdir unshares the folder from the user

1 files changed, 13 insertions, 11 deletions

diff --git a/apps/files_sharing/sharedstorage.php b/apps/files_sharing/sharedstorage.php
index a3f6bebb0f7..5f343331cde 100644
--- a/apps/files_sharing/sharedstorage.php
+++ b/apps/files_sharing/sharedstorage.php
@@ -53,22 +53,24 @@ class OC_FILESTORAGE_SHARED extends OC_FILESTORAGE {
 		}
 	}
 	
-	// TODO OC_SHARE::getPermissions()
 	public function mkdir($path) {
-		$source = $this->getSource($path);
-		if ($source) {
-			$storage = OC_FILESYSTEM::getStorage($source);
-			return $storage->mkdir($this->getInternalPath($source));
+		if ($path == "" || $path == "/") {
+			return false; 
+		} else {
+			$source = $this->getSource($path);
+			if ($source) {
+				if (OC_SHARE::isWriteable($path)) {
+					$storage = OC_FILESYSTEM::getStorage($source);
+					return $storage->mkdir($this->getInternalPath($source));
+				}
+			}
 		}
 	}
 	
-	// TODO OC_SHARE::getPermissions()
 	public function rmdir($path) {
-		$source = $this->getSource($path);
-		if ($source) {
-			$storage = OC_FILESYSTEM::getStorage($source);
-			return $storage->rmdir($this->getInternalPath($source));
-		}
+		// The folder will be removed from the database, but won't be deleted from the owner's filesystem
+		$target = OC_FILESYSTEM::getStorageMountPoint($this).$path;
+		OC_SHARE::unshareFromSelf($target);
 	}
 	
 	// TODO Change files within shared folders that are renamed