typeorm 0.3.20-dev.62f574b → 0.3.20-dev.8ebe769

package/driver/postgres/PostgresQueryRunner.d.ts

@@ -415,5 +415,5 @@ export declare class PostgresQueryRunner extends BaseQueryRunner implements Quer
     /**
      * Change table comment.
      */
-    changeTableComment(tableOrName: Table | string, newComment?: string): Promise<void>;
+    changeTableComment(tableOrName: Table | string, comment?: string): Promise<void>;
 }

package/driver/postgres/PostgresQueryRunner.js

@@ -418,10 +418,6 @@ class PostgresQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 downQueries.push(this.dropIndexSql(table, index));
             });
         }
-        if (table.comment) {
-            upQueries.push(new Query_1.Query("COMMENT ON TABLE " + this.escapePath(table) + " IS '" + table.comment + "'"));
-            downQueries.push(new Query_1.Query("COMMENT ON TABLE " + this.escapePath(table) + " IS NULL"));
-        }
         await this.executeQueries(upQueries, downQueries);
     }
     /**
@@ -1743,7 +1739,7 @@ class PostgresQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         const currentDatabase = await this.getCurrentDatabase();
         const dbTables = [];
         if (!tableNames) {
-            const tablesSql = `SELECT "table_schema", "table_name", obj_description(('"' || "table_schema" || '"."' || "table_name" || '"')::regclass, 'pg_class') AS table_comment FROM "information_schema"."tables"`;
+            const tablesSql = `SELECT "table_schema", "table_name" FROM "information_schema"."tables"`;
             dbTables.push(...(await this.query(tablesSql)));
         }
         else {
@@ -1753,7 +1749,7 @@ class PostgresQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 return `("table_schema" = '${schema || currentSchema}' AND "table_name" = '${tableName}')`;
             })
                 .join(" OR ");
-            const tablesSql = `SELECT "table_schema", "table_name", obj_description(('"' || "table_schema" || '"."' || "table_name" || '"')::regclass, 'pg_class') AS table_comment FROM "information_schema"."tables" WHERE ` +
+            const tablesSql = `SELECT "table_schema", "table_name" FROM "information_schema"."tables" WHERE ` +
                 tablesCondition;
             dbTables.push(...(await this.query(tablesSql)));
         }
@@ -1855,7 +1851,6 @@ class PostgresQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             const schema = getSchemaFromKey(dbTable, "table_schema");
             table.database = currentDatabase;
             table.schema = dbTable["table_schema"];
-            table.comment = dbTable["table_comment"];
             table.name = this.driver.buildTableName(dbTable["table_name"], schema);
             // create columns from the loaded columns
             table.columns = await Promise.all(dbColumns
@@ -2407,7 +2402,7 @@ class PostgresQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         const columns = index.columnNames
             .map((columnName) => `"${columnName}"`)
             .join(", ");
-        return new Query_1.Query(`CREATE ${index.isUnique ? "UNIQUE " : ""}INDEX${index.isConcurrent ? " CONCURRENTLY" : ""} "${index.name}" ON ${this.escapePath(table)} ${index.isSpatial ? "USING GiST " : ""}(${columns}) ${index.where ? "WHERE " + index.where : ""}`);
+        return new Query_1.Query(`CREATE ${index.isUnique ? "UNIQUE " : ""}${index.isConcurrent ? "CONCURRENTLY " : ""}INDEX "${index.name}" ON ${this.escapePath(table)} ${index.isSpatial ? "USING GiST " : ""}(${columns}) ${index.where ? "WHERE " + index.where : ""}`);
     }
     /**
      * Builds create view index sql.
@@ -2422,7 +2417,7 @@ class PostgresQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
      * Builds drop index sql.
      */
     dropIndexSql(table, indexOrName) {
-        const indexName = InstanceChecker_1.InstanceChecker.isTableIndex(indexOrName)
+        let indexName = InstanceChecker_1.InstanceChecker.isTableIndex(indexOrName)
             ? indexOrName.name
             : indexOrName;
         const concurrent = InstanceChecker_1.InstanceChecker.isTableIndex(indexOrName)
@@ -2430,8 +2425,8 @@ class PostgresQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             : false;
         const { schema } = this.driver.parseTableName(table);
         return schema
-            ? new Query_1.Query(`DROP INDEX ${concurrent ? "CONCURRENTLY " : ""}"${schema}"."${indexName}"`)
-            : new Query_1.Query(`DROP INDEX ${concurrent ? "CONCURRENTLY " : ""}"${indexName}"`);
+            ? new Query_1.Query(`DROP INDEX ${concurrent ? "CONCURRENTLY" : ""}"${schema}"."${indexName}"`)
+            : new Query_1.Query(`DROP INDEX ${concurrent ? "CONCURRENTLY" : ""}"${indexName}"`);
     }
     /**
      * Builds create primary key sql.
@@ -2694,23 +2689,8 @@ class PostgresQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     /**
      * Change table comment.
      */
-    async changeTableComment(tableOrName, newComment) {
-        const upQueries = [];
-        const downQueries = [];
-        const table = InstanceChecker_1.InstanceChecker.isTable(tableOrName)
-            ? tableOrName
-            : await this.getCachedTable(tableOrName);
-        newComment = this.escapeComment(newComment);
-        const comment = this.escapeComment(table.comment);
-        if (newComment === comment) {
-            return;
-        }
-        const newTable = table.clone();
-        upQueries.push(new Query_1.Query(`COMMENT ON TABLE ${this.escapePath(newTable)} IS ${newComment}`));
-        downQueries.push(new Query_1.Query(`COMMENT ON TABLE ${this.escapePath(table)} IS ${comment}`));
-        await this.executeQueries(upQueries, downQueries);
-        table.comment = newTable.comment;
-        this.replaceCachedTable(table, newTable);
+    changeTableComment(tableOrName, comment) {
+        throw new error_1.TypeORMError(`postgres driver does not support change table comment.`);
     }
 }
 exports.PostgresQueryRunner = PostgresQueryRunner;