typeorm 0.3.29-dev.a46eb0a → 0.3.29

package/query-builder/SelectQueryBuilder.js

@@ -499,25 +499,17 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
      * calling this function will override previously set ORDER BY conditions.
      */
     orderBy(sort, order = "ASC", nulls) {
-        if (order !== undefined && order !== "ASC" && order !== "DESC")
-            throw new error_1.TypeORMError(`SelectQueryBuilder.addOrderBy "order" can accept only "ASC" and "DESC" values.`);
-        if (nulls !== undefined &&
-            nulls !== "NULLS FIRST" &&
-            nulls !== "NULLS LAST")
-            throw new error_1.TypeORMError(`SelectQueryBuilder.addOrderBy "nulls" can accept only "NULLS FIRST" and "NULLS LAST" values.`);
         if (sort) {
             if (typeof sort === "object") {
+                this.validateOrderByCondition(sort);
                 this.expressionMap.orderBys = sort;
             }
             else {
-                if (nulls) {
-                    this.expressionMap.orderBys = {
-                        [sort]: { order, nulls },
-                    };
-                }
-                else {
-                    this.expressionMap.orderBys = { [sort]: order };
-                }
+                const condition = nulls
+                    ? { [sort]: { order, nulls } }
+                    : { [sort]: order };
+                this.validateOrderByCondition(condition);
+                this.expressionMap.orderBys = condition;
             }
         }
         else {
@@ -529,12 +521,10 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
      * Adds ORDER BY condition in the query builder.
      */
     addOrderBy(sort, order = "ASC", nulls) {
-        if (order !== undefined && order !== "ASC" && order !== "DESC")
-            throw new error_1.TypeORMError(`SelectQueryBuilder.addOrderBy "order" can accept only "ASC" and "DESC" values.`);
-        if (nulls !== undefined &&
-            nulls !== "NULLS FIRST" &&
-            nulls !== "NULLS LAST")
-            throw new error_1.TypeORMError(`SelectQueryBuilder.addOrderBy "nulls" can accept only "NULLS FIRST" and "NULLS LAST" values.`);
+        const condition = nulls
+            ? { [sort]: { order, nulls } }
+            : { [sort]: order };
+        this.validateOrderByCondition(condition);
         if (nulls) {
             this.expressionMap.orderBys[sort] = { order, nulls };
         }
@@ -545,48 +535,36 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
     }
     /**
      * Sets LIMIT - maximum number of rows to be selected.
-     * NOTE that it may not work as you expect if you are using joins.
-     * If you want to implement pagination, and you are having join in your query,
-     * then use the take method instead.
+     * When joins are present, a two-query distinct-id strategy is used
+     * so that LIMIT applies to root entities rather than raw joined rows.
+     * @param limit
      */
     limit(limit) {
-        this.expressionMap.limit = this.normalizeNumber(limit);
-        if (this.expressionMap.limit !== undefined &&
-            isNaN(this.expressionMap.limit))
-            throw new error_1.TypeORMError(`Provided "limit" value is not a number. Please provide a numeric value.`);
+        this.expressionMap.limit = this.validateNumericInput("limit", limit);
         return this;
     }
     /**
      * Sets OFFSET - selection offset.
-     * NOTE that it may not work as you expect if you are using joins.
-     * If you want to implement pagination, and you are having join in your query,
-     * then use the skip method instead.
+     * When joins are present, a two-query distinct-id strategy is used
+     * so that OFFSET applies to root entities rather than raw joined rows.
+     * @param offset
      */
     offset(offset) {
-        this.expressionMap.offset = this.normalizeNumber(offset);
-        if (this.expressionMap.offset !== undefined &&
-            isNaN(this.expressionMap.offset))
-            throw new error_1.TypeORMError(`Provided "offset" value is not a number. Please provide a numeric value.`);
+        this.expressionMap.offset = this.validateNumericInput("offset", offset);
         return this;
     }
     /**
      * Sets maximal number of entities to take.
      */
     take(take) {
-        this.expressionMap.take = this.normalizeNumber(take);
-        if (this.expressionMap.take !== undefined &&
-            isNaN(this.expressionMap.take))
-            throw new error_1.TypeORMError(`Provided "take" value is not a number. Please provide a numeric value.`);
+        this.expressionMap.take = this.validateNumericInput("take", take);
         return this;
     }
     /**
      * Sets number of entities to skip.
      */
     skip(skip) {
-        this.expressionMap.skip = this.normalizeNumber(skip);
-        if (this.expressionMap.skip !== undefined &&
-            isNaN(this.expressionMap.skip))
-            throw new error_1.TypeORMError(`Provided "skip" value is not a number. Please provide a numeric value.`);
+        this.expressionMap.skip = this.validateNumericInput("skip", skip);
         return this;
     }
     /**
@@ -1475,14 +1453,13 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
         return "";
     }
     /**
-     * Creates "LOCK" part of SQL query.
+     * @returns "LOCK" part of SQL query
      */
     createLockExpression() {
         const driver = this.connection.driver;
         let lockTablesClause = "";
         if (this.expressionMap.lockTables) {
-            if (!(DriverUtils_1.DriverUtils.isPostgresFamily(driver) ||
-                driver.options.type === "cockroachdb")) {
+            if (!DriverUtils_1.DriverUtils.isPostgresFamily(driver)) {
                 throw new error_1.TypeORMError("Lock tables not supported in selected driver");
             }
             if (this.expressionMap.lockTables.length < 1) {
@@ -1495,7 +1472,12 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
             onLockExpression = " NOWAIT";
         }
         else if (this.expressionMap.onLocked === "skip_locked") {
-            onLockExpression = " SKIP LOCKED";
+            if (driver.options.type === "sap") {
+                onLockExpression = " IGNORE LOCKED";
+            }
+            else {
+                onLockExpression = " SKIP LOCKED";
+            }
         }
         switch (this.expressionMap.lockMode) {
             case "pessimistic_read":
@@ -1514,6 +1496,9 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
                 else if (DriverUtils_1.DriverUtils.isPostgresFamily(driver)) {
                     return " FOR SHARE" + lockTablesClause + onLockExpression;
                 }
+                else if (driver.options.type === "sap") {
+                    return (" FOR SHARE LOCK" + lockTablesClause + onLockExpression);
+                }
                 else if (driver.options.type === "oracle") {
                     return " FOR UPDATE";
                 }
@@ -1530,7 +1515,7 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
                     return " FOR UPDATE" + onLockExpression;
                 }
                 else if (DriverUtils_1.DriverUtils.isPostgresFamily(driver) ||
-                    driver.options.type === "cockroachdb") {
+                    driver.options.type === "sap") {
                     return " FOR UPDATE" + lockTablesClause + onLockExpression;
                 }
                 else if (driver.options.type === "mssql") {
@@ -1539,19 +1524,24 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
                 else {
                     throw new LockNotSupportedOnGivenDriverError_1.LockNotSupportedOnGivenDriverError();
                 }
+            // deprecated, use pessimistic_write with onLocked = "skip_locked" instead
             case "pessimistic_partial_write":
                 if (DriverUtils_1.DriverUtils.isPostgresFamily(driver)) {
                     return " FOR UPDATE" + lockTablesClause + " SKIP LOCKED";
                 }
+                else if (driver.options.type === "sap") {
+                    return " FOR UPDATE" + lockTablesClause + " IGNORE LOCKED";
+                }
                 else if (DriverUtils_1.DriverUtils.isMySQLFamily(driver)) {
                     return " FOR UPDATE SKIP LOCKED";
                 }
                 else {
                     throw new LockNotSupportedOnGivenDriverError_1.LockNotSupportedOnGivenDriverError();
                 }
+            // deprecated, use pessimistic_write with onLocked = "nowait" instead
             case "pessimistic_write_or_fail":
                 if (DriverUtils_1.DriverUtils.isPostgresFamily(driver) ||
-                    driver.options.type === "cockroachdb") {
+                    driver.options.type === "sap") {
                     return " FOR UPDATE" + lockTablesClause + " NOWAIT";
                 }
                 else if (DriverUtils_1.DriverUtils.isMySQLFamily(driver)) {
@@ -1561,8 +1551,7 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
                     throw new LockNotSupportedOnGivenDriverError_1.LockNotSupportedOnGivenDriverError();
                 }
             case "for_no_key_update":
-                if (DriverUtils_1.DriverUtils.isPostgresFamily(driver) ||
-                    driver.options.type === "cockroachdb") {
+                if (DriverUtils_1.DriverUtils.isPostgresFamily(driver)) {
                     return (" FOR NO KEY UPDATE" +
                         lockTablesClause +
                         onLockExpression);
@@ -1996,7 +1985,10 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
         // where we make two queries to find the data we need
         // first query find ids in skip and take range
         // and second query loads the actual data in given ids range
-        if ((this.expressionMap.skip || this.expressionMap.take) &&
+        if ((this.expressionMap.skip ||
+            this.expressionMap.take ||
+            this.expressionMap.offset ||
+            this.expressionMap.limit) &&
             this.expressionMap.joinAttributes.length > 0) {
             // we are skipping order by here because its not working in subqueries anyway
             // to make order by working we need to apply it on a distinct query
@@ -2013,6 +2005,9 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
                 return `${distinctAlias}.${columnAlias} AS ${this.escape(alias)}`;
             });
             const originalQuery = this.clone();
+            // clear limit/offset from the inner query since pagination is handled by the outer distinct query
+            originalQuery.expressionMap.limit = undefined;
+            originalQuery.expressionMap.offset = undefined;
             // preserve original timeTravel value since we set it to "false" in subquery
             const originalQueryTimeTravel = originalQuery.expressionMap.timeTravel;
             rawResults = await new SelectQueryBuilder(this.connection, queryRunner)
@@ -2023,8 +2018,8 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
                 .timeTravelQuery(false) // set it to "false" since time travel clause must appear at the very end and applies to the entire SELECT clause.
                 .getQuery()})`, "distinctAlias")
                 .timeTravelQuery(originalQueryTimeTravel)
-                .offset(this.expressionMap.skip)
-                .limit(this.expressionMap.take)
+                .offset(this.expressionMap.skip ?? this.expressionMap.offset)
+                .limit(this.expressionMap.take ?? this.expressionMap.limit)
                 .orderBy(orderBys)
                 .cache(this.expressionMap.cache && this.expressionMap.cacheId
                 ? `${this.expressionMap.cacheId}-pagination`
@@ -2067,12 +2062,14 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
                                 " IN (:...orm_distinct_ids)";
                     }
                 }
-                rawResults = await this.clone()
+                const secondQuery = this.clone()
                     .mergeExpressionMap({
                     extraAppendedAndWhereCondition: condition,
                 })
-                    .setParameters(parameters)
-                    .loadRawResults(queryRunner);
+                    .setParameters(parameters);
+                secondQuery.expressionMap.limit = undefined;
+                secondQuery.expressionMap.offset = undefined;
+                rawResults = await secondQuery.loadRawResults(queryRunner);
             }
         }
         else {
@@ -2258,14 +2255,6 @@ class SelectQueryBuilder extends QueryBuilder_1.QueryBuilder {
         ObjectUtils_1.ObjectUtils.assign(this.expressionMap, expressionMap);
         return this;
     }
-    /**
-     * Normalizes a give number - converts to int if possible.
-     */
-    normalizeNumber(num) {
-        if (typeof num === "number" || num === undefined || num === null)
-            return num;
-        return Number(num);
-    }
     /**
      * Creates a query builder used to execute sql queries inside this query builder.
      */