Fix this issue and failing tests

Author: SOHELAHMED7

README.md

@@ -997,4 +997,3 @@ Professional support, consulting as well as software development services are av
 https://www.cebe.cc/en/contact
 
 Development of this library is sponsored by [cebe.:cloud: "Your Professional Deployment Platform"](https://cebe.cloud).
-

src/lib/migrations/PostgresMigrationBuilder.php

@@ -286,9 +286,12 @@ public function handleCommentsMigration()
     {
         $tableAlias = $this->model->getTableAlias();
         foreach ($this->newColumns as $column) {
+            /** @var ColumnSchema $column */
             if ($column->comment) {
-                $this->migration
-                    ->addUpCode($this->recordBuilder->addCommentOnColumn($tableAlias, $column->name, $column->comment));
+                if (!empty($column->xDbType) && is_string($column->xDbType)) { # see \cebe\yii2openapi\lib\migrations\MigrationRecordBuilder::createTable()
+                    $this->migration
+                        ->addUpCode($this->recordBuilder->addCommentOnColumn($tableAlias, $column->name, $column->comment));
+                }
             }
         }
     }

tests/specs/blog/migrations_pgsql_db/m200000_000002_create_table_blog_posts.php

@@ -21,8 +21,6 @@ public function safeUp()
         $this->createIndex('blog_posts_slug_key', '{{%blog_posts}}', 'slug', true);
         $this->addForeignKey('fk_blog_posts_category_id_categories_id', '{{%blog_posts}}', 'category_id', '{{%categories}}', 'id');
         $this->addForeignKey('fk_blog_posts_created_by_id_users_id', '{{%blog_posts}}', 'created_by_id', '{{%users}}', 'id');
-        $this->addCommentOnColumn('{{%blog_posts}}', 'category_id', 'Category of posts');
-        $this->addCommentOnColumn('{{%blog_posts}}', 'created_by_id', 'The User');
     }
 
     public function safeDown()

tests/specs/blog/migrations_pgsql_db/m200000_000004_create_table_post_comments.php

@@ -17,8 +17,6 @@ public function safeUp()
         ]);
         $this->addForeignKey('fk_post_comments_post_id_blog_posts_uid', '{{%post_comments}}', 'post_id', '{{%blog_posts}}', 'uid');
         $this->addForeignKey('fk_post_comments_author_id_users_id', '{{%post_comments}}', 'author_id', '{{%users}}', 'id');
-        $this->addCommentOnColumn('{{%post_comments}}', 'post_id', 'A blog post (uid used as pk for test purposes)');
-        $this->addCommentOnColumn('{{%post_comments}}', 'author_id', 'The User');
     }
 
     public function safeDown()

tests/specs/issue_fix/60_description_of_a_property_in_spec_must_correspond_to_db_table_column_comment/pgsql/migrations_pgsql_db/m200000_000001_create_table_fruits.php

@@ -12,7 +12,6 @@ public function safeUp()
             'name' => $this->text()->null()->defaultValue(null)->comment('desc with \' quote'),
             0 => '"description" double precision NULL DEFAULT NULL',
         ]);
-        $this->addCommentOnColumn('{{%fruits}}', 'name', 'desc with \' quote');
         $this->addCommentOnColumn('{{%fruits}}', 'description', 'desc \' 2');
     }
 

tests/specs/issue_fix/62_unnecessary_sql_statement_for_comment_on_column_in_pgsql/index.php

@@ -0,0 +1,14 @@
+<?php
+
+return [
+    'openApiPath' => '@specs/issue_fix/62_unnecessary_sql_statement_for_comment_on_column_in_pgsql/index.yml',
+    'generateUrls' => false,
+    'generateModels' => false,
+    'excludeModels' => [
+        'Error',
+    ],
+    'generateControllers' => false,
+    'generateMigrations' => true,
+    'generateModelFaker' => false, // `generateModels` must be `true` in order to use `generateModelFaker` as `true`
+];
+

tests/specs/issue_fix/62_unnecessary_sql_statement_for_comment_on_column_in_pgsql/index.yml

@@ -0,0 +1,27 @@
+openapi: 3.0.3
+x-description-is-comment: true
+info:
+  title: 'Description of a property in spec must correspond to DB TABLE COLUMN COMMENT #60'
+  version: 1.0.0
+
+components:
+  schemas:
+    Fruit:
+      type: object
+      properties:
+        id:
+          type: integer
+        name:
+          type: string
+          description: desc with ' quote
+        description:
+          type: number
+          x-db-type: double precision
+          description: desc ' 2
+
+paths:
+  '/':
+    get:
+      responses:
+        '200':
+          description: OK

tests/specs/issue_fix/62_unnecessary_sql_statement_for_comment_on_column_in_pgsql/pgsql/migrations_pgsql_db/m200000_000000_create_table_fruits.php

@@ -0,0 +1,22 @@
+<?php
+
+/**
+ * Table for Fruit
+ */
+class m200000_000000_create_table_fruits extends \yii\db\Migration
+{
+    public function safeUp()
+    {
+        $this->createTable('{{%fruits}}', [
+            'id' => $this->primaryKey(),
+            'name' => $this->text()->null()->defaultValue(null)->comment('desc with \' quote'),
+            0 => '"description" double precision NULL DEFAULT NULL',
+        ]);
+        $this->addCommentOnColumn('{{%fruits}}', 'description', 'desc \' 2');
+    }
+
+    public function safeDown()
+    {
+        $this->dropTable('{{%fruits}}');
+    }
+}

tests/specs/relations_in_faker/app/migrations_pgsql_db/m200000_000001_create_table_b123s.php

@@ -13,7 +13,6 @@ public function safeUp()
             'c123_id' => $this->integer()->null()->defaultValue(null)->comment('desc'),
         ]);
         $this->addForeignKey('fk_b123s_c123_id_c123s_id', '{{%b123s}}', 'c123_id', '{{%c123s}}', 'id');
-        $this->addCommentOnColumn('{{%b123s}}', 'c123_id', 'desc');
     }
 
     public function safeDown()

tests/specs/relations_in_faker/app/migrations_pgsql_db/m200000_000002_create_table_a123s.php

@@ -13,7 +13,6 @@ public function safeUp()
             'b123_id' => $this->integer()->null()->defaultValue(null)->comment('desc'),
         ]);
         $this->addForeignKey('fk_a123s_b123_id_b123s_id', '{{%a123s}}', 'b123_id', '{{%b123s}}', 'id');
-        $this->addCommentOnColumn('{{%a123s}}', 'b123_id', 'desc');
     }
 
     public function safeDown()