refactor: fix TypeError in ShowTableInfo::setYesOrNo()

kenjis · kenjis · commit ef2ba3e4f0f2 · 2023-11-29T13:03:52.000+09:00
1) CodeIgniter\Commands\Database\ShowTableInfoTest::testDbTableMetadata TypeError: CodeIgniter\Commands\Database\ShowTableInfo::setYesOrNo(): Argument #1 ($fieldValue) must be of type bool, int given, called in /home/runner/work/CodeIgniter4/CodeIgniter4/system/Commands/Database/ShowTableInfo.php on line 286 /home/runner/work/CodeIgniter4/CodeIgniter4/system/Commands/Database/ShowTableInfo.php:297 /home/runner/work/CodeIgniter4/CodeIgniter4/system/Commands/Database/ShowTableInfo.php:286 /home/runner/work/CodeIgniter4/CodeIgniter4/system/Commands/Database/ShowTableInfo.php:144 /home/runner/work/CodeIgniter4/CodeIgniter4/system/CLI/Commands.php:67 /home/runner/work/CodeIgniter4/CodeIgniter4/system/Common.php:198 /home/runner/work/CodeIgniter4/CodeIgniter4/tests/system/Commands/Database/ShowTableInfoTest.php:98
diff --git a/system/Commands/Database/ShowTableInfo.php b/system/Commands/Database/ShowTableInfo.php
@@ -292,9 +292,12 @@ private function showFieldMetaData(string $tableName): void
         CLI::table($this->tbody, $thead);
     }
 
-    private function setYesOrNo(bool $fieldValue): string
+    /**
+     * @param bool|int|string|null $fieldValue
+     */
+    private function setYesOrNo($fieldValue): string
     {
-        if ($fieldValue) {
+        if ((bool) $fieldValue) {
             return CLI::color('Yes', 'green');
         }
 

Original file line number	Diff line number	Diff line change
`@@ -292,9 +292,12 @@ private function showFieldMetaData(string $tableName): void`
`292`	`292`	`CLI::table($this->tbody, $thead);`
`293`	`293`	`}`
`294`	`294`
`295`		`- private function setYesOrNo(bool $fieldValue): string`
	`295`	`+ /**`
	`296`	`+ * @param bool\|int\|string\|null $fieldValue`
	`297`	`+ */`
	`298`	`+ private function setYesOrNo($fieldValue): string`
`296`	`299`	`{`
`297`		`- if ($fieldValue) {`
	`300`	`+ if ((bool) $fieldValue) {`
`298`	`301`	`return CLI::color('Yes', 'green');`
`299`	`302`	`}`
`300`	`303`