@sap/cds 7.0.3 → 7.1.1

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

@@ -16,7 +16,7 @@ const _isToConvert = type => type === 'cds.DateTime' || type === 'cds.Timestamp'
 
 const _convertDateTimeEntry = (entry, element, model) => {
   const { name, type, _target } = element
-  if (!(entry[name] === undefined || entry[name] === null)) {
+  if (!(entry[name] == null)) {
     if (_isToConvert(type) && entry[name] !== '$now') {
       entry[name] = _convertDateTimeElement(entry[name], element)
     }

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

@@ -11,7 +11,6 @@ const {
 const { readAndDeleteKeywords, isActiveEntityRequested, getKeyData } = require('../utils/where')
 const { isDraftRootEntity } = require('../../fiori/utils/csn')
 const { getColumns } = require('../../cds-services/services/utils/columns')
-
 const { DRAFT_COLUMNS_MAP } = require('../../common/constants/draft')
 
 const _getRootCQN = (context, requestActiveData) => {
@@ -120,52 +119,49 @@ const fioriGenericActivate = async function (req, next) {
   if (!cds.db) req.reject('NO_DATABASE_CONNECTION')
 
   const { draftData, activeData, adminData } = await _draftCompositionTree(this, req)
-
   if (!draftData) req.reject(404)
 
-  if (adminData.InProcessByUser !== req.user.id)
+  if (adminData.InProcessByUser !== req.user.id) {
     req.reject(403, 'DRAFT_LOCKED_BY_ANOTHER_USER', [adminData.InProcessByUser])
+  }
 
-  /*
-   * create or update
-   */
+  // create or update
   let query, event
   if (activeData) {
     readAndDeleteKeywords(['IsActiveEntity'], req.query.SELECT.from.ref[0].where)
     event = 'UPDATE'
-    // REVSIIT: setting data should be part of ql
+    // REVISIT: setting data should be part of ql
     query = UPDATE(req.target).where(req.query.SELECT.from.ref[0].where)
     query.UPDATE.data = draftData
-    query._activeData = activeData
   } else {
     event = 'CREATE'
     query = INSERT.into(req.target).entries(draftData)
   }
 
   // REVISIT: _draftMetadata
-  const r = new cds.Request({ event, query, data: draftData, _draftMetadata: adminData })
+  const request = new cds.Request({ event, query, data: draftData, _draftMetadata: adminData })
 
   // REVISIT: should not be necessary
-  r._ = Object.assign(r._, req._)
-  r._.params = req.params
-  r._.query = req.query
+  request._ = Object.assign(request._, req._)
+  request._.params = req.params
+  request._.query = req.query
 
   // use finally to preserve r.messages in success or error case
   let result
   try {
-    result = await this.dispatch(r)
+    result = await this.dispatch(request)
   } finally {
     // REVISIT: should not be necessary
-    if (r.messages) for (const m of r.messages) req.info(m)
+    if (request.messages) for (const m of request.messages) req.info(m)
   }
 
-  /*
-   * delete draft data
-   */
+  // delete draft data
   const deleteDraftAdminCqn = getDeleteDraftAdminCqn(adminData.DraftUUID)
   const draftTablesToDeleteFrom = [req.target.name + '_drafts']
-  for (const [entity] of getCompositionTargets(req.target, this).entries())
+  for (const [entity] of getCompositionTargets(req.target, this).entries()) {
     draftTablesToDeleteFrom.push(entity + '_drafts')
+  }
+
   await cds.db.tx(req).run([
     deleteDraftAdminCqn,
     ...draftTablesToDeleteFrom.map(dt => {
@@ -178,7 +174,6 @@ const fioriGenericActivate = async function (req, next) {
   // REVISIT: we need to use okra API here because it must be set in the batched request
   // status code must be set in handler to allow overriding for FE V2
   if (event === 'CREATE') req?._?.odataRes?.setStatusCode(201, { overwrite: true })
-
   return result
 }
 

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

@@ -33,10 +33,7 @@ const _getInsertAdminDataCQN = ({ user }, draftUUID, time) => {
 }
 
 const _getLockWhere = (where, columnsMap) => {
-  if (columnsMap.size === 0) {
-    return where
-  }
-
+  if (columnsMap.size === 0) return where
   const whereKeys = Object.keys(where)
   const lockWhere = {}
 
@@ -57,6 +54,7 @@ const _select = async (lockRecordCQN, draftExistsCQN, selectCQNs, req, dbtx) =>
     if (drafts.length) req.reject(409, 'DRAFT_ALREADY_EXISTS')
     req.reject(409, 'ENTITY_LOCKED')
   }
+
   const promisesResults = await Promise.allSettled([dbtx.run(draftExistsCQN), ...selectCQNs.map(cqn => dbtx.run(cqn))])
   const firstRejected = promisesResults.find(r => r.status === 'rejected')
   if (firstRejected) req.reject(firstRejected.reason)
@@ -117,7 +115,7 @@ const fioriGenericEdit = async function (req, next) {
   for (const q of selectCQNs) {
     const entity = definitions[q.SELECT.from.ref[0]]
     if (entity && !entity.name.match(/\.texts$/)) {
-      Object.defineProperty(q, '_suppressLocalization', { value: true })
+      q.SELECT.localized = false
     }
   }
 
@@ -165,7 +163,6 @@ const fioriGenericEdit = async function (req, next) {
   // REVISIT: we need to use okra API here because it must be set in the batched request
   // status code must be set in handler to allow overriding for FE V2
   req?._?.odataRes?.setStatusCode(201, { overwrite: true })
-
   return results[0][0]
 }
 

package/libx/_runtime/fiori/lean-draft.js

@@ -967,7 +967,7 @@ async function onEdit(req) {
   existingDraft[DRAFT_PARAMS] = draftParams
 
   const activeCQN = SELECT.one.from(req.target).columns(cols).where(targetWhere)
-  activeCQN._suppressLocalization = true // in the future we should be able to just set activeCQN.SELECT.localized = false
+  activeCQN.SELECT.localized = false
 
   const activeCheck = SELECT.one(req.target).columns([1]).where(targetWhere).forUpdate()
   activeCheck[DRAFT_PARAMS] = draftParams
@@ -979,7 +979,8 @@ async function onEdit(req) {
   } catch {} // eslint-disable-line no-empty
 
   const [res, draft] = await _promiseAll([
-    this._datasource.run(activeCQN),
+    // REVISIT: inofficial compat flag just in case it breaks something -> do not document
+    cds.env.fiori.read_actives_from_db ? this._datasource.run(activeCQN) : this.run(activeCQN),
     // no user check must be done here...
     existingDraft
   ])

package/libx/_runtime/hana/localized.js

@@ -21,7 +21,7 @@ const localizedHandler = function (req) {
   if (!req.locale) return
 
   // suppress localization by instruction
-  if (query._suppressLocalization) return
+  if (query.SELECT.localized === false) return
 
   // suppress localization for pure counts
   const columns = query.SELECT.columns

package/libx/_runtime/hana/search2cqn4sql.js

@@ -60,8 +60,10 @@ const search2cqn4sql = (query, entity, options) => {
   subQuery.where(expression)
 
   // suppress the localize handler from redirecting the subQuery's target to the localized view
-  Object.defineProperty(subQuery, '_suppressLocalization', { value: true })
+  subQuery.SELECT.localized = false
+
   query.where('exists', subQuery)
+
   return query
 }
 

package/libx/_runtime/remote/utils/client.js

@@ -49,11 +49,15 @@ const _executeHttpRequest = async ({ requestConfig, destination, destinationOpti
     requestOptions = { fetchCsrfToken: requestConfig._autoBatch ? csrfInBatch === true : csrf === true }
   }
 
-  LOG._debug &&
-    LOG.debug(`${requestConfig.method} ${destination.url || `<${destination.destinationName}>`}${requestConfig.url}`, {
-      headers: _sanitizeHeaders({ ...requestConfig.headers }),
-      data: requestConfig.data && SANITIZE_VALUES ? deepSanitize(requestConfig.data) : requestConfig.data
-    })
+  if (LOG._debug) {
+    const req2log = { headers: _sanitizeHeaders({ ...requestConfig.headers }) }
+    if (requestConfig.method !== 'GET' && requestConfig.method !== 'DELETE')
+      req2log.data = requestConfig.data && SANITIZE_VALUES ? deepSanitize(requestConfig.data) : requestConfig.data
+    LOG.debug(
+      `${requestConfig.method} ${destination.url || `<${destination.destinationName}>`}${requestConfig.url}`,
+      req2log
+    )
+  }
 
   // cloud sdk requires a new mechanism to differentiate the priority of headers
   // "custom" keeps the highest priority as before

package/libx/_runtime/sqlite/localized.js

@@ -33,7 +33,7 @@ const sqliteLocalized = function (req) {
   if (!req.locale) return
 
   // suppress localization by instruction
-  if (query._suppressLocalization) return
+  if (query.SELECT.localized === false) return
 
   // suppress localization for pure counts
   const columns = query.SELECT.columns

package/libx/odata/afterburner.js

@@ -3,6 +3,7 @@ const cds = require('../_runtime/cds')
 const { where2obj, resolveFromSelect } = require('../_runtime/common/utils/cqn')
 const { findCsnTargetFor } = require('../_runtime/common/utils/csn')
 const normalizeTimestamp = require('../_runtime/common/utils/normalizeTimestamp')
+const { rewriteExpandAsterisk } = require('../_runtime/common/utils/rewriteAsterisks')
 
 const _addKeysDeep = (keys, keysCollector, ignoreManagedBacklinks) => {
   for (const keyName in keys) {
@@ -400,7 +401,7 @@ function _addKeys(columns, target) {
 }
 
 // remove duplicate * in expand (e.g. expand=*,*)
-function _removeDuplicateAsterix(columns) {
+function _removeDuplicateAsterisk(columns) {
   let hasExpandStar = false
   for (let i = columns.length - 1; i > 0; i--) {
     const column = columns[i]
@@ -423,7 +424,9 @@ function _processColumns(cqn, target) {
     if (target.kind === 'entity') entity = target
     else if (target.kind === 'action' && target.returns?.kind === 'entity') entity = target.returns
     if (!entity) return
-    _removeDuplicateAsterix(columns)
+    _removeDuplicateAsterisk(columns)
+
+    rewriteExpandAsterisk(columns, entity)
     if (cds.env.features.odata_new_parser) _addKeys(columns, entity)
   }
 

package/libx/rest/middleware/operation.js

@@ -12,7 +12,7 @@ module.exports = async (_req, _res, next) => {
   try {
     const req = query
       ? new RestRequest({ query, event: operation.name, data, params: _params })
-      : new RestRequest({ event: operation.name.replace(`${srv.name}.`, ''), data, params: _params })
+      : new RestRequest({ event: operation.name.replace(`${srv.namespace}.`, ''), data, params: _params })
     result = await srv.dispatch(req)
   } catch (e) {
     return next(e)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sap/cds",
-  "version": "7.0.3",
+  "version": "7.1.1",
   "description": "SAP Cloud Application Programming Model - CDS for Node.js",
   "homepage": "https://cap.cloud.sap/",
   "keywords": [
@@ -33,8 +33,8 @@
     "node": ">=16"
   },
   "dependencies": {
-    "@sap/cds-compiler": ">=3.9",
-    "@sap/cds-fiori": "*",
+    "@sap/cds-compiler": "^4",
+    "@sap/cds-fiori": "^1",
     "@sap/cds-foss": "^4"
   },
   "cds": {

package/lib/srv/protocols/graphql.js

@@ -1,30 +0,0 @@
-const GraphQLAdapter = require('@cap-js/graphql') // eslint-disable-line cds/no-missing-dependencies
-const express = require ('express') // eslint-disable-line cds/no-missing-dependencies
-
-function CDSGraphQLAdapter (options) {
-  const { services } = options
-
-  return express.Router()
-  .use (express.json()) //> required in the slug handlers and logger below
-
-  // convenience slug route for /graphql/srv/entity/id
-  .use ('/:path/:entity/:id?(%20:query)?', (req, _, next) => {
-    // TODO: add filter by id -> then remove the // eslint-disable-line
-    const { entity, query } = req.params // eslint-disable-line no-unused-vars
-    if (query) req.body = { query }
-    if (entity) req.body.query = req.body.query.replace(/{/, `{ ${entity} {`) +'}'
-    next() //> goes on below
-  })
-
-  // convenience slug route for /graphql/srv
-  .use ('/:path', (req, res, next) => {
-    const srv = services [req.params.path]
-    if (req.body) req.body.query = req.body.query.replace(/{/, `{ ${srv.name} {`) +'}'
-    next() //> goes on below
-  })
-
-  // the global /graphql route
-  .use (new GraphQLAdapter (options))
-}
-
-module.exports = CDSGraphQLAdapter