@sap/cds-compiler 3.4.4 → 3.5.2

package/lib/transform/db/flattening.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const {
-  getUtils, walkCsnPath,
+  getUtils,
   applyTransformations, applyTransformationsOnNonDictionary,
   isBuiltinType, cloneCsnNonDict,
   copyAnnotations, implicitAs, isDeepEqual,
@@ -16,7 +16,7 @@ const { forEach } = require('../../utils/objectUtils');
  *
  * @param {CSN.Model} csn
  */
-function removeLeadingSelf(csn) {
+function removeLeadingSelf( csn ) {
   const magicVars = [ '$now', '$self', '$projection', '$user', '$session', '$at' ];
   applyTransformations(csn, {
     elements: (parent, prop, elements) => {
@@ -46,7 +46,7 @@ function removeLeadingSelf(csn) {
  * @param {string} pathDelimiter
  * @param {object} iterateOptions
  */
-function resolveTypeReferences(csn, options, resolved, pathDelimiter, iterateOptions = {}) {
+function resolveTypeReferences( csn, options, resolved, pathDelimiter, iterateOptions = {} ) {
   /**
    * Remove .localized from the element and any sub-elements
    *
@@ -55,7 +55,7 @@ function resolveTypeReferences(csn, options, resolved, pathDelimiter, iterateOpt
    *
    * @param {object} obj
    */
-  function removeLocalized(obj) {
+  function removeLocalized( obj ) {
     const stack = [ obj ];
     while (stack.length > 0) {
       const current = stack.pop();
@@ -151,7 +151,7 @@ function resolveTypeReferences(csn, options, resolved, pathDelimiter, iterateOpt
    * @param {CSN.Path} path
    * @returns {boolean}
    */
-  function isODataV4BuiltinFromService(typeName, path) {
+  function isODataV4BuiltinFromService( typeName, path ) {
     if (!options.toOdata || (options.odataVersion === 'v2') || typeof typeName !== 'string')
       return false;
 
@@ -170,7 +170,7 @@ function resolveTypeReferences(csn, options, resolved, pathDelimiter, iterateOpt
    * @param {string} typeName
    * @returns {boolean}
    */
-  function isODataItems(typeName) {
+  function isODataItems( typeName ) {
     const typeDef = csn.definitions[typeName];
     return !!(options.toOdata && typeDef && typeDef.items);
   }
@@ -183,7 +183,7 @@ function resolveTypeReferences(csn, options, resolved, pathDelimiter, iterateOpt
  * @param {string} pathDelimiter
  * @param {object} iterateOptions
  */
-function flattenAllStructStepsInRefs(csn, options, resolved, pathDelimiter, iterateOptions = {}) {
+function flattenAllStructStepsInRefs( csn, options, resolved, pathDelimiter, iterateOptions = {} ) {
   const { inspectRef, effectiveType } = csnRefs(csn);
   const { flattenStructStepsInRef } = transformUtils.getTransformers(csn, options, pathDelimiter);
   const adaptRefs = [];
@@ -196,7 +196,7 @@ function flattenAllStructStepsInRefs(csn, options, resolved, pathDelimiter, iter
    * @todo seems too hacky
    * @returns {WeakMap} A WeakMap where a link is the key and the type is the value
    */
-  function resolveLinkTypes(links = []) {
+  function resolveLinkTypes( links = [] ) {
     const resolvedLinkTypes = new WeakMap();
     links.forEach((link) => {
       const { art } = link;
@@ -244,7 +244,7 @@ function flattenAllStructStepsInRefs(csn, options, resolved, pathDelimiter, iter
    * @param {CSN.Path} path
    * @returns {boolean}
    */
-  function insideColumns(path) {
+  function insideColumns( path ) {
     return path.length >= 3 && (path[path.length - 3] === 'SELECT' || path[path.length - 3] === 'projection') && path[path.length - 2] === 'columns';
   }
   /**
@@ -253,7 +253,7 @@ function flattenAllStructStepsInRefs(csn, options, resolved, pathDelimiter, iter
    * @param {CSN.Path} path
    * @returns {boolean}
    */
-  function insideKeys(path) {
+  function insideKeys( path ) {
     return path.length >= 3 && path[path.length - 2] === 'keys' && typeof path[path.length - 1] === 'number';
   }
 }
@@ -265,7 +265,7 @@ function flattenAllStructStepsInRefs(csn, options, resolved, pathDelimiter, iter
  * @param {Function} error
  * @param {object} iterateOptions
  */
-function flattenElements(csn, options, pathDelimiter, error, iterateOptions = {}) {
+function flattenElements( csn, options, pathDelimiter, error, iterateOptions = {} ) {
   const { flattenStructuredElement, csnUtils } = transformUtils.getTransformers(csn, options, pathDelimiter);
   const { isAssocOrComposition, effectiveType } = csnUtils;
   const transformers = {
@@ -285,7 +285,7 @@ function flattenElements(csn, options, pathDelimiter, error, iterateOptions = {}
    * @param {object} dict
    * @param {CSN.Path} path
    */
-  function flatten(parent, prop, dict, path) {
+  function flatten( parent, prop, dict, path ) {
     if (!parent[prop].$orderedElements)
       setProp(parent[prop], '$orderedElements', []);
     forEach(dict, (elementName, element) => {
@@ -367,7 +367,7 @@ function flattenElements(csn, options, pathDelimiter, error, iterateOptions = {}
    * @param {string} elementName Name of the structured element
    * @returns {object} Returns a dictionary, where the key is the flat name of the branch and the value is an array of element-steps.
    */
-  function getBranches(element, elementName) {
+  function getBranches( element, elementName ) {
     const branches = {};
     const subbranchNames = [];
     const subbranchElements = [];
@@ -378,7 +378,7 @@ function flattenElements(csn, options, pathDelimiter, error, iterateOptions = {}
      * @param {object} e
      * @param {string} name
      */
-    function walkElements(e, name) {
+    function walkElements( e, name ) {
       if (isBuiltinType(e.type)) {
         branches[subbranchNames.concat(name).join(pathDelimiter)] = subbranchElements.concat(e);
       }
@@ -411,7 +411,7 @@ function flattenElements(csn, options, pathDelimiter, error, iterateOptions = {}
  * @param {boolean} flattenKeyRefs
  * @param {object} iterateOptions
  */
-function handleManagedAssociationsAndCreateForeignKeys(csn, options, error, pathDelimiter, flattenKeyRefs, iterateOptions = {}) {
+function handleManagedAssociationsAndCreateForeignKeys( csn, options, error, pathDelimiter, flattenKeyRefs, iterateOptions = {} ) {
   const { isManagedAssociation, inspectRef, isStructured } = getUtils(csn);
   const { flattenStructStepsInRef, flattenStructuredElement } = transformUtils.getTransformers(csn, options, pathDelimiter);
   if (flattenKeyRefs) {
@@ -445,7 +445,7 @@ function handleManagedAssociationsAndCreateForeignKeys(csn, options, error, path
    * @param {*} assocName
    * @param {*} path
    */
-  function flattenFKs(assoc, assocName, path) {
+  function flattenFKs( assoc, assocName, path ) {
     let finished = false;
     while (!finished) {
       const newKeys = [];
@@ -460,7 +460,7 @@ function handleManagedAssociationsAndCreateForeignKeys(csn, options, error, path
      * @param {object[]} collector New keys array to collect the flattened stuff in
      * @returns {boolean} True if all keys are scalar - false if there are things that still need to be processed.
      */
-    function processKeys(collector) {
+    function processKeys( collector ) {
       const inferredAlias = '$inferredAlias';
 
       let done = true;
@@ -538,7 +538,7 @@ function handleManagedAssociationsAndCreateForeignKeys(csn, options, error, path
      * @param {Array} ref
      * @returns {object} The clone of base
      */
-    function cloneAndExtendRef(key, base, ref) {
+    function cloneAndExtendRef( key, base, ref ) {
       const clone = cloneCsnNonDict(base, options);
       if (key.ref) {
         // We build a ref that contains the aliased fk - that element will be created later on, so this ref is not resolvable yet
@@ -590,12 +590,12 @@ function handleManagedAssociationsAndCreateForeignKeys(csn, options, error, path
      * @param {object} dict The params or elements thing
      * @param {CSN.Path} path
      */
-    function createFks(parent, prop, dict, path) {
+    function createFks( parent, prop, dict, path ) {
       const orderedElements = [];
       Object.entries(dict).forEach(([ elementName, element ]) => {
         orderedElements.push([ elementName, element ]);
         const eltPath = path.concat(prop, elementName);
-        const fks = createForeignKeysInternal(eltPath, element, elementName, csn, options, pathDelimiter);
+        const fks = createForeignKeys(eltPath, element, elementName, csn, options, pathDelimiter);
 
         // finalize the generated foreign keys
         const refCount = fks.reduce((acc, fk) => {
@@ -671,13 +671,12 @@ function handleManagedAssociationsAndCreateForeignKeys(csn, options, error, path
  * @param {number} lvl
  * @returns {Array[]} First element of every sub-array is the foreign key name, second is the foreign key definition
  */
-function createForeignKeysInternal(path, element, prefix, csn, options, pathDelimiter, lvl = 0) {
+function createForeignKeys( path, element, prefix, csn, options, pathDelimiter, lvl = 0 ) {
   const {
     effectiveType,
     inspectRef,
   } = getUtils(csn);
 
-
   const isInspectRefResult = !Array.isArray(path);
 
   let fks = [];
@@ -719,7 +718,7 @@ function createForeignKeysInternal(path, element, prefix, csn, options, pathDeli
       const continuePath = getContinuePath([ 'keys', keyIndex ]);
       const alias = key.as || implicitAs(key.ref);
       const result = inspectRef(continuePath);
-      fks = fks.concat(createForeignKeysInternal(result, result.art, alias, csn, options, pathDelimiter, lvl + 1));
+      fks = fks.concat(createForeignKeys(result, result.art, alias, csn, options, pathDelimiter, lvl + 1));
     });
     if (!hasKeys)
       delete finalElement.keys;
@@ -743,7 +742,7 @@ function createForeignKeysInternal(path, element, prefix, csn, options, pathDeli
       // Skip already produced foreign keys
       if (!elem['@odata.foreignKey4']) {
         const continuePath = getContinuePath([ 'elements', elemName ]);
-        fks = fks.concat(createForeignKeysInternal(continuePath, elem, elemName, csn, options, pathDelimiter, lvl + 1));
+        fks = fks.concat(createForeignKeys(continuePath, elem, elemName, csn, options, pathDelimiter, lvl + 1));
       }
     });
   }
@@ -759,7 +758,7 @@ function createForeignKeysInternal(path, element, prefix, csn, options, pathDeli
    * @param {Array} additions
    * @returns {CSN.Path}
    */
-  function getContinuePath(additions) {
+  function getContinuePath( additions ) {
     if (csn.definitions[finalElement.type])
       return [ 'definitions', finalElement.type, ...additions ];
     else if (finalTypeName)
@@ -790,27 +789,10 @@ function createForeignKeysInternal(path, element, prefix, csn, options, pathDeli
   return fks;
 }
 
-/**
- * This is the public createForeignKeys function that has no side effects
- *
- * @param {CSN.Path} path Path to the managed association
- * @param {CSN.Model} csn
- * @param {string} pathDelimiter
- * @returns {Array}
- */
-function createForeignKeys(path, csn, pathDelimiter = '_') {
-  if (!path || !Array.isArray(path))
-    throw Error('path must be a CSN path array');
-  if (!csn || typeof csn !== 'object')
-    throw Error('csn not provided');
-  return createForeignKeysInternal(path, walkCsnPath(csn, path), path[path.length - 1], csn, {}, pathDelimiter);
-}
-
 module.exports = {
   resolveTypeReferences,
   flattenAllStructStepsInRefs,
   flattenElements,
   removeLeadingSelf,
   handleManagedAssociationsAndCreateForeignKeys,
-  createForeignKeys,
 };

package/lib/transform/db/groupByOrderBy.js

@@ -13,7 +13,7 @@ const { ModelError } = require('../../base/error');
  * @param {Function} error
  * @param {CSN.Path} path
  */
-function replaceAssociationsInGroupByOrderBy(inputQuery, options, inspectRef, error, path) {
+function replaceAssociationsInGroupByOrderBy( inputQuery, options, inspectRef, error, path ) {
   const query = inputQuery.SET || inputQuery.SELECT;
 
   if (query.groupBy) {
@@ -29,7 +29,7 @@ function replaceAssociationsInGroupByOrderBy(inputQuery, options, inspectRef, er
             if (options.transformation === 'hdbcds' && options.sqlMapping === 'hdbcds') {
               error(null, groupByPath,
                     { $reviewed: true },
-                    'Unexpected managed association in GROUP BY for naming mode “hdbcds“');
+                    'Unexpected managed association in GROUP BY for naming mode “hdbcds”');
               continue;
             }
             const pathPrefix = query.groupBy[i].ref.slice(0, -1);
@@ -88,7 +88,7 @@ function replaceAssociationsInGroupByOrderBy(inputQuery, options, inspectRef, er
  * @param {CSN.Element} assoc
  * @returns {object[]}
  */
-function getForeignKeyRefs(assoc) {
+function getForeignKeyRefs( assoc ) {
   return assoc.keys.map((fk) => {
     if (!fk.$generatedFieldName)
       throw new ModelError(`Expecting generated field name for foreign key: ${JSON.stringify(fk)}`);

package/lib/transform/db/temporal.js

@@ -22,7 +22,7 @@ const validFromString = '@cds.valid.from';
  * @param {Function} messageFunctions.info
  * @returns {(artifact: CSN.Artifact, artifactName: string) => void} Callback for forEachDefinition applying the where-condition to views.
  */
-function getViewDecorator(csn, messageFunctions) {
+function getViewDecorator( csn, messageFunctions ) {
   const { info } = messageFunctions;
   const { get$combined } = getUtils(csn);
   return addTemporalWhereConditionToView;
@@ -37,7 +37,7 @@ function getViewDecorator(csn, messageFunctions) {
    * @param {CSN.Artifact} artifact
    * @param {string} artifactName
    */
-  function addTemporalWhereConditionToView(artifact, artifactName) {
+  function addTemporalWhereConditionToView( artifact, artifactName ) {
     const normalizedQuery = getNormalizedQuery(artifact);
     if (normalizedQuery && normalizedQuery.query && normalizedQuery.query.SELECT) {
       // BLOCKER: We need information to handle $combined
@@ -101,7 +101,7 @@ function getViewDecorator(csn, messageFunctions) {
    * @param {any} combined union of all entities of the from-clause
    * @returns {Array[]} Array where first field is array of elements with @cds.valid.from, second field is array of elements with @cds.valid.to.
    */
-  function getFromToElements(combined) {
+  function getFromToElements( combined ) {
     const from = [];
     const to = [];
     for (const name in combined) {
@@ -129,7 +129,7 @@ function getViewDecorator(csn, messageFunctions) {
    * @param {object} to
    * @returns {boolean} True if both are present and false.
    */
-  function hasFalsyTemporalAnnotations(SELECT, elements, from, to) {
+  function hasFalsyTemporalAnnotations( SELECT, elements, from, to ) {
     let fromElement = elements[from.name];
     let toElement = elements[to.name];
 
@@ -166,7 +166,7 @@ function getViewDecorator(csn, messageFunctions) {
  * @param {Function} messageFunctions.error
  * @returns {(artifact: CSN.Artifact, artifactName: string) => void} Callback for forEachDefinition processing the annotations.
  */
-function getAnnotationHandler(csn, options, pathDelimiter, messageFunctions) {
+function getAnnotationHandler( csn, options, pathDelimiter, messageFunctions ) {
   const { error } = messageFunctions;
   const {
     extractValidFromToKeyElement, checkAssignment, checkMultipleAssignments, recurseElements,
@@ -177,7 +177,7 @@ function getAnnotationHandler(csn, options, pathDelimiter, messageFunctions) {
    * @param {CSN.Artifact} artifact
    * @param {string} artifactName
    */
-  function handleTemporalAnnotations(artifact, artifactName) {
+  function handleTemporalAnnotations( artifact, artifactName ) {
     const validFrom = [];
     const validTo = [];
     const validKey = [];

package/lib/transform/db/transformExists.js

@@ -48,7 +48,7 @@ const { ModelError } = require('../../base/error');
  * @param {CSN.Options} options
  * @param {Function} error
  */
-function handleExists(csn, options, error) {
+function handleExists( csn, options, error ) {
   let { inspectRef } = csnRefs(csn);
   const generatedExists = new WeakMap();
   forEachDefinition(csn, (artifact, artifactName) => {
@@ -103,7 +103,7 @@ function handleExists(csn, options, error) {
    * @param {Array} args Join args
    * @returns {object}
    */
-  function getJoinSources(args) {
+  function getJoinSources( args ) {
     let sources = Object.create(null);
     for (const join of args) {
       if (join.as) {
@@ -127,7 +127,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Query} query
    * @returns {object}
    */
-  function getQuerySources(query) {
+  function getQuerySources( query ) {
     const sources = Object.create(null);
     if (query.from.as)
       sources[query.from.as] = query.from.as;
@@ -147,7 +147,7 @@ function handleExists(csn, options, error) {
    * @param {object[]} links links for a ref, produced by inspectRef
    * @returns {number|null} Null if no association was found
    */
-  function getFirstAssocIndex(startIndex, links) {
+  function getFirstAssocIndex( startIndex, links ) {
     for (let i = startIndex; i < links.length; i++) {
       if (links[i] && links[i].art && links[i].art.target)
         return i;
@@ -174,7 +174,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} path to the overall ref where _main is contained
    * @returns {Array} Return the now-nested ref-array
    */
-  function nestFilters(startIndex, startAssoc, startRest, path) {
+  function nestFilters( startIndex, startAssoc, startRest, path ) {
     let revert;
     if (!startAssoc.where) { // initialize first filter if not present
       if (typeof startAssoc === 'string') {
@@ -253,7 +253,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} exprPath
    * @returns {void}
    */
-  function forbidAssocInExists(exprPath) {
+  function forbidAssocInExists( exprPath ) {
     const expr = walkCsnPath(csn, exprPath);
     for (let i = 0; i < expr.length; i++) {
       if (i < expr.length - 1 && expr[i] === 'exists' && expr[i + 1].ref) {
@@ -272,7 +272,7 @@ function handleExists(csn, options, error) {
   /**
    * @param {object[]} assocs Array of refs of assocs - possibly with a .where to check
    */
-  function checkForInvalidAssoc(assocs) {
+  function checkForInvalidAssoc( assocs ) {
     for (const assoc of assocs) {
       if (assoc.where) {
         for (let i = 0; i < assoc.where.length; i++) {
@@ -311,7 +311,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} exprPath
    * @returns {Array}
    */
-  function nestExists(exprPath) {
+  function nestExists( exprPath ) {
     const expr = walkCsnPath(csn, exprPath);
     for (let i = 0; i < expr.length; i++) {
       if (i < expr.length - 1 && expr[i] === 'exists' && expr[i + 1].ref) {
@@ -344,7 +344,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} exprPath Path to the expression-array to process
    * @returns {{result: TokenStream, leftovers: Array[]}} result: A new token stream expression - the same as expr, but with the expanded EXISTS, leftovers: path-tuples to further subqueries to process.
    */
-  function processExists(queryPath, exprPath) {
+  function processExists( queryPath, exprPath ) {
     const toContinue = [];
     const newExpr = [];
     const query = walkCsnPath(csn, queryPath);
@@ -427,7 +427,7 @@ function handleExists(csn, options, error) {
    * @param {Token} current
    * @returns {object[]} The stuff to add to the where
    */
-  function translateManagedAssocToWhere(root, target, isPrefixedWithTableAlias, base, current) {
+  function translateManagedAssocToWhere( root, target, isPrefixedWithTableAlias, base, current ) {
     if (current.$scope === '$self') {
       error('ref-unexpected-exists-self', current.$path, { id: current.ref[0], name: 'exists' }, 'With $(NAME), path steps must not start with $(ID)');
       return [];
@@ -453,7 +453,7 @@ function handleExists(csn, options, error) {
    * @param {Array} ref Array of strings or objects with `id`
    * @returns {string[]}
    */
-  function toRawRef(ref) {
+  function toRawRef( ref ) {
     return ref.map(r => (r.id ? r.id : r));
   }
 
@@ -475,7 +475,7 @@ function handleExists(csn, options, error) {
    * @param {Token} current
    * @returns {object[]} The stuff to add to the where
    */
-  function translateUnmanagedAssocToWhere(root, target, isPrefixedWithTableAlias, base, current) {
+  function translateUnmanagedAssocToWhere( root, target, isPrefixedWithTableAlias, base, current ) {
     const whereExtension = [];
     for (let j = 0; j < root.on.length; j++) {
       const part = root.on[j];
@@ -532,7 +532,7 @@ function handleExists(csn, options, error) {
      * @param {...any} args
      * @returns {object} The merged args without an .as property
      */
-    function assignAndDeleteAs(...args) {
+    function assignAndDeleteAs( ...args ) {
       const obj = Object.assign.apply(null, args);
       delete obj.as;
       return obj;
@@ -546,7 +546,7 @@ function handleExists(csn, options, error) {
      * @param {object} obj
      * @returns {object}
      */
-    function translateToSourceSide(obj) {
+    function translateToSourceSide( obj ) {
       if (obj.ref) {
         if (obj.$scope === '$self') { // TODO: Check with this way down, do we keep the links?
           const column = obj._art._column;
@@ -580,7 +580,7 @@ function handleExists(csn, options, error) {
      * @param {CSN.Path} pathRight
      * @returns {boolean}
      */
-    function isValidDollarSelf(leftSide, pathLeft, middle, rightSide, pathRight) {
+    function isValidDollarSelf( leftSide, pathLeft, middle, rightSide, pathRight ) {
       if (leftSide && leftSide.ref && rightSide && rightSide.ref && middle === '=') {
         const right = inspectRef(pathRight);
         const left = inspectRef(pathLeft);
@@ -606,7 +606,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} path
    * @returns {{head: Array, root: CSN.Element, ref: string|object, tail: Array}} The first assoc (root), the corresponding ref (ref), anything before the ref (head) and the rest of the ref (tail).
    */
-  function getFirstAssoc(xprPart, path) {
+  function getFirstAssoc( xprPart, path ) {
     const { links, art } = inspectRef(path);
     for (let i = 0; i < xprPart.ref.length - 1; i++) {
       if (links[i].art && links[i].art.target) {
@@ -627,7 +627,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} path
    * @returns {{head: Array, root: CSN.Element, ref: string|object, tail: Array}} The last assoc (root), the corresponding ref (ref), anything before the ref (head) and the rest of the ref (tail).
    */
-  function getLastAssoc(xprPart, path) {
+  function getLastAssoc( xprPart, path ) {
     const { links, art } = inspectRef(path);
     for (let i = xprPart.ref.length - 1; i > -1; i--) {
       if (links[i].art && links[i].art.target) {
@@ -647,7 +647,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} path
    * @returns {boolean}
    */
-  function firstLinkIsEntityOrQuerySource(path) {
+  function firstLinkIsEntityOrQuerySource( path ) {
     const { links } = inspectRef(path);
     return links && (links[0].art.kind === 'entity' || links[0].art.query || links[0].art.from);
   }
@@ -675,7 +675,7 @@ function handleExists(csn, options, error) {
    * @todo $env is going to be removed from CSN, but csnRefs will provide it
    */
   // eslint-disable-next-line consistent-return
-  function getParent(xpr, path) {
+  function getParent( xpr, path ) {
     if (firstLinkIsEntityOrQuerySource(path)) {
       return xpr.ref[0];
     }
@@ -704,7 +704,7 @@ function handleExists(csn, options, error) {
    * @param {object} _sources Object containing the names of the query sources of the current query
    * @returns {CSN.Query}
    */
-  function getSubselect(target, assocRef, _sources) {
+  function getSubselect( target, assocRef, _sources ) {
     let subselectAlias = `_${assocRef.id ? assocRef.id : assocRef}_exists`;
 
     while (_sources[subselectAlias])
@@ -741,7 +741,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} path
    * @returns {string}
    */
-  function getBase(queryBase, isPrefixedWithTableAlias, current, path) {
+  function getBase( queryBase, isPrefixedWithTableAlias, current, path ) {
     if (typeof queryBase === 'string') // alias
       return queryBase;
     else if (queryBase) // ref
@@ -761,7 +761,7 @@ function handleExists(csn, options, error) {
    * @param {TokenStream} where
    * @returns {TokenStream} The input-where with the refs transformed to absolute ones
    */
-  function remapExistingWhere(target, where) {
+  function remapExistingWhere( target, where ) {
     return where.map((part) => {
       if (part.ref) {
         part.ref = [ target, ...part.ref ];
@@ -783,7 +783,7 @@ function handleExists(csn, options, error) {
    * @param {CSN.Path} path
    * @returns {TokenStream} The WHERE representing the $self comparison
    */
-  function translateDollarSelfToWhere(base, target, assoc, path) {
+  function translateDollarSelfToWhere( base, target, assoc, path ) {
     const where = [];
     const { art } = inspectRef(path);
     if (art.keys) {

package/lib/transform/db/views.js

@@ -15,7 +15,7 @@ const { ModelError } = require('../../base/error');
  * @param {string} associationName
  * @returns {object} The mixin association
  */
-function getMixinAssocOfQueryIfPublished(query, association, associationName) {
+function getMixinAssocOfQueryIfPublished( query, association, associationName ) {
   if (query && query.SELECT && query.SELECT.mixin) {
     const aliasedColumnsMap = Object.create(null);
     if (query.SELECT.columns) {
@@ -49,7 +49,7 @@ function getMixinAssocOfQueryIfPublished(query, association, associationName) {
  * @param {string} associationName
  * @returns {boolean} True if used
  */
-function usesMixinAssociation(query, association, associationName) {
+function usesMixinAssociation( query, association, associationName ) {
   if (query && query.SELECT && query.SELECT.columns) {
     for (const column of query.SELECT.columns) {
       if (typeof column === 'object' && column.ref && column.ref.length > 1 && (column.ref[0] === associationName || column.ref[0].id === associationName))
@@ -66,7 +66,7 @@ function usesMixinAssociation(query, association, associationName) {
  * @param {Function} transformCommon For the time being: Pass from outside
  * @returns {(query: CSN.Query, artifact: CSN.Artifact, artName: string, path: CSN.Path) => void} Transformer function for views
  */
-function getViewTransformer(csn, options, messageFunctions, transformCommon) {
+function getViewTransformer( csn, options, messageFunctions, transformCommon ) {
   const {
     get$combined, isAssocOrComposition,
     inspectRef, queryOrMain, // csnRefs
@@ -85,7 +85,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    * @param {CSN.Elements} elements
    * @param {CSN.Path} path
    */
-  function checkForMixinPublishing(query, elements, path) {
+  function checkForMixinPublishing( query, elements, path ) {
     for (const elementName in elements) {
       const element = elements[elementName];
       if (element.target) {
@@ -123,7 +123,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    * @param {CSN.Query} query
    * @returns {object}
    */
-  function getColumnMap(query) {
+  function getColumnMap( query ) {
     const map = Object.create(null);
     if (query && query.SELECT && query.SELECT.columns) {
       query.SELECT.columns.forEach((col) => {
@@ -169,7 +169,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    * @param {object} columnMap
    * @param {string} elemName
    */
-  function addProjectionOrStarElement(query, isProjection, isSelectStar, $combined, columnMap, elemName) {
+  function addProjectionOrStarElement( query, isProjection, isSelectStar, $combined, columnMap, elemName ) {
     // Prepend an alias if present
     let alias = (isProjection || isSelectStar) &&
     (query.SELECT.from.as || (query.SELECT.from.ref && implicitAs(query.SELECT.from.ref)));
@@ -205,7 +205,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    * @param {CSN.Element} elem
    * @param {string} elemName
    */
-  function addForeignKeysToColumns(columnMap, elem, elemName) {
+  function addForeignKeysToColumns( columnMap, elem, elemName ) {
     const assocCol = columnMap[elemName];
     if (assocCol && assocCol.ref) {
       elem.keys.forEach((foreignKey) => {
@@ -243,7 +243,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    * @param {CSN.Path} elementsPath Path pointing to elements
    * @param {CSN.Path} queryPath Path pointing to the query
    */
-  function handleAssociationElement(query, elements, columnMap, publishedMixins, elem, elemName, elementsPath, queryPath) {
+  function handleAssociationElement( query, elements, columnMap, publishedMixins, elem, elemName, elementsPath, queryPath ) {
     if (isUnion(queryPath) && options.transformation === 'hdbcds') {
       if (isBetaEnabled(options, 'ignoreAssocPublishingInUnion') && doA2J) {
         if (elem.keys)
@@ -328,7 +328,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    * @param {CSN.Column} col
    * @param {CSN.Path} path
    */
-  function addImplicitAliasWithAssoc(col, path) {
+  function addImplicitAliasWithAssoc( col, path ) {
     if (!col.as && col.ref && col.ref.length > 1) {
       const { links } = inspectRef(path);
       if (links && links.slice(0, -1).some(({ art }) => isAssocOrComposition(art && art.type || '')))
@@ -342,7 +342,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    *
    * @param {CSN.Column} col
    */
-  function addImplicitAliasWithLonelyParam(col) {
+  function addImplicitAliasWithLonelyParam( col ) {
     if (!col.as && col.param)
       col.as = getLastRefStepString(col.ref);
   }
@@ -355,7 +355,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    * @param {CSN.Column[]} columns
    * @param {CSN.Path} path
    */
-  function processColumns(functions, columns, path) {
+  function processColumns( functions, columns, path ) {
     for (let i = 0; i < columns.length; i++) {
       const col = columns[i];
       functions.forEach(fn => fn(col, path.concat(i)));
@@ -369,7 +369,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
    * @param {CSN.Path} path
    */
   // eslint-disable-next-line complexity
-  function transformViewOrEntity(query, artifact, artName, path) {
+  function transformViewOrEntity( query, artifact, artName, path ) {
     const { elements } = queryOrMain(query, artifact);
     // We use the elements from the leading query/main artifact - adapt the path
     const elementsPath = elements === artifact.elements ? path.slice(0, 2).concat('elements') : path.concat('elements');
@@ -453,7 +453,7 @@ function getViewTransformer(csn, options, messageFunctions, transformCommon) {
  * @param {CSN.Path} path
  * @returns {boolean}
  */
-function isUnion(path) {
+function isUnion( path ) {
   const subquery = path[path.length - 1];
   const queryIndex = path[path.length - 2];
   const args = path[path.length - 3];
@@ -467,7 +467,7 @@ function isUnion(path) {
  * @param {object} col A column
  * @returns {object}
  */
-function stripLeadingSelf(col) {
+function stripLeadingSelf( col ) {
   if (col.ref && col.ref.length > 1 && col.ref[0] === '$self')
     col.ref = col.ref.slice(1);
 
@@ -482,7 +482,7 @@ function stripLeadingSelf(col) {
  * @param {string} elementName
  * @returns {boolean}
  */
-function checkIsNotMixinByItself(query, columnMap, elementName) {
+function checkIsNotMixinByItself( query, columnMap, elementName ) {
   if (query && query.SELECT && query.SELECT.mixin) {
     const col = columnMap[elementName];
 
@@ -506,7 +506,7 @@ function checkIsNotMixinByItself(query, columnMap, elementName) {
  * @param {Array} ref
  * @returns {string}
  */
-function getLastRefStepString(ref) {
+function getLastRefStepString( ref ) {
   const last = ref[ref.length - 1];
   if (last.id)
     return last.id;