@sap/cds-compiler 3.1.2 → 3.4.0

package/lib/compiler/tweak-assocs.js

@@ -98,6 +98,7 @@ function tweakAssocs( model ) {
             } );
     }
     else {
+      // ID published! Used in stakeholder project; if renamed, add to oldMessageIds
       info( 'assoc-outside-service', [ elem.target.location, elem ],
             { target },
             'Association target $(TARGET) is outside any service' );
@@ -283,11 +284,12 @@ function tweakAssocs( model ) {
       // managed association as sub element not supported yet
       error( null, [ elem.location, elem ], {},
         // eslint-disable-next-line max-len
-             'Rewriting the ON condition of unmanaged association in sub element is not supported' );
+             'Rewriting the ON-condition of unmanaged association in sub element is not supported' );
       return;
     }
-    const nav = (elem._main && elem._main.query) ? pathNavigation( elem.value )
-      : { navigation: assoc };
+    const nav = (elem._main && elem._main.query && elem.value)
+      ? pathNavigation( elem.value ) // redirected source elem or mixin
+      : { navigation: assoc };       // redirected user-provided
     const cond = copyExpr( assoc.on,
       // replace location in ON except if from mixin element
                            nav.tableAlias && elem.name.location );
@@ -305,7 +307,7 @@ function tweakAssocs( model ) {
       // For "assoc1.assoc2" and "structelem1.assoc2"
       if (elem._redirected !== null) { // null = already reported
         error( 'rewrite-not-supported', [ elem.target.location, elem ], {},
-               'The ON condition is not rewritten here - provide an explicit ON condition' );
+               'The ON-condition is not rewritten here - provide an explicit ON-condition' );
       }
       return;
     }
@@ -314,7 +316,7 @@ function tweakAssocs( model ) {
     }
     else {
       // TODO: support that, now that the ON condition is rewritten in the right order
-      error( null, [ elem.value.location, elem ],
+      error( null, [ elem.value.location, elem ], {},
              'Selecting unmanaged associations from a sub query is not supported' );
     }
     cond.$inferred = 'rewrite';
@@ -325,7 +327,6 @@ function tweakAssocs( model ) {
     // 'assoc' in query or including entity from ON cond of mixin element /
     // element in included structure / element in source ref/d by table alias.
 
-    // TODO: re-check args in references, forbid parameter use for the moment
     // TODO: complain about $self (unclear semantics)
     // console.log( info(null, [assoc.name.location, assoc],
     //       { art: expr._artifact, names: expr.path.map(i=>i.id) }, 'A').toString(), expr.path)
@@ -341,13 +342,25 @@ function tweakAssocs( model ) {
       //                   { names: expr.path.map(i=>i.id), art: root }, 'TA').toString())
       if (!root || root._main !== source)
         return;                 // not $self or source element
+      if (expr.scope === 'param' || root.kind === '$parameters')
+        return;                 // are not allowed anyway - there was an error before
       const item = expr.path[root.kind === '$self' ? 1 : 0];
       // console.log('YE', assoc.name, item, root.name, expr.path)
-      rewritePath( expr, item, assoc,
-                   navProjection( item && tableAlias.elements[item.id], assoc ),
-                   assoc.value.location );
+      const elem = navProjection( item && tableAlias.elements[item.id], assoc );
+      rewritePath( expr, item, assoc, elem, assoc.value.location );
     }
     else if (assoc._main.query) { // from ON cond of mixin element in query
+      const root = expr.path[0]._navigation || expr.path[0]._artifact;
+      if (expr.scope === 'param' || root?.kind === '$parameters') {
+        if (assoc.$errorReported !== 'assoc-unexpected-scope') {
+          error( 'assoc-unexpected-scope', [ assoc.value.location, assoc ],
+                 { id: assoc.value._artifact.name.id },
+                 // eslint-disable-next-line max-len
+                 'Association $(ID) can\'t be projected because its ON-condition refers to a parameter' );
+          assoc.$errorReported = 'assoc-unexpected-scope';
+        }
+        return;
+      }
       const nav = pathNavigation( expr );
       if (nav.navigation || nav.tableAlias) { // rewrite src elem, mixin, $self[.elem]
         rewritePath( expr, nav.item, assoc,
@@ -372,9 +385,9 @@ function tweakAssocs( model ) {
                  { art: art && effectiveType( art ) },
                  {
                    // eslint-disable-next-line max-len
-                   std: 'This element is not originally referred to in the ON condition of association $(ART)',
+                   std: 'This element is not originally referred to in the ON-condition of association $(ART)',
                    // eslint-disable-next-line max-len
-                   element: 'This element is not originally referred to in the ON condition of association $(MEMBER) of $(ART)',
+                   element: 'This element is not originally referred to in the ON-condition of association $(MEMBER) of $(ART)',
                  } );
       }
       rewritePath( expr, item, assoc, (Array.isArray(elem) ? false : elem), null );
@@ -471,9 +484,8 @@ function tweakAssocs( model ) {
     if (elem && !Array.isArray(elem))
       return elem;
     // TODO: better (extra message), TODO: do it
-    error( 'query-undefined-element', [ item.location, assoc ], { id: name || item.id },
-           // eslint-disable-next-line max-len
-           'Element $(ID) has not been found in the elements of the query; please use REDIRECTED TO with an explicit ON condition' );
+    error( 'query-undefined-element', [ item.location, assoc ],
+           { id: name || item.id, '#': 'redirected' } );
     return (elem) ? false : null;
   }
 }

package/lib/compiler/utils.js

@@ -33,7 +33,17 @@ function annotationIsFalse( anno ) {                   // falsy, but not null (u
 }
 function annotationHasEllipsis( anno ) {
   const { val } = anno || {};
-  return Array.isArray( val ) && val.some( v => v.literal === 'token' && v.val === '...' );
+  return Array.isArray( val ) && val.find( v => v.literal === 'token' && v.val === '...' );
+}
+function annotationLocation( anno ) {
+  const { name, location } = anno;
+  return {
+    file: name.location.file,
+    line: name.location.line,
+    col: name.location.col,
+    endLine: location.endLine,
+    endCol: location.endCol,
+  };
 }
 
 /**
@@ -87,7 +97,7 @@ function linkToOrigin( origin, name, parent, prop, location, silentDep ) {
     setMemberParent( elem, name, parent, prop ); // TODO: redef in template
   setLink( elem, '_origin', origin );
   // TODO: should we use silent dependencies also for other things, like
-  // included elements?  (Currently for $inferred: 'expand-element' only)
+  // included elements?  (Currently for $inferred: 'expanded' only)
   if (silentDep)
     dependsOnSilent( elem, origin );
   else
@@ -395,6 +405,7 @@ module.exports = {
   annotationVal,
   annotationIsFalse,
   annotationHasEllipsis,
+  annotationLocation,
   annotateWith,
   setLink,
   setArtifactLink,

package/lib/edm/annotations/genericTranslation.js

@@ -6,7 +6,6 @@ const oDataDictionary = require('../../gen/Dictionary.json');
 const { forEachDefinition } = require('../../model/csnUtils');
 const { forEach } = require("../../utils/objectUtils");
 
-
 /*
    OASIS: https://github.com/oasis-tcs/odata-vocabularies/tree/master/vocabularies
    Aggregation (published)
@@ -562,6 +561,9 @@ function csn2annotationEdm(csn, serviceName, Edm = undefined, options=undefined,
           alternativeEdmTargetName = serviceName + '.EntityContainer/' + alternativeEdmTargetName.substr(lastDotIndex + 1);
         hasAlternativeCarrier = carrier.$hasEntitySet;
       }
+      else if(carrier.kind === 'type') {
+        testToStandardEdmTarget = (x => x ? x.includes(carrier.elements ? 'ComplexType' : 'TypeDefinition') : true);
+      }
       else if (carrier.kind === 'service') {
       // if annotated object is a service, annotation goes to EntityContainer,
       //   except if AppliesTo contains Schema but not EntityContainer, then annotation goes to Schema
@@ -653,14 +655,15 @@ function csn2annotationEdm(csn, serviceName, Edm = undefined, options=undefined,
           Insert $value into $edmJson with inner annotation as well.
         */
         if(innerAnnotation) {
-          if(carrier[prefix]) {
+          // != null => also != undefined
+          if(carrier[prefix] != null) {
             const valPrefix = prefix + '.$value';
             carrier[valPrefix] = carrier[prefix];
             delete carrier[prefix];
             rc = true;
           }
           const edmJsonPrefix = prefix + '.$edmJson';
-          if(carrier[edmJsonPrefix]) {
+          if(carrier[edmJsonPrefix] != null) {
             const valPrefix = prefix + '.$value.$edmJson';
             carrier[valPrefix] = carrier[edmJsonPrefix];
             delete carrier[edmJsonPrefix];
@@ -699,7 +702,7 @@ function csn2annotationEdm(csn, serviceName, Edm = undefined, options=undefined,
         }
         if (innerAnnotation) {
         // A voc annotation has two steps (Namespace+Name),
-        // any furter steps need to be rendered separately
+        // any further steps need to be rendered separately
           const innerAnnoSteps = innerAnnotation.split('.');
           const tailSteps = innerAnnoSteps.splice(2, innerAnnoSteps.length-2);
         // prepend annotation prefix (path) to tail steps
@@ -908,7 +911,7 @@ function csn2annotationEdm(csn, serviceName, Edm = undefined, options=undefined,
         oTarget.append(handleEdmJson(cAnnoValue['$edmJson'], context));
       }
       else if ( Object.keys(cAnnoValue).filter( x => x[0] !== '@' ).length === 0) {
-        // object consists only of properties starting with "@"
+        // object consists only of properties starting with "@", no $value
         message(warning, context, 'nested annotations without corresponding base annotation');
       }
       else {
@@ -1303,7 +1306,7 @@ function csn2annotationEdm(csn, serviceName, Edm = undefined, options=undefined,
   function handleEdmJson(obj, context, exprDef=undefined) {
 
     let edmNode = undefined;
-    if(obj === undefined)
+    if(obj === undefined || obj === null)
       return edmNode;
 
     const dynExprs = edmUtils.intersect(dynamicExpressionNames, Object.keys(obj));
@@ -1314,7 +1317,7 @@ function csn2annotationEdm(csn, serviceName, Edm = undefined, options=undefined,
     }
 
     if (dynExprs.length === 0) {
-      if (typeof obj === 'object' && obj !== null && !Array.isArray(obj) && Object.keys(obj).length === 1) {
+      if (typeof obj === 'object' && !Array.isArray(obj) && Object.keys(obj).length === 1) {
         const k = Object.keys(obj)[0];
         const val = obj[k];
         edmNode = new Edm.ValueThing(v, k[0] === '$' ? k.slice(1) : k, val );

package/lib/edm/csn2edm.js

@@ -305,9 +305,9 @@ function csn2edmAll(_csn, _options, serviceNames=undefined) {
       const reservedNames = ['Edm', 'odata', 'System', 'Transient'];
       const loc = ['definitions', schema.name];
       if(reservedNames.includes(schema.name))
-        warning('odata-spec-violation-namespace', loc, { names: reservedNames });
+        message('odata-spec-violation-namespace', loc, { names: reservedNames });
       if (schema.name.length > 511)
-        warning('odata-spec-violation-namespace', loc, { '#': 'length' });
+        message('odata-spec-violation-namespace', loc, { '#': 'length' });
       else {
         schema.name.split('.').forEach(id => {
           if (!edmUtils.isODataSimpleIdentifier(id))
@@ -413,10 +413,10 @@ function csn2edmAll(_csn, _options, serviceNames=undefined) {
           const pLoc = [...loc, 'elements', p._edmAttributes.Name];
           edmTypeCompatibilityCheck(p, pLoc);
           if(p._edmAttributes.Name === EntityTypeName)
-            warning('odata-spec-violation-property-name', pLoc, { kind: entityCsn.kind });
+            message('odata-spec-violation-property-name', pLoc, { kind: entityCsn.kind });
 
           if(options.isV2() && p._isCollection && !p._csn.target)
-            warning('odata-spec-violation-array', pLoc, { version: '2.0' });
+            message('odata-spec-violation-array', pLoc, { version: '2.0' });
 
           if(!edmUtils.isODataSimpleIdentifier(p._edmAttributes.Name))
             message('odata-spec-violation-id', pLoc, { id: p._edmAttributes.Name });
@@ -626,10 +626,10 @@ function csn2edmAll(_csn, _options, serviceNames=undefined) {
             !param._type.startsWith('Edm.') &&
             csn.definitions[param._type] &&
             !edmUtils.isStructuredType(csn.definitions[param._type]))
-            warning('odata-spec-violation-param', pLoc, { version: '2.0' });
+            message('odata-spec-violation-param', pLoc, { version: '2.0' });
 
           if(param._isCollection)
-            warning('odata-spec-violation-array', pLoc, { version: '2.0' });
+            message('odata-spec-violation-array', pLoc, { version: '2.0' });
 
           functionImport.append(param);
         });
@@ -645,7 +645,7 @@ function csn2edmAll(_csn, _options, serviceNames=undefined) {
           let type = returns.type;
           if (type) {
             if (!isBuiltinType(type) && csn.definitions[type].kind !== 'entity' && csn.definitions[type].kind !== 'type') {
-              warning('odata-spec-violation-returns', returnsLoc, { kind: action.kind, version: '2.0' });
+              message('odata-spec-violation-returns', returnsLoc, { kind: action.kind, version: '2.0' });
             }
             else if(isBuiltinType(type)) {
               type = edmUtils.mapCdsToEdmType(returns, messageFunctions, true);
@@ -731,7 +731,7 @@ function csn2edmAll(_csn, _options, serviceNames=undefined) {
 
         });
         if(options.isV2()) {
-          if(streamProps.length > 1) {
+          if(streamProps.length > 1) { // TODO: why not mention 2.0 in text?
             error(null, ['definitions', elementsCsn.name], { names: streamProps, version: '2.0', anno: '@Core.MediaType' },
                  `Expected only one element to be annotated with $(ANNO) for OData $(VERSION) but found $(NAMES)`);
           }
@@ -764,17 +764,17 @@ function csn2edmAll(_csn, _options, serviceNames=undefined) {
           const pLoc = [ ...loc, ...(structuredTypeCsn.items ? ['items', 'elements'] : [ 'elements' ]), p._edmAttributes.Name ];
           edmTypeCompatibilityCheck(p, pLoc);
           if(p._edmAttributes.Name === complexType._edmAttributes.Name)
-            warning('odata-spec-violation-property-name', pLoc, { kind: structuredTypeCsn.kind });
+            message('odata-spec-violation-property-name', pLoc, { kind: structuredTypeCsn.kind });
 
           if(!edmUtils.isODataSimpleIdentifier(p._edmAttributes.Name))
             message('odata-spec-violation-id', pLoc, { id: p._edmAttributes.Name });
 
           if(options.isV2()) {
             if(p._isCollection && !p._csn.target)
-              warning('odata-spec-violation-array', pLoc, { version: '2.0' });
+              message('odata-spec-violation-array', pLoc, { version: '2.0' });
 
             if(p._csn.target)
-              warning('odata-spec-violation-assoc', pLoc, { version: '2.0' });
+              message('odata-spec-violation-assoc', pLoc, { version: '2.0' });
           }
         });
 

package/lib/edm/edm.js

@@ -10,7 +10,7 @@ const { isBetaEnabled } = require('../base/model.js');
 const EdmTypeFacetMap = {
   'MaxLength': { v2: true, v4: true, remove: true, optional: true },
   'Precision': { v2: true, v4: true, remove: true, optional: true },
-  'Scale': { v2: true, v4: true, remove: true, optional: true },
+  'Scale': { v2: true, v4: true, remove: true, optional: true, extra: 'sap:variable-scale' },
   'SRID': { v4: true, remove: true, optional: true },
   //'FixedLength': { v2: true },
   //'Collation': { v2: true },
@@ -116,6 +116,8 @@ function getEdm(options, messageFunctions) {
     removeEdmAttribute(name) {
       if (name in this._edmAttributes)
         delete this._edmAttributes[name];
+      if (name in this._xmlOnlyAttributes)
+        delete this._xmlOnlyAttributes[name];
     }
 
     /**
@@ -695,14 +697,20 @@ function getEdm(options, messageFunctions) {
               // produce an unrecoverable error.
               if(td && (td.v2 === this.v2 || td.v4 === this.v4)) {
                 this.setEdmAttribute(typeName, odataType);
-                EdmTypeFacetNames.forEach(facet => {
-                  if(EdmTypeFacetMap[facet].remove)
-                    this.removeEdmAttribute(facet);
-                  if(td[facet] !== undefined && 
-                      (EdmTypeFacetMap[facet].v2 === this.v2 || 
-                      EdmTypeFacetMap[facet].v4 === this.v4)
-                    ) {
-                    this.setEdmAttribute(facet, csn['@odata.'+facet]);
+                EdmTypeFacetNames.forEach(facetName => {
+                  const facet = EdmTypeFacetMap[facetName];
+                  if(facet.remove) {
+                    this.removeEdmAttribute(facetName);
+                    this.removeEdmAttribute(facet.extra);
+                  }
+                  if(td[facetName] !== undefined &&
+                      (facet.v2 === this.v2 ||
+                      facet.v4 === this.v4))
+                  {
+                    if(this.v2 && facetName === 'Scale' && csn['@odata.'+facetName] === 'variable')
+                      this.setXml({ [facet.extra]: true });
+                    else
+                      this.setEdmAttribute(facetName, csn['@odata.'+facetName]);
                   }
                 });
               }

package/lib/edm/edmPreprocessor.js

@@ -1,10 +1,10 @@
 'use strict';
 /* eslint max-statements-per-line:off */
 const { setProp, isDeprecatedEnabled, isBetaEnabled } = require('../base/model');
-const { 
+const {
   forEachDefinition, forEachGeneric, forEachMemberRecursively,
-  isEdmPropertyRendered, getUtils, cloneCsnNonDict, cloneCsnDictionary,
-  isBuiltinType, applyTransformations
+  isEdmPropertyRendered, getUtils, cloneCsnNonDict,
+  isBuiltinType, applyTransformations, cloneAnnotationValue
 } = require('../model/csnUtils');
 const edmUtils = require('./edmUtils.js');
 const edmAnnoPreproc = require('./edmAnnoPreprocessor.js');
@@ -14,6 +14,12 @@ const expandCSNToFinalBaseType = require('../transform/odata/toFinalBaseType');
 
 const NavResAnno = '@Capabilities.NavigationRestrictions.RestrictedProperties';
 
+// Capabilities that can be pulled up to NavigationRestrictions
+const capabilities = Object.keys(require('../gen/Dictionary.json').
+  types['Capabilities.NavigationPropertyRestriction'].Properties).
+  filter(c => !['NavigationProperty', 'Navigability'].includes(c)).
+  map(c => `@Capabilities.`+c);
+
 /**
  *  edmPreprocessor warms up the model so that it can be converted into an EDM document and
  *  contains all late & application specific model transformations
@@ -203,7 +209,8 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
     // TODO: work on service basis (this requires post exposure renaming)
     let fallBackSchemaName = 'root';
     let i = 1;
-    while (Object.keys(defs).some(artName => {
+    const defNames = Object.keys(defs);
+    while (defNames.some(artName => {
       const p = artName.split('.');
       return p.length === 2 && p[0] === fallBackSchemaName;
     })) {
@@ -417,9 +424,11 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
         while(elt && !isBuiltinType(elt.type) && !elt.elements) {
           elt = csn.definitions[elt.type];
         }
-        if(elt && elt.elements) {
+        if(elt && elt.elements && !elt.$visited) {
+          setProp(elt, '$visited', true);
           forEachMemberRecursively(elt, initContainments,
             path, true, { pathWithoutProp: true, elementsOnly: true });
+          delete elt.$visited;
         }
       }
     }
@@ -437,9 +446,11 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
         while(elt && !(isBuiltinType(elt.type) || elt.elements)) {
           elt = csn.definitions[elt.type];
         }
-        if(elt && elt.elements) {
+        if(elt && elt.elements && !elt.$visited) {
+          setProp(elt, '$visited', true);
           forEachMemberRecursively(elt, markToContainer,
             [], true, { elementsOnly: true });
+          delete elt.$visited;
         }
       }
     }
@@ -1492,7 +1503,7 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
         and do not render associations (this will include the foreign keys of
         the _isToContainer association).
   */
-  function initEdmKeyRefPaths(def) {
+  function initEdmKeyRefPaths(def, defName) {
     if(def.$keys) {
       setProp(def, '$edmKeyPaths', []);
       // for all key elements that shouldn't be ignored produce the paths
@@ -1503,7 +1514,7 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
             // This is structured OData ONLY
             // if the foreign keys are explicitly requested, ignore associations and use the flat foreign keys instead
             if(!options.renderForeignKeys || (options.renderForeignKeys && !k.target))
-              def.$edmKeyPaths.push(...produceKeyRefPaths(k, kn));
+              def.$edmKeyPaths.push(...produceKeyRefPaths(k, kn, [ 'definitions', defName, 'elements', kn ]));
           }
         // In v2/v4 flat, associations are never rendered
           else if(!k.target) {
@@ -1525,7 +1536,7 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
         navprops to be key ref.
       If element is of scalar type, return it as an array.
     */
-    function produceKeyRefPaths(eltCsn, prefix) {
+    function produceKeyRefPaths(eltCsn, prefix, path) {
       const keyPaths = [];
       // we want to point to the element in the entity which is the first path step
       const location = def.$path.concat(['elements']).concat(prefix.split('/')[0]);
@@ -1537,18 +1548,25 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
       }
       // OData requires all elements along the path to be nullable: false (that is either key or notNull)
 
-      const finalType = csnUtils.getFinalTypeDef(eltCsn.items && eltCsn.items.type || eltCsn.type);
-      const elements = eltCsn.elements || eltCsn.items && eltCsn.items.elements ||
-      (finalType && (finalType.elements || finalType.items && finalType.items.elements));
+      const finalType = csnUtils.getFinalTypeDef(eltCsn.items?.type || eltCsn.type);
+      const elements = eltCsn.elements || eltCsn.items?.elements ||
+                     finalType?.elements || finalType?.items?.elements;
       if(elements) {
         Object.entries(elements).forEach(([eltName, elt]) => {
-          const newRefs = produceKeyRefPaths(elt, prefix + options.pathDelimiter + eltName);
-          if(newRefs.length) {
-            keyPaths.push(...newRefs);
+          if(!elt.$visited) {
+            setProp(elt, '$visited', true);
+            const newRefs = produceKeyRefPaths(elt, prefix + options.pathDelimiter + eltName, path);
+            if(newRefs.length) {
+              keyPaths.push(...newRefs);
             // check path step key for spec violations
-            const pathSegment = `${prefix}/${eltName}`;
-            checkKeySpecViolations(elt, location, pathSegment);
+              const pathSegment = `${prefix}/${eltName}`;
+              checkKeySpecViolations(elt, location, pathSegment);
+            }
+            delete elt.$visited;
+          } else {
+            error('odata-key-recursive', path, { name: prefix });
           }
+
         });
       }
       /* If element is a managed association (can't be anything else),
@@ -1572,7 +1590,7 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
               art = art._type || csnUtils.getCsnDef(art.type);
             }
           }
-          keyPaths.push(...produceKeyRefPaths(art, prefix + options.pathDelimiter + k.ref.join(options.pathDelimiter)));
+          keyPaths.push(...produceKeyRefPaths(art, prefix + options.pathDelimiter + k.ref.join(options.pathDelimiter), path));
         });
       }
       else {
@@ -1677,7 +1695,11 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
           while(elt && !(isBuiltinType(elt.type) || elt.elements)) {
             elt = csn.definitions[elt.type];
           }
-          elt && elt.elements && Object.values(elt.elements).forEach(e => produceTargetPath(newPrefix, e, curDef));
+          if(elt && elt.elements && !elt.$visited) {
+            setProp(elt, '$visited', true);
+            Object.values(elt.elements).forEach(e => produceTargetPath(newPrefix, e, curDef));
+            delete elt.$visited;
+          }
         }
       }
     }
@@ -1750,7 +1772,11 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
           while(elt && !(isBuiltinType(elt.type) || elt.elements)) {
             elt = csn.definitions[elt.type];
           }
-          elt && elt.elements && Object.values(elt.elements).forEach(e => npbs = npbs.concat(produceNavigationPath(e, curDef)));
+          if(elt && elt.elements && !elt.$visited) {
+            setProp(elt, '$visited', true);
+            Object.values(elt.elements).forEach(e => npbs = npbs.concat(produceNavigationPath(e, curDef)));
+            delete elt.$visited;
+          }
         }
       }
       npbs.forEach(p => p.Path = prefix + '/' + p.Path );
@@ -1832,7 +1858,7 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
       if(container.$containeeAssociations) {
         // copy or create container restrictions, don't modify original
         const localRestrictions = container[NavResAnno] ?
-        cloneCsnDictionary(container[NavResAnno], options) : []
+          cloneAnnotationValue(container[NavResAnno]) : []
 
         // prefix the existing navigation property restritictions on the container
         if(prefix.length) {
@@ -1867,13 +1893,10 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
 
             const props = Object.entries(containee);
             let newEntry = false;
-            ['@Capabilities.DeleteRestrictions',
-              '@Capabilities.InsertRestrictions',
-              '@Capabilities.UpdateRestrictions',
-              '@Capabilities.ReadRestrictions'].forEach(c => {
-                if(edmUtils.mergeIntoNavPropEntry(c, navPropEntry, prefix.concat(path), props))
-                  newEntry = true;
-              });
+            capabilities.forEach(c => {
+              if(edmUtils.mergeIntoNavPropEntry(c, navPropEntry, prefix.concat(path), props))
+                newEntry = true;
+            });
 
             if(newEntry && !hasEntry) {
               localRestrictions.push(navPropEntry);
@@ -1995,12 +2018,12 @@ function initializeModel(csn, _options, messageFunctions, requestedServiceNames=
     if (obj.type && isBuiltinType(obj.type) && !obj.target && !obj.targetAspect) {
       let edmType = edmUtils.mapCdsToEdmType(obj, messageFunctions, _options.odataVersion === 'v2', obj['@Core.MediaType']);
       edmUtils.assignProp(obj, '_edmType', edmType);
-    } else if (obj._isCollection && (obj.items && isBuiltinType(csnUtils.getFinalTypeDef(obj.items.type)))) {
+    } else if (obj._isCollection && (obj.items && isBuiltinType(csnUtils.getFinalBaseTypeWithProps(obj.items.type)?.type))) {
       let edmType = edmUtils.mapCdsToEdmType(obj.items, messageFunctions, _options.odataVersion === 'v2', obj['@Core.MediaType'], obj.$path);
       edmUtils.assignProp(obj, '_edmType', edmType);
     }
     // This is the special case when we have array of array, but will not be supported in the future
-    else if (obj._isCollection && obj.items && obj.items.type && obj.items.items && isBuiltinType(csnUtils.getFinalTypeDef(obj.items.items.type))) {
+    else if (obj._isCollection && obj.items && obj.items.type && obj.items.items && isBuiltinType(csnUtils.getFinalBaseTypeWithProps(obj.items.items.type)?.type)) {
       let edmType = edmUtils.mapCdsToEdmType(obj.items.items, messageFunctions, _options.odataVersion === 'v2', obj['@Core.MediaType']);
       edmUtils.assignProp(obj, '_edmType', edmType);
     }

package/lib/edm/edmUtils.js

@@ -1,6 +1,6 @@
 'use strict';
 const { setProp } = require('../base/model');
-const { isBuiltinType, isEdmPropertyRendered, applyTransformations } = require('../model/csnUtils');
+const { isBuiltinType, isEdmPropertyRendered, applyTransformations, cloneAnnotationValue } = require('../model/csnUtils');
 const { escapeString, hasControlCharacters, hasUnpairedUnicodeSurrogate } = require("../render/utils/stringEscapes");
 
 /* eslint max-statements-per-line:off */
@@ -508,6 +508,10 @@ function mapCdsToEdmType(csn, messageFunctions, isV2=false, isMediaType=false, l
     'cds.hana.SMALLDECIMAL': 'Edm.Decimal', // V4: Scale="floating" Precision="16"
     'cds.Integer64': 'Edm.Int64',
     'cds.Integer': 'Edm.Int32',
+    'cds.Int64': 'Edm.Int64',
+    'cds.Int32': 'Edm.Int32',
+    'cds.Int16': 'Edm.Int16',
+    'cds.UInt8': 'Edm.Byte',
     'cds.hana.SMALLINT': 'Edm.Int16',
     'cds.hana.TINYINT': 'Edm.Byte',
     'cds.Double': 'Edm.Double',
@@ -758,7 +762,8 @@ function mergeIntoNavPropEntry(annoPrefix, navPropEntry, prefix, props) {
 
   // Filter properties with prefix and reduce them into a new dictionary
   const o = props.filter(p => p[0].startsWith(annoPrefix+'.')).reduce((a,c) => {
-    a[c[0].replace(annoPrefix+'.', '')] = c[1];
+    // clone the annotation value to avoid side effects with rewritten paths
+    a[c[0].replace(annoPrefix+'.', '')] = cloneAnnotationValue(c[1]);
     return a;
   }, { });
 

package/lib/gen/Dictionary.json

@@ -1808,6 +1808,13 @@
         "Parameter"
       ]
     },
+    "UI.IsCopyAction": {
+      "Type": "Core.Tag",
+      "AppliesTo": [
+        "Record"
+      ],
+      "$experimental": true
+    },
     "UI.CreateHidden": {
       "Type": "Core.Tag",
       "AppliesTo": [
@@ -1897,6 +1904,13 @@
       ],
       "$experimental": true
     },
+    "UI.LeadingEntitySet": {
+      "Type": "Edm.String",
+      "AppliesTo": [
+        "EntityContainer"
+      ],
+      "$experimental": true
+    },
     "Validation.Pattern": {
       "Type": "Edm.String",
       "AppliesTo": [

package/lib/gen/language.checksum

@@ -1 +1 @@
-2067e213918678e2d85713a46e69d9cc
+5a5c0432924af9c8833ad5aa6b94724f