#1399 - Fix code style with minor refactor

Jeckerson · Jeckerson · commit e0429128069f · 2019-12-21T15:07:03.000Z
diff --git a/scripts/Phalcon/Mvc/Model/Migration.php b/scripts/Phalcon/Mvc/Model/Migration.php
@@ -930,9 +930,8 @@ public function getConnection()
      */
     public static function columnHasSize($type)
     {
-        $adapter = self::$databaseConfig->path('adapter');
-        if ($adapter == 'Postgresql' &&
-            in_array($type, [Column::TYPE_BOOLEAN, Column::TYPE_INTEGER, Column::TYPE_BIGINTEGER, Column::TYPE_DOUBLE])) {
+        $postgresTypes = [Column::TYPE_BOOLEAN, Column::TYPE_INTEGER, Column::TYPE_BIGINTEGER, Column::TYPE_DOUBLE];
+        if (self::$databaseConfig->path('adapter') == 'Postgresql' && in_array($type, $postgresTypes)) {
             return false;
         }
 

Original file line number	Diff line number	Diff line change
`@@ -930,9 +930,8 @@ public function getConnection()`
`930`	`930`	`*/`
`931`	`931`	`public static function columnHasSize($type)`
`932`	`932`	`{`
`933`		`- $adapter = self::$databaseConfig->path('adapter');`
`934`		`- if ($adapter == 'Postgresql' &&`
`935`		`- in_array($type, [Column::TYPE_BOOLEAN, Column::TYPE_INTEGER, Column::TYPE_BIGINTEGER, Column::TYPE_DOUBLE])) {`
	`933`	`+ $postgresTypes = [Column::TYPE_BOOLEAN, Column::TYPE_INTEGER, Column::TYPE_BIGINTEGER, Column::TYPE_DOUBLE];`
	`934`	`+ if (self::$databaseConfig->path('adapter') == 'Postgresql' && in_array($type, $postgresTypes)) {`
`936`	`935`	`return false;`
`937`	`936`	`}`
`938`	`937`