@openui5/sap.ui.rta 1.114.0 → 1.115.0

package/.reuse/dep5

@@ -192,11 +192,6 @@ Copyright: 2012 Matteo Spinelli
 License: MIT
 Comment: these files belong to: cubiq.org - swipeview
 
-Files: src/sap.ui.core/src/sap/ui/thirdparty/mobify-carousel.js
-Copyright: Mobify R&D Inc.
-License: MIT
-Comment: these files belong to: Mobify.js
-
 Files: src/sap.ui.core/src/sap/ui/thirdparty/zyngascroll.js
 Copyright: 2011 Zynga Inc.
 License: MIT

package/THIRDPARTY.txt

@@ -143,12 +143,6 @@ License: MIT
 License Text: https://github.com/SAP/openui5/blob/master/LICENSES/MIT.txt
 Contained in: src/sap.ui.core/src/sap/ui/thirdparty/swipe-view.js
 
-Component: Mobify.js, version: 1.1
-Copyright: Mobify R&D Inc.
-License: MIT
-License Text: https://github.com/SAP/openui5/blob/master/LICENSES/MIT.txt
-Contained in: src/sap.ui.core/src/sap/ui/thirdparty/mobify-carousel.js
-
 Component: Zynga Scroller, version: 1.2.1-0-g5d43806
 Copyright: 2011 Zynga Inc.
 License: MIT

package/package.json

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

package/src/sap/ui/rta/.library

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

package/src/sap/ui/rta/Client.js

@@ -57,7 +57,7 @@ sap.ui.define([
 	 * @alias sap.ui.rta.Client
 	 * @author SAP SE
 	 * @since 1.56.0
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @private
 	 * @ui5-restricted
 	 */

package/src/sap/ui/rta/RuntimeAuthoring.js

@@ -120,7 +120,7 @@ sap.ui.define([
 	 * @class The runtime authoring allows to adapt the fields of a running application.
 	 * @extends sap.ui.base.ManagedObject
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.30
@@ -441,6 +441,7 @@ sap.ui.define([
 	 * @public
 	 */
 	RuntimeAuthoring.prototype.start = function() {
+		var bIsAutomaticRestart = RuntimeAuthoring.needsRestart(this.getLayer());
 		var oDesignTimePromise;
 		var vError;
 		// Create DesignTime
@@ -454,8 +455,8 @@ sap.ui.define([
 			}
 
 			return this._loadUShellServicesPromise
-			.then(initVersioning.bind(this))
-			.then(initContextBasedAdaptations.bind(this))
+			.then(initVersioning.bind(this, bIsAutomaticRestart))
+			.then(initContextBasedAdaptations.bind(this, bIsAutomaticRestart))
 			/*
 			 Check if the application has personalized changes and reload without them;
 			 Also Check if the application has an available draft and if yes, reload with those changes.
@@ -575,7 +576,7 @@ sap.ui.define([
 					validateFlexEnabled(this);
 				}
 				this._sStatus = STARTED;
-				RuntimeAuthoring.disableRestart(Layer.CUSTOMER);
+				RuntimeAuthoring.disableRestart(this.getLayer());
 				this.fireStart({
 					editablePluginsCount: this.getPluginManager().getEditableOverlaysCount()
 				});
@@ -1063,33 +1064,30 @@ sap.ui.define([
 	 * @returns {Promise} Resolves as soon as the MessageBox is closed
 	 */
 	function onStackModified() {
-		var bBackEndDraftExists = this._oVersionsModel.getProperty("/backendDraft");
-		var bDraftDisplayed = this._oVersionsModel.getProperty("/displayedVersion") === Version.Number.Draft;
 		var oCommandStack = this.getCommandStack();
 		var bCanUndo = oCommandStack.canUndo();
 
+		// FIXME Missing check whether action is saveable e.g. switching a view should not trigger this
+		// bCanUndo and _bUserDiscardedDraft seem to be redundant logic,
+		// because displayedVersion would be draft already
 		if (
 			!this.getShowToolbars() ||
 			!bCanUndo ||
-			this._bUserDiscardedDraft ||
-			bDraftDisplayed ||
-			!bBackEndDraftExists
+			this._bUserDiscardedDraft
 		) {
-			return modifyStack.call(this);
+			modifyStack.call(this);
+			return;
 		}
 
 		// warn the user: the existing draft would be discarded in case the user saves
-		return Utils.showMessageBox("warning", "MSG_DRAFT_DISCARD_AND_CREATE_NEW_DIALOG", {
-			titleKey: "TIT_DRAFT_DISCARD_DIALOG",
-			actions: [MessageBox.Action.OK, MessageBox.Action.CANCEL],
-			emphasizedAction: MessageBox.Action.OK
-		})
-		.then(function(sAction) {
-			if (sAction === MessageBox.Action.OK) {
-				discardDraftConfirmed.call(this);
-			} else {
-				this.undo();
+		Utils.checkDraftOverwrite(this._oVersionsModel)
+		.then(function(bDialogShown) {
+			if (bDialogShown) {
+				this._bUserDiscardedDraft = true;
 			}
+			modifyStack.call(this);
+		}.bind(this), function() {
+			this.undo();
 		}.bind(this));
 	}
 
@@ -1211,6 +1209,10 @@ sap.ui.define([
 			// Save changes on the current layer and discard dirty changes on other layers
 			mPropertyBag.saveAsDraft = this.getLayer() === Layer.CUSTOMER;
 		}
+		if (this._oContextBasedAdaptationsModel.getProperty("/contextBasedAdaptationsEnabled")) {
+			// If an adaptation is being processed, saving without exiting must retrieve the updated state of the adaptation
+			mPropertyBag.adaptationId = bIsExit ? undefined : this._oContextBasedAdaptationsModel.getProperty("/displayedAdaptation/id");
+		}
 
 		return this._oSerializer.saveCommands(mPropertyBag)
 			.then(function() {
@@ -1344,18 +1346,18 @@ sap.ui.define([
 				emphasizedAction: MessageBox.Action.YES
 			}).then(function(sAction) {
 				if (sAction === MessageBox.Action.YES) {
-					this._serializeToLrep()
+					return this._serializeToLrep()
 						.then(callbackFn);
 				} else if (sAction === MessageBox.Action.NO) {
 					// avoids the data loss popup; a reload is triggered later and will destroy RTA & the command stack
 					this.getCommandStack().removeAllCommands(true);
-					callbackFn();
+					return callbackFn();
 				}
-				return undefined;
+				return Promise.resolve();
 			}.bind(this));
-			return;
+			return Promise.resolve();
 		}
-		callbackFn();
+		return callbackFn();
 	}
 
 	function onSwitchAdaptation(oEvent) {
@@ -1366,15 +1368,20 @@ sap.ui.define([
 		var sAdaptationId = oEvent.getParameter("adaptationId");
 		this._sSwitchToAdaptationId = sAdaptationId;
 		return handleDataLoss.call(this, "MSG_SWITCH_VERSION_DIALOG", "BTN_SWITCH_ADAPTATIONS",
-			switchAdaptation.bind(this, this._sSwitchToAdaptationId));
+			switchAdaptation.bind(this, this._sSwitchToAdaptationId))
+			.catch(function(oError) {
+				Utils.showMessageBox("error", "MSG_SWITCH_ADAPTATION_FAILED", {error: oError});
+				Log.error("sap.ui.rta: " + oError.stack || oError.message || oError);
+			});
 	}
 
 	function switchAdaptation(sAdaptationId) {
 		var sVersion = this._oVersionsModel.getProperty("/displayedVersion");
-		switchVersion.call(this, sVersion, sAdaptationId);
+		return switchVersion.call(this, sVersion, sAdaptationId);
 	}
 
 	function onSwitchVersion(oEvent) {
+		var fnCallback = oEvent.getParameter("callback") || function() {};
 		var sVersion = oEvent.getParameter("version");
 		var sDisplayedVersion = this._oVersionsModel.getProperty("/displayedVersion");
 
@@ -1384,24 +1391,30 @@ sap.ui.define([
 		}
 
 		this._sSwitchToVersion = sVersion;
-		return handleDataLoss.call(this, "MSG_SWITCH_VERSION_DIALOG", "TIT_SWITCH_VERSION_DIALOG",
-			switchVersion.bind(this, this._sSwitchToVersion));
+		handleDataLoss.call(this, "MSG_SWITCH_VERSION_DIALOG", "TIT_SWITCH_VERSION_DIALOG",
+			switchVersion.bind(this, this._sSwitchToVersion))
+			.then(fnCallback)
+			.catch(function(oError) {
+				Utils.showMessageBox("error", "MSG_SWITCH_VERSION_FAILED", {error: oError});
+				Log.error("sap.ui.rta: " + oError.stack || oError.message || oError);
+			});
 	}
 
 	function switchVersion(sVersion, sAdaptationId) {
 		RuntimeAuthoring.enableRestart(this.getLayer(), this.getRootControlInstance());
 
-		VersionsAPI.loadVersionForApplication({
+		return VersionsAPI.loadVersionForApplication({
 			control: this.getRootControlInstance(),
 			layer: this.getLayer(),
 			version: sVersion,
 			adaptationId: sAdaptationId
+		}).then(function() {
+			var oReloadInfo = {
+				versionSwitch: true,
+				version: sVersion
+			};
+			ReloadManager.triggerReload(oReloadInfo);
 		});
-		var oReloadInfo = {
-			versionSwitch: true,
-			version: sVersion
-		};
-		ReloadManager.triggerReload(oReloadInfo);
 	}
 
 	function onPublishVersion() {
@@ -1420,11 +1433,6 @@ sap.ui.define([
 		});
 	}
 
-	function discardDraftConfirmed() {
-		this._bUserDiscardedDraft = true;
-		modifyStack.call(this);
-	}
-
 	function isOldVersionDisplayed() {
 		return VersionsAPI.isOldVersionDisplayed({
 			control: this.getRootControlInstance(),
@@ -1439,7 +1447,15 @@ sap.ui.define([
 		});
 	}
 
-	function initVersioning() {
+	/**
+	 * Inits version models. Clears old state if RTA is starting from end user mode (no switch)
+	 * @param {boolean} bIsAutomaticRestart - If true this is not an RTA start but a reload due to version/adaptation switch
+	 * @returns {Promise<void>} - Promise
+	 */
+	function initVersioning(bIsAutomaticRestart) {
+		if (!bIsAutomaticRestart) {
+			VersionsAPI.clearInstances();
+		}
 		return VersionsAPI.initialize({
 			control: this.getRootControlInstance(),
 			layer: this.getLayer()
@@ -1448,7 +1464,15 @@ sap.ui.define([
 		}.bind(this));
 	}
 
-	function initContextBasedAdaptations() {
+	/**
+	 * Inits CBA models. Clears old state if RTA is starting from end user mode (no switch)
+	 * @param {boolean} bIsAutomaticRestart - If true this is not an RTA start but a reload due to version/adaptation switch
+	 * @returns {Promise<void>} - Promise
+	 */
+	function initContextBasedAdaptations(bIsAutomaticRestart) {
+		if (!bIsAutomaticRestart) {
+			ContextBasedAdaptationsAPI.clearInstances();
+		}
 		return ContextBasedAdaptationsAPI.initialize({
 			control: this.getRootControlInstance(),
 			layer: this.getLayer()
@@ -1754,7 +1778,7 @@ sap.ui.define([
 						if (oError && oError.message && oError.message.indexOf("The following Change cannot be applied because of a dependency") > -1) {
 							Utils.showMessageBox("error", "MSG_DEPENDENCY_ERROR", {error: oError});
 						}
-						Log.error("sap.ui.rta: " + oError.message);
+						Log.error("sap.ui.rta:", oError.message, oError.stack);
 					});
 			}
 			return undefined;

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

@@ -9,6 +9,7 @@ sap.ui.define([
 	"sap/ui/fl/Utils",
 	"sap/ui/fl/Layer",
 	"sap/ui/fl/LayerUtils",
+	"sap/ui/fl/write/api/Version",
 	"sap/ui/dt/OverlayUtil",
 	"sap/ui/dt/DOMUtil",
 	"sap/ui/dt/ElementUtil",
@@ -26,6 +27,7 @@ function(
 	FlexUtils,
 	Layer,
 	FlexLayerUtils,
+	Version,
 	OverlayUtil,
 	DOMUtil,
 	ElementUtil,
@@ -45,7 +47,7 @@ function(
 	 *
 	 * @namespace
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 *
 	 * @private
 	 * @since 1.30
@@ -407,7 +409,7 @@ function(
 		var oRenderer = Utils.getFiori2Renderer();
 		return oRenderer
 			&& oRenderer.getRootControl
-			&& oRenderer.getRootControl().getOUnifiedShell().getHeader();
+			&& oRenderer.getRootControl().getShellHeader();
 	};
 
 	/**
@@ -644,5 +646,37 @@ function(
 			});
 	};
 
+	/**
+	 * Check if an existing draft would be overwritten if a change is done on the currently shown version
+	 * If so it opens a confirmation dialog.
+	 * @param {object} oVersionsModel The versions model
+	 * @return {Promise.<boolean>} It either resolves with an indicator whether a confirmation
+	 * was shown or rejects with "cancel" if cancel was pressed
+	 */
+	Utils.checkDraftOverwrite = function(oVersionsModel) {
+		var bBackEndDraftExists = oVersionsModel.getProperty("/backendDraft");
+		var bDraftDisplayed = oVersionsModel.getProperty("/displayedVersion") === Version.Number.Draft;
+
+		if (
+			bDraftDisplayed ||
+			!bBackEndDraftExists
+		) {
+			return Promise.resolve(false);
+		}
+
+		// warn the user: the existing draft would be discarded in case the user saves
+		return Utils.showMessageBox("warning", "MSG_DRAFT_DISCARD_AND_CREATE_NEW_DIALOG", {
+			titleKey: "TIT_DRAFT_DISCARD_DIALOG",
+			actions: [MessageBox.Action.OK, MessageBox.Action.CANCEL],
+			emphasizedAction: MessageBox.Action.OK
+		})
+		.then(function(sAction) {
+			if (sAction !== MessageBox.Action.OK) {
+				throw "cancel";
+			}
+			return true;
+		});
+	};
+
 	return Utils;
 }, /* bExport= */true);

package/src/sap/ui/rta/appVariant/AppVariantManager.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.base.ManagedObject
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.53

package/src/sap/ui/rta/command/AddIFrame.js

@@ -16,7 +16,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.75

package/src/sap/ui/rta/command/AddProperty.js

@@ -14,7 +14,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.78

package/src/sap/ui/rta/command/AddXML.js

@@ -16,7 +16,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.54

package/src/sap/ui/rta/command/AddXMLAtExtensionPoint.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.76

package/src/sap/ui/rta/command/AppDescriptorCommand.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * @extends sap.ui.rta.command.BaseCommand
 	 *
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/ui/rta/command/BaseCommand.js

@@ -13,7 +13,7 @@ sap.ui.define(["sap/ui/base/ManagedObject"], function(ManagedObject) {
 	 * @extends sap.ui.base.ManagedObject
 	 *
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/ui/rta/command/BindProperty.js

@@ -26,7 +26,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.38

package/src/sap/ui/rta/command/Combine.js

@@ -16,7 +16,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.46

package/src/sap/ui/rta/command/CommandFactory.js

@@ -485,7 +485,7 @@ sap.ui.define([
 	 * @extends sap.ui.base.ManagedObject
 	 *
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/ui/rta/command/CompositeCommand.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * @extends sap.ui.rta.command.BaseCommand
 	 *
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 *
 	 * @constructor
 	 * @private

package/src/sap/ui/rta/command/ControlVariantConfigure.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.BaseCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.52
@@ -79,9 +79,7 @@ sap.ui.define([
 			this._aPreparedChanges.push(this.oModel.addVariantChange(this.sVariantManagementReference, mChangeProperties));
 		}.bind(this));
 
-		return Promise.resolve().then(function() {
-			this.oModel.checkUpdate(true);
-		}.bind(this));
+		return Promise.resolve();
 	};
 
 	/**
@@ -108,10 +106,8 @@ sap.ui.define([
 			this.oModel.deleteVariantChange(this.sVariantManagementReference, mPropertyBag, oChange);
 		}.bind(this));
 
-		return Promise.resolve().then(function() {
-			this.oModel.checkUpdate(true);
-			this._aPreparedChanges = null;
-		}.bind(this));
+		this._aPreparedChanges = null;
+		return Promise.resolve();
 	};
 
 	return ControlVariantConfigure;

package/src/sap/ui/rta/command/ControlVariantSave.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.BaseCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.86
@@ -59,11 +59,10 @@ sap.ui.define([
 		this._aDirtyChanges = this.getModel()._getDirtyChangesFromVariantChanges(this._aControlChanges);
 		this._aDirtyChanges.forEach(function(oChange) {
 			if (oChange.getFileType() === "change") {
-				oChange.assignedToVariant = true;
+				oChange.setSavedToVariant(true);
 			}
 		});
-		this.getModel().oData[this.sVariantManagementReference].modified = false;
-		this.getModel().checkUpdate(true);
+		this.getModel().invalidateMap();
 		return Promise.resolve();
 	};
 
@@ -75,10 +74,10 @@ sap.ui.define([
 	ControlVariantSave.prototype.undo = function() {
 		this._aDirtyChanges.forEach(function(oChange) {
 			if (oChange.getFileType() === "change") {
-				oChange.assignedToVariant = false;
+				oChange.setSavedToVariant(false);
 			}
 		});
-		this.getModel().checkDirtyStateForControlModels([this.sVariantManagementReference]);
+		this.getModel().invalidateMap();
 		return Promise.resolve();
 	};
 

package/src/sap/ui/rta/command/ControlVariantSaveAs.js

@@ -28,7 +28,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.BaseCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.86
@@ -66,6 +66,8 @@ sap.ui.define([
 		this.oModel = this.getModel();
 		this.setSourceDefaultVariant(this.oModel.getData()[this.sVariantManagementReference].defaultVariant);
 		this.sLayer = mFlexSettings.layer;
+		var mComponentPropertyBag = mFlexSettings;
+		mComponentPropertyBag.variantManagementControl = this.oVariantManagementControl;
 
 		function storeEventParameters(oEvent, oArgs) {
 			var mParameters = oEvent.getParameters();
@@ -84,7 +86,7 @@ sap.ui.define([
 			this.oVariantManagementControl.attachSave({resolve: resolve}, storeEventParameters, this);
 			this.oVariantManagementControl.attachCancel({resolve: resolve}, handleCancel, this);
 			this.oVariantManagementControl.openSaveAsDialogForKeyUser(rtaUtils.getRtaStyleClassName(),
-				ContextSharingAPI.createComponent(mFlexSettings));
+				ContextSharingAPI.createComponent(mComponentPropertyBag));
 		}.bind(this))
 			.then(function(bState) {
 				return bState;
@@ -117,12 +119,10 @@ sap.ui.define([
 				this.sNewVariantReference = this._oVariantChange.getId();
 				this._aPreparedChanges.forEach(function(oChange) {
 					if (oChange.getFileType() === "change") {
-						oChange.assignedToVariant = true;
+						oChange.setSavedToVariant(true);
 					}
 				});
-				// Assigning changes to the variant might have an impact on the modified state
-				// Call the check again to make sure it is up to date
-				this.getModel().checkDirtyStateForControlModels([this.sVariantManagementReference]);
+				this.getModel().invalidateMap();
 			}.bind(this));
 	};
 
@@ -135,7 +135,7 @@ sap.ui.define([
 		if (this._oVariantChange) {
 			this._aPreparedChanges.forEach(function(oChange) {
 				if (oChange.getFileType() === "ctrl_variant_management_change") {
-					this.oModel.oFlexController.deleteChange(oChange, this.oAppComponent);
+					this.oModel.oFlexController.deleteChange(oChange);
 				}
 			}.bind(this));
 
@@ -151,11 +151,8 @@ sap.ui.define([
 					return this.oModel.addAndApplyChangesOnVariant(this._aControlChanges);
 				}.bind(this))
 				.then(function() {
-					this.oModel.getData()[this.sVariantManagementReference].defaultVariant = this.getSourceDefaultVariant();
-					this.oModel.getData()[this.sVariantManagementReference].originalDefaultVariant = this.getSourceDefaultVariant();
 					this._aPreparedChanges = null;
 					this._oVariantChange = null;
-					this.getModel().checkUpdate(true);
 				}.bind(this));
 		}
 		return Promise.resolve();

package/src/sap/ui/rta/command/ControlVariantSetTitle.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.BaseCommand
 	 * @author SAP SE
-	 * @version 1.114.0
+	 * @version 1.115.0
 	 * @constructor
 	 * @private
 	 * @since 1.50
@@ -69,7 +69,6 @@ sap.ui.define([
 	 */
 	ControlVariantSetTitle.prototype.execute = function() {
 		var oVariantManagementControl = this.getElement();
-		var oVariantManagementControlBinding = oVariantManagementControl.getTitle().getBinding("text");
 
 		this.oAppComponent = flUtils.getAppComponentForControl(oVariantManagementControl);
 		this.oModel = this.oAppComponent.getModel(ControlVariantApplyAPI.getVariantModelName());
@@ -91,7 +90,6 @@ sap.ui.define([
 		return Promise.resolve(this.oModel.addVariantChange(this.sVariantManagementReference, mPropertyBag))
 			.then(function(oChange) {
 				this._oVariantChange = oChange;
-				oVariantManagementControlBinding.checkUpdate(true); /*Force Update as binding key stays same*/
 			}.bind(this));
 	};
 
@@ -101,7 +99,6 @@ sap.ui.define([
 	 * @returns {Promise} Returns resolve after undo
 	 */
 	ControlVariantSetTitle.prototype.undo = function() {
-		var oVariantManagementControlBinding = this.getElement().getTitle().getBinding("text");
 		var mPropertyBag = {
 			variantReference: this.sCurrentVariant,
 			changeType: "setTitle",
@@ -112,7 +109,6 @@ sap.ui.define([
 		return Promise.resolve(this.oModel.deleteVariantChange(this.sVariantManagementReference, mPropertyBag, oChange))
 			.then(function() {
 				this._oVariantChange = null;
-				oVariantManagementControlBinding.checkUpdate(true); /*Force Update as binding key stays same*/
 			}.bind(this));
 	};