@openui5/sap.ui.fl 1.92.0 → 1.93.3

package/src/sap/ui/fl/apply/_internal/changes/Reverter.js

@@ -17,6 +17,8 @@ sap.ui.define([
 ) {
 	"use strict";
 
+	var Reverter = {};
+
 	function _waitForApplyIfNecessary(oChange) {
 		if (!oChange.isApplyProcessFinished() && oChange.hasApplyProcessStarted()) {
 			// wait for the change to be applied
@@ -30,98 +32,102 @@ sap.ui.define([
 		return Promise.resolve();
 	}
 
-	var Reverter = {
-		/**
-		 * Reverts a specific change on the passed control if it is currently applied.
-		 *
-		 * @param {sap.ui.fl.Change} oChange - Change object that should be reverted on the passed control
-		 * @param {sap.ui.core.Control} oControl - Control which is the target of the passed change
-		 * @param {object} mPropertyBag - Object with parameters as properties
-		 * @param {sap.ui.core.Component} mPropertyBag.appComponent - Component instance that is currently loading
-		 * @param {object} mPropertyBag.appDescriptor - App descriptor containing the metadata of the current application
-		 * @param {sap.ui.core.util.reflection.BaseTreeModifier} mPropertyBag.modifier - Polymorph reuse operations handling the changes on the given view type
-		 * @param {sap.ui.core.mvc.View} mPropertyBag.view - View to process
-		 * @returns {Promise|sap.ui.fl.Utils.FakePromise} Resolving Promise/FakePromise with either the control (success) or <code>false</code> (failure) as value
-		 */
-		revertChangeOnControl: function(oChange, oControl, mPropertyBag) {
-			var mControl = Utils.getControlIfTemplateAffected(oChange, oControl, mPropertyBag);
-			var oChangeHandler;
-
-			return Utils.getChangeHandler(oChange, mControl, mPropertyBag).then(function(oReturnedChangeHandler) {
-				oChangeHandler = oReturnedChangeHandler;
-			})
-
-			.then(_waitForApplyIfNecessary.bind(null, oChange))
-
-			.then(function() {
-				if (oChange.isApplyProcessFinished()) {
-					oChange.startReverting();
-					return oChangeHandler.revertChange(oChange, mControl.control, mPropertyBag);
-				}
-				throw Error("Change was never applied");
+	function revertAndDeleteChangeOnControl(oChange, oControl, mRevertProperties, mPropertyBag) {
+		return Reverter.revertChangeOnControl(oChange, oControl, mRevertProperties)
+			.then(function(vRevertResult) {
+				return FlexCustomData.destroyAppliedCustomData(vRevertResult || oControl, oChange, mPropertyBag.modifier)
+					.then(function () {
+						return !!vRevertResult;
+					});
 			})
-
-			.then(function() {
-				// After revert the relevant control for the change might have changed, therefore it must be retrieved again (e.g. stashing)
-				mControl.control = mPropertyBag.modifier.bySelector(oChange.getSelector(), mPropertyBag.appComponent, mPropertyBag.view);
-				if (mControl.bTemplateAffected) {
-					mPropertyBag.modifier.updateAggregation(mControl.control, oChange.getContent().boundAggregation);
+			.then(function(bSuccess) {
+				if (bSuccess) {
+					// TODO should be changed as soon as new flex persistence is in place
+					mPropertyBag.flexController._oChangePersistence._deleteChangeInMap(oChange);
 				}
-				oChange.markRevertFinished();
-				return mControl.control;
-			})
-
-			.catch(function(oError) {
-				var sErrorMessage = "Change could not be reverted: " + oError.message;
-				Log.error(sErrorMessage);
-				oChange.markRevertFinished(sErrorMessage);
-				return false;
 			});
-		},
+	}
 
-		/**
-		 * Reverts all given changes in one app component.
-		 *
-		 * @param {sap.ui.fl.Change[]} aChanges - Array of changes to be reverted
-		 * @param {object} mPropertyBag - Object with additional properties
-		 * @param {sap.ui.core.Component} mPropertyBag.appComponent - Component instance that is currently loading
-		 * @param {sap.ui.core.util.reflection.BaseTreeModifier} mPropertyBag.modifier - Polymorph reuse operations handling the changes on the given view type
-		 * @param {sap.ui.fl.FlexController} mPropertyBag.flexController - Instance of the flex controller the change is saved in
-		 * @returns {Promise|sap.ui.fl.Utils.FakePromise} Promise/FakePromise that resolves as soon as all changes are reverted
-		 */
-		revertMultipleChanges: function(aChanges, mPropertyBag) {
-			var aPromiseStack = [];
-			aChanges.forEach(function(oChange) {
-				// Queued 'state' will be removed once the revert process is done
-				oChange.setQueuedForRevert();
-				aPromiseStack.push(function() {
-					var oSelector = oChange.getSelector && oChange.getSelector();
-					var oControl = mPropertyBag.modifier.bySelector(oSelector, mPropertyBag.appComponent);
-					if (!oControl) {
-						Log.warning("A flexibility change tries to revert changes on a nonexistent control with id " + oSelector.id);
-						return new FlUtils.FakePromise();
-					}
-					var mRevertProperties = {
-						modifier: mPropertyBag.modifier,
-						appComponent: mPropertyBag.appComponent,
-						view: FlUtils.getViewForControl(oControl)
-					};
+	/**
+	 * Reverts a specific change on the passed control if it is currently applied.
+	 *
+	 * @param {sap.ui.fl.Change} oChange - Change object that should be reverted on the passed control
+	 * @param {sap.ui.core.Control} oControl - Control which is the target of the passed change
+	 * @param {object} mPropertyBag - Object with parameters as properties
+	 * @param {sap.ui.core.Component} mPropertyBag.appComponent - Component instance that is currently loading
+	 * @param {object} mPropertyBag.appDescriptor - App descriptor containing the metadata of the current application
+	 * @param {sap.ui.core.util.reflection.BaseTreeModifier} mPropertyBag.modifier - Polymorph reuse operations handling the changes on the given view type
+	 * @param {sap.ui.core.mvc.View} mPropertyBag.view - View to process
+	 * @returns {Promise|sap.ui.fl.Utils.FakePromise} Resolving Promise/FakePromise with either the control (success) or <code>false</code> (failure) as value
+	 */
+	Reverter.revertChangeOnControl = function(oChange, oControl, mPropertyBag) {
+		var mControl = Utils.getControlIfTemplateAffected(oChange, oControl, mPropertyBag);
+		var oChangeHandler;
 
-					return Reverter.revertChangeOnControl(oChange, oControl, mRevertProperties).then(function(vRevertResult) {
-						FlexCustomData.destroyAppliedCustomData(vRevertResult || oControl, oChange, mPropertyBag.modifier);
-						return !!vRevertResult;
-					})
-					.then(function(bSuccess) {
-						if (bSuccess) {
-							// TODO should be changed as soon as new flex persistence is in place
-							mPropertyBag.flexController._oChangePersistence._deleteChangeInMap(oChange);
-						}
-					});
-				});
+		return Utils.getChangeHandler(oChange, mControl, mPropertyBag).then(function(oReturnedChangeHandler) {
+			oChangeHandler = oReturnedChangeHandler;
+		})
+		.then(_waitForApplyIfNecessary.bind(null, oChange))
+		.then(function() {
+			if (oChange.isApplyProcessFinished()) {
+				oChange.startReverting();
+				return oChangeHandler.revertChange(oChange, mControl.control, mPropertyBag);
+			}
+			throw Error("Change was never applied");
+		})
+		.then(function() {
+			// After revert the relevant control for the change might have changed, therefore it must be retrieved again (e.g. stashing)
+			mControl.control = mPropertyBag.modifier.bySelector(oChange.getSelector(), mPropertyBag.appComponent, mPropertyBag.view);
+			if (mControl.bTemplateAffected) {
+				return mPropertyBag.modifier.updateAggregation(mControl.control, oChange.getContent().boundAggregation);
+			}
+			return undefined;
+		})
+		.then(function () {
+			oChange.markRevertFinished();
+			return mControl.control;
+		})
+		.catch(function(oError) {
+			var sErrorMessage = "Change could not be reverted: " + oError.message;
+			Log.error(sErrorMessage);
+			oChange.markRevertFinished(sErrorMessage);
+			return false;
+		});
+	};
+
+	/**
+	 * Reverts all given changes in one app component.
+	 *
+	 * @param {sap.ui.fl.Change[]} aChanges - Array of changes to be reverted
+	 * @param {object} mPropertyBag - Object with additional properties
+	 * @param {sap.ui.core.Component} mPropertyBag.appComponent - Component instance that is currently loading
+	 * @param {sap.ui.core.util.reflection.BaseTreeModifier} mPropertyBag.modifier - Polymorph reuse operations handling the changes on the given view type
+	 * @param {sap.ui.fl.FlexController} mPropertyBag.flexController - Instance of the flex controller the change is saved in
+	 * @returns {Promise|sap.ui.fl.Utils.FakePromise} Promise/FakePromise that resolves as soon as all changes are reverted
+	 */
+	Reverter.revertMultipleChanges = function(aChanges, mPropertyBag) {
+		var aPromiseStack = [];
+		aChanges.forEach(function(oChange) {
+			// Queued 'state' will be removed once the revert process is done
+			oChange.setQueuedForRevert();
+			aPromiseStack.push(function() {
+				var oSelector = oChange.getSelector && oChange.getSelector();
+				var oControl = mPropertyBag.modifier.bySelector(oSelector, mPropertyBag.appComponent);
+				if (!oControl) {
+					Log.warning("A flexibility change tries to revert changes on a nonexistent control with id " + oSelector.id);
+					return new FlUtils.FakePromise();
+				}
+				var mRevertProperties = {
+					modifier: mPropertyBag.modifier,
+					appComponent: mPropertyBag.appComponent,
+					view: FlUtils.getViewForControl(oControl)
+				};
+				return revertAndDeleteChangeOnControl(oChange, oControl, mRevertProperties, mPropertyBag);
 			});
+		});
 
-			return FlUtils.execPromiseQueueSequentially(aPromiseStack);
-		}
+		return FlUtils.execPromiseQueueSequentially(aPromiseStack);
 	};
+
 	return Reverter;
 });

package/src/sap/ui/fl/apply/_internal/changes/Utils.js

@@ -20,11 +20,20 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.connectors.changes.Utils
 	 * @since 1.70
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal, sap.ui.fl.write._internal
 	 */
 
+	 function isDependencyStillValid(oChange, bHasChangeApplyFinishedCustomData) {
+		// if change is already applied OR if apply process has started,
+		// then dependency is no more valid
+		if (bHasChangeApplyFinishedCustomData || oChange.hasApplyProcessStarted()) {
+			return false;
+		}
+		return true;
+	}
+
 	var Utils = {
 		/**
 		 * Returns the control map containing control, controlType, bTemplateAffected and originalControl
@@ -87,16 +96,18 @@ sap.ui.define([
 			});
 		},
 
-		checkIfDependencyIsStillValid: function(oAppComponent, oModifier, mChangesMap, sChangeId) {
+		checkIfDependencyIsStillValidSync: function(oAppComponent, oModifier, mChangesMap, sChangeId) {
 			var oChange = FlUtils.getChangeFromChangesMap(mChangesMap.mChanges, sChangeId);
 			var oControl = oModifier.bySelector(oChange.getSelector(), oAppComponent);
+			var bHasChangeApplyFinishedCustomData = FlexCustomData.sync.hasChangeApplyFinishedCustomData(oControl, oChange);
+			return isDependencyStillValid(oChange, bHasChangeApplyFinishedCustomData);
+		},
 
-			// if change is already applied OR if apply process has started,
-			// then dependency is no more valid
-			if (FlexCustomData.hasChangeApplyFinishedCustomData(oControl, oChange, oModifier) || oChange.hasApplyProcessStarted()) {
-				return false;
-			}
-			return true;
+		checkIfDependencyIsStillValid: function(oAppComponent, oModifier, mChangesMap, sChangeId) {
+			var oChange = FlUtils.getChangeFromChangesMap(mChangesMap.mChanges, sChangeId);
+			var oControl = oModifier.bySelector(oChange.getSelector(), oAppComponent);
+			return FlexCustomData.hasChangeApplyFinishedCustomData(oControl, oChange, oModifier)
+				.then(isDependencyStillValid.bind(null, oChange));
 		}
 	};
 

package/src/sap/ui/fl/apply/_internal/changes/descriptor/Preprocessor.js

@@ -28,7 +28,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.Preprocessor
 	 * @experimental
 	 * @since 1.74
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/Registration.js

@@ -18,7 +18,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.Registration
 	 * @experimental
 	 * @since 1.74
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/RegistrationBuild.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.RegistrationBuild
 	 * @experimental
 	 * @since 1.77
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/app/AddAnnotationsToOData.js

@@ -139,7 +139,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.app.AddAnnotationsToOData
 	 * @experimental
 	 * @since 1.87
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/app/ChangeDataSource.js

@@ -26,7 +26,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.app.ChangeDataSouce
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/app/ChangeInbound.js

@@ -25,7 +25,7 @@ sap.ui.define([
      *
      * @namespace sap.ui.fl.apply._internal.changes.descriptor.app.ChangeInbound
      * @experimental
-     * @version 1.92.0
+     * @version 1.93.3
      * @private
      * @ui5-restricted sap.ui.fl.apply._internal
      */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/app/SetTitle.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.app.SetTitle
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/fiori/SetRegistrationIds.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.fiori.SetRegistrationIds
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ovp/AddNewCard.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ovp.AddNewCard
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ovp/ChangeCard.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ovp.ChangeCard
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ovp/DeleteCard.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ovp.DeleteCard
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ui5/AddComponentUsages.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ui5.AddComponentUsages
 	 * @experimental
 	 * @since 1.86
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ui5/AddLibrary.js

@@ -49,7 +49,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ui5.AddLibrary
 	 * @experimental
 	 * @since 1.74
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ui5/AddNewModel.js

@@ -97,7 +97,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ui5.AddNewModel
 	 * @experimental
 	 * @since 1.87
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ui5/AddNewModelEnhanceWith.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ui5.AddNewModelEnhanceWith
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ui5/SetFlexExtensionPointEnabled.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ui5.SetFlexExtensionPointEnabled
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/changes/descriptor/ui5/SetMinUI5Version.js

@@ -22,7 +22,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.changes.descriptor.ui5.SetMinUI5Version
 	 * @experimental
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */

package/src/sap/ui/fl/apply/_internal/connectors/ObjectStorageUtils.js

@@ -12,7 +12,7 @@ sap.ui.define([], function () {
 	 *
 	 * @namespace sap.ui.fl.initial._internal.connectors.Utils
 	 * @since 1.70
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.initial._internal, sap.ui.fl.write._internal
 	 */

package/src/sap/ui/fl/apply/_internal/extensionPoint/Processor.js

@@ -32,7 +32,7 @@ function(
 	 * @class
 	 * @constructor
 	 * @author SAP SE
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 */
 	var Processor;
 

package/src/sap/ui/fl/apply/_internal/flexState/FlexState.js

@@ -70,7 +70,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.flexState.FlexState
 	 * @experimental
 	 * @since 1.73
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal
 	 */
@@ -97,6 +97,7 @@ sap.ui.define([
 			pathInResponse: ["comp.variants", "comp.standardVariants", "comp.defaultVariants", "comp.changes"]
 		}
 	};
+	var _mExternalCompVariantInputData = {};
 
 	function updateComponentData(mPropertyBag) {
 		var oComponent = Component.get(mPropertyBag.componentId);
@@ -332,6 +333,17 @@ sap.ui.define([
 		}
 	};
 
+	FlexState.setInitialNonFlCompVariantData = function(sReference, sPersistencyKey, oStandardVariant, aVariants) {
+		_mExternalCompVariantInputData[sReference] = {};
+		_mExternalCompVariantInputData[sReference][sPersistencyKey] = {};
+		_mExternalCompVariantInputData[sReference][sPersistencyKey].standardVariant = oStandardVariant;
+		_mExternalCompVariantInputData[sReference][sPersistencyKey].variants = aVariants;
+	};
+
+	FlexState.getInitialNonFlCompVariantData = function(sReference) {
+		return _mExternalCompVariantInputData[sReference];
+	};
+
 	/**
 	 * Removes the saved filtered storage response and internal maps for the given reference.
 	 * The next initialize call will add it again.

package/src/sap/ui/fl/apply/_internal/flexState/Loader.js

@@ -29,7 +29,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.flexState.Loader
 	 * @experimental
 	 * @since 1.74
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl.apply._internal.flexState
 	 */

package/src/sap/ui/fl/apply/_internal/flexState/ManifestUtils.js

@@ -37,7 +37,7 @@ function(
 	 * @namespace sap.ui.fl.apply._internal.flexState.ManifestUtils
 	 * @experimental
 	 * @since 1.74
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 */
 	var ManifestUtils = {

package/src/sap/ui/fl/apply/_internal/flexState/UI2Personalization/UI2PersonalizationState.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.flexState.UI2Personalization.UI2PersonalizationState
 	 * @experimental Since 1.75
 	 * @since 1.75
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted
 	 */

package/src/sap/ui/fl/apply/_internal/flexState/changes/DependencyHandler.js

@@ -23,7 +23,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.flexState.changes.DependencyHandler
 	 * @experimental
 	 * @since 1.74
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl
 	 */

package/src/sap/ui/fl/apply/_internal/flexState/changes/ExtensionPointState.js

@@ -26,7 +26,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.flexState.changes.ExtensionPointState
 	 * @experimental Since 1.79
 	 * @since 1.79
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted
 	 */

package/src/sap/ui/fl/apply/_internal/flexState/compVariants/CompVariantMerger.js

@@ -114,7 +114,7 @@ sap.ui.define([
 	 *
 	 * @namespace sap.ui.fl.apply._internal.flexState.compVariants.CompVariantMerger
 	 * @since 1.86
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted sap.ui.fl
 	 */
@@ -179,7 +179,7 @@ sap.ui.define([
 		 *
 		 * @function
 		 * @since 1.89
-		 * @version 1.92.0
+		 * @version 1.93.3
 		 * @private
 		 * @ui5-restricted sap.ui.fl
 		 *

package/src/sap/ui/fl/apply/_internal/flexState/controlVariants/Switcher.js

@@ -95,7 +95,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.flexState.controlVariants.Switcher
 	 * @experimental Since 1.74
 	 * @since 1.74
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted
 	 */

package/src/sap/ui/fl/apply/_internal/flexState/controlVariants/VariantManagementState.js

@@ -45,7 +45,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply._internal.flexState.controlVariants.VariantManagementState
 	 * @experimental Since 1.74
 	 * @since 1.74
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @private
 	 * @ui5-restricted
 	 */

package/src/sap/ui/fl/apply/api/ControlVariantApplyAPI.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * @namespace sap.ui.fl.apply.api.ControlVariantApplyAPI
 	 * @experimental Since 1.67
 	 * @since 1.67
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @public
 	 */
 	var ControlVariantApplyAPI = /** @lends sap.ui.fl.apply.api.ControlVariantApplyAPI */{

package/src/sap/ui/fl/apply/api/SmartVariantManagementApplyAPI.js

@@ -49,6 +49,8 @@ sap.ui.define([
 		}).then(function () {
 			var sPersistencyKey = CompVariantUtils.getPersistencyKey(oControl);
 			var mCompVariantsMap = FlexState.getCompVariantsMap(sReference);
+			//Store external input data to FlexState so they can be restored after invalidating cache
+			FlexState.setInitialNonFlCompVariantData(sReference, sPersistencyKey, mPropertyBag.standardVariant, mPropertyBag.variants);
 			return mCompVariantsMap._initialize(sPersistencyKey, mPropertyBag.variants);
 		});
 	}

package/src/sap/ui/fl/changeHandler/AddIFrame.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 *
 	 * @alias sap.ui.fl.changeHandler.AddIFrame
 	 * @author SAP SE
-	 * @version 1.92.0
+	 * @version 1.93.3
 	 * @since 1.72
 	 * @private
 	 */
@@ -33,6 +33,7 @@ sap.ui.define([
 	 * @param {sap.ui.core.Control} oControl Control that matches the change selector for applying the change
 	 * @param {object} mPropertyBag Map of properties
 	 * @param {object} mPropertyBag.modifier Modifier for the controls
+	 * @returns {Promise} Promise resolving when the change is successfully applied
 	 * @ui5-restricted sap.ui.fl
 	 */
 	AddIFrame.applyChange = function(oChange, oControl, mPropertyBag) {
@@ -40,14 +41,27 @@ sap.ui.define([
 		var oChangeDefinition = oChange.getDefinition();
 		var oView = mPropertyBag.view;
 		var sAggregationName = oChangeDefinition.content.targetAggregation;
-		var oAggregationDefinition = oModifier.findAggregation(oControl, sAggregationName);
-		if (!oAggregationDefinition) {
-			throw new Error("The given Aggregation is not available in the given control: " + oModifier.getId(oControl));
-		}
-		var iIndex = getTargetAggregationIndex(oChange, oControl, mPropertyBag);
-		var oIFrame = createIFrame(oChange, mPropertyBag, oChangeDefinition.content.selector);
-		oModifier.insertAggregation(oControl, sAggregationName, oIFrame, iIndex, oView);
-		oChange.setRevertData([oModifier.getId(oIFrame)]);
+		var iIndex;
+		var oIFrame;
+		return Promise.resolve()
+			.then(oModifier.findAggregation.bind(oModifier, oControl, sAggregationName))
+			.then(function(oAggregationDefinition) {
+				if (!oAggregationDefinition) {
+					throw new Error("The given Aggregation is not available in the given control: " + oModifier.getId(oControl));
+				}
+				return getTargetAggregationIndex(oChange, oControl, mPropertyBag);
+			})
+			.then(function(iRetrievedIndex) {
+				iIndex = iRetrievedIndex;
+				return createIFrame(oChange, mPropertyBag, oChangeDefinition.content.selector);
+			})
+			.then(function(oCreatedIFrame) {
+				oIFrame = oCreatedIFrame;
+				return oModifier.insertAggregation(oControl, sAggregationName, oIFrame, iIndex, oView);
+			})
+			.then(function() {
+				oChange.setRevertData([oModifier.getId(oIFrame)]);
+			});
 	};
 
 	/**