@openui5/sap.ui.fl 1.129.0 → 1.130.0

package/THIRDPARTY.txt

@@ -6,7 +6,7 @@ The full text of all referenced licenses is appended at the end of this file.
 
 Library: sap.m:
 
-Component: purify.js, version: 2.3.8
+Component: purify.js, version: 3.1.7
 Copyright: Mario Heiderich
 License: Apache-2.0
 License Text: https://github.com/SAP/openui5/blob/master/LICENSES/Apache-2.0.txt

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openui5/sap.ui.fl",
-  "version": "1.129.0",
+  "version": "1.130.0",
   "description": "OpenUI5 UI Library sap.ui.fl",
   "author": "SAP SE (https://www.sap.com)",
   "license": "Apache-2.0",
@@ -14,7 +14,7 @@
     "url": "https://github.com/SAP/openui5.git"
   },
   "dependencies": {
-    "@openui5/sap.m": "1.129.0",
-    "@openui5/sap.ui.core": "1.129.0"
+    "@openui5/sap.m": "1.130.0",
+    "@openui5/sap.ui.core": "1.130.0"
   }
 }

package/src/sap/ui/fl/.library

@@ -6,7 +6,7 @@
   <copyright>OpenUI5
  * (c) Copyright 2009-2024 SAP SE or an SAP affiliate company.
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.</copyright>
-  <version>1.129.0</version>
+  <version>1.130.0</version>
 
   <documentation>SAPUI5 library with sap.ui.fl controls.</documentation>
 

package/src/sap/ui/fl/ChangePersistence.js

@@ -40,7 +40,7 @@ sap.ui.define([
 	 *
 	 * @constructor
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 * @since 1.25.0
 	 * @private
 	 * @param {object} mComponent - Component data to initiate <code>ChangePersistence</code> instance
@@ -111,13 +111,8 @@ sap.ui.define([
 		return bCondenserEnabled;
 	}
 
-	function isBackendCondensingEnabled(aChanges) {
-		var bEnabled = Settings.getInstanceOrUndef() && Settings.getInstanceOrUndef().isCondensingEnabled();
-		if (bEnabled && !checkIfOnlyOne(aChanges, "getNamespace")) {
-			bEnabled = false;
-		}
-
-		return bEnabled;
+	function isBackendCondensingEnabled() {
+		return Settings.getInstanceOrUndef()?.isCondensingEnabled();
 	}
 
 	function updateCacheAndDeleteUnsavedChanges(aAllChanges, aCondensedChanges, bSkipUpdateCache, bAlreadyDeletedViaCondense) {
@@ -155,19 +150,13 @@ sap.ui.define([
 	}
 
 	function executeWriteAndRemoveCalls(sCurrentLayer, sRequest, sParentVersion, bSkipUpdateCache, aAllChanges, aCondensedChanges) {
-		var aCondensedDeleteChanges = [];
-		var pRemoveCallsPromise = Promise.resolve();
-		var aNewChanges = aCondensedChanges.filter(function(oCondensedChange) {
-			if (oCondensedChange.getState() === States.LifecycleState.DELETED) {
-				aCondensedDeleteChanges.push(oCondensedChange);
-				return false;
-			}
-			return true;
-		});
+		let pRemoveCallsPromise = Promise.resolve();
+		const aDeletedChanges = aAllChanges.filter((oChange) => oChange.getState() === States.LifecycleState.DELETED);
+		const aNewChanges = aCondensedChanges.filter((oCondensedChange) => (oCondensedChange.getState() !== States.LifecycleState.DELETED));
 
 		// "remove" only supports a single change; multiple calls are required
-		if (aCondensedDeleteChanges.length) {
-			pRemoveCallsPromise = this.saveSequenceOfDirtyChanges(aCondensedDeleteChanges, bSkipUpdateCache, sParentVersion);
+		if (aDeletedChanges.length) {
+			pRemoveCallsPromise = this.saveSequenceOfDirtyChanges(aDeletedChanges, bSkipUpdateCache, sParentVersion);
 		}
 
 		// "write" supports multiple changes at once
@@ -216,52 +205,50 @@ sap.ui.define([
 		bCondenseAnyLayer,
 		sLayer
 	) {
-		var aDirtyChanges = aChanges || FlexObjectState.getDirtyFlexObjects(this._mComponent.name);
-		var sCurrentLayer = aDirtyChanges.length && aDirtyChanges[0].getLayer() || sLayer;
-		var aRelevantChangesForCondensing = getAllRelevantChangesForCondensing.call(
+		const aDirtyChanges = aChanges || FlexObjectState.getDirtyFlexObjects(this._mComponent.name);
+		const sCurrentLayer = aDirtyChanges.length && aDirtyChanges[0].getLayer() || sLayer;
+		const aRelevantChangesForCondensing = getAllRelevantChangesForCondensing.call(
 			this,
 			aDirtyChanges,
 			aDraftFilenames,
 			bCondenseAnyLayer,
 			sCurrentLayer
 		);
-		var bIsCondensingEnabled = (
-			isBackendCondensingEnabled(aRelevantChangesForCondensing)
+		const bIsCondensingEnabled = (
+			isBackendCondensingEnabled()
 			&& canGivenChangesBeCondensed(oAppComponent, aRelevantChangesForCondensing, bCondenseAnyLayer)
 		);
-		var aAllChanges = bIsCondensingEnabled ? aRelevantChangesForCondensing : aDirtyChanges;
-		var aChangesClone = aAllChanges.slice(0);
-		var aRequests = getRequests(aDirtyChanges);
+		const aAllFlexObjects = bIsCondensingEnabled ? aRelevantChangesForCondensing : aDirtyChanges;
+		const aChangesClone = aAllFlexObjects.slice(0);
+		const aRequests = getRequests(aDirtyChanges);
 
 		// Condensing is only allowed if all dirty changes belong to the same Transport Request
 		if (checkLayerAndSingleTransportRequest(aDirtyChanges)) {
-			var oCondensedChangesPromise = Promise.resolve(aChangesClone);
-			if (canGivenChangesBeCondensed(oAppComponent, aChangesClone, bCondenseAnyLayer)) {
-				oCondensedChangesPromise = Condenser.condense(oAppComponent, aChangesClone);
-			}
+			const oCondensedChangesPromise = canGivenChangesBeCondensed(oAppComponent, aChangesClone, bCondenseAnyLayer)
+				? Condenser.condense(oAppComponent, aChangesClone)
+				: Promise.resolve(aChangesClone);
 			return oCondensedChangesPromise.then(function(aCondensedChanges) {
-				var sRequest = aRequests[0];
+				const sRequest = aRequests[0];
 				if (bIsCondensingEnabled) {
 					return Storage.condense({
-						allChanges: aAllChanges,
+						allChanges: aAllFlexObjects,
 						condensedChanges: aCondensedChanges,
 						layer: sCurrentLayer,
 						transport: sRequest,
 						isLegacyVariant: false,
 						parentVersion: sParentVersion
 					}).then(function(oResponse) {
-						updateCacheAndDeleteUnsavedChanges.call(this, aAllChanges, aCondensedChanges, bSkipUpdateCache, true);
+						updateCacheAndDeleteUnsavedChanges.call(this, aAllFlexObjects, aCondensedChanges, bSkipUpdateCache, true);
 						return oResponse;
 					}.bind(this));
 				}
-				// Non-condensing route
 				return executeWriteAndRemoveCalls.call(
 					this,
 					sCurrentLayer,
 					sRequest,
 					sParentVersion,
 					bSkipUpdateCache,
-					aAllChanges,
+					aAllFlexObjects,
 					aCondensedChanges
 				);
 			}.bind(this));
@@ -285,7 +272,7 @@ sap.ui.define([
 		var oFirstNewChange;
 		if (sParentVersion) {
 			// in case of changes saved for a draft only the first writing operation must have the parentVersion targeting the basis
-			// followup changes must point the the existing draft created with the first request
+			// followup changes must point the existing draft created with the first request
 			var aNewChanges = aDirtyChanges.filter(function(oChange) {
 				return oChange.getState() === States.LifecycleState.NEW;
 			});
@@ -305,6 +292,7 @@ sap.ui.define([
 				oCollectedResponse.response.push(...oResponse.response);
 			}
 		}
+		FlexState.getFlexObjectsDataSelector().checkUpdate({reference: this._mComponent.name});
 		return oCollectedResponse;
 	};
 
@@ -362,7 +350,6 @@ sap.ui.define([
 				default:
 			}
 			oDirtyChange.setState(States.LifecycleState.PERSISTED);
-			FlexState.getFlexObjectsDataSelector().checkUpdate({reference: this._mComponent.name});
 		}
 	};
 
@@ -375,6 +362,7 @@ sap.ui.define([
 		aDirtyChanges.forEach(function(oDirtyChange) {
 			this._updateCacheAndDirtyState(oDirtyChange, bSkipUpdateCache);
 		}, this);
+		FlexState.getFlexObjectsDataSelector().checkUpdate({reference: this._mComponent.name});
 	};
 
 	function getRequests(aDirtyChanges) {

package/src/sap/ui/fl/ChangePersistenceFactory.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 * @alias sap.ui.fl.ChangePersistenceFactory
 	 * @since 1.27.0
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 * @private
 	 * @ui5-restricted sap.ui.fl
 	 */

package/src/sap/ui/fl/FakeLrepConnectorLocalStorage.js

@@ -20,7 +20,7 @@ function(
 	 * @namespace
 	 *
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 *
 	 * @private
 	 * @since 1.48

package/src/sap/ui/fl/FakeLrepConnectorSessionStorage.js

@@ -20,7 +20,7 @@ function(
 	 * @namespace
 	 *
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 *
 	 * @private
 	 * @since 1.58

package/src/sap/ui/fl/FakeLrepLocalStorage.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * @namespace
 	 *
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 *
 	 * @private
 	 * @since 1.48

package/src/sap/ui/fl/FlexController.js

@@ -51,7 +51,7 @@ sap.ui.define([
 	 * @since 1.27.0
 	 * @private
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 */
 	var FlexController = function(sComponentName) {
 		this._oChangePersistence = undefined;

package/src/sap/ui/fl/FlexControllerFactory.js

@@ -22,7 +22,7 @@ sap.ui.define([
 	 * @alias sap.ui.fl.FlexControllerFactory
 	 * @since 1.27.0
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.fl

package/src/sap/ui/fl/LayerUtils.js

@@ -41,7 +41,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.LayerUtils
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 */
 	var LayerUtils = {
 		_mLayersIndex: mLayersIndex,

package/src/sap/ui/fl/Utils.js

@@ -65,7 +65,7 @@ sap.ui.define([
 	 * @namespace
 	 * @alias sap.ui.fl.Utils
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.fl, sap.ui.rta
@@ -450,12 +450,12 @@ sap.ui.define([
 		 * Error message will be written in any case.
 		 *
 		 * @param {array.<function>} aPromiseQueue - List of asynchronous functions that returns promises
-		 * @param {boolean} bThrowError - true: errors will be rethrown and therefore break the execution
+		 * @param {boolean} bThrowError - true: errors will be re-thrown and therefore break the execution
 		 * @param {boolean} bAsync - true: asynchronous processing with Promise, false: synchronous processing with FakePromise
-		 * @param {boolean} bSupressAdditionalErrorMessage - true: additional error message will be suppressed
+		 * @param {boolean} bSuppressAdditionalErrorMessage - true: additional error message will be suppressed
 		 * @returns {Promise} Returns empty resolved Promise or FakePromise when all passed promises inside functions have been executed
 		 */
-		execPromiseQueueSequentially(aPromiseQueue, bThrowError, bAsync, bSupressAdditionalErrorMessage) {
+		execPromiseQueueSequentially(aPromiseQueue, bThrowError, bAsync, bSuppressAdditionalErrorMessage) {
 			if (aPromiseQueue.length === 0) {
 				if (bAsync) {
 					return Promise.resolve();
@@ -482,7 +482,7 @@ sap.ui.define([
 					Log.error(sErrorMessage, e);
 
 					if (bThrowError) {
-						if (bSupressAdditionalErrorMessage) {
+						if (bSuppressAdditionalErrorMessage) {
 							throw new Error(e.message);
 						} else {
 							throw new Error(sErrorMessage);
@@ -490,12 +490,12 @@ sap.ui.define([
 					}
 				})
 				.then(function() {
-					return this.execPromiseQueueSequentially(aPromiseQueue, bThrowError, bAsync, bSupressAdditionalErrorMessage);
+					return this.execPromiseQueueSequentially(aPromiseQueue, bThrowError, bAsync, bSuppressAdditionalErrorMessage);
 				}.bind(this));
 			}
 
 			Log.error("Changes could not be applied, promise not wrapped inside function.");
-			return this.execPromiseQueueSequentially(aPromiseQueue, bThrowError, bAsync, bSupressAdditionalErrorMessage);
+			return this.execPromiseQueueSequentially(aPromiseQueue, bThrowError, bAsync, bSuppressAdditionalErrorMessage);
 		},
 
 		/**
@@ -648,7 +648,7 @@ sap.ui.define([
 		},
 
 		/**
-		 * Searches in the control metadata for the aggregation defintion
+		 * Searches in the control metadata for the aggregation definition
 		 * @param {sap.ui.base.ManagedObject|Element} oControl - Control which has the aggregation
 		 * @param {string} sAggregationName - Aggregation name
 		 * @returns {object} Aggregation metadata

package/src/sap/ui/fl/apply/_internal/DelegateMediator.js

@@ -22,7 +22,7 @@ sap.ui.define([
 	 *
 	 * @private
 	 * @author SAP SE
-	 * @version 1.129.0
+	 * @version 1.130.0
 	 *
 	 */
 	const DelegateMediator = {};

package/src/sap/ui/fl/apply/_internal/appVariant/DescriptorChangeTypes.js

@@ -61,7 +61,8 @@ sap.ui.define([
 				"appdescr_fe_changePageConfiguration",
 				"appdescr_ui5_changeModel",
 				"appdescr_ui_generic_app_addNewObjectPage",
-				"appdescr_ui_generic_app_changePageConfiguration"
+				"appdescr_ui_generic_app_changePageConfiguration",
+				"appdescr_fe_addNewPage"
 			];
 		},
 		getCondensableChangeTypes() {