@sap/cds 6.2.3 → 6.3.1

package/libx/_runtime/cds-services/services/utils/compareJson.js

@@ -77,7 +77,7 @@ const _hasOpDeep = (entry, element) => {
   return false
 }
 
-const _addCompositionsToResult = (result, entity, prop, newValue, oldValue) => {
+const _addCompositionsToResult = (result, entity, prop, newValue, oldValue, opts) => {
   /*
    * REVISIT: the current impl results in {} instead of keeping null for compo to one.
    *          unfortunately, many follow-up errors occur (e.g., prop in null checks) if changed.
@@ -89,9 +89,9 @@ const _addCompositionsToResult = (result, entity, prop, newValue, oldValue) => {
     !Array.isArray(newValue[prop]) &&
     Object.keys(newValue[prop]).length === 0
   ) {
-    composition = compareJsonDeep(entity.elements[prop]._target, undefined, oldValue && oldValue[prop])
+    composition = compareJsonDeep(entity.elements[prop]._target, undefined, oldValue && oldValue[prop], opts)
   } else {
-    composition = compareJsonDeep(entity.elements[prop]._target, newValue[prop], oldValue && oldValue[prop])
+    composition = compareJsonDeep(entity.elements[prop]._target, newValue[prop], oldValue && oldValue[prop], opts)
   }
   if (composition.some(c => _hasOpDeep(c, entity.elements[prop]))) {
     result[prop] = entity.elements[prop].is2one ? composition[0] : composition
@@ -154,7 +154,7 @@ const _skipToMany = (entity, prop) => {
   return entity.elements[prop] && entity.elements[prop].is2many && _skip(entity, prop)
 }
 
-const _iteratePropsInNewEntry = (newEntry, keys, result, oldEntry, entity) => {
+const _iteratePropsInNewEntry = (newEntry, keys, result, oldEntry, entity, opts) => {
   // On app-service layer, generated foreign keys are not enumerable,
   // include them here too.
   for (const prop of Object.getOwnPropertyNames(newEntry)) {
@@ -164,7 +164,7 @@ const _iteratePropsInNewEntry = (newEntry, keys, result, oldEntry, entity) => {
     }
 
     // if value did not change --> ignored
-    if (newEntry[prop] === (oldEntry && oldEntry[prop]) || prop in DRAFT_COLUMNS_MAP) {
+    if (newEntry[prop] === (oldEntry && oldEntry[prop]) || (opts.ignoreDraftColumns && prop in DRAFT_COLUMNS_MAP)) {
       continue
     }
 
@@ -177,7 +177,7 @@ const _iteratePropsInNewEntry = (newEntry, keys, result, oldEntry, entity) => {
     }
 
     if (entity.elements[prop] && entity.elements[prop].isComposition) {
-      _addCompositionsToResult(result, entity, prop, newEntry, oldEntry)
+      _addCompositionsToResult(result, entity, prop, newEntry, oldEntry, opts)
       continue
     }
 
@@ -185,7 +185,7 @@ const _iteratePropsInNewEntry = (newEntry, keys, result, oldEntry, entity) => {
   }
 }
 
-const compareJsonDeep = (entity, newValue = [], oldValue = []) => {
+const compareJsonDeep = (entity, newValue = [], oldValue = [], opts) => {
   const resultsArray = []
   const keys = _getKeysOfEntity(entity)
 
@@ -200,7 +200,7 @@ const compareJsonDeep = (entity, newValue = [], oldValue = []) => {
 
     _addKeysToEntryIfNotExists(keys, newEntry)
 
-    _iteratePropsInNewEntry(newEntry, keys, result, oldEntry, entity)
+    _iteratePropsInNewEntry(newEntry, keys, result, oldEntry, entity, opts)
 
     resultsArray.push(result)
   }
@@ -259,8 +259,9 @@ const compareJsonDeep = (entity, newValue = [], oldValue = []) => {
  *
  * @returns {Array}
  */
-const compareJson = (newValue, oldValue, entity) => {
-  const result = compareJsonDeep(entity, newValue, oldValue)
+const compareJson = (newValue, oldValue, entity, opts = {}) => {
+  const options = Object.assign({ ignoreDraftColumns: false }, opts)
+  const result = compareJsonDeep(entity, newValue, oldValue, options)
 
   // in case of batch insert, result is an array
   // in all other cases it is an array with just one entry

package/libx/_runtime/cds-services/services/utils/differ.js

@@ -44,7 +44,7 @@ module.exports = class Differ {
     return cds
       .tx(req)
       .run(query)
-      .then(dbState => compareJson(undefined, dbState, req.target))
+      .then(dbState => compareJson(undefined, dbState, req.target, { ignoreDraftColumns: true }))
   }
 
   async _addPartialPersistentState(req) {
@@ -59,7 +59,7 @@ module.exports = class Differ {
     const combinedData = providedData || Object.assign({}, req.query.UPDATE.data || {}, req.query.UPDATE.with || {})
     const lastTransition = newQuery.UPDATE._transitions[newQuery.UPDATE._transitions.length - 1]
     const revertedPersistent = revertData(req._.partialPersistentState, lastTransition, this._srv)
-    return compareJson(combinedData, revertedPersistent, req.target)
+    return compareJson(combinedData, revertedPersistent, req.target, { ignoreDraftColumns: true })
   }
 
   async _diffPatch(req, providedData) {
@@ -73,7 +73,9 @@ module.exports = class Differ {
         .tx(req)
         .run(SELECT.from(draftRef).where(removeIsActiveEntityRecursively(where)).limit(1))
 
-      return compareJson(providedData || req.data, req._.partialPersistentState, req.target)
+      return compareJson(providedData || req.data, req._.partialPersistentState, req.target, {
+        ignoreDraftColumns: true
+      })
     }
   }
 
@@ -83,7 +85,7 @@ module.exports = class Differ {
         ? req.query.INSERT.entries[0]
         : req.query.INSERT.entries
 
-    return compareJson(originalData, undefined, req.target)
+    return compareJson(originalData, undefined, req.target, { ignoreDraftColumns: true })
   }
 
   async calculate(req, providedData) {

package/libx/_runtime/common/composition/data.js

@@ -8,6 +8,8 @@ const { cqn2cqn4sql } = require('../utils/cqn2cqn4sql')
 const cds = require('../../cds')
 const { SELECT } = cds.ql
 
+const CHUNK_SIZE = cds.env.features.chunk_deep || Number.MAX_VALUE
+
 /*
  * own utils
  */
@@ -62,13 +64,17 @@ const _getLinksOfCompTree = compositionTree => {
   return links
 }
 
-const _whereKeys = keys => {
-  const where = []
-  keys.forEach(key => {
-    if (where.length) where.push('or')
-    where.push({ xpr: [...ctUtils.whereKey(key)] })
-  })
-  return where
+const _whereKeys = keySet => {
+  if (!keySet.length || !Object.keys(keySet[0]).length) return []
+
+  const keys0 = Object.keys(keySet[0])
+  const keys = { list: keys0.map(pk => ({ ref: [pk] })) }
+  const values = {
+    list: keySet.map(row => ({
+      list: keys0.map(k => ({ val: row[k] }))
+    }))
+  }
+  return [keys, 'in', values]
 }
 
 const _parentKey = (element, key) => {
@@ -136,28 +142,19 @@ const _subWhere = (result, element) => {
   const links = [...element.backLinks, ...element.customBackLinks]
   if (result.length && links && links.length > 0) {
     where = {}
-    const chunkSize = cds.env.features.chunk_deep
     const keys0 = Object.keys(_getWhereObj(result[0], links))
-    if (chunkSize && result.length > chunkSize && keys0.length) {
+    if (keys0.length) {
       const keys = { list: keys0.map(pk => ({ ref: [pk] })) }
-      for (let i = 0; i < result.length; i += chunkSize) {
+      for (let i = 0; i < result.length; i += CHUNK_SIZE) {
         const values = {
-          list: result
-            .slice(i, i + chunkSize)
-            .map(row => ({ list: keys0.map(k => ({ val: _getWhereObj(row, links)[k] || null })) }))
+          list: result.slice(i, i + CHUNK_SIZE).map(row => ({
+            list: keys0.map(k => {
+              return { val: _getWhereObj(row, links)[k] }
+            })
+          }))
         }
         where[i] = [keys, 'in', values]
       }
-    } else {
-      where = []
-      for (const row of result) {
-        const whereObj = _getWhereObj(row, links)
-        const whereCQN = ctUtils.whereKey(whereObj)
-        if (whereCQN.length) {
-          if (where.length > 0) where.push('or')
-          where.push({ xpr: [...whereCQN] })
-        }
-      }
     }
   }
   return where
@@ -232,29 +229,22 @@ const _select = ({
 }
 
 const _selectDeepUpdateData = async args => {
-  const { model, compositionTree, entityName, data, root, selectData, tx, selectAllColumns } = args
+  const { model, compositionTree, entityName, data, root, selectData, tx, selectAllColumns, where, parentKeys } = args
   let result = []
-  const chunkSize = cds.env.features.chunk_deep
-  if (
-    chunkSize &&
-    !args.where &&
-    args.parentKeys &&
-    args.parentKeys.length > chunkSize &&
-    Object.keys(args.parentKeys[0]).length
-  ) {
-    const keys0 = Object.keys(args.parentKeys[0])
+  if (!where && parentKeys && parentKeys.length && Object.keys(parentKeys[0]).length) {
+    const keys0 = Object.keys(parentKeys[0])
     const keys = { list: keys0.map(pk => ({ ref: [pk] })) }
-    for (let i = 0; i < args.parentKeys.length; i += chunkSize) {
+    for (let i = 0; i < parentKeys.length; i += CHUNK_SIZE) {
       const values = {
-        list: args.parentKeys.slice(i, i + chunkSize).map(row => ({ list: keys0.map(k => ({ val: row[k] || null })) }))
+        list: parentKeys.slice(i, i + CHUNK_SIZE).map(row => ({ list: keys0.map(k => ({ val: row[k] })) }))
       }
-      const _args = { ...args, where: [keys, 'in', values] }
+      const _args = { ...args, where: [keys, 'in', values], parentKeys: undefined }
       const selectCQN = _select(_args)
       result.push(...(await tx.run(selectCQN)))
     }
-  } else if (chunkSize && args.where && !Array.isArray(args.where)) {
-    for (let where of Object.values(args.where)) {
-      const _args = { ...args, where }
+  } else if (where && !Array.isArray(where)) {
+    for (let w of Object.values(where)) {
+      const _args = { ...args, where: w }
       const selectCQN = _select(_args)
       result.push(...(await tx.run(selectCQN)))
     }
@@ -262,7 +252,6 @@ const _selectDeepUpdateData = async args => {
     const selectCQN = _select(args)
     result = await tx.run(selectCQN)
   }
-
   if (!result.length) return Promise.resolve(result)
 
   const keys = _keys(model.definitions[entityName], result)

package/libx/_runtime/common/composition/update.js

@@ -10,6 +10,8 @@ const { deepCopyObject } = require('../utils/copy')
 
 const getError = require('../../common/error')
 
+const CHUNK_SIZE = cds.env.features.chunk_deep || Number.MAX_VALUE
+
 /*
  * own utils
  */
@@ -35,13 +37,11 @@ const _dataByKey = (entity, data) => {
 }
 
 function _addSubDeepUpdateCQNForDelete({ entity, data, selectData, entityName, deleteCQNs }) {
-  const chunkSize = cds.env.features.chunk_deep
   const dataByKey = _dataByKey(entity, data)
-  if (chunkSize && selectData.length > chunkSize && Object.keys(selectData[0]).length) {
-    // REVISIT: Usage of "where in" syntax would be better
-    for (let j = 0; j < selectData.length; j += chunkSize) {
+  if (selectData.length && selectData[0] && Object.keys(selectData[0]).length) {
+    for (let j = 0; j < selectData.length; j += CHUNK_SIZE) {
       const deleteCQN = { DELETE: { from: entityName, where: [] } }
-      for (let i = j; i < j + chunkSize; i++) {
+      for (let i = j; i < j + CHUNK_SIZE && i < selectData.length; i++) {
         const selectEntry = selectData[i]
         if (!selectEntry) continue
         const dataEntry = dataByKey.get(_serializedKey(entity, selectEntry))
@@ -52,21 +52,8 @@ function _addSubDeepUpdateCQNForDelete({ entity, data, selectData, entityName, d
           deleteCQN.DELETE.where.push({ xpr: [...ctUtils.whereKey(ctUtils.key(entity, selectEntry))] })
         }
       }
-      deleteCQNs.push(deleteCQN)
-    }
-  } else {
-    const deleteCQN = { DELETE: { from: entityName, where: [] } }
-    for (const selectEntry of selectData) {
-      if (!selectEntry) continue
-      const dataEntry = dataByKey.get(_serializedKey(entity, selectEntry))
-      if (!dataEntry) {
-        if (deleteCQN.DELETE.where.length > 0) {
-          deleteCQN.DELETE.where.push('or')
-        }
-        deleteCQN.DELETE.where.push({ xpr: [...ctUtils.whereKey(ctUtils.key(entity, selectEntry))] })
-      }
+      if (deleteCQN.DELETE.where.length) deleteCQNs.push(deleteCQN)
     }
-    deleteCQNs.push(deleteCQN)
   }
 }
 

package/libx/_runtime/common/generic/paging.js

@@ -3,7 +3,7 @@ const { getDefaultPageSize, getMaxPageSize } = require('../utils/page')
 
 const commonGenericPaging = function (req) {
   // only if http request
-  if (!req._.req) return
+  if (!(req.http?.req || req._.req)) return
 
   // target === null if view with parameters
   if (!req.target || !req.query.SELECT || req.query.SELECT.one) return

package/libx/_runtime/common/utils/resolveView.js

@@ -353,7 +353,7 @@ const _newSelect = (query, transitions, service) => {
   if (!newSelect.columns && targetTransition.mapping.size) newSelect.columns = _initialColumns(targetTransition)
   if (newSelect.columns) {
     rewriteAsterisks({ SELECT: query.SELECT }, service.model, {
-      _4db: service instanceof cds.DatabaseService,
+      _4db: service instanceof cds.DatabaseService || service.kind === 'better-sqlite',
       target: targetTransition.queryTarget
     })
     newSelect.columns = _newColumns(newSelect.columns, targetTransition, service, service.kind !== 'app-service')
@@ -517,26 +517,20 @@ const _getTransitionData = (target, columns, service, skipForbiddenViewCheck) =>
   if (!skipForbiddenViewCheck) _checkForForbiddenViews(target)
   const targetStartsWithSrvName = service.namespace && target.name.startsWith(`${service.namespace}.`)
   const persistenceTable = _isPersistenceTable(target)
-  columns = _queryColumns(
-    target,
-    columns,
-    persistenceTable,
-    !(service instanceof cds.DatabaseService) && !targetStartsWithSrvName
-  )
-  if (persistenceTable && service instanceof cds.DatabaseService) {
+  const isDatabaseService = service instanceof cds.DatabaseService || service.kind === 'better-sqlite'
+  columns = _queryColumns(target, columns, persistenceTable, !isDatabaseService && !targetStartsWithSrvName)
+  // REVISIT: Change once we expose database service
+  if (persistenceTable && isDatabaseService) {
     return { target, transitionColumns: columns }
   }
   // stop projection resolving if it starts with the service name prefix
-  if (!(service instanceof cds.DatabaseService) && targetStartsWithSrvName) {
+  if (!isDatabaseService && targetStartsWithSrvName) {
     return { target, transitionColumns: columns }
   }
   // continue projection resolving if the target is a projection
   if (target.query && target.query._target) {
     const newTarget = target.query._target
-    if (
-      service instanceof cds.DatabaseService ||
-      !(service.namespace && newTarget.name.startsWith(`${service.namespace}.`))
-    ) {
+    if (isDatabaseService || !(service.namespace && newTarget.name.startsWith(`${service.namespace}.`))) {
       return _getTransitionData(newTarget, columns, service, skipForbiddenViewCheck)
     }
     return { target: newTarget, transitionColumns: columns }

package/libx/_runtime/db/utils/generateAliases.js

@@ -57,6 +57,7 @@ const _generateAliases = (partialCqn, aliasMap = new Map()) => {
     _redirectXpr(partialCqn.SELECT.having, selectMap)
     _redirectXpr(partialCqn.SELECT.columns, selectMap)
     _redirectXpr(partialCqn.SELECT.groupBy, selectMap)
+    _redirectXpr(partialCqn.SELECT.orderBy, selectMap)
     return
   }
 

package/libx/_runtime/fiori/generic/before.js

@@ -77,7 +77,7 @@ const _getRoot = req => {
   return root
 }
 
-const _getDraftDataFromExistingDraft = async (req, root) => {
+const _getDraftDataFromExistingDraft = async (req, root, isBoundAction) => {
   if (!root) return []
   if (root?.IsActiveEntity === false) {
     const query = _getSelectDraftDataCqn(root.entityName, root.where)
@@ -85,6 +85,9 @@ const _getDraftDataFromExistingDraft = async (req, root) => {
     return result
   }
 
+  // do not expect validate draft ownership for action call on active instances
+  if (isBoundAction) return []
+
   const rootWhere = getKeysCondition(req)
   const query = _getSelectDraftDataCqn(ensureNoDraftsSuffix(req.target.name), rootWhere)
   const result = await cds.tx(req).run(query)
@@ -147,8 +150,8 @@ const _deleteCancel = async function (req) {
 }
 
 const _validateDraftBoundAction = async function (req) {
-  const result = await _getDraftDataFromExistingDraft(req, _getRoot(req))
   const isBoundAction = true
+  const result = await _getDraftDataFromExistingDraft(req, _getRoot(req), isBoundAction)
   if (result && result.length > 0) _validateDraft(req, result, isBoundAction)
 }
 

package/libx/_runtime/fiori/generic/read.js

@@ -187,16 +187,23 @@ const _getDraftPropertiesDetermineDraft = (req, where, tableName, calcDraftUUID
 }
 
 function _copyCQNPartial(partial) {
-  if (partial.SELECT && partial.SELECT.where) {
+  if (partial.SELECT) {
     const newPartial = Object.assign({}, partial)
     const newSELECT = Object.assign({}, partial.SELECT)
     newSELECT.from = _copyCQNPartial(partial.SELECT.from)
     newPartial.SELECT = newSELECT
+    if (partial.SELECT._4odata) newSELECT._4odata = true
     if (partial.SELECT.columns) newPartial.SELECT.columns = _copyArray(partial.SELECT.columns)
     if (partial.SELECT.where) newPartial.SELECT.where = _copyArray(partial.SELECT.where)
     return newPartial
   }
 
+  if (partial.id) {
+    const res = Object.assign({}, partial)
+    if (res.where) res.where = _copyArray(res.where)
+    return res
+  }
+
   if (partial.ref) {
     return Object.assign({}, partial, { ref: _copyArray(partial.ref) })
   }
@@ -1272,12 +1279,12 @@ const fioriGenericRead = async function (req) {
   req.target = _getLocalizedEntity(this.model, req.target, req.user) || req.target
 
   // REVISIT DRAFT HANDLING: cqn2cqn4sql must not be called here
-  const query4sql = cqn2cqn4sql(req.query, this.model, { _4fiori: true })
+  const query4sql = cqn2cqn4sql(_copyCQNPartial(req.query), this.model, { _4fiori: true })
 
   // Clone the request. Do not clone with Object.assign as that would skip all non-enumerable properties.
   // REVISIT: query4sql.clone() doesn't really clone the original query, hence _generateCQN will heavily modify
   // it, e.g. IsActiveEntity is stripped. This is a problem for subsequent handlers which rely on this information.
-  const reqClone = { __proto__: req, query: query4sql.clone() }
+  const reqClone = { __proto__: req, query: query4sql }
   // Clone draft restrictions to the cloned query.
   reqClone.query._draftRestrictions = query._draftRestrictions
 
@@ -1320,7 +1327,7 @@ const fioriGenericRead = async function (req) {
   _adaptColumns4readAfterWrite(req, cqnScenario, query4sql)
 
   const result = await cds.tx(req).send({ query: cqnScenario.cqn, target: req.target })
-  return _postProcess(result, req, cqnScenario, enhancedWithLastChangeDateTime)
+  return _postProcess(result, reqClone, cqnScenario, enhancedWithLastChangeDateTime)
 }
 
 module.exports = cds.service.impl(function (srv, entity) {

package/libx/_runtime/hana/execute.js

@@ -51,8 +51,8 @@ const BASE64 = /^(?:[A-Za-z0-9+/]{4})*(?:[A-Za-z0-9+/]{4}|[A-Za-z0-9+/]{3}=|[A-Z
 
 function _getProcedureName(sql) {
   // name delimited with "" allows any character
-  const match = sql.trim().match(/^call \s*(("(?<delimited>.+)")|(?<undelimited>\w+))\s*\(/i)
-  return match && (match.groups.undelimited || match.groups.delimited)
+  const match = sql.trim().match(/^call \s*(("\w+"\.)?("(?<delimited>.+)")|(\w+\.)?(?<undelimited>\w+))\s*\(/i)
+  return match && (match.groups.undelimited ?? match.groups.delimited)
 }
 
 function _hdbGetResultForProcedure(rows, args, outParameters) {

package/libx/_runtime/hana/search2Contains.js

@@ -95,7 +95,9 @@ const isContainsPredicateSupported = (query, entity, columns2Search) => {
 const _isColumnFunc = (columns2Search, columnsDefs) =>
   columns2Search.some(column2Search => {
     if (column2Search.func) return true
-    return columnsDefs?.some(columnDef => columnDef.func && columnDef.as === column2Search.ref[0])
+    return columnsDefs?.some(
+      columnDef => (columnDef.func && columnDef.as === column2Search.ref[0]) || columnDef?.xpr?.some(xpr => xpr.func)
+    )
   })
 
 module.exports = {

package/libx/_runtime/hana/search2cqn4sql.js

@@ -82,6 +82,7 @@ const _addAliasToQuery = (query, entity, columnsToBeSearched) => {
   SELECT.columns = addAliasToExpression(SELECT.columns, getEntityName)
   columnsToBeSearched = addAliasToExpression(columnsToBeSearched, getEntityName)
   SELECT.groupBy = addAliasToExpression(SELECT.groupBy, getEntityName)
+  SELECT.orderBy = addAliasToExpression(SELECT.orderBy, getEntityName)
   SELECT.where = addAliasToExpression(SELECT.where, getEntityName)
   return columnsToBeSearched
 }

package/libx/_runtime/messaging/AMQPWebhookMessaging.js

@@ -38,7 +38,7 @@ class AMQPWebhookMessaging extends MessagingService {
   }
 
   startListening(opt = {}) {
-    if (!this.subscribedTopics.size) return
+    if (!this._listenToAll && !this.subscribedTopics.size) return
     if (!opt.doNotDeploy) {
       const management = this.getManagement()
       this.queued(management.createQueueAndSubscriptions.bind(management))()

package/libx/_runtime/messaging/enterprise-messaging-utils/EMManagement.js

@@ -87,12 +87,15 @@ class EMManagement {
         this.subdomain ? { queue: queueName, subdomain: this.subdomain } : { queue: queueName }
       )
     try {
+      const queueConfig = this.queueConfig && { ...this.queueConfig }
+      if (queueConfig?.deadMsgQueue)
+        queueConfig.deadMsgQueue = queueConfig.deadMsgQueue.replace(/\$namespace/g, this.namespace)
       const res = await authorizedRequest({
         method: 'PUT',
         uri: this.options.uri,
         path: `/hub/rest/api/v1/management/messaging/queues/${encodeURIComponent(queueName)}`,
         oa2: this.options.oa2,
-        dataObj: this.queueConfig,
+        dataObj: queueConfig,
         tokenStore: this
       })
       if (res.statusCode === 201) return true
@@ -389,7 +392,7 @@ class EMManagement {
         this.LOG._info && this.LOG.info('Unchanged subscriptions', unchangedSubs, ' ', this.subdomainInfo)
         await Promise.all([
           ...obsoleteSubs.map(s => this.deleteSubscription(s)),
-          ...additionalSubs.map(async s => this.createSubscription(s))
+          ...additionalSubs.map(async t => this.createSubscription(t))
         ])
         return
       }

package/libx/_runtime/messaging/enterprise-messaging.js

@@ -139,7 +139,7 @@ class EnterpriseMessaging extends AMQPWebhookMessaging {
     const doNotDeploy = _multitenancyEnabled() && !this.options.deployForProvider
     if (doNotDeploy) this.LOG._info && this.LOG.info('Skipping deployment of messaging artifacts for provider account')
     super.startListening({ doNotDeploy })
-    if (!doNotDeploy && this.subscribedTopics.size) {
+    if (!doNotDeploy && (this._listenToAll || this.subscribedTopics.size)) {
       const management = this.getManagement()
       // Webhooks will perform an OPTIONS call on creation to check the availability of the app.
       // On systems like Cloud Foundry the app URL will only be advertised once
@@ -218,6 +218,11 @@ class EnterpriseMessaging extends AMQPWebhookMessaging {
     const topic = msg.event
     const message = { ...(msg.headers || {}), data: msg.data }
 
+    const contentType =
+      msg.headers && ['id', 'source', 'specversion', 'type'].every(el => el in msg.headers)
+        ? 'application/cloudevents+json'
+        : 'application/json'
+
     await this.queued(() => {})()
 
     try {
@@ -229,6 +234,7 @@ class EnterpriseMessaging extends AMQPWebhookMessaging {
         tenant,
         dataObj: message,
         headers: {
+          'Content-Type': contentType,
           'x-qos': 1
         },
         tokenStore: {}

package/libx/_runtime/messaging/file-based.js

@@ -35,7 +35,7 @@ class FileBasedMessaging extends MessagingService {
   }
 
   startWatching() {
-    if (!this.subscribedTopics.size) return
+    if (!this._listenToAll && !this.subscribedTopics.size) return
     const watcher = async () => {
       if (!(await touched(this.file, this.recent))) return // > not touched since last check
       // REVISIT: Bad if lock file wasn't cleaned up (due to crashes...)

package/libx/_runtime/messaging/message-queuing.js

@@ -61,12 +61,15 @@ class MQManagement {
   async createQueue(queueName = this.queueName) {
     this.LOG._info && this.LOG.info('Create queue', { queue: queueName })
     try {
+      const queueConfig = this.queueConfig && { ...this.queueConfig }
+      if (queueConfig?.deadMessageQueue)
+        queueConfig.deadMessageQueue = queueConfig.deadMessageQueue.replace(/\$namespace/g, '')
       const res = await authorizedRequest({
         method: 'PUT',
         uri: this.options.url,
         path: `/v1/management/queues/${encodeURIComponent(queueName)}`,
         oa2: this.options.auth.oauth2,
-        dataObj: this.queueConfig,
+        dataObj: queueConfig,
         tokenStore: this
       })
       if (res.statusCode === 201) return true
@@ -182,7 +185,7 @@ class MQManagement {
           .filter(s => !existingSubscriptions.some(e => s === e))
         await Promise.all([
           ...obsoleteSubs.map(s => this.deleteSubscription(s)),
-          ...additionalSubs.map(s => this.createSubscription(s))
+          ...additionalSubs.map(t => this.createSubscription(t))
         ])
         return
       }

package/libx/_runtime/messaging/outbox/utils.js

@@ -50,7 +50,7 @@ const _processSingleMessage = async (service, message, succeededMessages) => {
   // Promise resolve is necessary because we want to set `cds.context` only
   // inside this call
   return Promise.resolve().then(async () => {
-    if (userId) cds.context.user = new cds.User.Privileged(userId)
+    if (userId) cds.context = { user: new cds.User.Privileged(userId) }
     try {
       service._emitImmediate && (await service._emitImmediate(msg))
       succeededMessages.push(message.ID)

package/libx/_runtime/messaging/service.js

@@ -97,7 +97,8 @@ class MessagingService extends OutboxService {
   on(event, cb) {
     const _event = _warnAndStripTopicPrefix(event, this.LOG)
     // save all subscribed topics (not needed for local-messaging)
-    this.subscribedTopics.set(this.prepareTopic(_event, true), _event)
+    if (event !== '*') this.subscribedTopics.set(this.prepareTopic(_event, true), _event)
+    else this._listenToAll = true
     return super.on(_event, cb)
   }
 
@@ -134,8 +135,9 @@ class MessagingService extends OutboxService {
       const subscribedEvent =
         this.subscribedTopics.get(_msg.event) ||
         (this.wildcarded && this.subscribedTopics.get(this.wildcarded(_msg.event)))
-      if (!subscribedEvent) throw new Error(`No handler for incoming message with topic '${_msg.event}' found.`)
-      _msg.event = subscribedEvent
+      if (!subscribedEvent && !this._listenToAll)
+        throw new Error(`No handler for incoming message with topic '${_msg.event}' found.`)
+      _msg.event = subscribedEvent || _msg.event
     }
     return _msg
   }

package/libx/odata/cqn2odata.js

@@ -516,12 +516,15 @@ const _select = (cqn, kind, model) => {
 const _insert = (cqn, kind, model) => {
   const INSERT = getProp(cqn, 'INSERT')
   const { url } = _from(getProp(INSERT, 'into'), kind, model)
-  const body = Array.isArray(INSERT.entries) && INSERT.entries.length === 1 ? INSERT.entries[0] : INSERT.entries
+  const body = _copyData(
+    Array.isArray(INSERT.entries) && INSERT.entries.length === 1 ? INSERT.entries[0] : INSERT.entries
+  )
   return { method: 'POST', path: url, body }
 }
 
 const _copyData = data => {
   // only works on flat structures
+  if (Array.isArray(data)) return data.map(_copyData)
   const copied = {}
   for (const property in data) {
     copied[property] =

package/libx/odata/utils.js

@@ -52,10 +52,9 @@ const formatVal = (val, elementName, csnTarget, kind) => {
   if (typeof val === 'boolean') return val
   if (typeof val === 'number') return getSafeNumber(val)
   if (!csnTarget && typeof val === 'string' && UUID.test(val)) return kind === 'odata-v2' ? `guid'${val}'` : val
-  const { type } = _getElement(csnTarget, elementName)
-
+  const element = _getElement(csnTarget, elementName)
   if (kind === 'odata-v2') {
-    switch (type) {
+    switch (element.type) {
       case 'cds.Decimal':
       case 'cds.Integer64':
       case 'cds.Int64':
@@ -64,20 +63,22 @@ const formatVal = (val, elementName, csnTarget, kind) => {
       case 'cds.LargeBinary':
         return `binary'${toBase64url(val)}'`
       case 'cds.Date':
-        return `datetime'${val}T00:00:00'`
+        return element['@odata.Type'] === 'Edm.DateTimeOffset'
+          ? `datetimeoffset'${val}T00:00:00'`
+          : `datetime'${val}T00:00:00'`
       case 'cds.DateTime':
-        return `datetime'${val}'`
+        return element['@odata.Type'] === 'Edm.DateTimeOffset' ? `datetimeoffset'${val}'` : `datetime'${val}'`
       case 'cds.Time':
         return `time'${_PT(val.split(':'))}'`
       case 'cds.Timestamp':
-        return `datetimeoffset'${val}'`
+        return element['@odata.Type'] === 'Edm.DateTime' ? `datetime'${val}'` : `datetimeoffset'${val}'`
       case 'cds.UUID':
         return `guid'${val}'`
       default:
         return `'${val}'`
     }
   } else {
-    switch (type) {
+    switch (element.type) {
       case 'cds.Binary':
       case 'cds.LargeBinary':
         return `binary'${toBase64url(val)}'`