@sap/cds-compiler 2.15.8 → 2.15.10

package/CHANGELOG.md

@@ -7,6 +7,14 @@
 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 2.15.10 - 2023-01-26
+
+### Fixed
+
+- If an entity with parameters is auto-exposed, the generated projection now has
+  the same formal parameters and its query forwards these parameters to the origin entity.
+- to.edm(x): Respect record type hint `$Type` in EDM JSON as full qualified `@type` URI property.
+
 ## Version 2.15.8 - 2022-08-02
 
 ### Fixed

package/lib/compiler/populate.js

@@ -1187,6 +1187,21 @@ function populate( model ) {
         $inferred: '$generated',
       },
     };
+    // forward target parameters to projection
+    if (target.params) {
+      art.params = Object.create(null);
+      // is art.query.from.path[0].$syntax: ':' required?
+      art.query.from.path[0].args = Object.create(null);
+      forEachGeneric(target, 'params', (p, pn) => {
+        art.params[pn] = linkToOrigin(p, pn, art, 'params', p.location);
+        art.query.from.path[0].args[pn] = {
+          name: { id: p.name.id, location: p.location },
+          location: p.location,
+          scope: 'param',
+          path: [ { id: pn, location: p.location } ],
+        };
+      });
+    }
     // TODO: do we need to tag the generated entity with elemScope = 'auto'?
     if (autoexposed) {
       Object.assign( autoexposed, art );

package/lib/edm/annotations/genericTranslation.js

@@ -1144,28 +1144,30 @@ function csn2annotationEdm(csn, serviceName, Edm = undefined, options=undefined,
         // this type doesn't exist
         message(warning, context, `explicitly specified type '${ actualTypeName }' not found in vocabulary`);
         // explicitly mentioned type, render in XML and JSON
-        newRecord.setEdmAttribute('Type', actualTypeName);
-      }
-      else if (dTypeName && !isDerivedFrom(actualTypeName, dTypeName)) {
-        // this type doesn't fit the expected one
-        message(warning, context, `explicitly specified type '${ actualTypeName
-        }' is not derived from expected type '${ dTypeName }'`);
-        actualTypeName = dTypeName;
-        // explicitly mentioned type, render in XML and JSON
-        newRecord.setEdmAttribute('Type', actualTypeName);
-      }
-      else if (isAbstractType(actualTypeName)) {
-        // this type is abstract
-        message(warning, context, `explicitly specified type '${ actualTypeName }' is abstract, specify a concrete type`);
-        if(dTypeName)
-          actualTypeName = dTypeName;
-        // set to definition name and render in XML and JSON
-        newRecord.setEdmAttribute('Type', actualTypeName);
+        newRecord.setXml({ 'Type': actualTypeName});
+        // unknown dictionary type: can't fully qualify it
+        newRecord.setJSON({ 'Type': actualTypeName});
       }
       else {
-        // ok
+        if (isAbstractType(actualTypeName)) {
+          // this type is abstract
+          message(warning, context, `explicitly specified type '${ actualTypeName }' is abstract, specify a concrete type`);
+          if(dTypeName)
+            actualTypeName = dTypeName;
+        }
+        else if (dTypeName && !isDerivedFrom(actualTypeName, dTypeName)) {
+          // this type doesn't fit the expected one
+          message(warning, context, `explicitly specified type '${ actualTypeName
+          }' is not derived from expected type '${ dTypeName }'`);
+          actualTypeName = dTypeName;
+        }
         // Dictionary Type, render in XML only for backward compatibility
         newRecord.setXml( { Type: actualTypeName });
+        const vocName = actualTypeName.slice(0, actualTypeName.indexOf('.'));
+        const vocDef = vocabularyDefinitions[vocName];
+        // Set full qualified type in JSON
+        if(vocDef)
+          newRecord.setJSON( { 'Type': `${vocDef.ref.Uri}#${actualTypeName}` });
       }
     }
     else if (dTypeName) { // there is an expected type name according to dictionary
@@ -1323,7 +1325,11 @@ function csn2annotationEdm(csn, serviceName, Edm = undefined, options=undefined,
           const props = Object.create(null);
           Object.entries(obj).forEach(([k, val]) => {
             if(k === '@type') {
-              edmNode.setEdmAttribute('Type', val);
+              edmNode.setJSON({ 'Type': val});
+              // try to shorten full qualified type URI to short type name
+              const parts = val.split('#');
+              const shortTypeName = parts[parts.length-1];
+              edmNode.setXml({ Type: shortTypeName });
             }
             else {
               let child = undefined;

package/lib/edm/edm.js

@@ -1351,8 +1351,8 @@ function getEdm(options, messageFunctions) {
   {
     toJSONattributes(json)
     {
-      if(this._edmAttributes.Type)
-        json['@type'] = this._edmAttributes.Type;
+      if(this._jsonOnlyAttributes.Type)
+        json['@type'] = this._jsonOnlyAttributes.Type;
       let keys = Object.keys(this._edmAttributes).filter(k => k !== 'Type');
       for(const key of keys)
         json['$'+key] = this._edmAttributes[key];

package/package.json

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