diff --git a/apps/files_trashbin/lib/Trash/TrashItem.php b/apps/files_trashbin/lib/Trash/TrashItem.php index 40ceb59aba..a0a0e1f67c 100644 --- a/apps/files_trashbin/lib/Trash/TrashItem.php +++ b/apps/files_trashbin/lib/Trash/TrashItem.php @@ -82,8 +82,8 @@ class TrashItem implements ITrashItem { return $this->fileInfo->getEtag(); } - public function getSize() { - return $this->fileInfo->getSize(); + public function getSize($includeMounts = true) { + return $this->fileInfo->getSize($includeMounts); } public function getMtime() { @@ -133,7 +133,7 @@ class TrashItem implements ITrashItem { public function isReadable() { return $this->fileInfo->isReadable(); } - + public function isUpdateable() { return $this->fileInfo->isUpdateable(); } diff --git a/lib/private/Files/FileInfo.php b/lib/private/Files/FileInfo.php index 575af56ceb..19b95cd035 100644 --- a/lib/private/Files/FileInfo.php +++ b/lib/private/Files/FileInfo.php @@ -80,6 +80,13 @@ class FileInfo implements \OCP\Files\FileInfo, \ArrayAccess { private $subMountsUsed = false; + /** + * The size of the file/folder without any sub mount + * + * @var int + */ + private $rawSize = 0; + /** * @param string|boolean $path * @param Storage\Storage $storage @@ -95,6 +102,7 @@ class FileInfo implements \OCP\Files\FileInfo, \ArrayAccess { $this->data = $data; $this->mount = $mount; $this->owner = $owner; + $this->rawSize = $this->data['size'] ?? 0; } public function offsetSet($offset, $value) { @@ -194,9 +202,13 @@ class FileInfo implements \OCP\Files\FileInfo, \ArrayAccess { /** * @return int */ - public function getSize() { - $this->updateEntryfromSubMounts(); - return isset($this->data['size']) ? 0 + $this->data['size'] : 0; + public function getSize($includeMounts = true) { + if ($includeMounts) { + $this->updateEntryfromSubMounts(); + return isset($this->data['size']) ? 0 + $this->data['size'] : 0; + } else { + return $this->rawSize; + } } /** diff --git a/lib/private/Files/Node/LazyRoot.php b/lib/private/Files/Node/LazyRoot.php index 389a1a9f0f..01b4ca5276 100644 --- a/lib/private/Files/Node/LazyRoot.php +++ b/lib/private/Files/Node/LazyRoot.php @@ -214,7 +214,7 @@ class LazyRoot implements IRootFolder { /** * @inheritDoc */ - public function getSize() { + public function getSize($includeMounts = true) { return $this->__call(__FUNCTION__, func_get_args()); } diff --git a/lib/private/Files/Node/Node.php b/lib/private/Files/Node/Node.php index 590f108061..41e8bf7524 100644 --- a/lib/private/Files/Node/Node.php +++ b/lib/private/Files/Node/Node.php @@ -190,12 +190,13 @@ class Node implements \OCP\Files\Node { } /** + * @param bool $includeMounts * @return int * @throws InvalidPathException * @throws NotFoundException */ - public function getSize() { - return $this->getFileInfo()->getSize(); + public function getSize($includeMounts = true) { + return $this->getFileInfo()->getSize($includeMounts); } /** diff --git a/lib/private/Files/Node/NonExistingFile.php b/lib/private/Files/Node/NonExistingFile.php index fbfb67749b..5183102483 100644 --- a/lib/private/Files/Node/NonExistingFile.php +++ b/lib/private/Files/Node/NonExistingFile.php @@ -66,9 +66,9 @@ class NonExistingFile extends File { } } - public function getSize() { + public function getSize($includeMounts = true) { if ($this->fileInfo) { - return parent::getSize(); + return parent::getSize($includeMounts); } else { throw new NotFoundException(); } diff --git a/lib/private/Files/Node/NonExistingFolder.php b/lib/private/Files/Node/NonExistingFolder.php index efc359181c..f6a1794a53 100644 --- a/lib/private/Files/Node/NonExistingFolder.php +++ b/lib/private/Files/Node/NonExistingFolder.php @@ -67,9 +67,9 @@ class NonExistingFolder extends Folder { } } - public function getSize() { + public function getSize($includeMounts = true) { if ($this->fileInfo) { - return parent::getSize(); + return parent::getSize($includeMounts); } else { throw new NotFoundException(); } diff --git a/lib/private/Files/Node/Root.php b/lib/private/Files/Node/Root.php index 189c2863a3..390d29edb3 100644 --- a/lib/private/Files/Node/Root.php +++ b/lib/private/Files/Node/Root.php @@ -282,9 +282,10 @@ class Root extends Folder implements IRootFolder { } /** + * @param bool $includeMounts * @return int */ - public function getSize() { + public function getSize($includeMounts = true) { return null; } diff --git a/lib/public/Files/FileInfo.php b/lib/public/Files/FileInfo.php index e25a47e83c..c256f0980f 100644 --- a/lib/public/Files/FileInfo.php +++ b/lib/public/Files/FileInfo.php @@ -81,10 +81,11 @@ interface FileInfo { /** * Get the size in bytes for the file or folder * + * @param bool $includeMounts whether or not to include the size of any sub mounts, since 16.0.0 * @return int * @since 7.0.0 */ - public function getSize(); + public function getSize($includeMounts = true); /** * Get the last modified date as timestamp for the file or folder diff --git a/lib/public/Files/Node.php b/lib/public/Files/Node.php index 59bd0e193c..016b217afc 100644 --- a/lib/public/Files/Node.php +++ b/lib/public/Files/Node.php @@ -136,12 +136,13 @@ interface Node extends FileInfo { /** * Get the size of the file or folder in bytes * + * @param bool $includeMounts * @return int * @throws InvalidPathException * @throws NotFoundException * @since 6.0.0 */ - public function getSize(); + public function getSize($includeMounts = true); /** * Get the Etag of the file or folder