@sap/cds-compiler 2.12.0 → 2.13.6

package/lib/transform/universalCsn/utils.js

@@ -0,0 +1,63 @@
+'use strict';
+
+const {
+  applyTransformations,
+} = require('../../model/csnUtils');
+const { setProp } = require('../../base/model');
+
+/**
+ * Set the annotation on the carrier if it is not already present.
+ *
+ * @param {object} carrier Object having/getting the annotation
+ * @param {string} name Name of the annotations
+ * @param {any} value Value of the annotation
+ */
+function setAnnotationIfNotDefined(carrier, name, value) {
+  if (carrier[name] === undefined)
+    carrier[name] = value;
+}
+
+/**
+ * Strip/change the CSN to ensure compatibility with what we have in the client CSN
+ * - Removes every occurrence of '$origin', '$generated' and '$source'
+ * - elements of subqueries become non-enumerable
+ * - remove `actions|params|virtual|notNull: null`, this stops the inheritance of the actions/params/virtual
+ *   along the $origin chain, this can be ignored for the comparison in our tests.
+ *
+ * @param {CSN.Model} csn
+ */
+function makeClientCompatible(csn) {
+  applyTransformations(csn, {
+    actions: removeNullProperty,
+    notNull: removeNullProperty,
+    params: removeNullProperty,
+    virtual: removeNullProperty,
+    $origin: (parent, prop) => delete parent[prop],
+    $generated: (parent, prop) => delete parent[prop],
+    $source: (parent, prop) => delete parent[prop],
+    SELECT: (parent, prop, query) => {
+      if (query.elements)
+        setProp(query, 'elements', query.elements);
+    },
+    SET: (parent, prop, query) => {
+      if (query.elements)
+        setProp(query, 'elements', query.elements);
+    },
+  });
+}
+
+/**
+ * Removes the `prop` from the `node` if `prop === null`
+ *
+ * @param {object} node
+ * @param {string} prop
+ */
+function removeNullProperty(node, prop) {
+  if (node[prop] === null)
+    delete node[prop];
+}
+
+module.exports = {
+  setAnnotationIfNotDefined,
+  makeClientCompatible,
+};

package/lib/utils/objectUtils.js

@@ -42,8 +42,38 @@ function forEach(obj, callback) {
   }
 }
 
+/**
+ * Loops over all elements in an object and calls the specified callback(o[key]) for each key
+ * --> can be used as substitute for `Object.values(…).forEach(…)`
+ *
+ * @param  {object} o the object which values should be iterated
+ * @param  {Function} callback
+ */
+function forEachValue(o, callback) {
+  for (const key in o) {
+    if (Object.hasOwnProperty.call(o, key))
+      callback(o[key]);
+  }
+}
+
+/**
+ * Loops over all elements in an object and calls the specified callback(key) for each key
+ * --> can be used as substitute for `Object.keys(…).forEach(…)`
+ *
+ * @param  {object} o the object which keys should be iterated
+ * @param  {Function} callback
+ */
+function forEachKey(o, callback) {
+  for (const key in o) {
+    if (Object.hasOwnProperty.call(o, key))
+      callback(key);
+  }
+}
+
 module.exports = {
   copyPropIfExist,
   createDict,
   forEach,
+  forEachValue,
+  forEachKey,
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sap/cds-compiler",
-  "version": "2.12.0",
+  "version": "2.13.6",
   "description": "CDS (Core Data Services) compiler and backends",
   "homepage": "https://cap.cloud.sap/",
   "author": "SAP SE (https://www.sap.com)",

package/share/messages/README.md

@@ -30,3 +30,29 @@ Furthermore:
   If users open the message in a terminal, it is often 80 characters wide.
   Even if that was not the case, longer lines are harder to read, so keep them
   short.
+
+## Example Structure
+
+```markdown
+# ‹message-id›
+
+Longer message text.  Some background info.
+Usually mentions the severity.
+
+## Example
+
+Code which also lead to the current message,
+with an explanation why it is problematic.
+
+## How to Fix
+
+Description of options how the issue can be fixed, using the example.
+
+## Notes on …
+
+Optional: more background info.
+
+## Related messages
+
+ - Optional: message ids for similar issues.
+```