@sap/cds 5.9.8 → 6.0.1

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

@@ -55,7 +55,7 @@ const _getParent = (req, service) => {
     keyPredicateText = keyPredicate.getText()
 
     if (keyPredicateName === 'IsActiveEntity') {
-      parent.IsActiveEntity = keyPredicateText
+      parent.IsActiveEntity = keyPredicateText === 'true'
     } else {
       parent.keyName = keyPredicateName
       parent.keyValue = keyPredicateText
@@ -104,7 +104,7 @@ const _getSelectDraftDataCqn = (entityName, where) => {
 
 const _getDraftDataFromExistingDraft = async (req, service, parent = _getParent(req, service)) => {
   if (parent) {
-    if (parent.IsActiveEntity === 'false') {
+    if (parent.IsActiveEntity === false) {
       const parentWhere = [{ ref: [parent.keyName] }, '=', { val: parent.keyValue }]
       const query = _getSelectDraftDataCqn(parent.entityName, parentWhere)
       const result = await cds.tx(req).run(query)
@@ -125,7 +125,7 @@ const _addDraftDataFromExistingDraft = async (req, service) => {
   const result = await _getDraftDataFromExistingDraft(req, service, parent)
 
   if (parent) {
-    if (parent.IsActiveEntity === 'false') {
+    if (parent.IsActiveEntity === false) {
       _validateDraft(req, result)
       _addDraftDataToContext(req, result)
       return result
@@ -139,21 +139,14 @@ const _addDraftDataFromExistingDraft = async (req, service) => {
   return result
 }
 
-const _addGeneratedDraftUUID = async req => {
-  req._draftMetadata = req._draftMetadata || {}
-  req.data.DraftAdministrativeData_DraftUUID = cds.utils.uuid()
-  req._draftMetadata.DraftUUID = req.data.DraftAdministrativeData_DraftUUID
-}
-
 /**
  * Generic Handler for before NEW requests.
- *
- * @param req
  */
 const _new = async function (req) {
-  if (isDraftActivateAction(req)) return
+  if (isDraftActivateAction(req)) return // REVISIT: How can NEW be draftActivate???
 
   if (isNavigationToMany(req)) {
+    // REVISIT: How can NEW be a navigation to many?
     const result = await _addDraftDataFromExistingDraft(req, this)
 
     // in order to fix corner case where active subitems are created in draft case
@@ -161,13 +154,13 @@ const _new = async function (req) {
     return
   }
 
-  _addGeneratedDraftUUID(req)
+  req._draftMetadata = req._draftMetadata || {}
+  req.data.DraftAdministrativeData_DraftUUID = cds.utils.uuid()
+  req._draftMetadata.DraftUUID = req.data.DraftAdministrativeData_DraftUUID
 }
 
 /**
  * Generic Handler for before PATCH and UPDATE requests.
- *
- * @param req
  */
 const _patchUpdate = async function (req) {
   if (isDraftActivateAction(req)) return
@@ -180,8 +173,6 @@ const _patchUpdate = async function (req) {
 
 /**
  * Generic Handler for before DELETE and CANCEL requests.
- *
- * @param req
  */
 const _deleteCancel = async function (req) {
   await _addDraftDataFromExistingDraft(req, this)
@@ -209,16 +200,13 @@ const _registerBoundActionHandlers = function (entityName, actions) {
   }
 }
 
-module.exports = cds.service.impl(function () {
-  _new._initial = true
-  _patchUpdate._initial = true
-  _deleteCancel._initial = true
-  const entities = Object.values(this.entities).filter(e => e._isDraftEnabled)
-
-  for (const entity of entities) {
-    this.before('NEW', entity, _new)
-    this.before(['PATCH', 'UPDATE'], entity, _patchUpdate)
-    this.before(['DELETE', 'CANCEL'], entity, _deleteCancel)
-    _registerBoundActionHandlers.call(this, entity.name, entity.actions)
-  }
+_new._initial = true
+_patchUpdate._initial = true
+_deleteCancel._initial = true
+
+module.exports = cds.service.impl((srv, entity) => {
+  srv.before('NEW', entity, _new)
+  srv.before(['PATCH', 'UPDATE'], entity, _patchUpdate)
+  srv.before(['DELETE', 'CANCEL'], entity, _deleteCancel)
+  _registerBoundActionHandlers.call(srv, entity.name, entity.actions)
 })

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

@@ -12,8 +12,6 @@ const _handler = function (req) {
   return deleteDraft(req, this)
 }
 
-module.exports = cds.service.impl(function () {
-  for (const entity of Object.values(this.entities).filter(e => e._isDraftEnabled)) {
-    this.on('CANCEL', entity, _handler)
-  }
+module.exports = cds.service.impl(function (srv, entity) {
+  srv.on('CANCEL', entity, _handler)
 })

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

@@ -14,8 +14,6 @@ const _handler = function (req) {
   return deleteDraft(req, this, true)
 }
 
-module.exports = cds.service.impl(function () {
-  for (const entity of Object.values(this.entities).filter(e => e._isDraftEnabled)) {
-    this.on('DELETE', entity, _handler)
-  }
+module.exports = cds.service.impl(function (srv, entity) {
+  srv.on('DELETE', entity, _handler)
 })

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

@@ -164,13 +164,9 @@ const _handler = async function (req) {
 
   await Promise.all(insertCQNs.map(CQN => dbtx.run(CQN)))
   setStatusCodeAndHeader(req._.odataRes, rootWhere, req.target.name.replace(`${this.name}.`, ''), false)
-  return Object.assign({}, results[0][0], { HasDraftEntity: false, HasActiveEntity: true, IsActiveEntity: false })
+  return results[0][0]
 }
 
-module.exports = cds.service.impl(function () {
-  const entities = Object.values(this.entities).filter(e => e._isDraftEnabled)
-
-  for (const entity of entities) {
-    this.on('EDIT', entity, _handler)
-  }
+module.exports = cds.service.impl(function (srv, entity) {
+  srv.on('EDIT', entity, _handler)
 })

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

@@ -0,0 +1,31 @@
+const cds = require('../../cds')
+
+exports.impl = cds.service.impl(function () {
+  const entities = Object.values(this.entities).filter(entity => entity._isDraftEnabled)
+  if (entities.length === 0) return
+
+  for (const each of entities) {
+    _before(this, each)
+    _new(this, each)
+    _patch(this, each)
+    _cancel(this, each)
+    _edit(this, each)
+    _prepare(this, each)
+    _activate(this, each)
+    _delete(this, each)
+  }
+
+  _readOverDraft(this) // registers an on READ * handler
+  for (const each of entities) _read(this, each) // have to go last
+})
+
+const _before = require('../../fiori/generic/before')
+const _new = require('../../fiori/generic/new')
+const _patch = require('../../fiori/generic/patch')
+const _cancel = require('../../fiori/generic/cancel')
+const _edit = require('../../fiori/generic/edit')
+const _prepare = require('../../fiori/generic/prepare')
+const _activate = require('../../fiori/generic/activate')
+const _readOverDraft = require('../../fiori/generic/readOverDraft')
+const _read = require('../../fiori/generic/read')
+const _delete = require('../../fiori/generic/delete')

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

@@ -96,8 +96,6 @@ const _handler = async function (req, next) {
   return result[0]
 }
 
-module.exports = cds.service.impl(function () {
-  for (const entity of Object.values(this.entities).filter(e => e._isDraftEnabled)) {
-    this.on('NEW', entity, _handler)
-  }
+module.exports = cds.service.impl(function (srv, entity) {
+  srv.on('NEW', entity, _handler)
 })

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

@@ -12,7 +12,7 @@ const { getKeysCondition } = require('../utils/where')
 const { getColumns } = require('../../cds-services/services/utils/columns')
 const { DRAFT_COLUMNS_CQN } = require('../../common/constants/draft')
 
-const _getSelectCQN = (model, { data, target: { name } }, keysCondition, checkUser = true) => {
+const _getSelectCQN = (model, { target: { name } }, keysCondition, checkUser = true) => {
   const activeName = ensureNoDraftsSuffix(name)
   const draftName = ensureDraftsSuffix(name)
 
@@ -64,7 +64,7 @@ const _getUpdateDraftCQN = ({ query, target: { name } }, keysCondition) => {
  * @param req
  */
 const _handler = async function (req) {
-  if (req.data.IsActiveEntity === 'true') req.reject(400, 'Patch can only be applied to a draft entity')
+  if (req.data.IsActiveEntity === true) req.reject(400, 'Patch can only be applied to a draft entity')
 
   const keysCondition = getKeysCondition(req.target, req.data)
   const dbtx = cds.tx(req)
@@ -86,10 +86,6 @@ const _handler = async function (req) {
   return result[0]
 }
 
-module.exports = cds.service.impl(function () {
-  const entities = Object.values(this.entities).filter(e => e._isDraftEnabled)
-
-  for (const entity of entities) {
-    this.on('PATCH', entity, _handler)
-  }
+module.exports = cds.service.impl(function (srv, entity) {
+  srv.on('PATCH', entity, _handler)
 })

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

@@ -50,8 +50,6 @@ const _handler = async function (req) {
   return result[0]
 }
 
-module.exports = cds.service.impl(function () {
-  for (const entity of Object.values(this.entities).filter(e => e._isDraftEnabled)) {
-    this.on('draftPrepare', entity, _handler)
-  }
+module.exports = cds.service.impl(function (srv, entity) {
+  srv.on('draftPrepare', entity, _handler)
 })