add option to list all files instead of limiting

Signed-off-by: Robin Appelman <[email protected]>

Author: icewind1991

core/Command/Info/File.php

@@ -7,10 +7,8 @@
 use OC\Files\ObjectStore\ObjectStoreStorage;
 use OCA\Files_External\Config\ExternalMountPoint;
 use OCA\GroupFolders\Mount\GroupMountPoint;
-use OCP\Files\Config\IUserMountCache;
 use OCP\Files\Folder;
 use OCP\Files\IHomeStorage;
-use OCP\Files\IRootFolder;
 use OCP\Files\Mount\IMountPoint;
 use OCP\Files\Node;
 use OCP\Files\NotFoundException;

core/Command/Info/FileUtils.php

@@ -23,7 +23,6 @@
 
 namespace OC\Core\Command\Info;
 
-
 use OC\Files\SetupManager;
 use OCA\Circles\MountManager\CircleMount;
 use OCA\Files_External\Config\ExternalMountPoint;
@@ -196,7 +195,8 @@ public function outputLargeFilesTree(
 		OutputInterface $output,
 		Folder $node,
 		string $prefix,
-		array &$sizeLimits
+		array &$sizeLimits,
+		bool $all,
 	): int {
 		/**
 		 * Algorithm to print the N largest items in a folder without requiring to query or sort the entire three
@@ -221,25 +221,31 @@ public function outputLargeFilesTree(
 			return $b->getSize() <=> $a->getSize();
 		});
 		foreach ($children as $i => $child) {
-			if (count($sizeLimits) === 0 || $child->getSize() < $sizeLimits[0]) {
-				return $count;
+			if (!$all) {
+				if (count($sizeLimits) === 0 || $child->getSize() < $sizeLimits[0]) {
+					return $count;
+				}
+				array_shift($sizeLimits);
 			}
-			array_shift($sizeLimits);
 			$count += 1;
 
 			/** @var Node $child */
 			$output->writeln("$prefix- " . $child->getName() . ": <info>" . Util::humanFileSize($child->getSize()) . "</info>");
 			if ($child instanceof Folder) {
 				$recurseSizeLimits = $sizeLimits;
-				for ($j = 0; $j < count($recurseSizeLimits); $j++) {
-					$nextChildSize = (int)$children[$i + $j + 1]?->getSize();
-					if ($nextChildSize > $recurseSizeLimits[0]) {
-						array_shift($recurseSizeLimits);
-						$recurseSizeLimits[] = $nextChildSize;
+				if (!$all) {
+					for ($j = 0; $j < count($recurseSizeLimits); $j++) {
+						if (isset($children[$i + $j + 1])) {
+							$nextChildSize = $children[$i + $j + 1]->getSize();
+							if ($nextChildSize > $recurseSizeLimits[0]) {
+								array_shift($recurseSizeLimits);
+								$recurseSizeLimits[] = $nextChildSize;
+							}
+						}
 					}
+					sort($recurseSizeLimits);
 				}
-				sort($recurseSizeLimits);
-				$recurseCount = $this->outputLargeFilesTree($output, $child, $prefix . "  ", $recurseSizeLimits);
+				$recurseCount = $this->outputLargeFilesTree($output, $child, $prefix . "  ", $recurseSizeLimits, $all);
 				$sizeLimits = array_slice($sizeLimits, $recurseCount);
 				$count += $recurseCount;
 			}

core/Command/Info/Space.php

@@ -23,7 +23,6 @@
 
 namespace OC\Core\Command\Info;
 
-
 use OCP\Files\Folder;
 use OCP\Util;
 use Symfony\Component\Console\Command\Command;
@@ -45,23 +44,24 @@ protected function configure(): void {
 			->setName('info:file:space')
 			->setDescription('Summarize space usage of specified folder')
 			->addArgument('file', InputArgument::REQUIRED, "File id or path")
-			->addOption('count', 'c', InputOption::VALUE_REQUIRED, "Number of items to display", 25);
+			->addOption('count', 'c', InputOption::VALUE_REQUIRED, "Number of items to display", 25)
+			->addOption('all', 'a', InputOption::VALUE_NONE, "Display all items");
 	}
 
 	public function execute(InputInterface $input, OutputInterface $output): int {
 		$fileInput = $input->getArgument('file');
 		$count = (int)$input->getOption('count');
+		$all = $input->getOption('all');
 		$node = $this->fileUtils->getNode($fileInput);
 		if (!$node) {
 			$output->writeln("<error>file $fileInput not found</error>");
 			return 1;
 		}
 		$output->writeln($node->getName() . ": <info>" . Util::humanFileSize($node->getSize()) . "</info>");
 		if ($node instanceof Folder) {
-			$limits = array_fill(0, $count - 1, 0);
-			$this->fileUtils->outputLargeFilesTree($output, $node, '', $limits);
+			$limits = $all ? [] : array_fill(0, $count - 1, 0);
+			$this->fileUtils->outputLargeFilesTree($output, $node, '', $limits, $all);
 		}
 		return 0;
 	}
-
 }

lib/composer/composer/autoload_classmap.php

@@ -960,6 +960,8 @@
     'OC\\Core\\Command\\Group\\ListCommand' => $baseDir . '/core/Command/Group/ListCommand.php',
     'OC\\Core\\Command\\Group\\RemoveUser' => $baseDir . '/core/Command/Group/RemoveUser.php',
     'OC\\Core\\Command\\Info\\File' => $baseDir . '/core/Command/Info/File.php',
+    'OC\\Core\\Command\\Info\\FileUtils' => $baseDir . '/core/Command/Info/FileUtils.php',
+    'OC\\Core\\Command\\Info\\Space' => $baseDir . '/core/Command/Info/Space.php',
     'OC\\Core\\Command\\Integrity\\CheckApp' => $baseDir . '/core/Command/Integrity/CheckApp.php',
     'OC\\Core\\Command\\Integrity\\CheckCore' => $baseDir . '/core/Command/Integrity/CheckCore.php',
     'OC\\Core\\Command\\Integrity\\SignApp' => $baseDir . '/core/Command/Integrity/SignApp.php',

lib/composer/composer/autoload_static.php

@@ -993,6 +993,8 @@ class ComposerStaticInit749170dad3f5e7f9ca158f5a9f04f6a2
         'OC\\Core\\Command\\Group\\ListCommand' => __DIR__ . '/../../..' . '/core/Command/Group/ListCommand.php',
         'OC\\Core\\Command\\Group\\RemoveUser' => __DIR__ . '/../../..' . '/core/Command/Group/RemoveUser.php',
         'OC\\Core\\Command\\Info\\File' => __DIR__ . '/../../..' . '/core/Command/Info/File.php',
+        'OC\\Core\\Command\\Info\\FileUtils' => __DIR__ . '/../../..' . '/core/Command/Info/FileUtils.php',
+        'OC\\Core\\Command\\Info\\Space' => __DIR__ . '/../../..' . '/core/Command/Info/Space.php',
         'OC\\Core\\Command\\Integrity\\CheckApp' => __DIR__ . '/../../..' . '/core/Command/Integrity/CheckApp.php',
         'OC\\Core\\Command\\Integrity\\CheckCore' => __DIR__ . '/../../..' . '/core/Command/Integrity/CheckCore.php',
         'OC\\Core\\Command\\Integrity\\SignApp' => __DIR__ . '/../../..' . '/core/Command/Integrity/SignApp.php',