refactor: add exit code to return

kenjis · kenjis · commit 367c5dbda543 · 2023-12-06T18:34:10.000+09:00
diff --git a/system/Commands/Database/ShowTableInfo.php b/system/Commands/Database/ShowTableInfo.php
@@ -127,13 +127,13 @@ public function run(array $params)
             CLI::error('Database has no tables!', 'light_gray', 'red');
             CLI::newLine();
 
-            return;
+            return EXIT_ERROR;
         }
 
         if (array_key_exists('show', $params)) {
             $this->showAllTables($tables);
 
-            return;
+            return EXIT_ERROR;
         }
 
         $tableName       = $params[0] ?? null;
@@ -154,10 +154,12 @@ public function run(array $params)
         if (array_key_exists('metadata', $params)) {
             $this->showFieldMetaData($tableName);
 
-            return;
+            return EXIT_SUCCESS;
         }
 
         $this->showDataOfTable($tableName, $limitRows, $limitFieldValue);
+
+        return EXIT_SUCCESS;
     }
 
     private function showDBConfig(): void

Original file line number	Diff line number	Diff line change
`@@ -127,13 +127,13 @@ public function run(array $params)`
`127`	`127`	`CLI::error('Database has no tables!', 'light_gray', 'red');`
`128`	`128`	`CLI::newLine();`
`129`	`129`
`130`		`- return;`
	`130`	`+ return EXIT_ERROR;`
`131`	`131`	`}`
`132`	`132`
`133`	`133`	`if (array_key_exists('show', $params)) {`
`134`	`134`	`$this->showAllTables($tables);`
`135`	`135`
`136`		`- return;`
	`136`	`+ return EXIT_ERROR;`
`137`	`137`	`}`
`138`	`138`
`139`	`139`	`$tableName = $params[0] ?? null;`
`@@ -154,10 +154,12 @@ public function run(array $params)`
`154`	`154`	`if (array_key_exists('metadata', $params)) {`
`155`	`155`	`$this->showFieldMetaData($tableName);`
`156`	`156`
`157`		`- return;`
	`157`	`+ return EXIT_SUCCESS;`
`158`	`158`	`}`
`159`	`159`
`160`	`160`	`$this->showDataOfTable($tableName, $limitRows, $limitFieldValue);`
	`161`	`+`
	`162`	`+ return EXIT_SUCCESS;`
`161`	`163`	`}`
`162`	`164`
`163`	`165`	`private function showDBConfig(): void`