typeorm 1.0.0-beta.1 → 1.0.0-beta.2

package/browser/driver/sap/SapQueryRunner.js

@@ -21,6 +21,7 @@ const BroadcasterResult_1 = require("../../subscriber/BroadcasterResult");
 const InstanceChecker_1 = require("../../util/InstanceChecker");
 const OrmUtils_1 = require("../../util/OrmUtils");
 const Query_1 = require("../Query");
+const validate_isolation_level_1 = require("../validate-isolation-level");
 const MetadataTableType_1 = require("../types/MetadataTableType");
 /**
  * Runs queries on a single SQL Server database connection.
@@ -69,9 +70,12 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Starts transaction.
+     *
      * @param isolationLevel
      */
     async startTransaction(isolationLevel) {
+        isolationLevel ??= this.dataSource.options.isolationLevel;
+        (0, validate_isolation_level_1.validateIsolationLevel)(this.driver.supportedIsolationLevels, isolationLevel);
         if (this.isReleased)
             throw new QueryRunnerAlreadyReleasedError_1.QueryRunnerAlreadyReleasedError();
         if (this.isTransactionActive &&
@@ -121,6 +125,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Switches AUTOCOMMIT mode on/off
+     *
      * @see https://help.sap.com/docs/HANA_SERVICE_CF/7c78579ce9b14a669c1f3295b0d8ca16/d538d11053bd4f3f847ec5ce817a3d4c.html?locale=en-US
      * @param options
      * @param options.status
@@ -139,6 +144,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Executes a given SQL query.
+     *
      * @param query
      * @param parameters
      * @param useStructuredResult
@@ -155,7 +161,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         const broadcasterResult = new BroadcasterResult_1.BroadcasterResult();
         try {
             const queryStartTime = Date.now();
-            const isInsertQuery = query.substr(0, 11) === "INSERT INTO";
+            const isInsertQuery = query.startsWith("INSERT INTO");
             if (parameters?.some(Array.isArray)) {
                 statement = await (0, node_util_1.promisify)(databaseConnection.prepare).call(databaseConnection, query);
             }
@@ -221,6 +227,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Returns raw data stream.
+     *
      * @param query
      * @param parameters
      * @param onEnd
@@ -277,6 +284,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     /**
      * Returns all available schema names including system schemas.
      * If database parameter specified, returns schemas of that database.
+     *
      * @param database
      */
     async getSchemas(database) {
@@ -288,6 +296,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Checks if database with the given name exist.
+     *
      * @param database
      */
     async hasDatabase(database) {
@@ -310,6 +319,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Checks if schema with the given name exist.
+     *
      * @param schema
      */
     async hasSchema(schema) {
@@ -325,13 +335,12 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Checks if table with the given name exist in the database.
+     *
      * @param tableOrName
      */
     async hasTable(tableOrName) {
         const parsedTableName = this.driver.parseTableName(tableOrName);
-        if (!parsedTableName.schema) {
-            parsedTableName.schema = await this.getCurrentSchema();
-        }
+        parsedTableName.schema ??= await this.getCurrentSchema();
         const sql = `SELECT COUNT(*) as "hasTable" FROM "SYS"."TABLES" WHERE "SCHEMA_NAME" = ? AND "TABLE_NAME" = ?`;
         const result = await this.query(sql, [
             parsedTableName.schema,
@@ -341,14 +350,13 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Checks if column with the given name exist in the given table.
+     *
      * @param tableOrName
      * @param columnName
      */
     async hasColumn(tableOrName, columnName) {
         const parsedTableName = this.driver.parseTableName(tableOrName);
-        if (!parsedTableName.schema) {
-            parsedTableName.schema = await this.getCurrentSchema();
-        }
+        parsedTableName.schema ??= await this.getCurrentSchema();
         const sql = `SELECT COUNT(*) as "hasColumn" FROM "SYS"."TABLE_COLUMNS" WHERE "SCHEMA_NAME" = ? AND "TABLE_NAME" = ? AND "COLUMN_NAME" = ?`;
         const result = await this.query(sql, [
             parsedTableName.schema,
@@ -359,6 +367,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new database.
+     *
      * @param database
      * @param ifNotExists
      */
@@ -367,6 +376,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops database.
+     *
      * @param database
      * @param ifExists
      */
@@ -375,6 +385,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new table schema.
+     *
      * @param schemaPath
      * @param ifNotExists
      */
@@ -396,6 +407,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops table schema
+     *
      * @param schemaPath
      * @param ifExists
      * @param isCascade
@@ -418,6 +430,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new table.
+     *
      * @param table
      * @param ifNotExists
      * @param createForeignKeys
@@ -440,8 +453,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         if (createIndices) {
             table.indices.forEach((index) => {
                 // new index may be passed without name. In this case we generate index name manually.
-                if (!index.name)
-                    index.name = this.dataSource.namingStrategy.indexName(table, index.columnNames, index.where);
+                index.name ??= this.dataSource.namingStrategy.indexName(table, index.columnNames, index.where);
                 upQueries.push(this.createIndexSql(table, index));
                 downQueries.push(this.dropIndexSql(table, index));
             });
@@ -450,6 +462,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops the table.
+     *
      * @param tableOrName
      * @param ifExists
      * @param dropForeignKeys
@@ -486,6 +499,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new view.
+     *
      * @param view
      * @param syncWithMetadata
      */
@@ -502,6 +516,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops the view.
+     *
      * @param target
      * @param ifExists
      */
@@ -523,6 +538,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Renames a table.
+     *
      * @param oldTableOrName
      * @param newTableName
      */
@@ -631,6 +647,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new column from the column in the table.
+     *
      * @param tableOrName
      * @param column
      */
@@ -639,9 +656,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             ? tableOrName
             : await this.getCachedTable(tableOrName);
         const parsedTableName = this.driver.parseTableName(table);
-        if (!parsedTableName.schema) {
-            parsedTableName.schema = await this.getCurrentSchema();
-        }
+        parsedTableName.schema ??= await this.getCurrentSchema();
         const clonedTable = table.clone();
         const upQueries = [];
         const downQueries = [];
@@ -739,6 +754,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new columns from the column in the table.
+     *
      * @param tableOrName
      * @param columns
      */
@@ -749,6 +765,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Renames column in the given table.
+     *
      * @param tableOrName
      * @param oldTableColumnOrName
      * @param newTableColumnOrName
@@ -774,6 +791,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Changes a column in the table.
+     *
      * @param tableOrName
      * @param oldTableColumnOrName
      * @param newColumn
@@ -961,6 +979,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Changes a column in the table.
+     *
      * @param tableOrName
      * @param changedColumns
      */
@@ -971,6 +990,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops column in the table.
+     *
      * @param tableOrName
      * @param columnOrName
      * @param ifExists
@@ -980,9 +1000,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             ? tableOrName
             : await this.getCachedTable(tableOrName);
         const parsedTableName = this.driver.parseTableName(table);
-        if (!parsedTableName.schema) {
-            parsedTableName.schema = await this.getCurrentSchema();
-        }
+        parsedTableName.schema ??= await this.getCurrentSchema();
         const column = InstanceChecker_1.InstanceChecker.isTableColumn(columnOrName)
             ? columnOrName
             : table.findColumnByName(columnOrName);
@@ -1104,6 +1122,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops the columns in the table.
+     *
      * @param tableOrName
      * @param columns
      * @param ifExists
@@ -1115,6 +1134,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new primary key.
+     *
      * @param tableOrName
      * @param columnNames
      */
@@ -1135,6 +1155,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Updates composite primary keys.
+     *
      * @param tableOrName
      * @param columns
      */
@@ -1143,9 +1164,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             ? tableOrName
             : await this.getCachedTable(tableOrName);
         const parsedTableName = this.driver.parseTableName(table);
-        if (!parsedTableName.schema) {
-            parsedTableName.schema = await this.getCurrentSchema();
-        }
+        parsedTableName.schema ??= await this.getCurrentSchema();
         const clonedTable = table.clone();
         const columnNames = columns.map((column) => column.name);
         const upQueries = [];
@@ -1220,6 +1239,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops a primary key.
+     *
      * @param tableOrName
      * @param constraintName
      * @param ifExists
@@ -1231,9 +1251,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         if (ifExists && table.primaryColumns.length === 0)
             return;
         const parsedTableName = this.driver.parseTableName(table);
-        if (!parsedTableName.schema) {
-            parsedTableName.schema = await this.getCurrentSchema();
-        }
+        parsedTableName.schema ??= await this.getCurrentSchema();
         const upQueries = [];
         const downQueries = [];
         // SAP HANA does not allow to drop PK's which is referenced by foreign keys.
@@ -1288,6 +1306,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new unique constraint.
+     *
      * @param tableOrName
      * @param uniqueConstraint
      */
@@ -1296,6 +1315,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new unique constraints.
+     *
      * @param tableOrName
      * @param uniqueConstraints
      */
@@ -1304,6 +1324,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops unique constraint.
+     *
      * @param tableOrName
      * @param uniqueOrName
      * @param ifExists
@@ -1313,6 +1334,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops unique constraints.
+     *
      * @param tableOrName
      * @param uniqueConstraints
      * @param ifExists
@@ -1322,6 +1344,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new check constraint.
+     *
      * @param tableOrName
      * @param checkConstraint
      */
@@ -1330,9 +1353,8 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             ? tableOrName
             : await this.getCachedTable(tableOrName);
         // new unique constraint may be passed without name. In this case we generate unique name manually.
-        if (!checkConstraint.name)
-            checkConstraint.name =
-                this.dataSource.namingStrategy.checkConstraintName(table, checkConstraint.expression);
+        checkConstraint.name ??=
+            this.dataSource.namingStrategy.checkConstraintName(table, checkConstraint.expression);
         const up = this.createCheckConstraintSql(table, checkConstraint);
         const down = this.dropCheckConstraintSql(table, checkConstraint);
         await this.executeQueries(up, down);
@@ -1340,6 +1362,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new check constraints.
+     *
      * @param tableOrName
      * @param checkConstraints
      */
@@ -1349,6 +1372,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops check constraint.
+     *
      * @param tableOrName
      * @param checkOrName
      * @param ifExists
@@ -1372,6 +1396,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops check constraints.
+     *
      * @param tableOrName
      * @param checkConstraints
      * @param ifExists
@@ -1382,6 +1407,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new exclusion constraint.
+     *
      * @param tableOrName
      * @param exclusionConstraint
      */
@@ -1390,6 +1416,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new exclusion constraints.
+     *
      * @param tableOrName
      * @param exclusionConstraints
      */
@@ -1398,6 +1425,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops exclusion constraint.
+     *
      * @param tableOrName
      * @param exclusionOrName
      * @param ifExists
@@ -1407,6 +1435,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops exclusion constraints.
+     *
      * @param tableOrName
      * @param exclusionConstraints
      * @param ifExists
@@ -1416,6 +1445,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new foreign key.
+     *
      * @param tableOrName
      * @param foreignKey
      */
@@ -1424,8 +1454,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             ? tableOrName
             : await this.getCachedTable(tableOrName);
         // new FK may be passed without name. In this case we generate FK name manually.
-        if (!foreignKey.name)
-            foreignKey.name = this.dataSource.namingStrategy.foreignKeyName(table, foreignKey.columnNames, this.getTablePath(foreignKey), foreignKey.referencedColumnNames);
+        foreignKey.name ??= this.dataSource.namingStrategy.foreignKeyName(table, foreignKey.columnNames, this.getTablePath(foreignKey), foreignKey.referencedColumnNames);
         const up = this.createForeignKeySql(table, foreignKey);
         const down = this.dropForeignKeySql(table, foreignKey);
         await this.executeQueries(up, down);
@@ -1433,6 +1462,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new foreign keys.
+     *
      * @param tableOrName
      * @param foreignKeys
      */
@@ -1442,6 +1472,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops a foreign key from the table.
+     *
      * @param tableOrName
      * @param foreignKeyOrName
      * @param ifExists
@@ -1458,9 +1489,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 return;
             throw new error_1.TypeORMError(`Supplied foreign key was not found in table ${table.name}`);
         }
-        if (!foreignKey.name) {
-            foreignKey.name = this.dataSource.namingStrategy.foreignKeyName(table, foreignKey.columnNames, this.getTablePath(foreignKey), foreignKey.referencedColumnNames);
-        }
+        foreignKey.name ??= this.dataSource.namingStrategy.foreignKeyName(table, foreignKey.columnNames, this.getTablePath(foreignKey), foreignKey.referencedColumnNames);
         const up = this.dropForeignKeySql(table, foreignKey);
         const down = this.createForeignKeySql(table, foreignKey);
         await this.executeQueries(up, down);
@@ -1468,6 +1497,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops a foreign keys from the table.
+     *
      * @param tableOrName
      * @param foreignKeys
      * @param ifExists
@@ -1478,6 +1508,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new index.
+     *
      * @param tableOrName
      * @param index
      */
@@ -1486,8 +1517,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             ? tableOrName
             : await this.getCachedTable(tableOrName);
         // new index may be passed without name. In this case we generate index name manually.
-        if (!index.name)
-            index.name = this.generateIndexName(table, index);
+        index.name ??= this.generateIndexName(table, index);
         const up = this.createIndexSql(table, index);
         const down = this.dropIndexSql(table, index);
         await this.executeQueries(up, down);
@@ -1495,6 +1525,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new indices
+     *
      * @param tableOrName
      * @param indices
      */
@@ -1504,6 +1535,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops an index.
+     *
      * @param tableOrName
      * @param indexOrName
      * @param ifExists
@@ -1521,8 +1553,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             throw new error_1.TypeORMError(`Supplied index ${indexOrName} was not found in table ${table.name}`);
         }
         // old index may be passed without name. In this case we generate index name manually.
-        if (!index.name)
-            index.name = this.generateIndexName(table, index);
+        index.name ??= this.generateIndexName(table, index);
         const up = this.dropIndexSql(table, index);
         const down = this.createIndexSql(table, index);
         await this.executeQueries(up, down);
@@ -1530,6 +1561,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops an indices from the table.
+     *
      * @param tableOrName
      * @param indices
      * @param ifExists
@@ -1541,6 +1573,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     /**
      * Clears all table contents.
      * Note: this operation uses SQL's TRUNCATE query which cannot be reverted in transactions.
+     *
      * @param tablePath
      * @param options
      * @param options.cascade
@@ -1599,17 +1632,13 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         if (!hasTable) {
             return [];
         }
-        if (!viewNames) {
-            viewNames = [];
-        }
+        viewNames ??= [];
         const currentDatabase = await this.getCurrentDatabase();
         const currentSchema = await this.getCurrentSchema();
         const viewsCondition = viewNames
             .map((viewName) => {
             let { schema, tableName: name } = this.driver.parseTableName(viewName);
-            if (!schema) {
-                schema = currentSchema;
-            }
+            schema ??= currentSchema;
             return `("t"."schema" = '${schema}' AND "t"."name" = '${name}')`;
         })
             .join(" OR ");
@@ -1630,10 +1659,11 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Loads all tables (with given names) from the database and creates a Table from them.
+     *
      * @param tableNames
      */
     async loadTables(tableNames) {
-        if (tableNames && tableNames.length === 0) {
+        if (tableNames?.length === 0) {
             return [];
         }
         const currentSchema = await this.getCurrentSchema();
@@ -1649,7 +1679,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 let [schema, name] = tableName.split(".");
                 if (!name) {
                     name = schema;
-                    schema = this.driver.options.schema || currentSchema;
+                    schema = this.driver.options.schema ?? currentSchema;
                 }
                 return `("SCHEMA_NAME" = '${schema}' AND "TABLE_NAME" = '${name}')`;
             })
@@ -1734,8 +1764,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 const tableMetadata = this.dataSource.entityMetadatas.find((metadata) => this.getTablePath(table) ===
                     this.getTablePath(metadata));
                 const hasIgnoredIndex = columnUniqueIndices.length > 0 &&
-                    tableMetadata &&
-                    tableMetadata.indices.some((index) => {
+                    tableMetadata?.indices.some((index) => {
                         return columnUniqueIndices.some((uniqueIndex) => {
                             return (index.name === uniqueIndex["INDEX_NAME"] &&
                                 index.synchronize === false);
@@ -1791,7 +1820,8 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 tableColumn.isNullable = dbColumn["IS_NULLABLE"] === "TRUE";
                 tableColumn.isPrimary = !!columnConstraints.find((constraint) => constraint["IS_PRIMARY_KEY"] === "TRUE");
                 tableColumn.isGenerated =
-                    dbColumn["GENERATION_TYPE"] === "ALWAYS AS IDENTITY";
+                    dbColumn["GENERATION_TYPE"] === "ALWAYS AS IDENTITY" ||
+                        dbColumn["GENERATION_TYPE"] === "BY DEFAULT AS IDENTITY";
                 if (tableColumn.isGenerated)
                     tableColumn.generationStrategy = "increment";
                 if (dbColumn["DEFAULT_VALUE"] === null ||
@@ -1885,6 +1915,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds and returns SQL for create table.
+     *
      * @param table
      * @param createForeignKeys
      */
@@ -1930,9 +1961,8 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         if (table.checks.length > 0) {
             const checksSql = table.checks
                 .map((check) => {
-                const checkName = check.name
-                    ? check.name
-                    : this.dataSource.namingStrategy.checkConstraintName(table, check.expression);
+                const checkName = check.name ??
+                    this.dataSource.namingStrategy.checkConstraintName(table, check.expression);
                 return `CONSTRAINT "${checkName}" CHECK (${check.expression})`;
             })
                 .join(", ");
@@ -1944,8 +1974,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 const columnNames = fk.columnNames
                     .map((columnName) => `"${columnName}"`)
                     .join(", ");
-                if (!fk.name)
-                    fk.name = this.dataSource.namingStrategy.foreignKeyName(table, fk.columnNames, this.getTablePath(fk), fk.referencedColumnNames);
+                fk.name ??= this.dataSource.namingStrategy.foreignKeyName(table, fk.columnNames, this.getTablePath(fk), fk.referencedColumnNames);
                 const referencedColumnNames = fk.referencedColumnNames
                     .map((columnName) => `"${columnName}"`)
                     .join(", ");
@@ -1987,6 +2016,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop table sql.
+     *
      * @param tableOrName
      * @param ifExists
      */
@@ -2008,9 +2038,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     async insertViewDefinitionSql(view) {
         let { schema, tableName: name } = this.driver.parseTableName(view);
-        if (!schema) {
-            schema = await this.getCurrentSchema();
-        }
+        schema ??= await this.getCurrentSchema();
         const expression = typeof view.expression === "string"
             ? view.expression.trim()
             : view.expression(this.dataSource).getQuery();
@@ -2023,6 +2051,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop view sql.
+     *
      * @param viewOrPath
      */
     dropViewSql(viewOrPath) {
@@ -2030,13 +2059,12 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds remove view sql.
+     *
      * @param viewOrPath
      */
     async deleteViewDefinitionSql(viewOrPath) {
         let { schema, tableName: name } = this.driver.parseTableName(viewOrPath);
-        if (!schema) {
-            schema = await this.getCurrentSchema();
-        }
+        schema ??= await this.getCurrentSchema();
         return this.deleteTypeormMetadataSql({
             type: MetadataTableType_1.MetadataTableType.VIEW,
             schema,
@@ -2051,6 +2079,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds create index sql.
+     *
      * @param table
      * @param index
      */
@@ -2069,6 +2098,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop index sql.
+     *
      * @param table
      * @param indexOrName
      */
@@ -2086,6 +2116,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds create primary key sql.
+     *
      * @param table
      * @param columnNames
      */
@@ -2098,6 +2129,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop primary key sql.
+     *
      * @param table
      */
     dropPrimaryKeySql(table) {
@@ -2107,6 +2139,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds create check constraint sql.
+     *
      * @param table
      * @param checkConstraint
      */
@@ -2115,6 +2148,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop check constraint sql.
+     *
      * @param table
      * @param checkOrName
      */
@@ -2126,6 +2160,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds create foreign key sql.
+     *
      * @param tableOrName
      * @param foreignKey
      */
@@ -2158,6 +2193,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop foreign key sql.
+     *
      * @param tableOrName
      * @param foreignKeyOrName
      */
@@ -2169,6 +2205,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Escapes a given comment so it's safe to include in a query.
+     *
      * @param comment
      */
     escapeComment(comment) {
@@ -2180,6 +2217,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Escapes given table or view path.
+     *
      * @param target
      */
     escapePath(target) {
@@ -2191,6 +2229,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds a query for create column.
+     *
      * @param column
      * @param explicitDefault
      * @param explicitNullable
@@ -2212,7 +2251,12 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         }
         if (column.isGenerated === true &&
             column.generationStrategy === "increment") {
-            c += " GENERATED ALWAYS AS IDENTITY";
+            // GENERATED BY DEFAULT allows TypeORM to explicitly insert values
+            // into identity columns when needed (e.g., synchronize, migrations).
+            // GENERATED ALWAYS rejects any explicit value — even the DEFAULT
+            // keyword — making it impossible to insert into tables whose only
+            // column is an identity column.
+            c += " GENERATED BY DEFAULT AS IDENTITY";
         }
         if (column.comment) {
             c += ` COMMENT ${this.escapeComment(column.comment)}`;
@@ -2221,6 +2265,7 @@ class SapQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Change table comment.
+     *
      * @param tableOrName
      * @param newComment
      */