@sap/cds-compiler - npm Package Compare versions

Comparing version 1.50.6 to 1.50.8

CHANGELOG.md

 # ChangeLog for cdx compiler and backends
 
 <!-- markdownlint-disable MD024 -->
+<!-- markdownlint-disable MD004 -->
 <!-- (no-duplicate-heading)-->
@@ -9,2 +10,8 @@
 
+## Version 1.50.8 - 2021-07-01
+
+### Fixed
+
+- to.hdi.migration: Don't generate `ALTER` for type change from association to composition or vice versa (if the rest stays the same), as the resulting SQL is identical.
+
 ## Version 1.50.6 - 2021-05-05
@@ -11,0 +18,0 @@

lib/modelCompare/compare.js

@@ -127,3 +127,3 @@ 'use strict';
       }
-      if (otherElement.type !== element.type || JSON.stringify(otherElement) !== JSON.stringify(element)) {
+      if (relevantTypeChange(element.type, otherElement.type) || typeParametersChanged(element, otherElement)) {
         // Type or parameters, e.g. association target, changed.
@@ -142,2 +142,46 @@ changedElements[name] = changedElement(element, otherElement);
 
+function relevantTypeChange(type, otherType) {
+  return otherType !== type && ![type, otherType].every(t => ['cds.Association', 'cds.Composition'].includes(t));
+}
+
+/**
+ * Returns whether two things are deeply equal.
+ * Function-type things are compared in terms of identity,
+ * object-type things in terms of deep equality of all of their properties,
+ * all other things in terms of strict equality (===).
+ *
+ * @param a {any} first thing
+ * @param b {any} second thing
+ * @param include {function} function of a key and a depth, returning true if and only if the given key at the given depth is to be included in comparison
+ * @param depth {number} the current depth in property hierarchy below each of the original arguments (positive, counting from 0; don't set)
+ * @returns {boolean}
+ */
+function deepEqual(a, b, include = () => true, depth = 0) {
+  function isObject(x) {
+    return x !== null && typeof x === 'object';
+  }
+  function samePropertyCount() {
+    return Object.keys(a).length === Object.keys(b).length;
+  }
+  function allPropertiesEqual() {
+    return Object.keys(a).reduce((prev, key) => prev && (!include(key, depth) || deepEqual(a[key], b[key], include, depth + 1)), true);
+  }
+
+  return isObject(a)
+      ? isObject(b)
+          ? samePropertyCount() && allPropertiesEqual()
+          : false
+      : a === b;
+}
+
+/**
+ * Returns whether any type parameters differ between two given elements. Ignores whether types themselves differ (`type` property).
+ * @param element {object} an element
+ * @param otherElement {object} another element
+ * @returns {boolean}
+ */
+function typeParametersChanged(element, otherElement) {
+  return !deepEqual(element, otherElement, (key, depth) => !(depth === 0 && key === 'type'));
+}
+
 function changedElement(element, otherElement) {
@@ -151,3 +195,4 @@ return {
 module.exports = {
-  compareModels
+  compareModels,
+  deepEqual
 };

package.json

 {
   "name": "@sap/cds-compiler",
-  "version": "1.50.6",
+  "version": "1.50.8",
   "description": "CDS (Core Data Services) compiler and backends",
@@ -5,0 +5,0 @@ "homepage": "https://cap.cloud.sap/",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

3857260

increased by0.05%

Lines of code

74277

increased by0.06%

No dependency changes