diff options
author | Roeland Jago Douma <roeland@famdouma.nl> | 2019-11-13 11:22:19 +0100 |
---|---|---|
committer | Roeland Jago Douma <roeland@famdouma.nl> | 2019-11-13 11:22:19 +0100 |
commit | fc4c1077882681b94bf02c9957fe2816ddeadbb7 (patch) | |
tree | 82b030bd9c1c88cc3b495d10bce792672486af92 /apps/files | |
parent | eee21c89d94e99c5170730ce085b27b27d69dd9f (diff) | |
download | nextcloud-server-fc4c1077882681b94bf02c9957fe2816ddeadbb7.tar.gz nextcloud-server-fc4c1077882681b94bf02c9957fe2816ddeadbb7.zip |
Check quota before transfer ownership
Fixes #15664
Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
Diffstat (limited to 'apps/files')
-rw-r--r-- | apps/files/lib/Command/TransferOwnership.php | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/apps/files/lib/Command/TransferOwnership.php b/apps/files/lib/Command/TransferOwnership.php index f417898f217..a77a02a4947 100644 --- a/apps/files/lib/Command/TransferOwnership.php +++ b/apps/files/lib/Command/TransferOwnership.php @@ -32,6 +32,7 @@ use OC\Files\Filesystem; use OC\Files\View; use OCP\Files\FileInfo; use OCP\Files\IHomeStorage; +use OCP\Files\IRootFolder; use OCP\Files\Mount\IMountManager; use OCP\IUser; use OCP\IUserManager; @@ -76,7 +77,10 @@ class TransferOwnership extends Command { /** @var string */ private $finalTarget; - public function __construct(IUserManager $userManager, IManager $shareManager, IMountManager $mountManager) { + public function __construct(IUserManager $userManager, + IManager $shareManager, + IMountManager $mountManager, + IRootFolder $rootFolder) { $this->userManager = $userManager; $this->shareManager = $shareManager; $this->mountManager = $mountManager; @@ -174,6 +178,15 @@ class TransferOwnership extends Command { */ protected function analyse(OutputInterface $output) { $view = new View(); + + $output->writeln('Validating quota'); + $size = $view->getFileInfo($this->sourcePath, false)->getSize(false); + $freeSpace = $view->free_space($this->destinationUser . '/files/'); + if ($size > $freeSpace) { + $output->writeln('<error>Target user does not have enough free space available</error>'); + throw new \Exception('Execution terminated'); + } + $output->writeln("Analysing files of $this->sourceUser ..."); $progress = new ProgressBar($output); $progress->start(); @@ -207,7 +220,6 @@ class TransferOwnership extends Command { } throw new \Exception('Execution terminated.'); } - } /** |