nextcloud / server

lib/private/Files/Storage/StorageFactory.php

Indentation +70 added lines, -70 removed lines

@@ -29,80 +29,80 @@
 use OCP\Files\Storage\IStorageFactory;
 
 class StorageFactory implements IStorageFactory {
-	/**
-	 * @var array[] [$name=>['priority'=>$priority, 'wrapper'=>$callable] $storageWrappers
-	 */
-	private $storageWrappers = [];
+    /**
+     * @var array[] [$name=>['priority'=>$priority, 'wrapper'=>$callable] $storageWrappers
+     */
+    private $storageWrappers = [];
 
-	/**
-	 * allow modifier storage behaviour by adding wrappers around storages
-	 *
-	 * $callback should be a function of type (string $mountPoint, Storage $storage) => Storage
-	 *
-	 * @param string $wrapperName name of the wrapper
-	 * @param callable $callback callback
-	 * @param int $priority wrappers with the lower priority are applied last (meaning they get called first)
-	 * @param \OCP\Files\Mount\IMountPoint[] $existingMounts existing mount points to apply the wrapper to
-	 * @return bool true if the wrapper was added, false if there was already a wrapper with this
-	 * name registered
-	 */
-	public function addStorageWrapper($wrapperName, $callback, $priority = 50, $existingMounts = []) {
-		if (isset($this->storageWrappers[$wrapperName])) {
-			return false;
-		}
+    /**
+     * allow modifier storage behaviour by adding wrappers around storages
+     *
+     * $callback should be a function of type (string $mountPoint, Storage $storage) => Storage
+     *
+     * @param string $wrapperName name of the wrapper
+     * @param callable $callback callback
+     * @param int $priority wrappers with the lower priority are applied last (meaning they get called first)
+     * @param \OCP\Files\Mount\IMountPoint[] $existingMounts existing mount points to apply the wrapper to
+     * @return bool true if the wrapper was added, false if there was already a wrapper with this
+     * name registered
+     */
+    public function addStorageWrapper($wrapperName, $callback, $priority = 50, $existingMounts = []) {
+        if (isset($this->storageWrappers[$wrapperName])) {
+            return false;
+        }
 
-		// apply to existing mounts before registering it to prevent applying it double in MountPoint::createStorage
-		foreach ($existingMounts as $mount) {
-			$mount->wrapStorage($callback);
-		}
+        // apply to existing mounts before registering it to prevent applying it double in MountPoint::createStorage
+        foreach ($existingMounts as $mount) {
+            $mount->wrapStorage($callback);
+        }
 
-		$this->storageWrappers[$wrapperName] = ['wrapper' => $callback, 'priority' => $priority];
-		return true;
-	}
+        $this->storageWrappers[$wrapperName] = ['wrapper' => $callback, 'priority' => $priority];
+        return true;
+    }
 
-	/**
-	 * Remove a storage wrapper by name.
-	 * Note: internal method only to be used for cleanup
-	 *
-	 * @param string $wrapperName name of the wrapper
-	 * @internal
-	 */
-	public function removeStorageWrapper($wrapperName) {
-		unset($this->storageWrappers[$wrapperName]);
-	}
+    /**
+     * Remove a storage wrapper by name.
+     * Note: internal method only to be used for cleanup
+     *
+     * @param string $wrapperName name of the wrapper
+     * @internal
+     */
+    public function removeStorageWrapper($wrapperName) {
+        unset($this->storageWrappers[$wrapperName]);
+    }
 
-	/**
-	 * Create an instance of a storage and apply the registered storage wrappers
-	 *
-	 * @param \OCP\Files\Mount\IMountPoint $mountPoint
-	 * @param string $class
-	 * @param array $arguments
-	 * @return \OCP\Files\Storage
-	 */
-	public function getInstance(IMountPoint $mountPoint, $class, $arguments) {
-		return $this->wrap($mountPoint, new $class($arguments));
-	}
+    /**
+     * Create an instance of a storage and apply the registered storage wrappers
+     *
+     * @param \OCP\Files\Mount\IMountPoint $mountPoint
+     * @param string $class
+     * @param array $arguments
+     * @return \OCP\Files\Storage
+     */
+    public function getInstance(IMountPoint $mountPoint, $class, $arguments) {
+        return $this->wrap($mountPoint, new $class($arguments));
+    }
 
-	/**
-	 * @param \OCP\Files\Mount\IMountPoint $mountPoint
-	 * @param \OCP\Files\Storage $storage
-	 * @return \OCP\Files\Storage
-	 */
-	public function wrap(IMountPoint $mountPoint, $storage) {
-		$wrappers = array_values($this->storageWrappers);
-		usort($wrappers, function ($a, $b) {
-			return $b['priority'] - $a['priority'];
-		});
-		/** @var callable[] $wrappers */
-		$wrappers = array_map(function ($wrapper) {
-			return $wrapper['wrapper'];
-		}, $wrappers);
-		foreach ($wrappers as $wrapper) {
-			$storage = $wrapper($mountPoint->getMountPoint(), $storage, $mountPoint);
-			if (!($storage instanceof \OCP\Files\Storage)) {
-				throw new \Exception('Invalid result from storage wrapper');
-			}
-		}
-		return $storage;
-	}
+    /**
+     * @param \OCP\Files\Mount\IMountPoint $mountPoint
+     * @param \OCP\Files\Storage $storage
+     * @return \OCP\Files\Storage
+     */
+    public function wrap(IMountPoint $mountPoint, $storage) {
+        $wrappers = array_values($this->storageWrappers);
+        usort($wrappers, function ($a, $b) {
+            return $b['priority'] - $a['priority'];
+        });
+        /** @var callable[] $wrappers */
+        $wrappers = array_map(function ($wrapper) {
+            return $wrapper['wrapper'];
+        }, $wrappers);
+        foreach ($wrappers as $wrapper) {
+            $storage = $wrapper($mountPoint->getMountPoint(), $storage, $mountPoint);
+            if (!($storage instanceof \OCP\Files\Storage)) {
+                throw new \Exception('Invalid result from storage wrapper');
+            }
+        }
+        return $storage;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -90,11 +90,11 @@
 	 */
 	public function wrap(IMountPoint $mountPoint, $storage) {
 		$wrappers = array_values($this->storageWrappers);
-		usort($wrappers, function ($a, $b) {
+		usort($wrappers, function($a, $b) {
 			return $b['priority'] - $a['priority'];
 		});
 		/** @var callable[] $wrappers */
-		$wrappers = array_map(function ($wrapper) {
+		$wrappers = array_map(function($wrapper) {
 			return $wrapper['wrapper'];
 		}, $wrappers);
 		foreach ($wrappers as $wrapper) {

lib/private/Files/Storage/PolyFill/CopyDirectory.php

Indentation +71 added lines, -71 removed lines

@@ -25,81 +25,81 @@
 namespace OC\Files\Storage\PolyFill;
 
 trait CopyDirectory {
-	/**
-	 * Check if a path is a directory
-	 *
-	 * @param string $path
-	 * @return bool
-	 */
-	abstract public function is_dir($path);
+    /**
+     * Check if a path is a directory
+     *
+     * @param string $path
+     * @return bool
+     */
+    abstract public function is_dir($path);
 
-	/**
-	 * Check if a file or folder exists
-	 *
-	 * @param string $path
-	 * @return bool
-	 */
-	abstract public function file_exists($path);
+    /**
+     * Check if a file or folder exists
+     *
+     * @param string $path
+     * @return bool
+     */
+    abstract public function file_exists($path);
 
-	/**
-	 * Delete a file or folder
-	 *
-	 * @param string $path
-	 * @return bool
-	 */
-	abstract public function unlink($path);
+    /**
+     * Delete a file or folder
+     *
+     * @param string $path
+     * @return bool
+     */
+    abstract public function unlink($path);
 
-	/**
-	 * Open a directory handle for a folder
-	 *
-	 * @param string $path
-	 * @return resource | bool
-	 */
-	abstract public function opendir($path);
+    /**
+     * Open a directory handle for a folder
+     *
+     * @param string $path
+     * @return resource | bool
+     */
+    abstract public function opendir($path);
 
-	/**
-	 * Create a new folder
-	 *
-	 * @param string $path
-	 * @return bool
-	 */
-	abstract public function mkdir($path);
+    /**
+     * Create a new folder
+     *
+     * @param string $path
+     * @return bool
+     */
+    abstract public function mkdir($path);
 
-	public function copy($source, $target) {
-		if ($this->is_dir($source)) {
-			if ($this->file_exists($target)) {
-				$this->unlink($target);
-			}
-			$this->mkdir($target);
-			return $this->copyRecursive($source, $target);
-		} else {
-			return parent::copy($source, $target);
-		}
-	}
+    public function copy($source, $target) {
+        if ($this->is_dir($source)) {
+            if ($this->file_exists($target)) {
+                $this->unlink($target);
+            }
+            $this->mkdir($target);
+            return $this->copyRecursive($source, $target);
+        } else {
+            return parent::copy($source, $target);
+        }
+    }
 
-	/**
-	 * For adapters that don't support copying folders natively
-	 *
-	 * @param $source
-	 * @param $target
-	 * @return bool
-	 */
-	protected function copyRecursive($source, $target) {
-		$dh = $this->opendir($source);
-		$result = true;
-		while ($file = readdir($dh)) {
-			if (!\OC\Files\Filesystem::isIgnoredDir($file)) {
-				if ($this->is_dir($source . '/' . $file)) {
-					$this->mkdir($target . '/' . $file);
-					$result = $this->copyRecursive($source . '/' . $file, $target . '/' . $file);
-				} else {
-					$result = parent::copy($source . '/' . $file, $target . '/' . $file);
-				}
-				if (!$result) {
-					break;
-				}
-			}
-		}
-		return $result;
-	}
+    /**
+     * For adapters that don't support copying folders natively
+     *
+     * @param $source
+     * @param $target
+     * @return bool
+     */
+    protected function copyRecursive($source, $target) {
+        $dh = $this->opendir($source);
+        $result = true;
+        while ($file = readdir($dh)) {
+            if (!\OC\Files\Filesystem::isIgnoredDir($file)) {
+                if ($this->is_dir($source . '/' . $file)) {
+                    $this->mkdir($target . '/' . $file);
+                    $result = $this->copyRecursive($source . '/' . $file, $target . '/' . $file);
+                } else {
+                    $result = parent::copy($source . '/' . $file, $target . '/' . $file);
+                }
+                if (!$result) {
+                    break;
+                }
+            }
+        }
+        return $result;
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -89,11 +89,11 @@
 		$result = true;
 		while ($file = readdir($dh)) {
 			if (!\OC\Files\Filesystem::isIgnoredDir($file)) {
-				if ($this->is_dir($source . '/' . $file)) {
-					$this->mkdir($target . '/' . $file);
-					$result = $this->copyRecursive($source . '/' . $file, $target . '/' . $file);
+				if ($this->is_dir($source.'/'.$file)) {
+					$this->mkdir($target.'/'.$file);
+					$result = $this->copyRecursive($source.'/'.$file, $target.'/'.$file);
 				} else {
-					$result = parent::copy($source . '/' . $file, $target . '/' . $file);
+					$result = parent::copy($source.'/'.$file, $target.'/'.$file);
 				}
 				if (!$result) {
 					break;

lib/private/Files/Storage/CommonTest.php

Indentation +47 added lines, -47 removed lines

@@ -33,53 +33,53 @@
 namespace OC\Files\Storage;
 
 class CommonTest extends \OC\Files\Storage\Common{
-	/**
-	 * underlying local storage used for missing functions
-	 * @var \OC\Files\Storage\Local
-	 */
-	private $storage;
+    /**
+     * underlying local storage used for missing functions
+     * @var \OC\Files\Storage\Local
+     */
+    private $storage;
 
-	public function __construct($params) {
-		$this->storage=new \OC\Files\Storage\Local($params);
-	}
+    public function __construct($params) {
+        $this->storage=new \OC\Files\Storage\Local($params);
+    }
 
-	public function getId(){
-		return 'test::'.$this->storage->getId();
-	}
-	public function mkdir($path) {
-		return $this->storage->mkdir($path);
-	}
-	public function rmdir($path) {
-		return $this->storage->rmdir($path);
-	}
-	public function opendir($path) {
-		return $this->storage->opendir($path);
-	}
-	public function stat($path) {
-		return $this->storage->stat($path);
-	}
-	public function filetype($path) {
-		return @$this->storage->filetype($path);
-	}
-	public function isReadable($path) {
-		return $this->storage->isReadable($path);
-	}
-	public function isUpdatable($path) {
-		return $this->storage->isUpdatable($path);
-	}
-	public function file_exists($path) {
-		return $this->storage->file_exists($path);
-	}
-	public function unlink($path) {
-		return $this->storage->unlink($path);
-	}
-	public function fopen($path, $mode) {
-		return $this->storage->fopen($path, $mode);
-	}
-	public function free_space($path) {
-		return $this->storage->free_space($path);
-	}
-	public function touch($path, $mtime=null) {
-		return $this->storage->touch($path, $mtime);
-	}
+    public function getId(){
+        return 'test::'.$this->storage->getId();
+    }
+    public function mkdir($path) {
+        return $this->storage->mkdir($path);
+    }
+    public function rmdir($path) {
+        return $this->storage->rmdir($path);
+    }
+    public function opendir($path) {
+        return $this->storage->opendir($path);
+    }
+    public function stat($path) {
+        return $this->storage->stat($path);
+    }
+    public function filetype($path) {
+        return @$this->storage->filetype($path);
+    }
+    public function isReadable($path) {
+        return $this->storage->isReadable($path);
+    }
+    public function isUpdatable($path) {
+        return $this->storage->isUpdatable($path);
+    }
+    public function file_exists($path) {
+        return $this->storage->file_exists($path);
+    }
+    public function unlink($path) {
+        return $this->storage->unlink($path);
+    }
+    public function fopen($path, $mode) {
+        return $this->storage->fopen($path, $mode);
+    }
+    public function free_space($path) {
+        return $this->storage->free_space($path);
+    }
+    public function touch($path, $mtime=null) {
+        return $this->storage->touch($path, $mtime);
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -32,7 +32,7 @@ 
 
 namespace OC\Files\Storage;
 
-class CommonTest extends \OC\Files\Storage\Common{
+class CommonTest extends \OC\Files\Storage\Common {
 	/**
 	 * underlying local storage used for missing functions
 	 * @var \OC\Files\Storage\Local
@@ -40,10 +40,10 @@ 
 	private $storage;
 
 	public function __construct($params) {
-		$this->storage=new \OC\Files\Storage\Local($params);
+		$this->storage = new \OC\Files\Storage\Local($params);
 	}
 
-	public function getId(){
+	public function getId() {
 		return 'test::'.$this->storage->getId();
 	}
 	public function mkdir($path) {
@@ -79,7 +79,7 @@ 
 	public function free_space($path) {
 		return $this->storage->free_space($path);
 	}
-	public function touch($path, $mtime=null) {
+	public function touch($path, $mtime = null) {
 		return $this->storage->touch($path, $mtime);
 	}
 }

lib/private/Files/Notify/RenameChange.php

Indentation +21 added lines, -21 removed lines

@@ -26,27 +26,27 @@
 use OCP\Files\Notify\IRenameChange;
 
 class RenameChange extends Change implements IRenameChange {
-	/** @var string */
-	private $targetPath;
+    /** @var string */
+    private $targetPath;
 
-	/**
-	 * Change constructor.
-	 *
-	 * @param int $type
-	 * @param string $path
-	 * @param string $targetPath
-	 */
-	public function __construct($type, $path, $targetPath) {
-		parent::__construct($type, $path);
-		$this->targetPath = $targetPath;
-	}
+    /**
+     * Change constructor.
+     *
+     * @param int $type
+     * @param string $path
+     * @param string $targetPath
+     */
+    public function __construct($type, $path, $targetPath) {
+        parent::__construct($type, $path);
+        $this->targetPath = $targetPath;
+    }
 
-	/**
-	 * Get the new path of the renamed file relative to the storage root
-	 *
-	 * @return string
-	 */
-	public function getTargetPath() {
-		return $this->targetPath;
-	}
+    /**
+     * Get the new path of the renamed file relative to the storage root
+     *
+     * @return string
+     */
+    public function getTargetPath() {
+        return $this->targetPath;
+    }
 }

lib/private/Files/Notify/Change.php

Indentation +32 added lines, -32 removed lines

@@ -26,40 +26,40 @@
 use OCP\Files\Notify\IChange;
 
 class Change implements IChange {
-	/** @var int */
-	private $type;
+    /** @var int */
+    private $type;
 
-	/** @var string */
-	private $path;
+    /** @var string */
+    private $path;
 
-	/**
-	 * Change constructor.
-	 *
-	 * @param int $type
-	 * @param string $path
-	 */
-	public function __construct($type, $path) {
-		$this->type = $type;
-		$this->path = $path;
-	}
+    /**
+     * Change constructor.
+     *
+     * @param int $type
+     * @param string $path
+     */
+    public function __construct($type, $path) {
+        $this->type = $type;
+        $this->path = $path;
+    }
 
-	/**
-	 * Get the type of the change
-	 *
-	 * @return int IChange::ADDED, IChange::REMOVED, IChange::MODIFIED or IChange::RENAMED
-	 */
-	public function getType() {
-		return $this->type;
-	}
+    /**
+     * Get the type of the change
+     *
+     * @return int IChange::ADDED, IChange::REMOVED, IChange::MODIFIED or IChange::RENAMED
+     */
+    public function getType() {
+        return $this->type;
+    }
 
-	/**
-	 * Get the path of the file that was changed relative to the root of the storage
-	 *
-	 * Note, for rename changes this path is the old path for the file
-	 *
-	 * @return mixed
-	 */
-	public function getPath() {
-		return $this->path;
-	}
+    /**
+     * Get the path of the file that was changed relative to the root of the storage
+     *
+     * Note, for rename changes this path is the old path for the file
+     *
+     * @return mixed
+     */
+    public function getPath() {
+        return $this->path;
+    }
 }

lib/private/Files/Cache/HomePropagator.php

Indentation +21 added lines, -21 removed lines

@@ -25,28 +25,28 @@
 use OCP\IDBConnection;
 
 class HomePropagator extends Propagator {
-	private $ignoredBaseFolders;
+    private $ignoredBaseFolders;
 
-	/**
-	 * @param \OC\Files\Storage\Storage $storage
-	 */
-	public function __construct(\OC\Files\Storage\Storage $storage, IDBConnection $connection) {
-		parent::__construct($storage, $connection);
-		$this->ignoredBaseFolders = ['files_encryption'];
-	}
+    /**
+     * @param \OC\Files\Storage\Storage $storage
+     */
+    public function __construct(\OC\Files\Storage\Storage $storage, IDBConnection $connection) {
+        parent::__construct($storage, $connection);
+        $this->ignoredBaseFolders = ['files_encryption'];
+    }
 
 
-	/**
-	 * @param string $internalPath
-	 * @param int $time
-	 * @param int $sizeDifference number of bytes the file has grown
-	 */
-	public function propagateChange($internalPath, $time, $sizeDifference = 0) {
-		list($baseFolder) = explode('/', $internalPath, 2);
-		if (in_array($baseFolder, $this->ignoredBaseFolders)) {
-			return [];
-		} else {
-			parent::propagateChange($internalPath, $time, $sizeDifference);
-		}
-	}
+    /**
+     * @param string $internalPath
+     * @param int $time
+     * @param int $sizeDifference number of bytes the file has grown
+     */
+    public function propagateChange($internalPath, $time, $sizeDifference = 0) {
+        list($baseFolder) = explode('/', $internalPath, 2);
+        if (in_array($baseFolder, $this->ignoredBaseFolders)) {
+            return [];
+        } else {
+            parent::propagateChange($internalPath, $time, $sizeDifference);
+        }
+    }
 }

lib/private/Files/Cache/CacheEntry.php

Spacing +1 added lines, -1 removed lines

@@ -58,7 +58,7 @@
 	}
 
 	public function getId() {
-		return (int)$this->data['fileid'];
+		return (int) $this->data['fileid'];
 	}
 
 	public function getStorageId() {

Indentation +71 added lines, -71 removed lines

@@ -28,100 +28,100 @@
  * meta data for a file or folder
  */
 class CacheEntry implements ICacheEntry, \ArrayAccess {
-	/**
-	 * @var array
-	 */
-	private $data;
+    /**
+     * @var array
+     */
+    private $data;
 
-	public function __construct(array $data) {
-		$this->data = $data;
-	}
+    public function __construct(array $data) {
+        $this->data = $data;
+    }
 
-	public function offsetSet($offset, $value) {
-		$this->data[$offset] = $value;
-	}
+    public function offsetSet($offset, $value) {
+        $this->data[$offset] = $value;
+    }
 
-	public function offsetExists($offset) {
-		return isset($this->data[$offset]);
-	}
+    public function offsetExists($offset) {
+        return isset($this->data[$offset]);
+    }
 
-	public function offsetUnset($offset) {
-		unset($this->data[$offset]);
-	}
+    public function offsetUnset($offset) {
+        unset($this->data[$offset]);
+    }
 
-	public function offsetGet($offset) {
-		if (isset($this->data[$offset])) {
-			return $this->data[$offset];
-		} else {
-			return null;
-		}
-	}
+    public function offsetGet($offset) {
+        if (isset($this->data[$offset])) {
+            return $this->data[$offset];
+        } else {
+            return null;
+        }
+    }
 
-	public function getId() {
-		return (int)$this->data['fileid'];
-	}
+    public function getId() {
+        return (int)$this->data['fileid'];
+    }
 
-	public function getStorageId() {
-		return $this->data['storage'];
-	}
+    public function getStorageId() {
+        return $this->data['storage'];
+    }
 
 
-	public function getPath() {
-		return $this->data['path'];
-	}
+    public function getPath() {
+        return $this->data['path'];
+    }
 
 
-	public function getName() {
-		return $this->data['name'];
-	}
+    public function getName() {
+        return $this->data['name'];
+    }
 
 
-	public function getMimeType() {
-		return $this->data['mimetype'];
-	}
+    public function getMimeType() {
+        return $this->data['mimetype'];
+    }
 
 
-	public function getMimePart() {
-		return $this->data['mimepart'];
-	}
+    public function getMimePart() {
+        return $this->data['mimepart'];
+    }
 
-	public function getSize() {
-		return $this->data['size'];
-	}
+    public function getSize() {
+        return $this->data['size'];
+    }
 
-	public function getMTime() {
-		return $this->data['mtime'];
-	}
+    public function getMTime() {
+        return $this->data['mtime'];
+    }
 
-	public function getStorageMTime() {
-		return $this->data['storage_mtime'];
-	}
+    public function getStorageMTime() {
+        return $this->data['storage_mtime'];
+    }
 
-	public function getEtag() {
-		return $this->data['etag'];
-	}
+    public function getEtag() {
+        return $this->data['etag'];
+    }
 
-	public function getPermissions() {
-		return $this->data['permissions'];
-	}
+    public function getPermissions() {
+        return $this->data['permissions'];
+    }
 
-	public function isEncrypted() {
-		return isset($this->data['encrypted']) && $this->data['encrypted'];
-	}
+    public function isEncrypted() {
+        return isset($this->data['encrypted']) && $this->data['encrypted'];
+    }
 
-	public function getMetadataEtag(): ?string {
-		return $this->data['metadata_etag'];
-	}
+    public function getMetadataEtag(): ?string {
+        return $this->data['metadata_etag'];
+    }
 
-	public function getCreationTime(): ?int {
-		return $this->data['creation_time'];
-	}
+    public function getCreationTime(): ?int {
+        return $this->data['creation_time'];
+    }
 
-	public function getUploadTime(): ?int {
-		return $this->data['upload_time'];
-	}
+    public function getUploadTime(): ?int {
+        return $this->data['upload_time'];
+    }
 
-	public function getData() {
-		return $this->data;
-	}
+    public function getData() {
+        return $this->data;
+    }
 }

lib/private/Files/Cache/HomeCache.php

Indentation +51 added lines, -51 removed lines

@@ -30,58 +30,58 @@
 use OCP\Files\Cache\ICacheEntry;
 
 class HomeCache extends Cache {
-	/**
-	 * get the size of a folder and set it in the cache
-	 *
-	 * @param string $path
-	 * @param array $entry (optional) meta data of the folder
-	 * @return int
-	 */
-	public function calculateFolderSize($path, $entry = null) {
-		if ($path !== '/' and $path !== '' and $path !== 'files' and $path !== 'files_trashbin' and $path !== 'files_versions') {
-			return parent::calculateFolderSize($path, $entry);
-		} elseif ($path === '' or $path === '/') {
-			// since the size of / isn't used (the size of /files is used instead) there is no use in calculating it
-			return 0;
-		}
+    /**
+     * get the size of a folder and set it in the cache
+     *
+     * @param string $path
+     * @param array $entry (optional) meta data of the folder
+     * @return int
+     */
+    public function calculateFolderSize($path, $entry = null) {
+        if ($path !== '/' and $path !== '' and $path !== 'files' and $path !== 'files_trashbin' and $path !== 'files_versions') {
+            return parent::calculateFolderSize($path, $entry);
+        } elseif ($path === '' or $path === '/') {
+            // since the size of / isn't used (the size of /files is used instead) there is no use in calculating it
+            return 0;
+        }
 
-		$totalSize = 0;
-		if (is_null($entry)) {
-			$entry = $this->get($path);
-		}
-		if ($entry && $entry['mimetype'] === 'httpd/unix-directory') {
-			$id = $entry['fileid'];
-			$sql = 'SELECT SUM(`size`) AS f1 ' .
-			   'FROM `*PREFIX*filecache` ' .
-				'WHERE `parent` = ? AND `storage` = ? AND `size` >= 0';
-			$result = \OC_DB::executeAudited($sql, array($id, $this->getNumericStorageId()));
-			if ($row = $result->fetchRow()) {
-				$result->closeCursor();
-				list($sum) = array_values($row);
-				$totalSize = 0 + $sum;
-				$entry['size'] += 0;
-				if ($entry['size'] !== $totalSize) {
-					$this->update($id, array('size' => $totalSize));
-				}
-			}
-		}
-		return $totalSize;
-	}
+        $totalSize = 0;
+        if (is_null($entry)) {
+            $entry = $this->get($path);
+        }
+        if ($entry && $entry['mimetype'] === 'httpd/unix-directory') {
+            $id = $entry['fileid'];
+            $sql = 'SELECT SUM(`size`) AS f1 ' .
+                'FROM `*PREFIX*filecache` ' .
+                'WHERE `parent` = ? AND `storage` = ? AND `size` >= 0';
+            $result = \OC_DB::executeAudited($sql, array($id, $this->getNumericStorageId()));
+            if ($row = $result->fetchRow()) {
+                $result->closeCursor();
+                list($sum) = array_values($row);
+                $totalSize = 0 + $sum;
+                $entry['size'] += 0;
+                if ($entry['size'] !== $totalSize) {
+                    $this->update($id, array('size' => $totalSize));
+                }
+            }
+        }
+        return $totalSize;
+    }
 
-	/**
-	 * @param string $path
-	 * @return ICacheEntry
-	 */
-	public function get($path) {
-		$data = parent::get($path);
-		if ($path === '' or $path === '/') {
-			// only the size of the "files" dir counts
-			$filesData = parent::get('files');
+    /**
+     * @param string $path
+     * @return ICacheEntry
+     */
+    public function get($path) {
+        $data = parent::get($path);
+        if ($path === '' or $path === '/') {
+            // only the size of the "files" dir counts
+            $filesData = parent::get('files');
 
-			if (isset($filesData['size'])) {
-				$data['size'] = $filesData['size'];
-			}
-		}
-		return $data;
-	}
+            if (isset($filesData['size'])) {
+                $data['size'] = $filesData['size'];
+            }
+        }
+        return $data;
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -51,8 +51,8 @@
 		}
 		if ($entry && $entry['mimetype'] === 'httpd/unix-directory') {
 			$id = $entry['fileid'];
-			$sql = 'SELECT SUM(`size`) AS f1 ' .
-			   'FROM `*PREFIX*filecache` ' .
+			$sql = 'SELECT SUM(`size`) AS f1 '.
+			   'FROM `*PREFIX*filecache` '.
 				'WHERE `parent` = ? AND `storage` = ? AND `size` >= 0';
 			$result = \OC_DB::executeAudited($sql, array($id, $this->getNumericStorageId()));
 			if ($row = $result->fetchRow()) {

lib/private/Files/Cache/MoveFromCacheTrait.php

Spacing +1 added lines, -1 removed lines

@@ -67,7 +67,7 @@
 		if ($sourceEntry->getMimeType() === ICacheEntry::DIRECTORY_MIMETYPE) {
 			$folderContent = $sourceCache->getFolderContentsById($sourceEntry->getId());
 			foreach ($folderContent as $subEntry) {
-				$subTargetPath = $targetPath . '/' . $subEntry->getName();
+				$subTargetPath = $targetPath.'/'.$subEntry->getName();
 				$this->copyFromCache($sourceCache, $subEntry, $subTargetPath);
 			}
 		}

Indentation +54 added lines, -54 removed lines

@@ -29,63 +29,63 @@
  * Fallback implementation for moveFromCache
  */
 trait MoveFromCacheTrait {
-	/**
-	 * store meta data for a file or folder
-	 *
-	 * @param string $file
-	 * @param array $data
-	 *
-	 * @return int file id
-	 * @throws \RuntimeException
-	 */
-	abstract public function put($file, array $data);
+    /**
+     * store meta data for a file or folder
+     *
+     * @param string $file
+     * @param array $data
+     *
+     * @return int file id
+     * @throws \RuntimeException
+     */
+    abstract public function put($file, array $data);
 
-	/**
-	 * Move a file or folder in the cache
-	 *
-	 * @param \OCP\Files\Cache\ICache $sourceCache
-	 * @param string $sourcePath
-	 * @param string $targetPath
-	 */
-	public function moveFromCache(ICache $sourceCache, $sourcePath, $targetPath) {
-		$sourceEntry = $sourceCache->get($sourcePath);
+    /**
+     * Move a file or folder in the cache
+     *
+     * @param \OCP\Files\Cache\ICache $sourceCache
+     * @param string $sourcePath
+     * @param string $targetPath
+     */
+    public function moveFromCache(ICache $sourceCache, $sourcePath, $targetPath) {
+        $sourceEntry = $sourceCache->get($sourcePath);
 
-		$this->copyFromCache($sourceCache, $sourceEntry, $targetPath);
+        $this->copyFromCache($sourceCache, $sourceEntry, $targetPath);
 
-		$sourceCache->remove($sourcePath);
-	}
+        $sourceCache->remove($sourcePath);
+    }
 
-	/**
-	 * Copy a file or folder in the cache
-	 *
-	 * @param \OCP\Files\Cache\ICache $sourceCache
-	 * @param ICacheEntry $sourceEntry
-	 * @param string $targetPath
-	 */
-	public function copyFromCache(ICache $sourceCache, ICacheEntry $sourceEntry, $targetPath) {
-		$this->put($targetPath, $this->cacheEntryToArray($sourceEntry));
-		if ($sourceEntry->getMimeType() === ICacheEntry::DIRECTORY_MIMETYPE) {
-			$folderContent = $sourceCache->getFolderContentsById($sourceEntry->getId());
-			foreach ($folderContent as $subEntry) {
-				$subTargetPath = $targetPath . '/' . $subEntry->getName();
-				$this->copyFromCache($sourceCache, $subEntry, $subTargetPath);
-			}
-		}
-	}
+    /**
+     * Copy a file or folder in the cache
+     *
+     * @param \OCP\Files\Cache\ICache $sourceCache
+     * @param ICacheEntry $sourceEntry
+     * @param string $targetPath
+     */
+    public function copyFromCache(ICache $sourceCache, ICacheEntry $sourceEntry, $targetPath) {
+        $this->put($targetPath, $this->cacheEntryToArray($sourceEntry));
+        if ($sourceEntry->getMimeType() === ICacheEntry::DIRECTORY_MIMETYPE) {
+            $folderContent = $sourceCache->getFolderContentsById($sourceEntry->getId());
+            foreach ($folderContent as $subEntry) {
+                $subTargetPath = $targetPath . '/' . $subEntry->getName();
+                $this->copyFromCache($sourceCache, $subEntry, $subTargetPath);
+            }
+        }
+    }
 
-	private function cacheEntryToArray(ICacheEntry $entry) {
-		return [
-			'size' => $entry->getSize(),
-			'mtime' => $entry->getMTime(),
-			'storage_mtime' => $entry->getStorageMTime(),
-			'mimetype' => $entry->getMimeType(),
-			'mimepart' => $entry->getMimePart(),
-			'etag' => $entry->getEtag(),
-			'permissions' => $entry->getPermissions(),
-			'encrypted' => $entry->isEncrypted(),
-			'creation_time' => $entry->getCreationTime(),
-			'upload_time' => $entry->getUploadTime(),
-			'metadata_etag' => $entry->getMetadataEtag(),
-		];
-	}
+    private function cacheEntryToArray(ICacheEntry $entry) {
+        return [
+            'size' => $entry->getSize(),
+            'mtime' => $entry->getMTime(),
+            'storage_mtime' => $entry->getStorageMTime(),
+            'mimetype' => $entry->getMimeType(),
+            'mimepart' => $entry->getMimePart(),
+            'etag' => $entry->getEtag(),
+            'permissions' => $entry->getPermissions(),
+            'encrypted' => $entry->isEncrypted(),
+            'creation_time' => $entry->getCreationTime(),
+            'upload_time' => $entry->getUploadTime(),
+            'metadata_etag' => $entry->getMetadataEtag(),
+        ];
+    }
 }