@sap/cds-compiler 3.4.4 → 3.5.2

package/lib/model/csnUtils.js

@@ -43,7 +43,7 @@ const { version } = require('../../package.json');
  * Get utility functions for a given CSN. Re-exports functions of `csnRefs()`.
  * @param {CSN.Model} model (Compact) CSN model
  */
-function getUtils(model, universalReady) {
+function getUtils( model, universalReady ) {
   const _csnRefs = csnRefs(model, universalReady);
   const { artifactRef } = _csnRefs;
   /** Cache for getFinalBaseTypeWithProps(). Specific to the current model. */
@@ -77,7 +77,7 @@ function getUtils(model, universalReady) {
    * @param {CSN.Query} query
    * @returns {object}
    */
-  function get$combined(query) {
+  function get$combined( query ) {
     return getSources(query);
   }
 
@@ -91,7 +91,7 @@ function getUtils(model, universalReady) {
    * @param {boolean} [isSubquery]
    * @returns {object} Map of sources
    */
-  function getSources(query, isSubquery = false) {
+  function getSources( query, isSubquery = false ) {
     // Remark CW: better just a while along query.SET.args[0]
     if (query.SET) {
       if (query.SET.args[0].SELECT && query.SET.args[0].SELECT.elements)
@@ -124,7 +124,7 @@ function getUtils(model, universalReady) {
     return {};
   }
 
-  function walkArgs(args) {
+  function walkArgs( args ) {
     let elements = Object.create(null);
     for (const arg of args) {
       if (arg.args) {
@@ -149,7 +149,7 @@ function getUtils(model, universalReady) {
    * @param {object} mapB Map b - will not be returned
    * @returns {object} mapA
    */
-  function mergeElementMaps(mapA, mapB) {
+  function mergeElementMaps( mapA, mapB ) {
     for (const elementName in mapB) {
       if (!mapA[elementName])
         mapA[elementName] = [];
@@ -170,7 +170,7 @@ function getUtils(model, universalReady) {
    * @param {any} [errorParent] Parent name to use for error messages, ref before alias
    * @returns {object} existingMap
    */
-  function mergeElementsIntoMap(existingMap, elements, $location, parent, errorParent) {
+  function mergeElementsIntoMap( existingMap, elements, $location, parent, errorParent ) {
     for (const elementName in elements) {
       const element = elements[elementName];
       if (!existingMap[elementName])
@@ -188,7 +188,7 @@ function getUtils(model, universalReady) {
    * Return the name part of the artifact name - no namespace etc.
    * @param {string|object} name Absolute name of the artifact
    */
-  function getBaseName(name) {
+  function getBaseName( name ) {
     if (!name)
       return name;
 
@@ -202,7 +202,7 @@ function getUtils(model, universalReady) {
    * Return the left-most, primary source of the given query.
    * @param {*} query Definition's query object
    */
-  function getQueryPrimarySource(query) {
+  function getQueryPrimarySource( query ) {
     if (!query)
       return undefined;
     else if (query.SELECT)
@@ -224,7 +224,7 @@ function getUtils(model, universalReady) {
    * Create an object to track visited objects identified by a unique string.
    * @param {string} [initialId] Initial entry (optional)
    */
-  function createVisited(initialId) {
+  function createVisited( initialId ) {
     const visited = Object.create(null);
     check(initialId);
     return { check };
@@ -234,7 +234,7 @@ function getUtils(model, universalReady) {
      * add it to the list of visited identifiers.
      * @param {string} id unique identifier
      */
-    function check(id) {
+    function check( id ) {
       if (!id)
         return;
       if (visited[id])
@@ -248,7 +248,7 @@ function getUtils(model, universalReady) {
    * Get the CSN definition for an artifact name.
    * @param {string} defName Absolute name of the artifact
    */
-  function getCsnDef(defName) {
+  function getCsnDef( defName ) {
     if (model.definitions[defName])
       return model.definitions[defName];
     throw new ModelError(`Nonexistent definition in the model: '${ defName }'`);
@@ -261,7 +261,7 @@ function getUtils(model, universalReady) {
    * @param {object} obj
    * @returns {boolean}
    */
-  function isStructured(obj) {
+  function isStructured( obj ) {
     return !!(obj.elements || (obj.type && getFinalBaseTypeWithProps(obj.type)?.elements));
   }
 
@@ -272,7 +272,7 @@ function getUtils(model, universalReady) {
    * @param {string} typeName Absolute type name
    * @returns {object}
    */
-  function getFinalTypeDef(typeName) {
+  function getFinalTypeDef( typeName ) {
     const visited = createVisited(typeName);
     let type = model.definitions[typeName];
     if (!type)
@@ -291,7 +291,7 @@ function getUtils(model, universalReady) {
    * @param {string} typeName Absolute type name
    * @returns {string}
    */
-  function getFinalType(typeName) {
+  function getFinalType( typeName ) {
     const visited = createVisited(typeName);
     let type = model.definitions[typeName];
     while (type && type.type) {
@@ -304,7 +304,7 @@ function getUtils(model, universalReady) {
 
   // Return true if 'node' is a managed association element
   // TODO: what about elements having a type, which (finally) is an assoc?
-  function isManagedAssociation(node) {
+  function isManagedAssociation( node ) {
     return node.target !== undefined && node.on === undefined && node.keys;
   }
 
@@ -313,7 +313,7 @@ function getUtils(model, universalReady) {
    * to any of them.
    * @param {string} typeName Absolute type name
    */
-  function isAssocOrComposition(typeName) {
+  function isAssocOrComposition( typeName ) {
     if (typeName === 'cds.Association' || typeName === 'cds.Composition')
       return true;
     const visited = createVisited(typeName);
@@ -331,7 +331,7 @@ function getUtils(model, universalReady) {
    * Returns if a type is an association or a typedef to it.
    * @param {string} typeName Absolute type name
    */
-  function isAssociation(typeName) {
+  function isAssociation( typeName ) {
     if (typeName === 'cds.Association')
       return true;
     const visited = createVisited(typeName);
@@ -349,7 +349,7 @@ function getUtils(model, universalReady) {
    * Returns if a type is an composition or a typedef to it.
    * @param {string} typeName Absolute type name
    */
-  function isComposition(typeName) {
+  function isComposition( typeName ) {
     if (typeName === 'cds.Composition')
       return true;
     const visited = createVisited(typeName);
@@ -367,7 +367,7 @@ function getUtils(model, universalReady) {
    * Return the namespace part of the artifact name.
    * @param {string} name Absolute name of artifact
    */
-  function getNamespaceOfArtifact(name) {
+  function getNamespaceOfArtifact( name ) {
     let lastDotIdx = name.lastIndexOf('.');
     if (lastDotIdx === -1)
       return undefined;
@@ -386,7 +386,7 @@ function getUtils(model, universalReady) {
    * Return the context part of the artifact name if any.
    * @param {string} name Absolute name of artifact
    */
-  function getContextOfArtifact(name) {
+  function getContextOfArtifact( name ) {
     let lastDotIdx = name.lastIndexOf('.');
     while (model.definitions[name]) {
       if (model.definitions[name].kind === 'context' || model.definitions[name].kind === 'service')
@@ -406,7 +406,7 @@ function getUtils(model, universalReady) {
    * @param {any} theValue string value of the annotation
    * @param {any} node Node to add the annotation to
    */
-  function addStringAnnotationTo(absoluteName, theValue, node) {
+  function addStringAnnotationTo( absoluteName, theValue, node ) {
     // Sanity check
     if (!absoluteName.startsWith('@'))
       throw Error(`Annotation name should start with "@": ${ absoluteName }`);
@@ -425,7 +425,7 @@ function getUtils(model, universalReady) {
    * @param {string} artifactName Absolute name of artifact
    * @returns {string|null}
    */
-  function getServiceName(artifactName) {
+  function getServiceName( artifactName ) {
     for (;;) {
       const idx = artifactName.lastIndexOf('.');
       if (idx === -1)
@@ -455,11 +455,11 @@ function getUtils(model, universalReady) {
    * @param {any} transformers Object defining transformer functions
    * @returns {object}
    */
-  function cloneWithTransformations(rootNode, transformers) {
+  function cloneWithTransformations( rootNode, transformers ) {
     return transformNode(rootNode);
 
     // This general transformation function will be applied to each node recursively
-    function transformNode(node) {
+    function transformNode( node ) {
     // Return primitive values and null unchanged, but let objects and dictionaries through
     // (Note that 'node instanceof Object' would be false for dictionaries).
       if (node === null || typeof node !== 'object')
@@ -488,7 +488,7 @@ function getUtils(model, universalReady) {
     }
   }
 
-  function _normalizeTypeRef(type) {
+  function _normalizeTypeRef( type ) {
     if (type && typeof type === 'object' && type.ref?.length === 1)
       type = type.ref[0]; // simplify type: no element -> simple string can be used
     return type;
@@ -512,7 +512,7 @@ function getUtils(model, universalReady) {
    * @param {string|object} type Type as string or type ref, i.e. `{ ref: [...] }`
    * @returns {object|null}
    */
-  function getFinalBaseTypeWithProps(type) {
+  function getFinalBaseTypeWithProps( type ) {
     type = _normalizeTypeRef(type);
     if (!type)
       return null;
@@ -566,7 +566,7 @@ function getUtils(model, universalReady) {
      *
      * @param {object} typeProps
      */
-    function _cacheResolved(typeProps) {
+    function _cacheResolved( typeProps ) {
       finalBaseTypeCache[resolvedKey] = typeProps;
       return typeProps;
     }
@@ -577,7 +577,7 @@ function getUtils(model, universalReady) {
      * @param obj
      * @returns {boolean} True, if structured/arrayed/invalid, false if scalar.
      */
-    function _cacheNonScalar(obj) {
+    function _cacheNonScalar( obj ) {
       if (!obj) { // Reference has no proper type, e.g. due to `type of View:calculated`.
         _cacheResolved(null);
         return true;
@@ -597,7 +597,7 @@ function getUtils(model, universalReady) {
      * @param {object} target
      * @param {object} source
      */
-    function _copyTypeProps(target, source) {
+    function _copyTypeProps( target, source ) {
       target.type = source.type;
       const typeProps = [ ...typeParameters.list, 'enum', 'default', 'localized' ];
       for (const param of typeProps) {
@@ -622,7 +622,7 @@ function getUtils(model, universalReady) {
  * @param {object} csn Top-level CSN.  You can pass non-dictionary values.
  * @param {CSN.Options} options CSN Options, only used for `dictionaryPrototype`, `testMode`, and `testSortCsn`
  */
-function cloneCsnNonDict(csn, options) {
+function cloneCsnNonDict( csn, options ) {
   return sortCsn(csn, options);
 }
 
@@ -641,7 +641,7 @@ function cloneCsnNonDict(csn, options) {
  * @param {CSN.Options} options Only cloneOptions.dictionaryPrototype is
  *                              used and cloneOptions are passed to sortCsn().
  */
-function cloneCsnDictionary(csn, options) {
+function cloneCsnDictionary( csn, options ) {
   return _cloneCsnDictionary(csn, options);
 }
 
@@ -652,7 +652,7 @@ function cloneCsnDictionary(csn, options) {
  * @param {any} value
  * @returns {any}
  */
-function cloneAnnotationValue(value) {
+function cloneAnnotationValue( value ) {
   return _cloneAnnotationValue( value );
 }
 
@@ -685,7 +685,7 @@ function forEachDefinition( csn, callback, iterateOptions = {} ) {
  * @param constructCallback
  */
 function forEachMember( construct, callback, path = [], ignoreIgnore = true, iterateOptions = {},
-                        constructCallback = (_construct, _prop, _path) => {}) {
+                        constructCallback = (_construct, _prop, _path) => {} ) {
   // Allow processing _ignored elements if requested
   if (ignoreIgnore && construct._ignore)
     return;
@@ -728,7 +728,7 @@ function forEachMember( construct, callback, path = [], ignoreIgnore = true, ite
  * @param {constructCallback|constructCallback[]} callback
  */
 function forEachMemberWithQuery( construct, callback, path = [], ignoreIgnore = true, iterateOptions = {},
-                                 constructCallback = (_construct, _prop, _path) => {}) {
+                                 constructCallback = (_construct, _prop, _path) => {} ) {
   forEachMember(construct, callback, path, ignoreIgnore, iterateOptions, constructCallback);
   if (construct.query) {
     forAllQueries(construct.query, (q, p) => {
@@ -751,7 +751,7 @@ function forEachMemberWithQuery( construct, callback, path = [], ignoreIgnore =
  * @param {constructCallback|constructCallback[]} callback
  */
 function forEachMemberRecursively( construct, callback, path = [], ignoreIgnore = true, iterateOptions = {},
-                                   constructCallback = (_construct, _prop, _path) => {}) {
+                                   constructCallback = (_construct, _prop, _path) => {} ) {
   forEachMember( construct, ( member, memberName, prop, subpath, parent ) => {
     if (Array.isArray(callback))
       callback.forEach(cb => cb( member, memberName, prop, subpath, parent ));
@@ -775,7 +775,7 @@ function forEachMemberRecursively( construct, callback, path = [], ignoreIgnore
  * @param {constructCallback|constructCallback[]} callback
  */
 function forEachMemberRecursivelyWithQuery( construct, callback, path = [], ignoreIgnore = true, iterateOptions = {},
-                                            constructCallback = (_construct, _prop, _path) => {}) {
+                                            constructCallback = (_construct, _prop, _path) => {} ) {
   forEachMemberRecursively(construct, callback, path, ignoreIgnore, iterateOptions, constructCallback);
   if (construct.query) {
     forAllQueries(construct.query, (q, p) => {
@@ -798,7 +798,7 @@ function forEachMemberRecursivelyWithQuery( construct, callback, path = [], igno
  * @param {CSN.Path} path
  * @param {object} iterateOptions can be used to skip certain kinds from being iterated
  */
-function forEachGeneric( construct, prop, callback, path = [], iterateOptions = {}) {
+function forEachGeneric( construct, prop, callback, path = [], iterateOptions = {} ) {
   const dict = construct[prop];
   for (const name in dict) {
     if (!Object.prototype.hasOwnProperty.call(dict, name))
@@ -810,7 +810,7 @@ function forEachGeneric( construct, prop, callback, path = [], iterateOptions =
       continue;
     executeCallbacks( dictObj, name );
   }
-  function executeCallbacks(o, name ) {
+  function executeCallbacks( o, name ) {
     const p = iterateOptions.pathWithoutProp ? [ name ] : [ prop, name ];
 
     if (Array.isArray(callback))
@@ -820,72 +820,66 @@ function forEachGeneric( construct, prop, callback, path = [], iterateOptions =
   }
 }
 
+const queryTraversalProperties = [ 'args', 'xpr', 'columns', 'where', 'having' ];
+
 /**
- * @param {CSN.Query} mainQuery
- * @param {queryCallback|queryCallback[]} queryCallback
+ * @param {CSN.Query} query
+ * @param {queryCallback} queryCallback
  * @param {CSN.Path} path
  */
-function forAllQueries(mainQuery, queryCallback, path = []) {
-  return traverseQuery(mainQuery, queryCallback, path);
-  function traverseQuery( query, callback, queryPath ) {
-    if (query.SELECT) {
-      // The projection is turned into a normalized query - there
-      // is no real SELECT, it is fake
-      if (!(path.length === 3 && path[2] === 'projection'))
-        queryPath.push('SELECT');
-      executeCallbacks();
-      query = query.SELECT;
-    }
-    else if (query.SET) {
-      queryPath.push('SET');
-      executeCallbacks();
-      query = query.SET;
-    }
+function forAllQueries( query, queryCallback, path ) {
+  if (query.SELECT) {
+    // The projection is turned into a normalized query - there
+    // is no real SELECT, it is fake
+    if (!(path.length === 3 && path[2] === 'projection'))
+      path.push('SELECT');
+    queryCallback( query, path );
+    query = query.SELECT;
+  }
+  else if (query.SET) {
+    path.push('SET');
+    queryCallback( query, path );
+    query = query.SET;
+  }
 
-    if (query.from)
-      traverseFrom( query.from, callback, queryPath.concat([ 'from' ]) );
+  if (query.from)
+    traverseFrom( query.from, queryCallback, [ ...path, 'from' ] );
 
-    for (const prop of [ 'args', 'xpr', 'columns', 'where', 'having' ]) {
-      // all properties which could have sub queries (directly or indirectly)
-      const expr = query[prop];
-      if (expr && typeof expr === 'object') {
-        if (Array.isArray(expr)) {
-          for (let i = 0; i < expr.length; i++)
-            traverseQuery(expr[i], callback, queryPath.concat([ prop, i ]));
-        }
-        else {
-          for (const argName of Object.keys( expr ))
-            traverseQuery(expr[argName], callback, queryPath.concat([ prop, argName ]));
-        }
+  for (const prop of queryTraversalProperties) {
+    // all properties which could have sub queries (directly or indirectly)
+    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 ]);
+      }
+      else {
+        for (const argName of Object.keys( expr ))
+          forAllQueries(expr[argName], queryCallback, [ ...path, prop, argName ]);
       }
-    }
-    function executeCallbacks() {
-      if (Array.isArray(callback))
-        callback.forEach(cb => cb( query, queryPath ));
-      else
-        callback( query, queryPath );
     }
   }
+}
 
-  /**
-   * @param {CSN.QueryFrom} from
-   * @param {Function} callback
-   * @param {CSN.Path} csnPath
-   */
-  function traverseFrom( from, callback, csnPath = [] ) {
-    if (from.ref) { // ignore
-      return;
-    }
-    else if (from.args) { // join
-      for (let i = 0; i < from.args.length; i++)
-        traverseFrom(from.args[i], callback, csnPath.concat([ 'args', i ]));
-    }
-    else {
-      traverseQuery( from, callback, csnPath );
-    } // sub query in FROM
+/**
+ * @param {CSN.QueryFrom} from
+ * @param {queryCallback} queryCallback
+ * @param {CSN.Path} path
+ */
+function traverseFrom( from, queryCallback, path = [] ) {
+  if (from.ref) {
+    // ignore
+  }
+  else if (from.args) { // join
+    for (let i = 0; i < from.args.length; i++)
+      traverseFrom(from.args[i], queryCallback, [ ...path, 'args', i ]);
+  }
+  else {
+    forAllQueries( from, queryCallback, path ); // sub query in FROM
   }
 }
 
+
 /**
  * Compare a given annotation value with an expectation value and return
  *
@@ -907,7 +901,7 @@ function forAllQueries(mainQuery, queryCallback, path = []) {
  * @param {boolean} caseInsensitive
  * @returns {boolean}
  */
-function hasAnnotationValue(artifact, annotationName, expected = true, caseInsensitive = false) {
+function hasAnnotationValue( artifact, annotationName, expected = true, caseInsensitive = false ) {
   if (expected === false)
     return artifact[annotationName] === expected || artifact[annotationName] === null;
   else if (typeof artifact[annotationName] === 'string' && caseInsensitive === true)
@@ -925,7 +919,7 @@ function hasAnnotationValue(artifact, annotationName, expected = true, caseInsen
  * @param {CSN.Element} elementCsn
  * @param {ODataOptions} options EDM specific options
  */
-function isEdmPropertyRendered(elementCsn, options) {
+function isEdmPropertyRendered( elementCsn, options ) {
   // FKs are rendered in
   // V2/V4 flat: always on
   // V4 struct: on/off
@@ -962,7 +956,7 @@ function isEdmPropertyRendered(elementCsn, options) {
  * @returns {string} The resulting database name for (absolute) 'artifactName', depending on the current naming mode.
  */
 // eslint-disable-next-line no-unused-vars
-function getArtifactDatabaseNameOf(artifactName, sqlMapping, csn, sqlDialect = 'plain') {
+function getArtifactDatabaseNameOf( artifactName, sqlMapping, csn, sqlDialect = 'plain' ) {
   if (csn && typeof csn === 'object' && csn.definitions) {
     isValidMappingDialectCombi(sqlDialect, sqlMapping);
     if (sqlMapping === 'quoted' || sqlMapping === 'hdbcds')
@@ -992,7 +986,7 @@ function getArtifactDatabaseNameOf(artifactName, sqlMapping, csn, sqlDialect = '
  * @param {string} artifactName Artifact name to use
  * @returns {string} The resulting name
  */
-function getResultingName(csn, namingMode, artifactName) {
+function getResultingName( csn, namingMode, artifactName ) {
   if (namingMode === 'plain' || artifactName.indexOf('.') === -1)
     return artifactName;
 
@@ -1026,7 +1020,7 @@ function getResultingName(csn, namingMode, artifactName) {
  * @param {CSN.Model} csn
  * @returns {string[]|null} Array of at most 2 strings: if both: [prefix, suffix], otherwise just one - or null
  */
-function getUnderscoredName(startIndex, parts, csn) {
+function getUnderscoredName( startIndex, parts, csn ) {
   for (let i = startIndex; i < parts.length; i++) {
     const namePart = parts.slice(0, i).join('.');
     const art = csn.definitions[namePart];
@@ -1058,7 +1052,7 @@ function getUnderscoredName(startIndex, parts, csn) {
   return null;
 }
 
-function isValidMappingDialectCombi(sqlDialect, sqlMapping) {
+function isValidMappingDialectCombi( sqlDialect, sqlMapping ) {
   if (sqlMapping === 'hdbcds' && sqlDialect !== 'hana')
     throw new Error(`sqlMapping "hdbcds" must only be used with sqlDialect "hana" - found: ${ sqlDialect }`);
   return true;
@@ -1082,7 +1076,7 @@ function isValidMappingDialectCombi(sqlDialect, sqlMapping) {
  * @returns {string} The resulting database element name for 'elemName', depending on the current naming mode.
  */
 // eslint-disable-next-line no-unused-vars
-function getElementDatabaseNameOf(elemName, sqlMapping, sqlDialect = 'plain') {
+function getElementDatabaseNameOf( elemName, sqlMapping, sqlDialect = 'plain' ) {
   isValidMappingDialectCombi(sqlDialect, sqlMapping);
   if (sqlMapping === 'hdbcds')
     return elemName;
@@ -1134,7 +1128,7 @@ function setDependencies( csn, refs = csnRefs(csn) ) {
     cleanup, csn, _dependents, _dependencies,
   };
 
-  function handleArgs(artifact, artifactName, args) {
+  function handleArgs( artifact, artifactName, args ) {
     for (const arg of args) {
       if (arg.args)
         handleArgs(artifact, artifactName, arg.args);
@@ -1143,20 +1137,20 @@ function setDependencies( csn, refs = csnRefs(csn) ) {
     }
   }
 
-  function handleDependency(dependency, dependant, dependantName) {
+  function handleDependency( dependency, dependant, dependantName ) {
     dependant[_dependencies].add(dependency);
     initDependents(dependency);
     dependency[_dependents][dependantName] = dependant;
   }
 
-  function initDependents(obj) {
+  function initDependents( obj ) {
     if (!obj[_dependents]) {
       obj[_dependents] = Object.create(null);
       cleanup.push(() => delete obj[_dependents]);
     }
   }
 
-  function initDependencies(obj) {
+  function initDependencies( obj ) {
     if (!obj[_dependencies]) {
       obj[_dependencies] = new Set();
       cleanup.push(() => delete obj[_dependencies]);
@@ -1171,7 +1165,7 @@ function setDependencies( csn, refs = csnRefs(csn) ) {
  * @param {CSN.Artifact} art
  * @returns {boolean}
  */
-function isPersistedOnDatabase(art) {
+function isPersistedOnDatabase( art ) {
   return !(art.kind === 'entity' && (art.abstract || hasAnnotationValue(art, '@cds.persistence.skip')));
 }
 /**
@@ -1180,7 +1174,7 @@ function isPersistedOnDatabase(art) {
  * @param {CSN.Artifact} artifact
  * @returns {boolean}
  */
-function isPersistedAsView(artifact) {
+function isPersistedAsView( artifact ) {
   return artifact && artifact.kind === 'entity' &&
       !artifact._ignore &&
       !artifact.abstract &&
@@ -1194,7 +1188,7 @@ function isPersistedAsView(artifact) {
  * @param {CSN.Artifact} artifact
  * @returns {boolean}
  */
-function isPersistedAsTable(artifact) {
+function isPersistedAsTable( artifact ) {
   return artifact.kind === 'entity' &&
       !artifact._ignore &&
       !artifact.abstract &&
@@ -1219,7 +1213,7 @@ function generatedByCompilerVersion() {
  * @param {CSN.Artifact} art Artifact with a query or a projection
  * @returns {object} Object with a query property.
  */
-function getNormalizedQuery(art) {
+function getNormalizedQuery( art ) {
   if (art.projection)
     return { query: { SELECT: art.projection } };
 
@@ -1234,7 +1228,7 @@ function getNormalizedQuery(art) {
 * @param {CSN.Model} csn
 * @returns {string} Name of the root
 */
-function getRootArtifactName(artifactName, csn) {
+function getRootArtifactName( artifactName, csn ) {
   const parts = artifactName.split('.');
 
   if (parts.length === 1)
@@ -1262,7 +1256,7 @@ function getRootArtifactName(artifactName, csn) {
 //   'foo.bar.wiz' => 'wiz'
 //   'foo' => 'foo';
 //   'foo::bar' => 'bar'
-function getLastPartOf(name) {
+function getLastPartOf( name ) {
   return name.substring(name.search(/[^.:]+$/));
 }
 
@@ -1272,7 +1266,7 @@ function getLastPartOf(name) {
 //   ['foo.bar.wiz'] => 'wiz'
 //   ['foo'] => 'foo';
 //   ['foo::bar'] => 'bar'
-function getLastPartOfRef(ref) {
+function getLastPartOfRef( ref ) {
   const lastPathStep = ref[ref.length - 1];
   return getLastPartOf(lastPathStep.id || lastPathStep);
 }
@@ -1286,7 +1280,7 @@ function getLastPartOfRef(ref) {
  * @param {object} toNode
  * @param {boolean} [overwrite]
  */
-function copyAnnotations(fromNode, toNode, overwrite = false) {
+function copyAnnotations( fromNode, toNode, overwrite = false ) {
   // Ignore if no toNode (in case of errors)
   if (!toNode)
     return;
@@ -1310,7 +1304,7 @@ function copyAnnotations(fromNode, toNode, overwrite = false) {
  * @param {object} toNode
  * @param {boolean} [overwrite]
  */
-function copyAnnotationsAndDoc(fromNode, toNode, overwrite = false) {
+function copyAnnotationsAndDoc( fromNode, toNode, overwrite = false ) {
   // Ignore if no toNode (in case of errors)
   if (!toNode)
     return;
@@ -1340,7 +1334,7 @@ function copyAnnotationsAndDoc(fromNode, toNode, overwrite = false) {
  *       filter:   Positive filter. If it returns true, annotations for the referenced artifact
  *                 will be applied.
  */
-function applyAnnotationsFromExtensions(csn, config) {
+function applyAnnotationsFromExtensions( csn, config ) {
   if (!csn.extensions)
     return;
 
@@ -1360,7 +1354,7 @@ function applyAnnotationsFromExtensions(csn, config) {
     }
   }
 
-  function applyAnnotationsToElements(ext, def) {
+  function applyAnnotationsToElements( ext, def ) {
     // Only the definition is arrayed but the extension is not since
     // `items` is not expected in `extensions` by the CSN frontend and not
     // generated by the CDL parser for `annotate E:arrayed.elem`.
@@ -1380,7 +1374,7 @@ function applyAnnotationsFromExtensions(csn, config) {
   }
 }
 
-function isAspect(node) {
+function isAspect( node ) {
   return node && node.kind === 'aspect';
 }
 
@@ -1390,7 +1384,7 @@ function isAspect(node) {
  * @param {CSN.Artifact} artifact
  * @returns {boolean}
  */
-function hasValidSkipOrExists(artifact) {
+function hasValidSkipOrExists( artifact ) {
   return artifact.kind === 'entity' &&
          (hasAnnotationValue(artifact, '@cds.persistence.exists', true) || hasAnnotationValue(artifact, '@cds.persistence.skip', true));
 }
@@ -1402,7 +1396,7 @@ function hasValidSkipOrExists(artifact) {
  * @param {string} artifactName artifact name to get the namespace for
  * @returns {string | null} The namespace name
  */
-function getNamespace(csn, artifactName) {
+function getNamespace( csn, artifactName ) {
   const parts = artifactName.split('.');
   let seen = parts[0];
   const art = csn.definitions[seen];
@@ -1432,7 +1426,7 @@ function getNamespace(csn, artifactName) {
  * @param {CSN.Model} csn
  * @param {CSN.Options} options
  */
-function sortCsnDefinitionsForTests(csn, options) {
+function sortCsnDefinitionsForTests( csn, options ) {
   if (!options.testMode && !options.testSortCsn)
     return;
   const sorted = Object.create(null);
@@ -1448,7 +1442,7 @@ function sortCsnDefinitionsForTests(csn, options) {
  * @param {CSN.Model} csn
  * @returns {string[]}
  */
-function getServiceNames(csn) {
+function getServiceNames( csn ) {
   const result = [];
   forEachDefinition(csn, (artifact, artifactName) => {
     if (artifact.kind === 'service' && !artifact.abstract)
@@ -1464,7 +1458,7 @@ function getServiceNames(csn) {
  * @param {CSN.Path} path
  * @returns {object} Whatever is at the end of path
  */
-function walkCsnPath(csn, path) {
+function walkCsnPath( csn, path ) {
   /** @type {object} */
   let obj = csn;
   for (const segment of path)
@@ -1482,7 +1476,7 @@ function walkCsnPath(csn, path) {
  * @param {CSN.Options} options
  * @returns {string|null}
  */
-function getVariableReplacement(ref, options) {
+function getVariableReplacement( ref, options ) {
   if (options && options.variableReplacements) {
     let replacement = options.variableReplacements;
     for (const segment of ref) {
@@ -1511,7 +1505,7 @@ function getVariableReplacement(ref, options) {
  * the comparison. This eliminates false negatives such as
  *  mismatching $locations or @odata.foreignKey4.
  */
-function isDeepEqual(obj, other, noExtendedProps) {
+function isDeepEqual( obj, other, noExtendedProps ) {
   let objectKeys = Object.keys(obj);
   let otherKeys = Object.keys(other);
 
@@ -1537,6 +1531,19 @@ function isDeepEqual(obj, other, noExtendedProps) {
   return true;
 }
 
+/**
+ * Returns a function that, if called, calls all functions inside
+ * the given `functions` array with the same arguments.
+ *
+ * @param {Function[]} functions
+ * @param {object} thisArg Argument that will be passed to all functions as `this`.
+ */
+function functionList( functions, thisArg ) {
+  return function iterateFunctions(...args) {
+    return functions.map(f => f.apply(thisArg, args));
+  };
+}
+
 module.exports = {
   getUtils,
   cloneCsn: cloneCsnNonDict, // Umbrella relies on this name
@@ -1581,4 +1588,5 @@ module.exports = {
   getVariableReplacement,
   implicitAs,
   isDeepEqual,
+  functionList,
 };