@openui5/sap.ui.rta 1.139.0 → 1.141.0

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

@@ -23,7 +23,7 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.base.ManagedObject
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.42
@@ -52,7 +52,7 @@ sap.ui.define([
 	}
 
 	/**
-	 * Promise to ensure that the event triggered methods are executed sequentionally.
+	 * Promise to ensure that the event triggered methods are executed sequentially.
 	 */
 	LREPSerializer.prototype._lastPromise = Promise.resolve();
 
@@ -67,14 +67,14 @@ sap.ui.define([
 	 * @public
 	 */
 	LREPSerializer.prototype.needsReload = function() {
-		this._lastPromise = this._lastPromise.catch(function() {
+		this._lastPromise = this._lastPromise
+		.catch(() => {
 			// _lastPromise chain must not be interrupted
-		}).then(function() {
+		})
+		.then(() => {
 			const aCommands = this.getCommandStack().getAllExecutedCommands();
-			return aCommands.some(function(oCommand) {
-				return !!oCommand.needsReload;
-			});
-		}.bind(this));
+			return aCommands.some((oCommand) => !!oCommand.needsReload);
+		});
 		return this._lastPromise;
 	};
 	/**
@@ -92,15 +92,17 @@ sap.ui.define([
 	 * @public
 	 */
 	LREPSerializer.prototype.saveCommands = function(mPropertyBag) {
-		this._lastPromise = this._lastPromise.catch(function(oError) {
+		this._lastPromise = this._lastPromise
+		.catch((oError) => {
 			Log.error(oError);
 			// _lastPromise chain must not be interrupted
-		}).then(function() {
+		})
+		.then(async () => {
 			const oRootControl = getRootControlInstance(this.getRootControl());
 			if (!oRootControl) {
 				throw new Error("Can't save commands without root control instance!");
 			}
-			return PersistenceWriteAPI.save({
+			await PersistenceWriteAPI.save({
 				selector: oRootControl,
 				skipUpdateCache: false,
 				draft: !!mPropertyBag.saveAsDraft,
@@ -110,12 +112,10 @@ sap.ui.define([
 				adaptationId: mPropertyBag.adaptationId,
 				condenseAnyLayer: mPropertyBag.condenseAnyLayer
 			});
-		}.bind(this))
-		.then(function() {
 			Log.info("UI adaptation successfully wrote changes to the persistence");
 			this.getCommandStack().setSaved(true);
 			this.getCommandStack().removeAllCommands();
-		}.bind(this));
+		});
 
 		return this._lastPromise;
 	};
@@ -145,21 +145,18 @@ sap.ui.define([
 	};
 
 	/**
-	 * @description
 	 * At this point command stack is not aware if the changes have been already booked for the new app variant.
 	 * Therefore if there shall be some UI changes present in command stack, we undo all the changes till the beginning.
 	 * In the last when user presses 'Save and Exit', there will be no change registered for the current app.
-	 * @param {boolean} bRemoveChanges if LREPSerializer should clear the dirty changes in the persistence
-	 * @returns {Promise} returns a promise with true or false
+	 * @param {boolean} bRemoveChanges - Whether LREPSerializer should clear the dirty changes in the persistence
+	 * @returns {Promise} Resolves with true or false
 	 */
-	LREPSerializer.prototype.clearCommandStack = function(bRemoveChanges) {
+	LREPSerializer.prototype.clearCommandStack = async function(bRemoveChanges) {
 		const oCommandStack = this.getCommandStack();
 
-		return this._triggerUndoChanges(bRemoveChanges)
-		.then(function() {
-			oCommandStack.removeAllCommands();
-			return true;
-		});
+		await this._triggerUndoChanges(bRemoveChanges);
+		oCommandStack.removeAllCommands();
+		return true;
 	};
 
 	return LREPSerializer;

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

@@ -20,13 +20,13 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.BaseCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.90
 	 * @alias sap.ui.rta.command.LocalReset
 	 */
-	var LocalReset = BaseCommand.extend("sap.ui.rta.command.LocalReset", {
+	const LocalReset = BaseCommand.extend("sap.ui.rta.command.LocalReset", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -49,7 +49,7 @@ sap.ui.define([
 	 * @override
 	 */
 	LocalReset.prototype.prepare = function(mFlexSettings) {
-		var oContainerControl = this.getElement();
+		const oContainerControl = this.getElement();
 		this._oAppComponent = FlUtils.getAppComponentForControl(oContainerControl);
 		this._aAffectedChanges = LocalResetAPI.getNestedUIChangesForControl(
 			oContainerControl,

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

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

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

@@ -17,13 +17,13 @@ function(
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.34
 	 * @alias sap.ui.rta.command.Move
 	 */
-	var Move = FlexCommand.extend("sap.ui.rta.command.Move", {
+	const Move = FlexCommand.extend("sap.ui.rta.command.Move", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -49,8 +49,8 @@ function(
 	 * @override
 	 */
 	Move.prototype._getChangeSpecificData = function() {
-		var mSource = this.getSource();
-		var mTarget = this.getTarget();
+		const mSource = this.getSource();
+		const mTarget = this.getTarget();
 
 		// replace elements by their id, unify format and help with serialization
 		if (mSource.parent) {
@@ -61,7 +61,7 @@ function(
 			mTarget.id = mTarget.parent.getId();
 			delete mTarget.parent;
 		}
-		var mSpecificInfo = {
+		const mSpecificInfo = {
 			changeType: this.getChangeType(),
 			content: {
 				source: mSource,

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

@@ -16,13 +16,13 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.34
 	 * @alias sap.ui.rta.command.Property
 	 */
-	var Property = FlexCommand.extend("sap.ui.rta.command.Property", {
+	const Property = FlexCommand.extend("sap.ui.rta.command.Property", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -49,7 +49,7 @@ sap.ui.define([
 	});
 
 	Property.prototype._getChangeSpecificData = function() {
-		var oElement = this.getElement();
+		const oElement = this.getElement();
 		// general format
 		return {
 			changeType: this.getChangeType(),

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

@@ -12,13 +12,13 @@ sap.ui.define(["sap/ui/rta/command/FlexCommand"], function(FlexCommand) {
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.42
 	 * @alias sap.ui.rta.command.Remove
 	 */
-	var Remove = FlexCommand.extend("sap.ui.rta.command.Remove", {
+	const Remove = FlexCommand.extend("sap.ui.rta.command.Remove", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -33,9 +33,9 @@ sap.ui.define(["sap/ui/rta/command/FlexCommand"], function(FlexCommand) {
 	});
 
 	Remove.prototype._getChangeSpecificData = function() {
-		var oElement = this.getRemovedElement() || this.getElement();
+		const oElement = this.getRemovedElement() || this.getElement();
 
-		var mSpecificInfo = {
+		const mSpecificInfo = {
 			changeType: this.getChangeType(),
 			removedElement: {
 				id: oElement.getId()

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

@@ -12,13 +12,13 @@ sap.ui.define(["sap/ui/rta/command/FlexCommand"], function(FlexCommand) {
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.34
 	 * @alias sap.ui.rta.command.Rename
 	 */
-	var Rename = FlexCommand.extend("sap.ui.rta.command.Rename", {
+	const Rename = FlexCommand.extend("sap.ui.rta.command.Rename", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -38,7 +38,7 @@ sap.ui.define(["sap/ui/rta/command/FlexCommand"], function(FlexCommand) {
 	});
 
 	Rename.prototype._getChangeSpecificData = function() {
-		var mSpecificInfo = {
+		const mSpecificInfo = {
 			changeType: this.getChangeType(),
 			content: {
 				renamedElement: {

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

@@ -16,13 +16,13 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.101
 	 * @alias sap.ui.rta.command.Resize
 	 */
-	var Resize = FlexCommand.extend("sap.ui.rta.command.Resize", {
+	const Resize = FlexCommand.extend("sap.ui.rta.command.Resize", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {

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

@@ -12,13 +12,13 @@ sap.ui.define(["sap/ui/rta/command/FlexCommand"], function(FlexCommand) {
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.44
 	 * @alias sap.ui.rta.command.Reveal
 	 */
-	var Reveal = FlexCommand.extend("sap.ui.rta.command.Reveal", {
+	const Reveal = FlexCommand.extend("sap.ui.rta.command.Reveal", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -32,7 +32,7 @@ sap.ui.define(["sap/ui/rta/command/FlexCommand"], function(FlexCommand) {
 	});
 
 	Reveal.prototype._getChangeSpecificData = function() {
-		var mSpecificChangeInfo = {
+		const mSpecificChangeInfo = {
 			changeType: this.getChangeType(),
 			content: {}
 		};

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

@@ -12,13 +12,13 @@ sap.ui.define(["sap/ui/rta/command/FlexCommand"], function(FlexCommand) {
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.44
 	 * @alias sap.ui.rta.command.Settings
 	 */
-	var Settings = FlexCommand.extend("sap.ui.rta.command.Settings", {
+	const Settings = FlexCommand.extend("sap.ui.rta.command.Settings", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {

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

@@ -14,13 +14,13 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.FlexCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.46
 	 * @alias sap.ui.rta.command.Split
 	 */
-	var Split = FlexCommand.extend("sap.ui.rta.command.Split", {
+	const Split = FlexCommand.extend("sap.ui.rta.command.Split", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -46,7 +46,7 @@ sap.ui.define([
 	 * @override
 	 */
 	Split.prototype._getChangeSpecificData = function() {
-		var mSpecificInfo = {
+		const mSpecificInfo = {
 			changeType: this.getChangeType(),
 			content: {
 				newElementIds: this.getNewElementIds(),

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

@@ -28,24 +28,16 @@ sap.ui.define([
 ) {
 	"use strict";
 
-	function toAvailableChanges(mChanges, aChanges, sFileName) {
-		var oChange = mChanges[sFileName];
-		if (oChange) {
-			aChanges.push(oChange);
-		}
-		return aChanges;
-	}
-
 	function pushToStack(oComponent, mComposite, oStack, oChange) {
-		var oSelector = oChange.getSelector && oChange.getSelector();
-		var oCommand = new Settings({
+		const oSelector = oChange.getSelector && oChange.getSelector();
+		const oCommand = new Settings({
 			selector: oSelector,
 			changeType: oChange.getChangeType(),
 			element: JsControlTreeModifier.bySelector(oSelector, oComponent)
 		});
 		oCommand._oPreparedChange = oChange;
 		// check if change belongs to a composite command
-		var sCompositeId = oChange.getSupportInformation().compositeCommand;
+		const sCompositeId = oChange.getSupportInformation().compositeCommand;
 		if (sCompositeId) {
 			if (!mComposite[sCompositeId]) {
 				mComposite[sCompositeId] = new CompositeCommand();
@@ -92,13 +84,13 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.base.ManagedObject
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.34
 	 * @alias sap.ui.rta.command.Stack
 	 */
-	var Stack = ManagedObject.extend("sap.ui.rta.command.Stack", {
+	const Stack = ManagedObject.extend("sap.ui.rta.command.Stack", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -146,33 +138,35 @@ sap.ui.define([
 	 * Creates a stack prefilled with Settings commands. Every command contains a change from the given file name list
 	 *
 	 * @param {sap.ui.base.ManagedObject} oControl - Used to get the component
-	 * @param {string[]} aFileNames - Array of file names of changes the stack should be initialized with
+	 * @param {string[]} [aFileNames] - Array of file names of changes the stack should be initialized with
 	 * @returns {Promise} Resolves with a stack as parameter
 	 */
-	Stack.initializeWithChanges = function(oControl, aFileNames) {
-		var oStack = new Stack();
+	Stack.initializeWithChanges = async function(oControl, aFileNames) {
+		const oStack = new Stack();
 		oStack._aPersistedChanges = aFileNames;
-		if (aFileNames && aFileNames.length > 0) {
-			var oComponent = FlUtils.getAppComponentForControl(oControl);
-			var mPropertyBag = {
+		if (aFileNames?.length > 0) {
+			const oComponent = FlUtils.getAppComponentForControl(oControl);
+			const mPropertyBag = {
 				selector: oComponent,
 				invalidateCache: false
 			};
-			return PersistenceWriteAPI._getUIChanges(mPropertyBag)
-
-			.then(function(aChanges) {
-				var mComposite = {};
-				var mChanges = {};
-				aChanges.forEach(function(oChange) {
-					mChanges[oChange.getId()] = oChange;
-				});
-				aFileNames
-				.reduce(toAvailableChanges.bind(null, mChanges), [])
-				.forEach(pushToStack.bind(null, oComponent, mComposite, oStack));
-				return oStack;
+			const aChanges = await PersistenceWriteAPI._getUIChanges(mPropertyBag);
+			const mComposite = {};
+			const mChanges = {};
+			aChanges.forEach((oChange) => {
+				mChanges[oChange.getId()] = oChange;
 			});
+			aFileNames
+			.reduce((aChanges, sFileName) => {
+				const oChange = mChanges[sFileName];
+				if (oChange) {
+					aChanges.push(oChange);
+				}
+				return aChanges;
+			}, [])
+			.forEach(pushToStack.bind(null, oComponent, mComposite, oStack));
 		}
-		return Promise.resolve(oStack);
+		return oStack;
 	};
 
 	Stack.prototype.init = function() {
@@ -221,12 +215,12 @@ sap.ui.define([
 	};
 
 	Stack.prototype.removeCommand = function(vObject, bSuppressInvalidate) {
-		var oRemovedCommand = this.removeAggregation("commands", vObject, bSuppressInvalidate);
+		const oRemovedCommand = this.removeAggregation("commands", vObject, bSuppressInvalidate);
 		return oRemovedCommand;
 	};
 
 	Stack.prototype.removeAllCommands = function(bSuppressInvalidate) {
-		var aCommands = this.removeAllAggregation("commands", bSuppressInvalidate);
+		const aCommands = this.removeAllAggregation("commands", bSuppressInvalidate);
 		this._toBeExecuted = -1;
 		this.fireModified();
 		return aCommands;
@@ -297,10 +291,12 @@ sap.ui.define([
 
 	Stack.prototype.execute = function() {
 		this.setLastCommandExecuted(
-			this.getLastCommandExecuted().catch(() => {
+			this.getLastCommandExecuted()
+			.catch(() => {
 			// continue also if previous command failed
-			}).then(async () => {
-				var oCommand = this._getCommandToBeExecuted();
+			})
+			.then(async () => {
+				const oCommand = this._getCommandToBeExecuted();
 				if (oCommand) {
 					try {
 						await addCommandChangesToPersistence.call(this, oCommand);
@@ -318,7 +314,7 @@ sap.ui.define([
 						this._toBeExecuted--;
 						// Remove Flex Changes for failed command from persistence
 						removeCommandChangesFromPersistence.call(this, oError.command);
-						var oRtaResourceBundle = Lib.getResourceBundleFor("sap.ui.rta");
+						const oRtaResourceBundle = Lib.getResourceBundleFor("sap.ui.rta");
 						// AddXMLAtExtensionPoint errors explain to the user what they did wrong, so they shouldn't open an incident
 						const sErrorMessage = oCommand.isA("sap.ui.rta.command.AddXMLAtExtensionPoint") ?
 							oError.message : oRtaResourceBundle.getText("MSG_GENERIC_ERROR_MESSAGE", [oError.message]);
@@ -338,13 +334,15 @@ sap.ui.define([
 
 	Stack.prototype._unExecute = function() {
 		this.setLastCommandUnExecuted(
-			this.getLastCommandUnExecuted().catch(() => {
+			this.getLastCommandUnExecuted()
+			.catch(() => {
 			// continue also if previous undo failed
-			}).then(async () => {
+			})
+			.then(async () => {
 				if (this.canUndo()) {
 					this._bUndoneCommands = true;
 					this._toBeExecuted++;
-					var oCommand = this._getCommandToBeExecuted();
+					const oCommand = this._getCommandToBeExecuted();
 					const aDiscardedChanges = oCommand.getDiscardedChanges?.();
 					if (oCommand) {
 						await oCommand.undo();
@@ -365,9 +363,7 @@ sap.ui.define([
 	};
 
 	Stack.prototype.canSave = function() {
-		return this.canUndo() && this.getAllExecutedCommands().some(function(oCommand) {
-			return oCommand.getRelevantForSave();
-		});
+		return this.canUndo() && this.getAllExecutedCommands().some((oCommand) => oCommand.getRelevantForSave());
 	};
 
 	Stack.prototype.undo = function() {
@@ -394,10 +390,10 @@ sap.ui.define([
 	 * @public
 	 */
 	Stack.prototype.getAllExecutedCommands = function() {
-		var aAllExecutedCommands = [];
-		var aCommands = this.getCommands();
-		for (var i = aCommands.length - 1; i > this._toBeExecuted; i--) {
-			var aSubCommands = this.getSubCommands(aCommands[i]);
+		let aAllExecutedCommands = [];
+		const aCommands = this.getCommands();
+		for (let i = aCommands.length - 1; i > this._toBeExecuted; i--) {
+			const aSubCommands = this.getSubCommands(aCommands[i]);
 			aAllExecutedCommands = aAllExecutedCommands.concat(aSubCommands);
 		}
 		return aAllExecutedCommands;
@@ -411,12 +407,12 @@ sap.ui.define([
 	 * @private
 	 */
 	Stack.prototype.getSubCommands = function(oCommand) {
-		var aCommands = [];
+		let aCommands = [];
 		if (oCommand.getCommands) {
-			oCommand.getCommands().forEach(function(oSubCommand) {
-				var aSubCommands = this.getSubCommands(oSubCommand);
+			oCommand.getCommands().forEach((oSubCommand) => {
+				const aSubCommands = this.getSubCommands(oSubCommand);
 				aCommands = aCommands.concat(aSubCommands);
-			}, this);
+			});
 		} else {
 			aCommands.push(oCommand);
 		}
@@ -430,10 +426,10 @@ sap.ui.define([
 	 * @private
 	 */
 	Stack.prototype.compositeLastTwoCommands = function() {
-		var oLastCommand = this.pop();
-		var oSecondLastCommand = this.pop();
+		const oLastCommand = this.pop();
+		const oSecondLastCommand = this.pop();
 
-		var oCompositeCommand = new CompositeCommand();
+		const oCompositeCommand = new CompositeCommand();
 		oCompositeCommand.addCommand(oSecondLastCommand);
 		oCompositeCommand.addCommand(oLastCommand);
 		this.push(oCompositeCommand);

package/src/sap/ui/rta/command/compVariant/CompVariantContent.js

@@ -20,13 +20,13 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.BaseCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.102
 	 * @alias sap.ui.rta.command.compVariant.CompVariantContent
 	 */
-	var CompVariantContent = BaseCommand.extend("sap.ui.rta.command.compVariant.CompVariantContent", {
+	const CompVariantContent = BaseCommand.extend("sap.ui.rta.command.compVariant.CompVariantContent", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -47,7 +47,7 @@ sap.ui.define([
 	});
 
 	function callFlAPIFunction(sFunctionName, sKey, oValue) {
-		var mPropertyBag = {
+		const mPropertyBag = {
 			...oValue,
 			...this.mInformation,
 			id: sKey,
@@ -58,11 +58,11 @@ sap.ui.define([
 
 	// private function of the SmartVariantManagement are approved and documented to be used from here
 	function setVariantContent(oContent) {
-		var oVariantManagementControl = this.getElement();
-		var oCurrentVariantContent = oVariantManagementControl._getVariantContent(this.getVariantId());
-		var oNewVariantContent = { ...oCurrentVariantContent };
+		const oVariantManagementControl = this.getElement();
+		const oCurrentVariantContent = oVariantManagementControl._getVariantContent(this.getVariantId());
+		let oNewVariantContent = { ...oCurrentVariantContent };
 		if (oVariantManagementControl.isPageVariant()) {
-			var oContentToBeSet = {};
+			const oContentToBeSet = {};
 			oContentToBeSet[this.getPersistencyKey()] = oContent;
 			oVariantManagementControl._applyVariantByPersistencyKey(this.getPersistencyKey(), oContentToBeSet, "KEY_USER");
 			oNewVariantContent[this.getPersistencyKey()] = oContent;
@@ -84,15 +84,15 @@ sap.ui.define([
 
 	CompVariantContent.prototype.execute = function() {
 		this.getElement().setModified(true);
-		var oNewVariantContent = setVariantContent.call(this, this.getNewContent());
+		const oNewVariantContent = setVariantContent.call(this, this.getNewContent());
 		callFlAPIFunction.call(this, "updateVariantContent", this.getVariantId(), {content: oNewVariantContent});
 		return Promise.resolve();
 	};
 
 	CompVariantContent.prototype.undo = function() {
-		var oVariant = callFlAPIFunction.call(this, "revert", this.getVariantId(), {});
+		const oVariant = callFlAPIFunction.call(this, "revert", this.getVariantId(), {});
 		this.getElement().setModified(this.getIsModifiedBefore());
-		var oVariantManagementControl = this.getElement();
+		const oVariantManagementControl = this.getElement();
 		if (oVariantManagementControl.isPageVariant()) {
 			setVariantContent.call(this, oVariant.getContent()[this.getPersistencyKey()]);
 		} else {

package/src/sap/ui/rta/command/compVariant/CompVariantSaveAs.js

@@ -20,13 +20,13 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.BaseCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.87
 	 * @alias sap.ui.rta.command.compVariant.CompVariantSaveAs
 	 */
-	var CompVariantSaveAs = BaseCommand.extend("sap.ui.rta.command.compVariant.CompVariantSaveAs", {
+	const CompVariantSaveAs = BaseCommand.extend("sap.ui.rta.command.compVariant.CompVariantSaveAs", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {
@@ -71,8 +71,8 @@ sap.ui.define([
 	 * @returns {Promise} Returns resolve after execution
 	 */
 	CompVariantSaveAs.prototype.execute = function() {
-		var oNewVariantProperties = this.getNewVariantProperties();
-		var mPropertyBag = {
+		const oNewVariantProperties = this.getNewVariantProperties();
+		const mPropertyBag = {
 			changeSpecificData: {
 				// in case of redo the variant is still saved and the ID should be reused
 				id: this._oVariant ? this._oVariant.getVariantId() : undefined,

package/src/sap/ui/rta/command/compVariant/CompVariantSwitch.js

@@ -18,13 +18,13 @@ sap.ui.define([
 	 * @class
 	 * @extends sap.ui.rta.command.BaseCommand
 	 * @author SAP SE
-	 * @version 1.139.0
+	 * @version 1.141.0
 	 * @constructor
 	 * @private
 	 * @since 1.87
 	 * @alias sap.ui.rta.command.compVariant.CompVariantSwitch
 	 */
-	var CompVariantSwitch = BaseCommand.extend("sap.ui.rta.command.compVariant.CompVariantSwitch", {
+	const CompVariantSwitch = BaseCommand.extend("sap.ui.rta.command.compVariant.CompVariantSwitch", {
 		metadata: {
 			library: "sap.ui.rta",
 			properties: {