nextcloud / server

apps/files_trashbin/lib/Storage.php

Indentation +192 added lines, -192 removed lines

@@ -43,220 +43,220 @@
 use Symfony\Component\EventDispatcher\EventDispatcherInterface;
 
 class Storage extends Wrapper {
-	/** @var IMountPoint */
-	private $mountPoint;
+    /** @var IMountPoint */
+    private $mountPoint;
 
-	/** @var  IUserManager */
-	private $userManager;
+    /** @var  IUserManager */
+    private $userManager;
 
-	/** @var ILogger */
-	private $logger;
+    /** @var ILogger */
+    private $logger;
 
-	/** @var EventDispatcherInterface */
-	private $eventDispatcher;
+    /** @var EventDispatcherInterface */
+    private $eventDispatcher;
 
-	/** @var IRootFolder */
-	private $rootFolder;
+    /** @var IRootFolder */
+    private $rootFolder;
 
-	/** @var ITrashManager */
-	private $trashManager;
+    /** @var ITrashManager */
+    private $trashManager;
 
-	private $trashEnabled = true;
+    private $trashEnabled = true;
 
-	/**
-	 * Storage constructor.
-	 *
-	 * @param array $parameters
-	 * @param ITrashManager $trashManager
-	 * @param IUserManager|null $userManager
-	 * @param ILogger|null $logger
-	 * @param EventDispatcherInterface|null $eventDispatcher
-	 * @param IRootFolder|null $rootFolder
-	 */
-	public function __construct(
-		$parameters,
-		ITrashManager $trashManager = null,
-		IUserManager $userManager = null,
-		ILogger $logger = null,
-		EventDispatcherInterface $eventDispatcher = null,
-		IRootFolder $rootFolder = null
-	) {
-		$this->mountPoint = $parameters['mountPoint'];
-		$this->trashManager = $trashManager;
-		$this->userManager = $userManager;
-		$this->logger = $logger;
-		$this->eventDispatcher = $eventDispatcher;
-		$this->rootFolder = $rootFolder;
-		parent::__construct($parameters);
-	}
+    /**
+     * Storage constructor.
+     *
+     * @param array $parameters
+     * @param ITrashManager $trashManager
+     * @param IUserManager|null $userManager
+     * @param ILogger|null $logger
+     * @param EventDispatcherInterface|null $eventDispatcher
+     * @param IRootFolder|null $rootFolder
+     */
+    public function __construct(
+        $parameters,
+        ITrashManager $trashManager = null,
+        IUserManager $userManager = null,
+        ILogger $logger = null,
+        EventDispatcherInterface $eventDispatcher = null,
+        IRootFolder $rootFolder = null
+    ) {
+        $this->mountPoint = $parameters['mountPoint'];
+        $this->trashManager = $trashManager;
+        $this->userManager = $userManager;
+        $this->logger = $logger;
+        $this->eventDispatcher = $eventDispatcher;
+        $this->rootFolder = $rootFolder;
+        parent::__construct($parameters);
+    }
 
-	/**
-	 * Deletes the given file by moving it into the trashbin.
-	 *
-	 * @param string $path path of file or folder to delete
-	 *
-	 * @return bool true if the operation succeeded, false otherwise
-	 */
-	public function unlink($path) {
-		if ($this->trashEnabled) {
-			try {
-				return $this->doDelete($path, 'unlink');
-			} catch (GenericEncryptionException $e) {
-				// in case of a encryption exception we delete the file right away
-				$this->logger->info(
-					"Can't move file " . $path .
-					" to the trash bin, therefore it was deleted right away");
+    /**
+     * Deletes the given file by moving it into the trashbin.
+     *
+     * @param string $path path of file or folder to delete
+     *
+     * @return bool true if the operation succeeded, false otherwise
+     */
+    public function unlink($path) {
+        if ($this->trashEnabled) {
+            try {
+                return $this->doDelete($path, 'unlink');
+            } catch (GenericEncryptionException $e) {
+                // in case of a encryption exception we delete the file right away
+                $this->logger->info(
+                    "Can't move file " . $path .
+                    " to the trash bin, therefore it was deleted right away");
 
-				return $this->storage->unlink($path);
-			}
-		} else {
-			return $this->storage->unlink($path);
-		}
-	}
+                return $this->storage->unlink($path);
+            }
+        } else {
+            return $this->storage->unlink($path);
+        }
+    }
 
-	/**
-	 * Deletes the given folder by moving it into the trashbin.
-	 *
-	 * @param string $path path of folder to delete
-	 *
-	 * @return bool true if the operation succeeded, false otherwise
-	 */
-	public function rmdir($path) {
-		if ($this->trashEnabled) {
-			return $this->doDelete($path, 'rmdir');
-		} else {
-			return $this->storage->rmdir($path);
-		}
-	}
+    /**
+     * Deletes the given folder by moving it into the trashbin.
+     *
+     * @param string $path path of folder to delete
+     *
+     * @return bool true if the operation succeeded, false otherwise
+     */
+    public function rmdir($path) {
+        if ($this->trashEnabled) {
+            return $this->doDelete($path, 'rmdir');
+        } else {
+            return $this->storage->rmdir($path);
+        }
+    }
 
-	/**
-	 * check if it is a file located in data/user/files only files in the
-	 * 'files' directory should be moved to the trash
-	 *
-	 * @param $path
-	 * @return bool
-	 */
-	protected function shouldMoveToTrash($path) {
-		$normalized = Filesystem::normalizePath($this->mountPoint . '/' . $path);
-		$parts = explode('/', $normalized);
-		if (count($parts) < 4 || strpos($normalized, '/appdata_') === 0) {
-			return false;
-		}
+    /**
+     * check if it is a file located in data/user/files only files in the
+     * 'files' directory should be moved to the trash
+     *
+     * @param $path
+     * @return bool
+     */
+    protected function shouldMoveToTrash($path) {
+        $normalized = Filesystem::normalizePath($this->mountPoint . '/' . $path);
+        $parts = explode('/', $normalized);
+        if (count($parts) < 4 || strpos($normalized, '/appdata_') === 0) {
+            return false;
+        }
 
-		// check if there is a app which want to disable the trash bin for this file
-		$fileId = $this->storage->getCache()->getId($path);
-		$owner = $this->storage->getOwner($path);
-		if ($owner === false || $this->storage->instanceOfStorage(\OCA\Files_Sharing\External\Storage::class)) {
-			$nodes = $this->rootFolder->getById($fileId);
-		} else {
-			$nodes = $this->rootFolder->getUserFolder($owner)->getById($fileId);
-		}
+        // check if there is a app which want to disable the trash bin for this file
+        $fileId = $this->storage->getCache()->getId($path);
+        $owner = $this->storage->getOwner($path);
+        if ($owner === false || $this->storage->instanceOfStorage(\OCA\Files_Sharing\External\Storage::class)) {
+            $nodes = $this->rootFolder->getById($fileId);
+        } else {
+            $nodes = $this->rootFolder->getUserFolder($owner)->getById($fileId);
+        }
 
-		foreach ($nodes as $node) {
-			$event = $this->createMoveToTrashEvent($node);
-			$this->eventDispatcher->dispatch('OCA\Files_Trashbin::moveToTrash', $event);
-			if ($event->shouldMoveToTrashBin() === false) {
-				return false;
-			}
-		}
+        foreach ($nodes as $node) {
+            $event = $this->createMoveToTrashEvent($node);
+            $this->eventDispatcher->dispatch('OCA\Files_Trashbin::moveToTrash', $event);
+            if ($event->shouldMoveToTrashBin() === false) {
+                return false;
+            }
+        }
 
-		if ($parts[2] === 'files' && $this->userManager->userExists($parts[1])) {
-			return true;
-		}
+        if ($parts[2] === 'files' && $this->userManager->userExists($parts[1])) {
+            return true;
+        }
 
-		return false;
-	}
+        return false;
+    }
 
-	/**
-	 * get move to trash event
-	 *
-	 * @param Node $node
-	 * @return MoveToTrashEvent
-	 */
-	protected function createMoveToTrashEvent(Node $node) {
-		return new MoveToTrashEvent($node);
-	}
+    /**
+     * get move to trash event
+     *
+     * @param Node $node
+     * @return MoveToTrashEvent
+     */
+    protected function createMoveToTrashEvent(Node $node) {
+        return new MoveToTrashEvent($node);
+    }
 
-	/**
-	 * Run the delete operation with the given method
-	 *
-	 * @param string $path path of file or folder to delete
-	 * @param string $method either "unlink" or "rmdir"
-	 *
-	 * @return bool true if the operation succeeded, false otherwise
-	 */
-	private function doDelete($path, $method) {
-		if (
-			!\OC::$server->getAppManager()->isEnabledForUser('files_trashbin')
-			|| (pathinfo($path, PATHINFO_EXTENSION) === 'part')
-			|| $this->shouldMoveToTrash($path) === false
-		) {
-			return call_user_func([$this->storage, $method], $path);
-		}
+    /**
+     * Run the delete operation with the given method
+     *
+     * @param string $path path of file or folder to delete
+     * @param string $method either "unlink" or "rmdir"
+     *
+     * @return bool true if the operation succeeded, false otherwise
+     */
+    private function doDelete($path, $method) {
+        if (
+            !\OC::$server->getAppManager()->isEnabledForUser('files_trashbin')
+            || (pathinfo($path, PATHINFO_EXTENSION) === 'part')
+            || $this->shouldMoveToTrash($path) === false
+        ) {
+            return call_user_func([$this->storage, $method], $path);
+        }
 
-		// check permissions before we continue, this is especially important for
-		// shared files
-		if (!$this->isDeletable($path)) {
-			return false;
-		}
+        // check permissions before we continue, this is especially important for
+        // shared files
+        if (!$this->isDeletable($path)) {
+            return false;
+        }
 
-		$isMovedToTrash = $this->trashManager->moveToTrash($this, $path);
-		if (!$isMovedToTrash) {
-			return call_user_func([$this->storage, $method], $path);
-		} else {
-			return true;
-		}
-	}
+        $isMovedToTrash = $this->trashManager->moveToTrash($this, $path);
+        if (!$isMovedToTrash) {
+            return call_user_func([$this->storage, $method], $path);
+        } else {
+            return true;
+        }
+    }
 
-	/**
-	 * Setup the storate wrapper callback
-	 */
-	public static function setupStorage() {
-		\OC\Files\Filesystem::addStorageWrapper('oc_trashbin', function ($mountPoint, $storage) {
-			return new \OCA\Files_Trashbin\Storage(
-				['storage' => $storage, 'mountPoint' => $mountPoint],
-				\OC::$server->query(ITrashManager::class),
-				\OC::$server->getUserManager(),
-				\OC::$server->getLogger(),
-				\OC::$server->getEventDispatcher(),
-				\OC::$server->getLazyRootFolder()
-			);
-		}, 1);
-	}
+    /**
+     * Setup the storate wrapper callback
+     */
+    public static function setupStorage() {
+        \OC\Files\Filesystem::addStorageWrapper('oc_trashbin', function ($mountPoint, $storage) {
+            return new \OCA\Files_Trashbin\Storage(
+                ['storage' => $storage, 'mountPoint' => $mountPoint],
+                \OC::$server->query(ITrashManager::class),
+                \OC::$server->getUserManager(),
+                \OC::$server->getLogger(),
+                \OC::$server->getEventDispatcher(),
+                \OC::$server->getLazyRootFolder()
+            );
+        }, 1);
+    }
 
-	public function getMountPoint() {
-		return $this->mountPoint;
-	}
+    public function getMountPoint() {
+        return $this->mountPoint;
+    }
 
-	public function moveFromStorage(IStorage $sourceStorage, $sourceInternalPath, $targetInternalPath) {
-		$sourceIsTrashbin = $sourceStorage->instanceOfStorage(Storage::class);
-		try {
-			// the fallback for moving between storage involves a copy+delete
-			// we don't want to trigger the trashbin when doing the delete
-			if ($sourceIsTrashbin) {
-				/** @var Storage $sourceStorage */
-				$sourceStorage->disableTrash();
-			}
-			$result = parent::moveFromStorage($sourceStorage, $sourceInternalPath, $targetInternalPath);
-			if ($sourceIsTrashbin) {
-				/** @var Storage $sourceStorage */
-				$sourceStorage->enableTrash();
-			}
-			return $result;
-		} catch (\Exception $e) {
-			if ($sourceIsTrashbin) {
-				/** @var Storage $sourceStorage */
-				$sourceStorage->enableTrash();
-			}
-			throw $e;
-		}
-	}
+    public function moveFromStorage(IStorage $sourceStorage, $sourceInternalPath, $targetInternalPath) {
+        $sourceIsTrashbin = $sourceStorage->instanceOfStorage(Storage::class);
+        try {
+            // the fallback for moving between storage involves a copy+delete
+            // we don't want to trigger the trashbin when doing the delete
+            if ($sourceIsTrashbin) {
+                /** @var Storage $sourceStorage */
+                $sourceStorage->disableTrash();
+            }
+            $result = parent::moveFromStorage($sourceStorage, $sourceInternalPath, $targetInternalPath);
+            if ($sourceIsTrashbin) {
+                /** @var Storage $sourceStorage */
+                $sourceStorage->enableTrash();
+            }
+            return $result;
+        } catch (\Exception $e) {
+            if ($sourceIsTrashbin) {
+                /** @var Storage $sourceStorage */
+                $sourceStorage->enableTrash();
+            }
+            throw $e;
+        }
+    }
 
-	protected function disableTrash() {
-		$this->trashEnabled = false;
-	}
+    protected function disableTrash() {
+        $this->trashEnabled = false;
+    }
 
-	protected function enableTrash() {
-		$this->trashEnabled = true;
-	}
+    protected function enableTrash() {
+        $this->trashEnabled = true;
+    }
 }