@sap/cds-compiler 2.12.0 → 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,
+      });
+    }
 
 
     /**
@@ -269,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}
@@ -301,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) {
@@ -435,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) {