@sap/cds-compiler 2.11.2 → 2.13.6

package/lib/transform/db/expansion.js

@@ -20,8 +20,9 @@ const { setProp, isBetaEnabled } = require('../../base/model');
  * @param {Function} messageFunctions.error
  * @param {Function} messageFunctions.info
  * @param {Function} messageFunctions.throwWithError
+ * @param {object} iterateOptions
  */
-function expandStructureReferences(csn, options, pathDelimiter, { error, info, throwWithError }) {
+function expandStructureReferences(csn, options, pathDelimiter, { error, info, throwWithError }, iterateOptions = {}) {
   const {
     isStructured, get$combined, getFinalBaseType, getServiceName,
   } = getUtils(csn);
@@ -39,7 +40,11 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
       const artifact = csn.definitions[path[1]];
       if (!hasAnnotationValue(artifact, '@cds.persistence.table')) {
         const root = get$combined({ SELECT: parent });
-        parent.columns = replaceStar(root, columns, parent.excluding);
+        // TODO: replace with the correct options.transformation?
+        // Do not expand the * in OData for a moment, not to introduce changes
+        // while the OData CSN is still official
+        if (!options.toOdata)
+          parent.columns = replaceStar(root, columns, parent.excluding);
         parent.columns = expand(parent.columns, path.concat('columns'), true);
       }
     },
@@ -49,7 +54,7 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
     orderBy: (parent, name, orderBy, path) => {
       parent.orderBy = expand(orderBy, path.concat('orderBy'));
     },
-  });
+  }, [], iterateOptions);
 
   /**
    * Turn .expand/.inline into normal refs. @cds.persistence.skip .expand with to-many (and all transitive views).
@@ -71,16 +76,25 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
         if (!hasAnnotationValue(artifact, '@cds.persistence.table')) {
           const rewritten = rewrite(root, parent.columns, parent.excluding);
           parent.columns = rewritten.columns;
-          if (rewritten.toMany.length > 0) {
+          /*
+           * Do not remove unexpandable many columns in OData
+          */
+          if (rewritten.toMany.length > 0 && !options.toOdata) {
             markAsToDummyfy(artifact, path[1]);
             if (getServiceName(path[1]) === null)
               error( null, [ 'definitions', path[1] ], { name: path[1] }, 'Unexpected .expand with to-many association in entity $(NAME), which is outside any service');
           }
+          else {
+            parent.columns = rewritten.columns;
+          }
         }
       },
     });
 
-    dummyfy();
+    // OData must keep @cds.persistence.skip definitions
+    // to present them in the API (and CSN)
+    if (!options.toOdata)
+      dummyfy();
 
     cleanup.forEach(fn => fn());
 
@@ -88,23 +102,26 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
 
 
     const publishing = [];
-
-    applyTransformations(csn, {
-      target: (parent, name, target, path) => {
-        if (toDummyfy.indexOf(target) !== -1) {
-          publishing.push({
-            parent, name, target, path: [ ...path ],
-          });
-        }
-      },
-      from: check,
-      columns: check,
-      where: check,
-      groupBy: check,
-      orderBy: check,
-      having: check,
-      limit: check,
-    });
+    // OData must allow navigations to @cds.persistence.skip targets
+    // as valid navigations in the API
+    if (!options.toOdata) {
+      applyTransformations(csn, {
+        target: (parent, name, target, path) => {
+          if (toDummyfy.indexOf(target) !== -1) {
+            publishing.push({
+              parent, name, target, path: [ ...path ],
+            });
+          }
+        },
+        from: check,
+        columns: check,
+        where: check,
+        groupBy: check,
+        orderBy: check,
+        having: check,
+        limit: check,
+      });
+    }
 
 
     /**
@@ -239,15 +256,14 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
      * @param {CSN.Artifact} root All elements visible fromt he query source ($combined)
      * @param {CSN.Column[]} columns
      * @param {string[]} excluding
-     * @returns {{columns: Array, toManys: Array}} Object with rewritten columns (.expand/.inline) and with any .expand + to-many
+     * @returns {{columns: Array, toMany: Array}} Object with rewritten columns (.expand/.inline) and with any .expand + to-many
      */
     function rewrite(root, columns, excluding) {
       const allToMany = [];
       const newThing = [];
       // Replace stars - needs to happen here since the .expand/.inline first path step affects the root *
       columns = replaceStar(root, columns, excluding);
-      for (let i = 0; i < columns.length; i++) {
-        const col = columns[i];
+      for (const col of columns) {
         if (col.expand) {
           // TODO: Can col.ref be empty without an as? Assumption is it cannot - if it has, it's an error, we throw, compiler checks.
           const { expanded, toManys } = expandInline(root, col, col.ref || [], [ dbName(col) ]);
@@ -270,7 +286,7 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
     }
 
     /**
-     * Check wether the given object is a to-many association
+     * Check whether the given object is a to-many association
      *
      * @param {CSN.Element} obj
      * @returns {boolean}
@@ -302,6 +318,11 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
       while (stack.length > 0) {
         const [ base, current, currentRef, currentAlias ] = stack.pop();
         if (isToMany(current) && current.expand) {
+          expanded.push({
+            expand: current.expand,
+            ref: currentRef,
+            as: currentAlias.join(pathDelimiter),
+          });
           toManys.push({ art: current, ref: currentRef, as: currentAlias.join(pathDelimiter) });
         }
         else if (current.expand) {
@@ -436,7 +457,7 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
    *
    * @param {Array} thing
    * @param {CSN.Path} path
-   * @param {boolean} [withAlias=false] Wether to "expand" the (implicit) alias aswell.
+   * @param {boolean} [withAlias=false] Whether to "expand" the (implicit) alias aswell.
    * @returns {Array} New array - with all structured things expanded
    */
   function expand(thing, path, withAlias = false) {
@@ -477,6 +498,7 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
    */
   function expandRef(art, ref, alias, isKey, withAlias) {
     const expanded = [];
+    /** @type {Array<[CSN.Element, any[], any[]]>} */
     const stack = [ [ art, ref, [ alias || ref[ref.length - 1] ] ] ];
     while (stack.length > 0) {
       const [ current, currentRef, currentAlias ] = stack.pop();
@@ -564,8 +586,7 @@ function expandStructureReferences(csn, options, pathDelimiter, { error, info, t
         }
       }
       // Finally: Replace the stars and leave out the shadowed things
-      for (let i = 0; i < subs.length; i++) {
-        const sub = subs[i];
+      for (const sub of subs) {
         if (sub !== '*' && !replaced[dbName(sub)])
           final.push(sub);
         else if (sub === '*')