[TASK] Improve CLI output

b13 · Aug 1, 2024 · 36e628f · 36e628f
1 parent 9e4003a
commit 36e628f
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 7 deletions.
diff --git a/Classes/Command/SortingCommand.php b/Classes/Command/SortingCommand.php
@@ -59,12 +59,17 @@ public function execute(InputInterface $input, OutputInterface $output): int
             $input->getOption('enable-logging'),
             $pid
         );
-        foreach ($errors as $error) {
-            $output->writeln($error);
-        }
-        if (empty($errors)) {
-            $output->writeln('migration finished');
+
+        if ($output->isVerbose()) {
+            $output->writeln('Checking ' . ($pid > 0 ? 'page ' . $pid : 'entire pagetree') . ($dryrun ? ' [DRYRUN]' : ''));
+            foreach ($errors as $error) {
+                $output->writeln($error);
+            }
+            if (empty($errors)) {
+                $output->writeln('- all good, nothing to do here');
+            }
         }
-        return 0;
+
+        return self::SUCCESS;
     }
 }
diff --git a/Classes/Integrity/Sorting.php b/Classes/Integrity/Sorting.php
@@ -92,7 +92,7 @@ protected function fixChildrenSortingUpdateRequired(Container $container, array
             $children = $container->getChildrenByColPos($colPos);
             foreach ($children as $child) {
                 if ($child['sorting'] <= $prevSorting) {
-                    $this->errors[] = 'container uid: ' . $containerRecord['uid'] . ', pid ' . $containerRecord['pid'] . ' must be fixed';
+                    $this->errors[] = '- pid ' . $containerRecord['pid'] . ', container uid ' . $containerRecord['uid'] . ' must be fixed';
                     return true;
                 }
                 $prevSorting = $child['sorting'];