typeorm 0.3.0-rc.9 → 0.3.0

package/browser/query-builder/UpdateQueryBuilder.js

@@ -1,255 +1,224 @@
-import * as tslib_1 from "tslib";
-import { CockroachDriver } from "../driver/cockroachdb/CockroachDriver";
-import { ObserverExecutor } from "../observer/ObserverExecutor";
 import { QueryBuilder } from "./QueryBuilder";
-import { SqlServerDriver } from "../driver/sqlserver/SqlServerDriver";
-import { PostgresDriver } from "../driver/postgres/PostgresDriver";
-import { EntityMetadata } from "../metadata/EntityMetadata";
 import { UpdateResult } from "./result/UpdateResult";
 import { ReturningStatementNotSupportedError } from "../error/ReturningStatementNotSupportedError";
 import { ReturningResultsEntityUpdator } from "./ReturningResultsEntityUpdator";
-import { SqljsDriver } from "../driver/sqljs/SqljsDriver";
-import { MysqlDriver } from "../driver/mysql/MysqlDriver";
-import { BroadcasterResult } from "../subscriber/BroadcasterResult";
-import { AbstractSqliteDriver } from "../driver/sqlite-abstract/AbstractSqliteDriver";
 import { LimitOnUpdateNotSupportedError } from "../error/LimitOnUpdateNotSupportedError";
-import { OracleDriver } from "../driver/oracle/OracleDriver";
 import { UpdateValuesMissingError } from "../error/UpdateValuesMissingError";
-import { EntityColumnNotFound } from "../error/EntityColumnNotFound";
-import { AuroraDataApiDriver } from "../driver/aurora-data-api/AuroraDataApiDriver";
+import { TypeORMError } from "../error";
+import { EntityPropertyNotFoundError } from "../error/EntityPropertyNotFoundError";
+import { DriverUtils } from "../driver/DriverUtils";
 /**
  * Allows to build complex sql queries in a fashion way and execute those queries.
  */
-var UpdateQueryBuilder = /** @class */ (function (_super) {
-    tslib_1.__extends(UpdateQueryBuilder, _super);
+export class UpdateQueryBuilder extends QueryBuilder {
     // -------------------------------------------------------------------------
     // Constructor
     // -------------------------------------------------------------------------
-    function UpdateQueryBuilder(connectionOrQueryBuilder, queryRunner) {
-        var _this = _super.call(this, connectionOrQueryBuilder, queryRunner) || this;
-        _this.expressionMap.aliasNamePrefixingEnabled = false;
-        return _this;
+    constructor(connectionOrQueryBuilder, queryRunner) {
+        super(connectionOrQueryBuilder, queryRunner);
+        this["@instanceof"] = Symbol.for("UpdateQueryBuilder");
+        this.expressionMap.aliasNamePrefixingEnabled = false;
     }
     // -------------------------------------------------------------------------
     // Public Implemented Methods
     // -------------------------------------------------------------------------
     /**
-     * Gets generated sql query without parameters being replaced.
+     * Gets generated SQL query without parameters being replaced.
      */
-    UpdateQueryBuilder.prototype.getQuery = function () {
-        var sql = this.createUpdateExpression();
+    getQuery() {
+        let sql = this.createComment();
+        sql += this.createUpdateExpression();
         sql += this.createOrderByExpression();
         sql += this.createLimitExpression();
         return sql.trim();
-    };
+    }
     /**
      * Executes sql generated by query builder and returns raw database results.
      */
-    UpdateQueryBuilder.prototype.execute = function () {
-        return tslib_1.__awaiter(this, void 0, void 0, function () {
-            var queryRunner, transactionStartedByUs, broadcastResult, returningResultsEntityUpdator, _a, sql, parameters, updateResult, result, driver, broadcastResult, error_1, rollbackError_1;
-            return tslib_1.__generator(this, function (_b) {
-                switch (_b.label) {
-                    case 0:
-                        queryRunner = this.obtainQueryRunner();
-                        transactionStartedByUs = false;
-                        _b.label = 1;
-                    case 1:
-                        _b.trys.push([1, 15, 20, 25]);
-                        if (!(this.expressionMap.useTransaction === true && queryRunner.isTransactionActive === false)) return [3 /*break*/, 3];
-                        return [4 /*yield*/, queryRunner.startTransaction()];
-                    case 2:
-                        _b.sent();
-                        transactionStartedByUs = true;
-                        _b.label = 3;
-                    case 3:
-                        if (!(this.expressionMap.callListeners === true && this.expressionMap.mainAlias.hasMetadata)) return [3 /*break*/, 5];
-                        broadcastResult = new BroadcasterResult();
-                        queryRunner.broadcaster.broadcastBeforeUpdateEvent(broadcastResult, this.expressionMap.mainAlias.metadata);
-                        if (!(broadcastResult.promises.length > 0)) return [3 /*break*/, 5];
-                        return [4 /*yield*/, Promise.all(broadcastResult.promises)];
-                    case 4:
-                        _b.sent();
-                        _b.label = 5;
-                    case 5:
-                        returningResultsEntityUpdator = new ReturningResultsEntityUpdator(queryRunner, this.expressionMap);
-                        if (this.expressionMap.updateEntity === true &&
-                            this.expressionMap.mainAlias.hasMetadata &&
-                            this.expressionMap.whereEntities.length > 0) {
-                            this.expressionMap.extraReturningColumns = returningResultsEntityUpdator.getUpdationReturningColumns();
-                        }
-                        _a = tslib_1.__read(this.getQueryAndParameters(), 2), sql = _a[0], parameters = _a[1];
-                        updateResult = new UpdateResult();
-                        return [4 /*yield*/, queryRunner.query(sql, parameters)];
-                    case 6:
-                        result = _b.sent();
-                        driver = queryRunner.connection.driver;
-                        if (driver instanceof PostgresDriver) {
-                            updateResult.raw = result[0];
-                            updateResult.affected = result[1];
-                        }
-                        else {
-                            updateResult.raw = result;
-                        }
-                        if (!(this.expressionMap.updateEntity === true &&
-                            this.expressionMap.mainAlias.hasMetadata &&
-                            this.expressionMap.whereEntities.length > 0)) return [3 /*break*/, 8];
-                        return [4 /*yield*/, returningResultsEntityUpdator.update(updateResult, this.expressionMap.whereEntities)];
-                    case 7:
-                        _b.sent();
-                        _b.label = 8;
-                    case 8:
-                        if (!(this.expressionMap.callListeners === true && this.expressionMap.mainAlias.hasMetadata)) return [3 /*break*/, 10];
-                        broadcastResult = new BroadcasterResult();
-                        queryRunner.broadcaster.broadcastAfterUpdateEvent(broadcastResult, this.expressionMap.mainAlias.metadata);
-                        if (!(broadcastResult.promises.length > 0)) return [3 /*break*/, 10];
-                        return [4 /*yield*/, Promise.all(broadcastResult.promises)];
-                    case 9:
-                        _b.sent();
-                        _b.label = 10;
-                    case 10:
-                        if (!transactionStartedByUs) return [3 /*break*/, 12];
-                        return [4 /*yield*/, queryRunner.commitTransaction()];
-                    case 11:
-                        _b.sent();
-                        _b.label = 12;
-                    case 12:
-                        if (!this.expressionMap.callObservers) return [3 /*break*/, 14];
-                        if (!(transactionStartedByUs || (this.expressionMap.useTransaction === false && queryRunner.isTransactionActive === false))) return [3 /*break*/, 14];
-                        return [4 /*yield*/, new ObserverExecutor(this.connection.observers).execute()];
-                    case 13:
-                        _b.sent();
-                        _b.label = 14;
-                    case 14: return [2 /*return*/, updateResult];
-                    case 15:
-                        error_1 = _b.sent();
-                        if (!transactionStartedByUs) return [3 /*break*/, 19];
-                        _b.label = 16;
-                    case 16:
-                        _b.trys.push([16, 18, , 19]);
-                        return [4 /*yield*/, queryRunner.rollbackTransaction()];
-                    case 17:
-                        _b.sent();
-                        return [3 /*break*/, 19];
-                    case 18:
-                        rollbackError_1 = _b.sent();
-                        return [3 /*break*/, 19];
-                    case 19: throw error_1;
-                    case 20:
-                        if (!(queryRunner !== this.queryRunner)) return [3 /*break*/, 22];
-                        return [4 /*yield*/, queryRunner.release()];
-                    case 21:
-                        _b.sent();
-                        _b.label = 22;
-                    case 22:
-                        if (!(this.connection.driver instanceof SqljsDriver && !queryRunner.isTransactionActive)) return [3 /*break*/, 24];
-                        return [4 /*yield*/, this.connection.driver.autoSave()];
-                    case 23:
-                        _b.sent();
-                        _b.label = 24;
-                    case 24: return [7 /*endfinally*/];
-                    case 25: return [2 /*return*/];
+    async execute() {
+        const queryRunner = this.obtainQueryRunner();
+        let transactionStartedByUs = false;
+        try {
+            // start transaction if it was enabled
+            if (this.expressionMap.useTransaction === true &&
+                queryRunner.isTransactionActive === false) {
+                await queryRunner.startTransaction();
+                transactionStartedByUs = true;
+            }
+            // call before updation methods in listeners and subscribers
+            if (this.expressionMap.callListeners === true &&
+                this.expressionMap.mainAlias.hasMetadata) {
+                await queryRunner.broadcaster.broadcast("BeforeUpdate", this.expressionMap.mainAlias.metadata, this.expressionMap.valuesSet);
+            }
+            let declareSql = null;
+            let selectOutputSql = null;
+            // if update entity mode is enabled we may need extra columns for the returning statement
+            const returningResultsEntityUpdator = new ReturningResultsEntityUpdator(queryRunner, this.expressionMap);
+            const returningColumns = [];
+            if (Array.isArray(this.expressionMap.returning) &&
+                this.expressionMap.mainAlias.hasMetadata) {
+                for (const columnPath of this.expressionMap.returning) {
+                    returningColumns.push(...this.expressionMap.mainAlias.metadata.findColumnsWithPropertyPath(columnPath));
                 }
-            });
-        });
-    };
+            }
+            if (this.expressionMap.updateEntity === true &&
+                this.expressionMap.mainAlias.hasMetadata &&
+                this.expressionMap.whereEntities.length > 0) {
+                this.expressionMap.extraReturningColumns =
+                    returningResultsEntityUpdator.getUpdationReturningColumns();
+                returningColumns.push(...this.expressionMap.extraReturningColumns.filter((c) => !returningColumns.includes(c)));
+            }
+            if (returningColumns.length > 0 &&
+                this.connection.driver.options.type === "mssql") {
+                declareSql = this.connection.driver.buildTableVariableDeclaration("@OutputTable", returningColumns);
+                selectOutputSql = `SELECT * FROM @OutputTable`;
+            }
+            // execute update query
+            const [updateSql, parameters] = this.getQueryAndParameters();
+            const statements = [declareSql, updateSql, selectOutputSql];
+            const queryResult = await queryRunner.query(statements.filter((sql) => sql != null).join(";\n\n"), parameters, true);
+            const updateResult = UpdateResult.from(queryResult);
+            // if we are updating entities and entity updation is enabled we must update some of entity columns (like version, update date, etc.)
+            if (this.expressionMap.updateEntity === true &&
+                this.expressionMap.mainAlias.hasMetadata &&
+                this.expressionMap.whereEntities.length > 0) {
+                await returningResultsEntityUpdator.update(updateResult, this.expressionMap.whereEntities);
+            }
+            // call after updation methods in listeners and subscribers
+            if (this.expressionMap.callListeners === true &&
+                this.expressionMap.mainAlias.hasMetadata) {
+                await queryRunner.broadcaster.broadcast("AfterUpdate", this.expressionMap.mainAlias.metadata, this.expressionMap.valuesSet);
+            }
+            // close transaction if we started it
+            if (transactionStartedByUs)
+                await queryRunner.commitTransaction();
+            return updateResult;
+        }
+        catch (error) {
+            // rollback transaction if we started it
+            if (transactionStartedByUs) {
+                try {
+                    await queryRunner.rollbackTransaction();
+                }
+                catch (rollbackError) { }
+            }
+            throw error;
+        }
+        finally {
+            if (queryRunner !== this.queryRunner) {
+                // means we created our own query runner
+                await queryRunner.release();
+            }
+        }
+    }
     // -------------------------------------------------------------------------
     // Public Methods
     // -------------------------------------------------------------------------
     /**
      * Values needs to be updated.
      */
-    UpdateQueryBuilder.prototype.set = function (values) {
+    set(values) {
         this.expressionMap.valuesSet = values;
         return this;
-    };
+    }
     /**
      * Sets WHERE condition in the query builder.
      * If you had previously WHERE expression defined,
      * calling this function will override previously set WHERE conditions.
      * Additionally you can add parameters used in where expression.
      */
-    UpdateQueryBuilder.prototype.where = function (where, parameters) {
+    where(where, parameters) {
         this.expressionMap.wheres = []; // don't move this block below since computeWhereParameter can add where expressions
-        var condition = this.computeWhereParameter(where);
+        const condition = this.getWhereCondition(where);
         if (condition)
-            this.expressionMap.wheres = [{ type: "simple", condition: condition }];
+            this.expressionMap.wheres = [
+                { type: "simple", condition: condition },
+            ];
         if (parameters)
             this.setParameters(parameters);
         return this;
-    };
+    }
     /**
      * Adds new AND WHERE condition in the query builder.
      * Additionally you can add parameters used in where expression.
      */
-    UpdateQueryBuilder.prototype.andWhere = function (where, parameters) {
-        this.expressionMap.wheres.push({ type: "and", condition: this.computeWhereParameter(where) });
+    andWhere(where, parameters) {
+        this.expressionMap.wheres.push({
+            type: "and",
+            condition: this.getWhereCondition(where),
+        });
         if (parameters)
             this.setParameters(parameters);
         return this;
-    };
+    }
     /**
      * Adds new OR WHERE condition in the query builder.
      * Additionally you can add parameters used in where expression.
      */
-    UpdateQueryBuilder.prototype.orWhere = function (where, parameters) {
-        this.expressionMap.wheres.push({ type: "or", condition: this.computeWhereParameter(where) });
+    orWhere(where, parameters) {
+        this.expressionMap.wheres.push({
+            type: "or",
+            condition: this.getWhereCondition(where),
+        });
         if (parameters)
             this.setParameters(parameters);
         return this;
-    };
+    }
     /**
-     * Adds new AND WHERE with conditions for the given ids.
+     * Sets WHERE condition in the query builder with a condition for the given ids.
+     * If you had previously WHERE expression defined,
+     * calling this function will override previously set WHERE conditions.
      */
-    UpdateQueryBuilder.prototype.whereInIds = function (ids) {
-        return this.where(this.createWhereIdsExpression(ids));
-    };
+    whereInIds(ids) {
+        return this.where(this.getWhereInIdsCondition(ids));
+    }
     /**
      * Adds new AND WHERE with conditions for the given ids.
      */
-    UpdateQueryBuilder.prototype.andWhereInIds = function (ids) {
-        return this.andWhere(this.createWhereIdsExpression(ids));
-    };
+    andWhereInIds(ids) {
+        return this.andWhere(this.getWhereInIdsCondition(ids));
+    }
     /**
      * Adds new OR WHERE with conditions for the given ids.
      */
-    UpdateQueryBuilder.prototype.orWhereInIds = function (ids) {
-        return this.orWhere(this.createWhereIdsExpression(ids));
-    };
+    orWhereInIds(ids) {
+        return this.orWhere(this.getWhereInIdsCondition(ids));
+    }
     /**
      * Optional returning/output clause.
      */
-    UpdateQueryBuilder.prototype.output = function (output) {
+    output(output) {
         return this.returning(output);
-    };
+    }
     /**
      * Optional returning/output clause.
      */
-    UpdateQueryBuilder.prototype.returning = function (returning) {
+    returning(returning) {
         // not all databases support returning/output cause
-        if (!this.connection.driver.isReturningSqlSupported())
+        if (!this.connection.driver.isReturningSqlSupported("update")) {
             throw new ReturningStatementNotSupportedError();
+        }
         this.expressionMap.returning = returning;
         return this;
-    };
+    }
     /**
      * Sets ORDER BY condition in the query builder.
      * If you had previously ORDER BY expression defined,
      * calling this function will override previously set ORDER BY conditions.
      */
-    UpdateQueryBuilder.prototype.orderBy = function (sort, order, nulls) {
-        if (order === void 0) { order = "ASC"; }
-        var _a, _b;
+    orderBy(sort, order = "ASC", nulls) {
         if (sort) {
-            if (sort instanceof Object) {
+            if (typeof sort === "object") {
                 this.expressionMap.orderBys = sort;
             }
             else {
                 if (nulls) {
-                    this.expressionMap.orderBys = (_a = {}, _a[sort] = { order: order, nulls: nulls }, _a);
+                    this.expressionMap.orderBys = {
+                        [sort]: { order, nulls },
+                    };
                 }
                 else {
-                    this.expressionMap.orderBys = (_b = {}, _b[sort] = order, _b);
+                    this.expressionMap.orderBys = { [sort]: order };
                 }
             }
         }
@@ -257,228 +226,238 @@ var UpdateQueryBuilder = /** @class */ (function (_super) {
             this.expressionMap.orderBys = {};
         }
         return this;
-    };
+    }
     /**
      * Adds ORDER BY condition in the query builder.
      */
-    UpdateQueryBuilder.prototype.addOrderBy = function (sort, order, nulls) {
-        if (order === void 0) { order = "ASC"; }
+    addOrderBy(sort, order = "ASC", nulls) {
         if (nulls) {
-            this.expressionMap.orderBys[sort] = { order: order, nulls: nulls };
+            this.expressionMap.orderBys[sort] = { order, nulls };
         }
         else {
             this.expressionMap.orderBys[sort] = order;
         }
         return this;
-    };
+    }
     /**
      * Sets LIMIT - maximum number of rows to be selected.
      */
-    UpdateQueryBuilder.prototype.limit = function (limit) {
+    limit(limit) {
         this.expressionMap.limit = limit;
         return this;
-    };
+    }
     /**
      * Indicates if entity must be updated after update operation.
      * This may produce extra query or use RETURNING / OUTPUT statement (depend on database).
      * Enabled by default.
      */
-    UpdateQueryBuilder.prototype.whereEntity = function (entity) {
-        var _this = this;
+    whereEntity(entity) {
         if (!this.expressionMap.mainAlias.hasMetadata)
-            throw new Error(".whereEntity method can only be used on queries which update real entity table.");
+            throw new TypeORMError(`.whereEntity method can only be used on queries which update real entity table.`);
         this.expressionMap.wheres = [];
-        var entities = entity instanceof Array ? entity : [entity];
-        entities.forEach(function (entity) {
-            var entityIdMap = _this.expressionMap.mainAlias.metadata.getEntityIdMap(entity);
+        const entities = Array.isArray(entity) ? entity : [entity];
+        entities.forEach((entity) => {
+            const entityIdMap = this.expressionMap.mainAlias.metadata.getEntityIdMap(entity);
             if (!entityIdMap)
-                throw new Error("Provided entity does not have ids set, cannot perform operation.");
-            _this.orWhereInIds(entityIdMap);
+                throw new TypeORMError(`Provided entity does not have ids set, cannot perform operation.`);
+            this.orWhereInIds(entityIdMap);
         });
         this.expressionMap.whereEntities = entities;
         return this;
-    };
+    }
     /**
      * Indicates if entity must be updated after update operation.
      * This may produce extra query or use RETURNING / OUTPUT statement (depend on database).
      * Enabled by default.
      */
-    UpdateQueryBuilder.prototype.updateEntity = function (enabled) {
+    updateEntity(enabled) {
         this.expressionMap.updateEntity = enabled;
         return this;
-    };
+    }
     // -------------------------------------------------------------------------
     // Protected Methods
     // -------------------------------------------------------------------------
     /**
      * Creates UPDATE express used to perform insert query.
      */
-    UpdateQueryBuilder.prototype.createUpdateExpression = function () {
-        var _this = this;
-        var valuesSet = this.getValueSet();
-        var metadata = this.expressionMap.mainAlias.hasMetadata ? this.expressionMap.mainAlias.metadata : undefined;
+    createUpdateExpression() {
+        const valuesSet = this.getValueSet();
+        const metadata = this.expressionMap.mainAlias.hasMetadata
+            ? this.expressionMap.mainAlias.metadata
+            : undefined;
         // prepare columns and values to be updated
-        var updateColumnAndValues = [];
-        var newParameters = {};
-        var parametersCount = this.connection.driver instanceof MysqlDriver ||
-            this.connection.driver instanceof AuroraDataApiDriver ||
-            this.connection.driver instanceof OracleDriver ||
-            this.connection.driver instanceof AbstractSqliteDriver
-            ? 0 : Object.keys(this.expressionMap.nativeParameters).length;
+        const updateColumnAndValues = [];
+        const updatedColumns = [];
         if (metadata) {
-            EntityMetadata.createPropertyPath(metadata, valuesSet).forEach(function (propertyPath) {
+            this.createPropertyPath(metadata, valuesSet).forEach((propertyPath) => {
                 // todo: make this and other query builder to work with properly with tables without metadata
-                var columns = metadata.findColumnsWithPropertyPath(propertyPath);
+                const columns = metadata.findColumnsWithPropertyPath(propertyPath);
                 if (columns.length <= 0) {
-                    throw new EntityColumnNotFound(propertyPath);
+                    throw new EntityPropertyNotFoundError(propertyPath, metadata);
                 }
-                columns.forEach(function (column) {
-                    if (!column.isUpdate) {
+                columns.forEach((column) => {
+                    if (!column.isUpdate ||
+                        updatedColumns.includes(column)) {
                         return;
                     }
-                    // skip weird cases when we send wrong relations in update map
-                    if (column.relationMetadata &&
-                        (column.relationMetadata.isManyToMany || column.relationMetadata.isOneToMany))
-                        return;
-                    // skip generated columns updation since we can't update them - only database update them
-                    if (column.isGenerated)
-                        return;
-                    var paramName = "upd_" + column.databaseName;
+                    updatedColumns.push(column);
                     //
-                    var value = column.getEntityValue(valuesSet);
-                    if (column.referencedColumn && value instanceof Object) {
-                        value = column.referencedColumn.getEntityValue(value);
+                    let value = column.getEntityValue(valuesSet);
+                    if (column.referencedColumn &&
+                        typeof value === "object" &&
+                        !Buffer.isBuffer(value)) {
+                        value =
+                            column.referencedColumn.getEntityValue(value);
                     }
-                    else if (!(value instanceof Function)) {
-                        value = _this.connection.driver.preparePersistentValue(value, column);
+                    else if (!(typeof value === "function")) {
+                        value =
+                            this.connection.driver.preparePersistentValue(value, column);
                     }
                     // todo: duplication zone
-                    if (value instanceof Function) { // support for SQL expressions in update query
-                        updateColumnAndValues.push(_this.escape(column.databaseName) + " = " + value());
+                    if (typeof value === "function") {
+                        // support for SQL expressions in update query
+                        updateColumnAndValues.push(this.escape(column.databaseName) +
+                            " = " +
+                            value());
+                    }
+                    else if (this.connection.driver.options.type === "sap" &&
+                        value === null) {
+                        updateColumnAndValues.push(this.escape(column.databaseName) + " = NULL");
                     }
                     else {
-                        if (_this.connection.driver instanceof SqlServerDriver) {
-                            value = _this.connection.driver.parametrizeValue(column, value);
-                            // } else if (value instanceof Array) {
-                            //     value = new ArrayParameter(value);
-                        }
-                        if (_this.connection.driver instanceof MysqlDriver ||
-                            _this.connection.driver instanceof AuroraDataApiDriver ||
-                            _this.connection.driver instanceof OracleDriver ||
-                            _this.connection.driver instanceof AbstractSqliteDriver) {
-                            newParameters[paramName] = value;
-                        }
-                        else {
-                            _this.expressionMap.nativeParameters[paramName] = value;
+                        if (this.connection.driver.options.type === "mssql") {
+                            value = this.connection.driver.parametrizeValue(column, value);
                         }
-                        var expression = null;
-                        if ((_this.connection.driver instanceof MysqlDriver || _this.connection.driver instanceof AuroraDataApiDriver) && _this.connection.driver.spatialTypes.indexOf(column.type) !== -1) {
-                            var useLegacy = _this.connection.driver.options.legacySpatialSupport;
-                            var geomFromText = useLegacy ? "GeomFromText" : "ST_GeomFromText";
+                        const paramName = this.createParameter(value);
+                        let expression = null;
+                        if ((DriverUtils.isMySQLFamily(this.connection.driver) ||
+                            this.connection.driver.options.type ===
+                                "aurora-mysql") &&
+                            this.connection.driver.spatialTypes.indexOf(column.type) !== -1) {
+                            const useLegacy = this.connection.driver.options.legacySpatialSupport;
+                            const geomFromText = useLegacy
+                                ? "GeomFromText"
+                                : "ST_GeomFromText";
                             if (column.srid != null) {
-                                expression = geomFromText + "(" + _this.connection.driver.createParameter(paramName, parametersCount) + ", " + column.srid + ")";
+                                expression = `${geomFromText}(${paramName}, ${column.srid})`;
                             }
                             else {
-                                expression = geomFromText + "(" + _this.connection.driver.createParameter(paramName, parametersCount) + ")";
+                                expression = `${geomFromText}(${paramName})`;
                             }
                         }
-                        else if (_this.connection.driver instanceof PostgresDriver && _this.connection.driver.spatialTypes.indexOf(column.type) !== -1) {
+                        else if (this.connection.driver.options.type ===
+                            "postgres" &&
+                            this.connection.driver.spatialTypes.indexOf(column.type) !== -1) {
                             if (column.srid != null) {
-                                expression = "ST_SetSRID(ST_GeomFromGeoJSON(" + _this.connection.driver.createParameter(paramName, parametersCount) + "), " + column.srid + ")::" + column.type;
+                                expression = `ST_SetSRID(ST_GeomFromGeoJSON(${paramName}), ${column.srid})::${column.type}`;
                             }
                             else {
-                                expression = "ST_GeomFromGeoJSON(" + _this.connection.driver.createParameter(paramName, parametersCount) + ")::" + column.type;
+                                expression = `ST_GeomFromGeoJSON(${paramName})::${column.type}`;
                             }
                         }
+                        else if (this.connection.driver.options.type ===
+                            "mssql" &&
+                            this.connection.driver.spatialTypes.indexOf(column.type) !== -1) {
+                            expression =
+                                column.type +
+                                    "::STGeomFromText(" +
+                                    paramName +
+                                    ", " +
+                                    (column.srid || "0") +
+                                    ")";
+                        }
                         else {
-                            expression = _this.connection.driver.createParameter(paramName, parametersCount);
+                            expression = paramName;
                         }
-                        updateColumnAndValues.push(_this.escape(column.databaseName) + " = " + expression);
-                        parametersCount++;
+                        updateColumnAndValues.push(this.escape(column.databaseName) +
+                            " = " +
+                            expression);
                     }
                 });
             });
-            if (metadata.versionColumn)
-                updateColumnAndValues.push(this.escape(metadata.versionColumn.databaseName) + " = " + this.escape(metadata.versionColumn.databaseName) + " + 1");
-            if (metadata.updateDateColumn)
-                updateColumnAndValues.push(this.escape(metadata.updateDateColumn.databaseName) + " = CURRENT_TIMESTAMP"); // todo: fix issue with CURRENT_TIMESTAMP(6) being used, can "DEFAULT" be used?!
+            // Don't allow calling update only with columns that are `update: false`
+            if (updateColumnAndValues.length > 0 ||
+                Object.keys(valuesSet).length === 0) {
+                if (metadata.versionColumn &&
+                    updatedColumns.indexOf(metadata.versionColumn) === -1)
+                    updateColumnAndValues.push(this.escape(metadata.versionColumn.databaseName) +
+                        " = " +
+                        this.escape(metadata.versionColumn.databaseName) +
+                        " + 1");
+                if (metadata.updateDateColumn &&
+                    updatedColumns.indexOf(metadata.updateDateColumn) === -1)
+                    updateColumnAndValues.push(this.escape(metadata.updateDateColumn.databaseName) +
+                        " = CURRENT_TIMESTAMP"); // todo: fix issue with CURRENT_TIMESTAMP(6) being used, can "DEFAULT" be used?!
+            }
         }
         else {
-            Object.keys(valuesSet).map(function (key) {
-                var value = valuesSet[key];
+            Object.keys(valuesSet).map((key) => {
+                let value = valuesSet[key];
                 // todo: duplication zone
-                if (value instanceof Function) { // support for SQL expressions in update query
-                    updateColumnAndValues.push(_this.escape(key) + " = " + value());
+                if (typeof value === "function") {
+                    // support for SQL expressions in update query
+                    updateColumnAndValues.push(this.escape(key) + " = " + value());
+                }
+                else if (this.connection.driver.options.type === "sap" &&
+                    value === null) {
+                    updateColumnAndValues.push(this.escape(key) + " = NULL");
                 }
                 else {
                     // we need to store array values in a special class to make sure parameter replacement will work correctly
                     // if (value instanceof Array)
                     //     value = new ArrayParameter(value);
-                    if (_this.connection.driver instanceof MysqlDriver ||
-                        _this.connection.driver instanceof AuroraDataApiDriver ||
-                        _this.connection.driver instanceof OracleDriver ||
-                        _this.connection.driver instanceof AbstractSqliteDriver) {
-                        newParameters[key] = value;
-                    }
-                    else {
-                        _this.expressionMap.nativeParameters[key] = value;
-                    }
-                    updateColumnAndValues.push(_this.escape(key) + " = " + _this.connection.driver.createParameter(key, parametersCount));
-                    parametersCount++;
+                    const paramName = this.createParameter(value);
+                    updateColumnAndValues.push(this.escape(key) + " = " + paramName);
                 }
             });
         }
         if (updateColumnAndValues.length <= 0) {
             throw new UpdateValuesMissingError();
         }
-        // we re-write parameters this way because we want our "UPDATE ... SET" parameters to be first in the list of "nativeParameters"
-        // because some drivers like mysql depend on order of parameters
-        if (this.connection.driver instanceof MysqlDriver ||
-            this.connection.driver instanceof AuroraDataApiDriver ||
-            this.connection.driver instanceof OracleDriver ||
-            this.connection.driver instanceof AbstractSqliteDriver) {
-            this.expressionMap.nativeParameters = Object.assign(newParameters, this.expressionMap.nativeParameters);
-        }
         // get a table name and all column database names
-        var whereExpression = this.createWhereExpression();
-        var returningExpression = this.createReturningExpression();
-        // generate and return sql update query
-        if (returningExpression && (this.connection.driver instanceof PostgresDriver || this.connection.driver instanceof OracleDriver || this.connection.driver instanceof CockroachDriver)) {
-            return "UPDATE " + this.getTableName(this.getMainTableName()) + " SET " + updateColumnAndValues.join(", ") + whereExpression + " RETURNING " + returningExpression;
+        const whereExpression = this.createWhereExpression();
+        const returningExpression = this.createReturningExpression("update");
+        if (returningExpression === "") {
+            return `UPDATE ${this.getTableName(this.getMainTableName())} SET ${updateColumnAndValues.join(", ")}${whereExpression}`; // todo: how do we replace aliases in where to nothing?
         }
-        else if (returningExpression && this.connection.driver instanceof SqlServerDriver) {
-            return "UPDATE " + this.getTableName(this.getMainTableName()) + " SET " + updateColumnAndValues.join(", ") + " OUTPUT " + returningExpression + whereExpression;
+        if (this.connection.driver.options.type === "mssql") {
+            return `UPDATE ${this.getTableName(this.getMainTableName())} SET ${updateColumnAndValues.join(", ")} OUTPUT ${returningExpression}${whereExpression}`;
         }
-        else {
-            return "UPDATE " + this.getTableName(this.getMainTableName()) + " SET " + updateColumnAndValues.join(", ") + whereExpression; // todo: how do we replace aliases in where to nothing?
-        }
-    };
+        return `UPDATE ${this.getTableName(this.getMainTableName())} SET ${updateColumnAndValues.join(", ")}${whereExpression} RETURNING ${returningExpression}`;
+    }
     /**
      * Creates "ORDER BY" part of SQL query.
      */
-    UpdateQueryBuilder.prototype.createOrderByExpression = function () {
-        var _this = this;
-        var orderBys = this.expressionMap.orderBys;
+    createOrderByExpression() {
+        const orderBys = this.expressionMap.orderBys;
         if (Object.keys(orderBys).length > 0)
-            return " ORDER BY " + Object.keys(orderBys)
-                .map(function (columnName) {
-                if (typeof orderBys[columnName] === "string") {
-                    return _this.replacePropertyNames(columnName) + " " + orderBys[columnName];
-                }
-                else {
-                    return _this.replacePropertyNames(columnName) + " " + orderBys[columnName].order + " " + orderBys[columnName].nulls;
-                }
-            })
-                .join(", ");
+            return (" ORDER BY " +
+                Object.keys(orderBys)
+                    .map((columnName) => {
+                    if (typeof orderBys[columnName] === "string") {
+                        return (this.replacePropertyNames(columnName) +
+                            " " +
+                            orderBys[columnName]);
+                    }
+                    else {
+                        return (this.replacePropertyNames(columnName) +
+                            " " +
+                            orderBys[columnName].order +
+                            " " +
+                            orderBys[columnName].nulls);
+                    }
+                })
+                    .join(", "));
         return "";
-    };
+    }
     /**
      * Creates "LIMIT" parts of SQL query.
      */
-    UpdateQueryBuilder.prototype.createLimitExpression = function () {
-        var limit = this.expressionMap.limit;
+    createLimitExpression() {
+        let limit = this.expressionMap.limit;
         if (limit) {
-            if (this.connection.driver instanceof MysqlDriver || this.connection.driver instanceof AuroraDataApiDriver) {
+            if (DriverUtils.isMySQLFamily(this.connection.driver) ||
+                this.connection.driver.options.type === "aurora-mysql") {
                 return " LIMIT " + limit;
             }
             else {
@@ -486,17 +465,15 @@ var UpdateQueryBuilder = /** @class */ (function (_super) {
             }
         }
         return "";
-    };
+    }
     /**
      * Gets array of values need to be inserted into the target table.
      */
-    UpdateQueryBuilder.prototype.getValueSet = function () {
-        if (this.expressionMap.valuesSet instanceof Object)
+    getValueSet() {
+        if (typeof this.expressionMap.valuesSet === "object")
             return this.expressionMap.valuesSet;
         throw new UpdateValuesMissingError();
-    };
-    return UpdateQueryBuilder;
-}(QueryBuilder));
-export { UpdateQueryBuilder };
+    }
+}
 
 //# sourceMappingURL=UpdateQueryBuilder.js.map