typeorm 0.3.22-dev.b0ea913 → 0.3.22-dev.bcaa0bf

package/browser/driver/postgres/PostgresQueryRunner.js

@@ -13,12 +13,12 @@ import { TableIndex } from "../../schema-builder/table/TableIndex";
 import { TableUnique } from "../../schema-builder/table/TableUnique";
 import { View } from "../../schema-builder/view/View";
 import { Broadcaster } from "../../subscriber/Broadcaster";
+import { BroadcasterResult } from "../../subscriber/BroadcasterResult";
 import { InstanceChecker } from "../../util/InstanceChecker";
 import { OrmUtils } from "../../util/OrmUtils";
-import { VersionUtils } from "../../util/VersionUtils";
+import { DriverUtils } from "../DriverUtils";
 import { Query } from "../Query";
 import { MetadataTableType } from "../types/MetadataTableType";
-import { BroadcasterResult } from "../../subscriber/BroadcasterResult";
 /**
  * Runs queries on a single postgres database connection.
  */
@@ -592,7 +592,7 @@ export class PostgresQueryRunner extends BaseQueryRunner {
         });
         // rename ENUM types
         const enumColumns = newTable.columns.filter((column) => column.type === "enum" || column.type === "simple-enum");
-        for (let column of enumColumns) {
+        for (const column of enumColumns) {
             // skip renaming for user-defined enum name
             if (column.enumName)
                 continue;
@@ -1603,7 +1603,6 @@ export class PostgresQueryRunner extends BaseQueryRunner {
         if (!isAnotherTransactionActive)
             await this.startTransaction();
         try {
-            const version = await this.getVersion();
             // drop views
             const selectViewDropsQuery = `SELECT 'DROP VIEW IF EXISTS "' || schemaname || '"."' || viewname || '" CASCADE;' as "query" ` +
                 `FROM "pg_views" WHERE "schemaname" IN (${schemaNamesString}) AND "viewname" NOT IN ('geography_columns', 'geometry_columns', 'raster_columns', 'raster_overviews')`;
@@ -1611,7 +1610,7 @@ export class PostgresQueryRunner extends BaseQueryRunner {
             await Promise.all(dropViewQueries.map((q) => this.query(q["query"])));
             // drop materialized views
             // Note: materialized views introduced in Postgres 9.3
-            if (VersionUtils.isGreaterOrEqual(version, "9.3")) {
+            if (DriverUtils.isReleaseVersionOrGreater(this.driver, "9.3")) {
                 const selectMatViewDropsQuery = `SELECT 'DROP MATERIALIZED VIEW IF EXISTS "' || schemaname || '"."' || matviewname || '" CASCADE;' as "query" ` +
                     `FROM "pg_matviews" WHERE "schemaname" IN (${schemaNamesString})`;
                 const dropMatViewQueries = await this.query(selectMatViewDropsQuery);
@@ -1636,7 +1635,9 @@ export class PostgresQueryRunner extends BaseQueryRunner {
                     await this.rollbackTransaction();
                 }
             }
-            catch (rollbackError) { }
+            catch {
+                // no-op
+            }
             throw error;
         }
     }
@@ -2293,7 +2294,7 @@ export class PostgresQueryRunner extends BaseQueryRunner {
      */
     async getVersion() {
         const result = await this.query(`SELECT version()`);
-        return result[0]["version"].replace(/^PostgreSQL ([\d.]+) .*$/, "$1");
+        return result[0].version.replace(/^PostgreSQL ([\d.]+) .*$/, "$1");
     }
     /**
      * Builds drop table sql.