Fix psalm issues

yiisoft · Jan 29, 2024 · 334661b · 334661b
1 parent 05ebf54
commit 334661b
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/DDLQueryBuilder.php b/src/DDLQueryBuilder.php
@@ -105,8 +105,8 @@ public function createIndex(
             [$schema, $table] = $tableParts;
         }
 
-        return 'CREATE ' . ($indexType ? ($indexType . ' ') : '') . 'INDEX '
-            . $this->quoter->quoteTableName(($schema ? $schema . '.' : '') . $name)
+        return 'CREATE ' . ($indexType !== null ? $indexType . ' ' : '') . 'INDEX '
+            . $this->quoter->quoteTableName(($schema !== null ? $schema . '.' : '') . $name)
             . ' ON '
             . $this->quoter->quoteTableName($table)
             . ' (' . $this->queryBuilder->buildColumns($columns) . ')';

diff --git a/src/Schema.php b/src/Schema.php
@@ -752,7 +752,7 @@ private function getJsonColumns(TableSchemaInterface $table): array
         $regexp = '/\bjson_valid\(\s*["`\[]?(.+?)["`\]]?\s*\)/i';
 
         foreach ($checks as $check) {
-            if (preg_match_all($regexp, $check->getExpression(), $matches, PREG_SET_ORDER)) {
+            if (preg_match_all($regexp, $check->getExpression(), $matches, PREG_SET_ORDER) > 0) {
                 foreach ($matches as $match) {
                     $result[] = $match[1];
                 }