From f757a76040509eac5215828a4b9df8c2fda882b3 Mon Sep 17 00:00:00 2001 From: Yurii Date: Fri, 26 Jun 2026 11:11:16 +0300 Subject: [PATCH] Commands exist status --- .../Espo/Classes/ConsoleCommands/CheckFilePermissions.php | 4 ++++ application/Espo/Classes/ConsoleCommands/Import.php | 6 ++++++ application/Espo/Core/Console/Commands/AppInfo.php | 1 + application/Espo/Core/Console/Commands/EntityUtil.php | 1 + 4 files changed, 12 insertions(+) diff --git a/application/Espo/Classes/ConsoleCommands/CheckFilePermissions.php b/application/Espo/Classes/ConsoleCommands/CheckFilePermissions.php index bfe06da978..a93714aba1 100644 --- a/application/Espo/Classes/ConsoleCommands/CheckFilePermissions.php +++ b/application/Espo/Classes/ConsoleCommands/CheckFilePermissions.php @@ -60,6 +60,10 @@ class CheckFilePermissions implements Command $msg = " " . ($isWritable ? "OK" : "FAIL") . " : $path"; + if (!$isWritable) { + $io->setExitStatus(1); + } + $io->writeLine($msg); } } diff --git a/application/Espo/Classes/ConsoleCommands/Import.php b/application/Espo/Classes/ConsoleCommands/Import.php index 89fb1b8140..a45dc6cab6 100644 --- a/application/Espo/Classes/ConsoleCommands/Import.php +++ b/application/Espo/Classes/ConsoleCommands/Import.php @@ -55,12 +55,14 @@ class Import implements Command if (!$id && $filePath) { if (!$paramsId) { $io->writeLine("You need to specify --params-id option."); + $io->setExitStatus(1); return; } if (!$this->fileManager->isFile($filePath)) { $io->writeLine("File not found."); + $io->setExitStatus(1); return; } @@ -77,6 +79,7 @@ class Import implements Command $countDuplicate = $result->getCountDuplicate(); } catch (Throwable $e) { $io->writeLine("Error occurred: " . $e->getMessage()); + $io->setExitStatus(1); return; } @@ -98,6 +101,7 @@ class Import implements Command $this->service->revert($id); } catch (Throwable $e) { $io->writeLine("Error occurred: " . $e->getMessage()); + $io->setExitStatus(1); return; } @@ -114,6 +118,7 @@ class Import implements Command $result = $this->service->importById($id, true, $forceResume); } catch (Throwable $e) { $io->writeLine("Error occurred: " . $e->getMessage()); + $io->setExitStatus(1); return; } @@ -133,5 +138,6 @@ class Import implements Command } $io->writeLine("Not enough params passed."); + $io->setExitStatus(1); } } diff --git a/application/Espo/Core/Console/Commands/AppInfo.php b/application/Espo/Core/Console/Commands/AppInfo.php index 2fe3569a96..7e285e9ec7 100644 --- a/application/Espo/Core/Console/Commands/AppInfo.php +++ b/application/Espo/Core/Console/Commands/AppInfo.php @@ -80,6 +80,7 @@ class AppInfo implements Command } $io->writeLine("Not supported flag specified."); + $io->setExitStatus(1); } protected function processType(IO $io, string $type, Params $params): void diff --git a/application/Espo/Core/Console/Commands/EntityUtil.php b/application/Espo/Core/Console/Commands/EntityUtil.php index 54e7f4e199..83496d7178 100644 --- a/application/Espo/Core/Console/Commands/EntityUtil.php +++ b/application/Espo/Core/Console/Commands/EntityUtil.php @@ -50,6 +50,7 @@ class EntityUtil implements Command if (!$subCommand) { $io->writeLine("No sub-command specified."); + $io->setExitStatus(1); return; }