[TASK] Add exit code to cli commands

All cli commands will now return an exit code depending on whether
they found any problems:

- 0 : everything ok, nothing found
- 1 : found problems

Resolves: #619

Author: sypets

Classes/Command/DeleteChildrenWithNonExistingParentCommand.php

@@ -51,6 +51,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 $this->integrityFix->deleteChildrenWithNonExistingParent($warning);
             }
         }
-        return 0;
+        // return with exit code !0 if errors found
+        return count($res['warnings']) > 0 ? 1 : 0;
     }
 }

Classes/Command/DeleteChildrenWithWrongPidCommand.php

@@ -51,6 +51,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 $this->integrityFix->deleteChildrenWithWrongPid($error);
             }
         }
-        return 0;
+        // return with exit code !0 if errors found
+        return count($res['errors']) > 0 ? 1 : 0;
     }
 }

Classes/Command/FixContainerParentForConnectedModeCommand.php

@@ -46,6 +46,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 $this->integrityFix->changeContainerParentToDefaultLanguageContainer($error);
             }
         }
-        return 0;
+        // return with exit code !0 if errors found
+        return count($res['errors']) > 0 ? 1 : 0;
     }
 }

Classes/Command/FixLanguageModeCommand.php

@@ -48,6 +48,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
             }
         }
         $this->integrityFix->languageMode($errors);
-        return 0;
+        // return with exit code !0 if errors found
+        return count($res['errors']) > 0 ? 1 : 0;
     }
 }

Classes/Command/IntegrityCommand.php

@@ -51,7 +51,9 @@ public function execute(InputInterface $input, OutputInterface $output): int
         }
         if (count($res['warnings']) === 0 && count($res['errors']) === 0) {
             $io->success('Good Job, no errors/warnings!');
+            return 0;
         }
-        return 0;
+        // return with exit code !0 if errors found
+        return 1;
     }
 }

Classes/Command/SortingCommand.php

@@ -69,7 +69,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
                 $output->writeln('- all good, nothing to do here');
             }
         }
-
-        return self::SUCCESS;
+        // return with exit code !0 if errors found
+        return count($errors) > 0 ? 1 : 0;
     }
 }

Classes/Command/SortingInPageCommand.php

@@ -65,6 +65,7 @@ public function execute(InputInterface $input, OutputInterface $output): int
         if (empty($errors)) {
             $output->writeln('migration finished');
         }
-        return 0;
+        // return with exit code !0 if errors found
+        return count($errors) > 0 ? 1 : 0;
     }
 }

README.md

@@ -208,6 +208,8 @@ bin/typo3 container:deleteChildrenWithWrongPid
 bin/typo3 container:deleteChildrenWithNonExistingParent
 ```
 
+Commands will generally exit with 0 on non-error and exit with !0 if an error was found.
+
 ## TODOs
 - Integrity proofs
 - List module actions