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

package/browser/driver/oracle/OracleQueryRunner.js

@@ -19,6 +19,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 oracle database connection.
@@ -78,16 +79,15 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Starts transaction.
+     *
      * @param isolationLevel
      */
-    async startTransaction(isolationLevel = "READ COMMITTED") {
+    async startTransaction(isolationLevel) {
+        isolationLevel ??=
+            this.dataSource.options.isolationLevel ?? "READ COMMITTED";
+        (0, validate_isolation_level_1.validateIsolationLevel)(this.driver.supportedIsolationLevels, isolationLevel);
         if (this.isReleased)
             throw new QueryRunnerAlreadyReleasedError_1.QueryRunnerAlreadyReleasedError();
-        // await this.query("START TRANSACTION");
-        if (isolationLevel !== "SERIALIZABLE" &&
-            isolationLevel !== "READ COMMITTED") {
-            throw new error_1.TypeORMError(`Oracle only supports SERIALIZABLE and READ COMMITTED isolation`);
-        }
         this.isTransactionActive = true;
         try {
             await this.broadcaster.broadcast("BeforeTransactionStart");
@@ -140,6 +140,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Executes a given SQL query.
+     *
      * @param query
      * @param parameters
      * @param useStructuredResult
@@ -157,7 +158,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 autoCommit: !this.isTransactionActive,
                 outFormat: this.driver.oracle.OUT_FORMAT_OBJECT,
             };
-            const raw = await databaseConnection.execute(query, parameters || {}, executionOptions);
+            const raw = await databaseConnection.execute(query, parameters ?? {}, executionOptions);
             // log slow queries if maxQueryExecution time is set
             const maxQueryExecutionTime = this.driver.options.maxQueryExecutionTime;
             const queryEndTime = Date.now();
@@ -168,9 +169,9 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 this.driver.dataSource.logger.logQuerySlow(queryExecutionTime, query, parameters, this);
             const result = new QueryResult_1.QueryResult();
             result.raw =
-                raw.rows ||
-                    raw.outBinds ||
-                    raw.rowsAffected ||
+                raw.rows ??
+                    raw.outBinds ??
+                    raw.rowsAffected ??
                     raw.implicitResults;
             if (raw?.hasOwnProperty("rows") && Array.isArray(raw.rows)) {
                 result.records = raw.rows;
@@ -204,6 +205,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Returns raw data stream.
+     *
      * @param query
      * @param parameters
      * @param onEnd
@@ -243,6 +245,7 @@ class OracleQueryRunner 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) {
@@ -250,6 +253,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Checks if database with the given name exist.
+     *
      * @param database
      */
     async hasDatabase(database) {
@@ -270,6 +274,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Checks if schema with the given name exist.
+     *
      * @param schema
      */
     async hasSchema(schema) {
@@ -284,6 +289,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Checks if table with the given name exist in the database.
+     *
      * @param tableOrName
      */
     async hasTable(tableOrName) {
@@ -294,6 +300,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Checks if column with the given name exist in the given table.
+     *
      * @param tableOrName
      * @param columnName
      */
@@ -305,6 +312,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new database.
+     *
      * @param database
      * @param ifNotExists
      */
@@ -331,6 +339,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops database.
+     *
      * @param database
      * @param ifExists
      */
@@ -339,6 +348,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new table schema.
+     *
      * @param schemaPath
      * @param ifNotExists
      */
@@ -347,6 +357,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops table schema.
+     *
      * @param schemaPath
      * @param ifExists
      */
@@ -355,6 +366,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new table.
+     *
      * @param table
      * @param ifNotExists
      * @param createForeignKeys
@@ -377,8 +389,7 @@ class OracleQueryRunner 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(index));
             });
@@ -404,6 +415,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops the table.
+     *
      * @param tableOrName
      * @param ifExists
      * @param dropForeignKeys
@@ -457,6 +469,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new view.
+     *
      * @param view
      * @param syncWithMetadata
      */
@@ -473,6 +486,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops the view.
+     *
      * @param target
      * @param ifExists
      */
@@ -493,6 +507,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Renames the given table.
+     *
      * @param oldTableOrName
      * @param newTableName
      */
@@ -567,6 +582,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new column from the column in the table.
+     *
      * @param tableOrName
      * @param column
      */
@@ -584,9 +600,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             const primaryColumns = clonedTable.primaryColumns;
             // if table already have primary key, me must drop it and recreate again
             if (primaryColumns.length > 0) {
-                const pkName = primaryColumns[0].primaryKeyConstraintName
-                    ? primaryColumns[0].primaryKeyConstraintName
-                    : this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
+                const pkName = primaryColumns[0].primaryKeyConstraintName ??
+                    this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
                 const columnNames = primaryColumns
                     .map((column) => `"${column.name}"`)
                     .join(", ");
@@ -594,9 +609,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 downQueries.push(new Query_1.Query(`ALTER TABLE ${this.escapePath(table)} ADD CONSTRAINT "${pkName}" PRIMARY KEY (${columnNames})`));
             }
             primaryColumns.push(column);
-            const pkName = primaryColumns[0].primaryKeyConstraintName
-                ? primaryColumns[0].primaryKeyConstraintName
-                : this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
+            const pkName = primaryColumns[0].primaryKeyConstraintName ??
+                this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
             const columnNames = primaryColumns
                 .map((column) => `"${column.name}"`)
                 .join(", ");
@@ -642,6 +656,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new columns from the column in the table.
+     *
      * @param tableOrName
      * @param columns
      */
@@ -652,6 +667,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Renames column in the given table.
+     *
      * @param tableOrName
      * @param oldTableColumnOrName
      * @param newTableColumnOrName
@@ -677,6 +693,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Changes a column in the table.
+     *
      * @param tableOrName
      * @param oldTableColumnOrName
      * @param newColumn
@@ -821,9 +838,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                 const primaryColumns = clonedTable.primaryColumns;
                 // if primary column state changed, we must always drop existed constraint.
                 if (primaryColumns.length > 0) {
-                    const pkName = primaryColumns[0].primaryKeyConstraintName
-                        ? primaryColumns[0].primaryKeyConstraintName
-                        : this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
+                    const pkName = primaryColumns[0].primaryKeyConstraintName ??
+                        this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
                     const columnNames = primaryColumns
                         .map((column) => `"${column.name}"`)
                         .join(", ");
@@ -835,9 +851,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                     // update column in table
                     const column = clonedTable.columns.find((column) => column.name === newColumn.name);
                     column.isPrimary = true;
-                    const pkName = primaryColumns[0].primaryKeyConstraintName
-                        ? primaryColumns[0].primaryKeyConstraintName
-                        : this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
+                    const pkName = primaryColumns[0].primaryKeyConstraintName ??
+                        this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
                     const columnNames = primaryColumns
                         .map((column) => `"${column.name}"`)
                         .join(", ");
@@ -852,10 +867,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                     column.isPrimary = false;
                     // if we have another primary keys, we must recreate constraint.
                     if (primaryColumns.length > 0) {
-                        const pkName = primaryColumns[0]
-                            .primaryKeyConstraintName
-                            ? primaryColumns[0].primaryKeyConstraintName
-                            : this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
+                        const pkName = primaryColumns[0].primaryKeyConstraintName ??
+                            this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
                         const columnNames = primaryColumns
                             .map((column) => `"${column.name}"`)
                             .join(", ");
@@ -890,6 +903,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Changes a column in the table.
+     *
      * @param tableOrName
      * @param changedColumns
      */
@@ -900,6 +914,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops column in the table.
+     *
      * @param tableOrName
      * @param columnOrName
      * @param ifExists
@@ -921,9 +936,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         const downQueries = [];
         // drop primary key constraint
         if (column.isPrimary) {
-            const pkName = column.primaryKeyConstraintName
-                ? column.primaryKeyConstraintName
-                : this.dataSource.namingStrategy.primaryKeyName(clonedTable, clonedTable.primaryColumns.map((column) => column.name));
+            const pkName = column.primaryKeyConstraintName ??
+                this.dataSource.namingStrategy.primaryKeyName(clonedTable, clonedTable.primaryColumns.map((column) => column.name));
             const columnNames = clonedTable.primaryColumns
                 .map((primaryColumn) => `"${primaryColumn.name}"`)
                 .join(", ");
@@ -934,10 +948,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             tableColumn.isPrimary = false;
             // if primary key have multiple columns, we must recreate it without dropped column
             if (clonedTable.primaryColumns.length > 0) {
-                const pkName = clonedTable.primaryColumns[0]
-                    .primaryKeyConstraintName
-                    ? clonedTable.primaryColumns[0].primaryKeyConstraintName
-                    : this.dataSource.namingStrategy.primaryKeyName(clonedTable, clonedTable.primaryColumns.map((column) => column.name));
+                const pkName = clonedTable.primaryColumns[0].primaryKeyConstraintName ??
+                    this.dataSource.namingStrategy.primaryKeyName(clonedTable, clonedTable.primaryColumns.map((column) => column.name));
                 const columnNames = clonedTable.primaryColumns
                     .map((primaryColumn) => `"${primaryColumn.name}"`)
                     .join(", ");
@@ -992,6 +1004,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops the columns in the table.
+     *
      * @param tableOrName
      * @param columns
      * @param ifExists
@@ -1003,6 +1016,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new primary key.
+     *
      * @param tableOrName
      * @param columnNames
      * @param constraintName
@@ -1024,6 +1038,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Updates composite primary keys.
+     *
      * @param tableOrName
      * @param columns
      */
@@ -1038,9 +1053,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         // if table already have primary columns, we must drop them.
         const primaryColumns = clonedTable.primaryColumns;
         if (primaryColumns.length > 0) {
-            const pkName = primaryColumns[0].primaryKeyConstraintName
-                ? primaryColumns[0].primaryKeyConstraintName
-                : this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
+            const pkName = primaryColumns[0].primaryKeyConstraintName ??
+                this.dataSource.namingStrategy.primaryKeyName(clonedTable, primaryColumns.map((column) => column.name));
             const columnNamesString = primaryColumns
                 .map((column) => `"${column.name}"`)
                 .join(", ");
@@ -1053,9 +1067,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             .forEach((column) => {
             column.isPrimary = true;
         });
-        const pkName = primaryColumns[0].primaryKeyConstraintName
-            ? primaryColumns[0].primaryKeyConstraintName
-            : this.dataSource.namingStrategy.primaryKeyName(clonedTable, columnNames);
+        const pkName = primaryColumns[0].primaryKeyConstraintName ??
+            this.dataSource.namingStrategy.primaryKeyName(clonedTable, columnNames);
         const columnNamesString = columnNames
             .map((columnName) => `"${columnName}"`)
             .join(", ");
@@ -1066,6 +1079,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops a primary key.
+     *
      * @param tableOrName
      * @param constraintName
      * @param ifExists
@@ -1085,6 +1099,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new unique constraint.
+     *
      * @param tableOrName
      * @param uniqueConstraint
      */
@@ -1093,9 +1108,8 @@ class OracleQueryRunner 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 (!uniqueConstraint.name)
-            uniqueConstraint.name =
-                this.dataSource.namingStrategy.uniqueConstraintName(table, uniqueConstraint.columnNames);
+        uniqueConstraint.name ??=
+            this.dataSource.namingStrategy.uniqueConstraintName(table, uniqueConstraint.columnNames);
         const up = this.createUniqueConstraintSql(table, uniqueConstraint);
         const down = this.dropUniqueConstraintSql(table, uniqueConstraint);
         await this.executeQueries(up, down);
@@ -1103,6 +1117,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new unique constraints.
+     *
      * @param tableOrName
      * @param uniqueConstraints
      */
@@ -1112,6 +1127,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops a unique constraint.
+     *
      * @param tableOrName
      * @param uniqueOrName
      * @param ifExists
@@ -1135,6 +1151,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops unique constraints.
+     *
      * @param tableOrName
      * @param uniqueConstraints
      * @param ifExists
@@ -1145,6 +1162,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates new check constraint.
+     *
      * @param tableOrName
      * @param checkConstraint
      */
@@ -1153,9 +1171,8 @@ class OracleQueryRunner 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);
@@ -1163,6 +1180,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates new check constraints.
+     *
      * @param tableOrName
      * @param checkConstraints
      */
@@ -1172,6 +1190,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops check constraint.
+     *
      * @param tableOrName
      * @param checkOrName
      * @param ifExists
@@ -1195,6 +1214,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops check constraints.
+     *
      * @param tableOrName
      * @param checkConstraints
      * @param ifExists
@@ -1205,6 +1225,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new exclusion constraint.
+     *
      * @param tableOrName
      * @param exclusionConstraint
      */
@@ -1213,6 +1234,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new exclusion constraints.
+     *
      * @param tableOrName
      * @param exclusionConstraints
      */
@@ -1221,6 +1243,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops exclusion constraint.
+     *
      * @param tableOrName
      * @param exclusionOrName
      * @param ifExists
@@ -1230,6 +1253,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops exclusion constraints.
+     *
      * @param tableOrName
      * @param exclusionConstraints
      * @param ifExists
@@ -1239,6 +1263,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new foreign key.
+     *
      * @param tableOrName
      * @param foreignKey
      */
@@ -1247,8 +1272,7 @@ class OracleQueryRunner 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);
@@ -1256,6 +1280,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new foreign keys.
+     *
      * @param tableOrName
      * @param foreignKeys
      */
@@ -1265,6 +1290,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops a foreign key from the table.
+     *
      * @param tableOrName
      * @param foreignKeyOrName
      * @param ifExists
@@ -1281,9 +1307,7 @@ class OracleQueryRunner 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);
@@ -1291,6 +1315,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops a foreign keys from the table.
+     *
      * @param tableOrName
      * @param foreignKeys
      * @param ifExists
@@ -1301,6 +1326,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new index.
+     *
      * @param tableOrName
      * @param index
      */
@@ -1309,8 +1335,7 @@ class OracleQueryRunner 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(index);
         await this.executeQueries(up, down);
@@ -1318,6 +1343,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Creates a new indices
+     *
      * @param tableOrName
      * @param indices
      */
@@ -1327,6 +1353,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops an index from the table.
+     *
      * @param tableOrName
      * @param indexOrName
      * @param ifExists
@@ -1344,8 +1371,7 @@ class OracleQueryRunner 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(index);
         const down = this.createIndexSql(table, index);
         await this.executeQueries(up, down);
@@ -1353,6 +1379,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Drops an indices from the table.
+     *
      * @param tableOrName
      * @param indices
      * @param ifExists
@@ -1364,6 +1391,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     /**
      * Clears all table contents.
      * Note: this operation uses SQL's TRUNCATE query which cannot be reverted in transactions.
+     *
      * @param tableName
      * @param options
      * @param options.cascade
@@ -1413,17 +1441,13 @@ class OracleQueryRunner 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) => this.driver.parseTableName(viewName))
             .map(({ schema, tableName }) => {
-            if (!schema) {
-                schema = this.driver.options.schema || currentSchema;
-            }
+            schema ??= this.driver.options.schema ?? currentSchema;
             return `("T"."schema" = '${schema}' AND "T"."name" = '${tableName}')`;
         })
             .join(" OR ");
@@ -1437,8 +1461,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             const parsedName = this.driver.parseTableName(dbView["name"]);
             const view = new View_1.View();
             view.database =
-                parsedName.database || dbView["database"] || currentDatabase;
-            view.schema = parsedName.schema || dbView["schema"] || currentSchema;
+                parsedName.database ?? dbView["database"] ?? currentDatabase;
+            view.schema = parsedName.schema ?? dbView["schema"] ?? currentSchema;
             view.name = parsedName.tableName;
             view.expression = dbView["value"];
             view.materialized =
@@ -1448,10 +1472,11 @@ class OracleQueryRunner 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 dbTables = [];
@@ -1649,7 +1674,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
                         name: tableColumn.name,
                     });
                     const results = await this.query(asExpressionQuery.query, asExpressionQuery.parameters);
-                    if (results[0] && results[0].value) {
+                    if (results[0]?.value) {
                         tableColumn.asExpression = results[0].value;
                     }
                     else {
@@ -1754,6 +1779,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds and returns SQL for create table.
+     *
      * @param table
      * @param createForeignKeys
      */
@@ -1776,9 +1802,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         if (table.uniques.length > 0) {
             const uniquesSql = table.uniques
                 .map((unique) => {
-                const uniqueName = unique.name
-                    ? unique.name
-                    : this.dataSource.namingStrategy.uniqueConstraintName(table, unique.columnNames);
+                const uniqueName = unique.name ??
+                    this.dataSource.namingStrategy.uniqueConstraintName(table, unique.columnNames);
                 const columnNames = unique.columnNames
                     .map((columnName) => `"${columnName}"`)
                     .join(", ");
@@ -1790,9 +1815,8 @@ class OracleQueryRunner 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(", ");
@@ -1804,8 +1828,7 @@ class OracleQueryRunner 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(", ");
@@ -1821,9 +1844,8 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
         }
         const primaryColumns = table.columns.filter((column) => column.isPrimary);
         if (primaryColumns.length > 0) {
-            const primaryKeyName = primaryColumns[0].primaryKeyConstraintName
-                ? primaryColumns[0].primaryKeyConstraintName
-                : this.dataSource.namingStrategy.primaryKeyName(table, primaryColumns.map((column) => column.name));
+            const primaryKeyName = primaryColumns[0].primaryKeyConstraintName ??
+                this.dataSource.namingStrategy.primaryKeyName(table, primaryColumns.map((column) => column.name));
             const columnNames = primaryColumns
                 .map((column) => `"${column.name}"`)
                 .join(", ");
@@ -1834,6 +1856,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop table sql.
+     *
      * @param tableOrName
      * @param ifExists
      */
@@ -1869,6 +1892,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop view sql.
+     *
      * @param view
      */
     dropViewSql(view) {
@@ -1877,6 +1901,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds remove view sql.
+     *
      * @param view
      */
     deleteViewDefinitionSql(view) {
@@ -1887,6 +1912,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds create index sql.
+     *
      * @param table
      * @param index
      */
@@ -1898,6 +1924,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop index sql.
+     *
      * @param indexOrName
      */
     dropIndexSql(indexOrName) {
@@ -1908,14 +1935,14 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds create primary key sql.
+     *
      * @param table
      * @param columnNames
      * @param constraintName
      */
     createPrimaryKeySql(table, columnNames, constraintName) {
-        const primaryKeyName = constraintName
-            ? constraintName
-            : this.dataSource.namingStrategy.primaryKeyName(table, columnNames);
+        const primaryKeyName = constraintName ??
+            this.dataSource.namingStrategy.primaryKeyName(table, columnNames);
         const columnNamesString = columnNames
             .map((columnName) => `"${columnName}"`)
             .join(", ");
@@ -1923,6 +1950,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop primary key sql.
+     *
      * @param table
      */
     dropPrimaryKeySql(table) {
@@ -1930,13 +1958,13 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
             throw new error_1.TypeORMError(`Table ${table} has no primary keys.`);
         const columnNames = table.primaryColumns.map((column) => column.name);
         const constraintName = table.primaryColumns[0].primaryKeyConstraintName;
-        const primaryKeyName = constraintName
-            ? constraintName
-            : this.dataSource.namingStrategy.primaryKeyName(table, columnNames);
+        const primaryKeyName = constraintName ??
+            this.dataSource.namingStrategy.primaryKeyName(table, columnNames);
         return new Query_1.Query(`ALTER TABLE ${this.escapePath(table)} DROP CONSTRAINT "${primaryKeyName}"`);
     }
     /**
      * Builds create unique constraint sql.
+     *
      * @param table
      * @param uniqueConstraint
      */
@@ -1948,6 +1976,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop unique constraint sql.
+     *
      * @param table
      * @param uniqueOrName
      */
@@ -1959,6 +1988,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds create check constraint sql.
+     *
      * @param table
      * @param checkConstraint
      */
@@ -1967,6 +1997,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop check constraint sql.
+     *
      * @param table
      * @param checkOrName
      */
@@ -1978,6 +2009,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds create foreign key sql.
+     *
      * @param table
      * @param foreignKey
      */
@@ -1998,6 +2030,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds drop foreign key sql.
+     *
      * @param table
      * @param foreignKeyOrName
      */
@@ -2009,6 +2042,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Builds a query for create column.
+     *
      * @param column
      */
     buildCreateColumnSql(column) {
@@ -2032,6 +2066,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Escapes given table or view path.
+     *
      * @param target
      */
     escapePath(target) {
@@ -2044,6 +2079,7 @@ class OracleQueryRunner extends BaseQueryRunner_1.BaseQueryRunner {
     }
     /**
      * Change table comment.
+     *
      * @param tableOrName
      * @param comment
      */