@sap/cds-compiler 5.9.10 → 5.9.14

package/CHANGELOG.md

@@ -7,18 +7,36 @@
 Note: `beta` fixes, changes and features are usually not listed in this ChangeLog but [here](doc/CHANGELOG_BETA.md).
 The compiler behavior concerning `beta` features can change at any time without notice.
 
+## Verion 5.9.14 - 2025-12-03
+
+### Fixed
+
+- to.sql|hdi: Don't add superfluous and sometimes wrong (SQLite) parentheses around `UNION`s
+
+## Version 5.9.12 - 2025-09-19
+
+### Fixed
+
+- to.sql: Topological ordering of views did not always account for subqueries (fixes regression from v5.9.0)
+
 ## Version 5.9.10 - 2025-09-11
 
+### Fixed
+
 - parser: Keep parentheses around lists on the right side of an `in` operator.
 - compiler: For calculated elements using associations with filters and cardinality, CSN recompilation could
             fail for `gensrc` CSN, as happens for MTX.
 
 ## Version 5.9.8 - 2025-07-14
 
+### Fixed
+
 - compiler: Calculated elements can now have a localized type
 
 ## Version 5.9.6 - 2025-06-18
 
+### Fixed
+
 - to.sql: Fix error when calculated element refers to a localized element.
 - to.edm(x):
   + Fix errors for service entities containing multiple path steps (e.g. `Service.Prefix.MyEntity`).
@@ -26,6 +44,8 @@ The compiler behavior concerning `beta` features can change at any time without
 
 ## Version 5.9.4 - 2025-05-22
 
+### Fixed
+
 - to.edm(x): Parameters are marked optional unless explicitly marked as `not null`.
   Annotation `Core.OptionalParameter` will be added to optional parameters.
 

package/lib/model/csnUtils.js

@@ -602,8 +602,7 @@ function forAllQueries( query, queryCallback, path ) {
     const expr = query[prop];
     if (expr && typeof expr === 'object') {
       if (Array.isArray(expr)) {
-        for (let i = 0; i < expr.length; i++)
-          forAllQueries(expr[i], queryCallback, [ ...path, prop, i ]);
+        traverseStructurizedExpression(expr, [ ...path, prop ]);
       }
       else {
         for (const argName of Object.keys( expr ))
@@ -611,6 +610,23 @@ function forAllQueries( query, queryCallback, path ) {
       }
     }
   }
+
+  /**
+   * Traverse the possibly structured xpr. A structured expression may contain arrays inside arrays
+   * to show precedence of operators.
+   *
+   * @param {any} xpr
+   * @param {CSN.Path} csnPath
+   */
+  function traverseStructurizedExpression( xpr, csnPath ) {
+    if (Array.isArray(xpr)) {
+      for (let i = 0; i < xpr.length; i++)
+        traverseStructurizedExpression(xpr[i], [ ...csnPath, i ]);
+    }
+    else {
+      forAllQueries(xpr, queryCallback, csnPath);
+    }
+  }
 }
 
 /**

package/lib/render/toSql.js

@@ -1307,11 +1307,7 @@ function toSqlDdl( csn, options, messageFunctions ) {
 
       // Set operation may also have an ORDER BY and LIMIT/OFFSET (in contrast to the ones belonging to
       // each SELECT)
-      // If the whole SET has an ORDER BY/LIMIT, wrap the part before that in parentheses
-      // (otherwise some SQL implementations (e.g. sqlite) would interpret the ORDER BY/LIMIT as belonging
-      // to the last SET argument, not to the whole SET)
       if (query.SET.orderBy || query.SET.limit) {
-        result = `(${result})`;
         if (query.SET.orderBy) {
           const orderBy = query.SET.orderBy.map(entry => renderOrderByEntry(entry, env.withSubPath([ 'orderBy' ]))).join(', ');
           result += `\n${env.indent}ORDER BY ${orderBy}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sap/cds-compiler",
-  "version": "5.9.10",
+  "version": "5.9.14",
   "description": "CDS (Core Data Services) compiler and backends",
   "homepage": "https://cap.cloud.sap/",
   "author": "SAP SE (https://www.sap.com)",