@sap/cds-compiler 5.9.10 → 5.9.12

package/CHANGELOG.md

@@ -7,18 +7,30 @@
 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.
 
+## 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 +38,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/package.json

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