@openui5/sap.ui.rta 1.128.0 → 1.130.0

package/src/sap/ui/rta/plugin/additionalElements/AdditionalElementsPlugin.js

@@ -39,11 +39,11 @@ sap.ui.define([
 ) {
 	"use strict";
 
-	var SINGULAR = true;
-	var PLURAL = false;
+	const SINGULAR = true;
+	const PLURAL = false;
 
 	function isThereAnAggregationActionForSameAggregation(mActions, mParents) {
-		var sResponsibleElementsParentAggregation = mParents.responsibleElementOverlay.getParentAggregationOverlay().getAggregationName();
+		const sResponsibleElementsParentAggregation = mParents.responsibleElementOverlay.getParentAggregationOverlay().getAggregationName();
 		return Object.keys(mActions).some(function(sAggregationName) {
 			return sAggregationName === sResponsibleElementsParentAggregation;
 		});
@@ -119,13 +119,13 @@ sap.ui.define([
 	 * @class The plugin allows to add additional elements that exist either hidden in the UI or in the OData service
 	 * @extends sap.ui.rta.plugin.Plugin
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @constructor
 	 * @private
 	 * @since 1.44
 	 * @alias sap.ui.rta.plugin.additionalElements.AdditionalElementsPlugin
 	 */
-	var AdditionalElementsPlugin = Plugin.extend("sap.ui.rta.plugin.additionalElements.AdditionalElementsPlugin", {
+	const AdditionalElementsPlugin = Plugin.extend("sap.ui.rta.plugin.additionalElements.AdditionalElementsPlugin", {
 		// eslint-disable-next-line object-shorthand
 		constructor: function(...aArgs) {
 			this._getMenuItemsPromise = Promise.resolve();
@@ -150,14 +150,14 @@ sap.ui.define([
 
 		// For add elements plugin, include other aggregations which are potentially valid targets for hidden elements
 		_getRelevantOverlays(oOverlay) {
-			var aRelevantOverlays = OverlayUtil.findAllOverlaysInContainer(oOverlay, /* bIncludeOtherAggregations= */true);
+			const aRelevantOverlays = OverlayUtil.findAllOverlaysInContainer(oOverlay, /* bIncludeOtherAggregations= */true);
 
 			oOverlay.setRelevantOverlays(aRelevantOverlays);
 			return aRelevantOverlays;
 		},
 
 		getContextMenuText(bOverlayIsSibling, oOverlay, sAggregationName, bHasSubMenu) {
-			var oTextResources;
+			let oTextResources;
 
 			function getGenericText() {
 				oTextResources = Lib.getResourceBundleFor("sap.ui.rta");
@@ -175,10 +175,10 @@ sap.ui.define([
 				oTextResources = Lib.getResourceBundleFor("sap.ui.rta");
 				return oTextResources.getText("CTX_ADD_ELEMENTS_WITH_SUBMENU");
 			}
-			var mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oOverlay, this);
-			var mAllActions = ActionExtractor.getActionsOrUndef(bOverlayIsSibling, oOverlay);
+			const mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oOverlay, this);
+			const mAllActions = ActionExtractor.getActionsOrUndef(bOverlayIsSibling, oOverlay);
 			sAggregationName ||= Object.keys(mAllActions)[0];
-			var mActions = mAllActions[sAggregationName];
+			const mActions = mAllActions[sAggregationName];
 
 			// Safeguarding
 			if (!mActions) {
@@ -203,17 +203,17 @@ sap.ui.define([
 				return true;
 			}
 
-			var oOverlay = this.getResponsibleElementOverlay(aElementOverlays[0]);
-			var oParentOverlay;
-			var bIsEnabled = false;
+			const oOverlay = this.getResponsibleElementOverlay(aElementOverlays[0]);
+			let oParentOverlay;
+			let bIsEnabled = false;
 			if (bOverlayIsSibling) {
 				oParentOverlay = oOverlay.getParentElementOverlay();
 				if (oParentOverlay) {
 					bIsEnabled = true;
 				}
 			} else {
-				var mAllActions = ActionExtractor.getActionsOrUndef(bOverlayIsSibling, oOverlay);
-				var mActions = mAllActions[sAggregationName];
+				const mAllActions = ActionExtractor.getActionsOrUndef(bOverlayIsSibling, oOverlay);
+				const mActions = mAllActions[sAggregationName];
 				if (
 					mActions &&
 					((mActions.reveal && mActions.reveal.elements.length > 0)
@@ -223,8 +223,8 @@ sap.ui.define([
 				}
 			}
 
-			var oCachedElements = this.getCachedElements(bOverlayIsSibling);
-			var bElementsAvailable = !!(oCachedElements && oCachedElements.length > 0);
+			const oCachedElements = this.getCachedElements(bOverlayIsSibling);
+			const bElementsAvailable = !!(oCachedElements && oCachedElements.length > 0);
 			bIsEnabled &&= bElementsAvailable;
 			return bIsEnabled;
 		},
@@ -239,9 +239,9 @@ sap.ui.define([
 		 */
 		registerElementOverlay(...aArgs) {
 			const [oOverlay] = aArgs;
-			var oModel = oOverlay.getElement().getModel();
+			const oModel = oOverlay.getElement().getModel();
 			if (oModel) {
-				var oMetaModel = oModel.getMetaModel();
+				const oMetaModel = oModel.getMetaModel();
 				if (oMetaModel && oMetaModel.loaded) {
 					oMetaModel.loaded().then(function() {
 						this.evaluateEditable([oOverlay], {onRegistration: true});
@@ -273,11 +273,11 @@ sap.ui.define([
 		 * @private
 		 */
 		showAvailableElements(bOverlayIsSibling, sAggregationName, aResponsibleElementOverlays, iIndex, sControlName, sDisplayText) {
-			var oResponsibleElementOverlay = aResponsibleElementOverlays[0];
-			var mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oResponsibleElementOverlay, this);
-			var vSiblingElement = bOverlayIsSibling && oResponsibleElementOverlay.getElement();
-			var mActions;
-			var aAllElements = [];
+			const oResponsibleElementOverlay = aResponsibleElementOverlays[0];
+			const mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oResponsibleElementOverlay, this);
+			const vSiblingElement = bOverlayIsSibling && oResponsibleElementOverlay.getElement();
+			let mActions;
+			let aAllElements = [];
 
 			return ActionExtractor.getActions(bOverlayIsSibling, oResponsibleElementOverlay, this, undefined, this.getDesignTime())
 			.then(function(mRetrievedActions) {
@@ -291,16 +291,16 @@ sap.ui.define([
 			.then(function(aCollectedElements) {
 				aAllElements = aCollectedElements;
 				configureExtensibility.call(this, bOverlayIsSibling);
-				var oAggregationWithElements = aAllElements.filter(function(mElementsPerAggregation) {
+				const oAggregationWithElements = aAllElements.filter(function(mElementsPerAggregation) {
 					return mElementsPerAggregation.aggregation === sAggregationName;
 				})[0];
-				var aElementsPerAggregation = oAggregationWithElements ? oAggregationWithElements.elements : [];
+				const aElementsPerAggregation = oAggregationWithElements ? oAggregationWithElements.elements : [];
 
 				this.getDialog().setElements(aElementsPerAggregation);
 				if (sDisplayText) {
 					// Aggregation is part of title
-					var oTextResources = Lib.getResourceBundleFor("sap.ui.rta");
-					var sDialogTitle = oTextResources.getText("HEADER_ADDITIONAL_ELEMENTS_WITH_AGGREGATION", [sDisplayText]);
+					const oTextResources = Lib.getResourceBundleFor("sap.ui.rta");
+					const sDialogTitle = oTextResources.getText("HEADER_ADDITIONAL_ELEMENTS_WITH_AGGREGATION", [sDisplayText]);
 					this.getDialog().setTitle(sDialogTitle);
 				} else if (sAggregationName || sControlName) {
 					// Only one aggregation, no aggregation in title
@@ -310,7 +310,7 @@ sap.ui.define([
 				return this.getDialog().open()
 
 				.then(function() {
-					var aSelectedElements = this.getDialog().getSelectedElements();
+					const aSelectedElements = this.getDialog().getSelectedElements();
 					return CommandBuilder.createCommands(mParents,
 						vSiblingElement,
 						mActions,
@@ -322,7 +322,7 @@ sap.ui.define([
 				}.bind(this))
 
 				.then(function() {
-					var oOverlayToFocus = OverlayRegistry.getOverlay(vSiblingElement) || oResponsibleElementOverlay;
+					const oOverlayToFocus = OverlayRegistry.getOverlay(vSiblingElement) || oResponsibleElementOverlay;
 					oOverlayToFocus.focus();
 				})
 
@@ -344,7 +344,7 @@ sap.ui.define([
 		},
 
 		_setDialogTitle(mActions, oParentElement, sControlName) {
-			var sDialogTitle = AdditionalElementsUtils.getText(
+			const sDialogTitle = AdditionalElementsUtils.getText(
 				"HEADER_ADDITIONAL_ELEMENTS",
 				mActions,
 				oParentElement,
@@ -389,7 +389,7 @@ sap.ui.define([
 		},
 
 		async _isEditableCheck(oOverlay, bOverlayIsSibling) {
-			var mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oOverlay, this);
+			const mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oOverlay, this);
 
 			if (!mParents.relevantContainerOverlay) {
 				return false;
@@ -400,7 +400,7 @@ sap.ui.define([
 			await this._getMenuItemsPromise;
 			this.clearCachedElements();
 			return Utils.doIfAllControlsAreAvailable([oOverlay, mParents.parentOverlay], function() {
-				var bEditable = false;
+				let bEditable = false;
 				// For the sibling case, check if anything is available for the same aggregation
 				if (bOverlayIsSibling) {
 					bEditable = isThereAnAggregationActionForSameAggregation(mActions, mParents);
@@ -429,12 +429,12 @@ sap.ui.define([
 		 * @protected
 		 */
 		getAllElements(bOverlayIsSibling, aElementOverlays) {
-			var oElementOverlay = aElementOverlays[0];
-			var mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oElementOverlay, this);
-			var mActions;
-			var aPromises = [];
-			var bCheckExtensibility = false;
-			var aCachedElements = this.getCachedElements(bOverlayIsSibling);
+			const oElementOverlay = aElementOverlays[0];
+			const mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oElementOverlay, this);
+			let mActions;
+			const aPromises = [];
+			let bCheckExtensibility = false;
+			const aCachedElements = this.getCachedElements(bOverlayIsSibling);
 
 			if (aCachedElements) {
 				return aCachedElements;
@@ -492,8 +492,8 @@ sap.ui.define([
 		 * @return {object[]} Array containing the items with required data
 		 */
 		getMenuItems(aElementOverlays) {
-			var aMenuItems = [];
-			var oMenuItem;
+			const aMenuItems = [];
+			let oMenuItem;
 			this.clearCachedElements();
 			// Prevents clearing of cached elements before menu items are build. This happens when evaluate editable is running in parallel
 			this._getMenuItemsPromise =
@@ -503,11 +503,11 @@ sap.ui.define([
 				this.getAllElements(true, aElementOverlays)
 			])
 			.then(function(aElementsWithAggregations) {
-				var bHasChildren = aElementsWithAggregations[0].length > 0;
-				var bHasMultipleAggregations = aElementsWithAggregations[0].length > 1;
-				var bHasSiblings = aElementsWithAggregations[1].length > 0;
-				var bIsAvailableForChildren = this.isAvailable(aElementOverlays, false);
-				var bIsAvailableForSibling = this.isAvailable(aElementOverlays, true);
+				const bHasChildren = aElementsWithAggregations[0].length > 0;
+				const bHasMultipleAggregations = aElementsWithAggregations[0].length > 1;
+				const bHasSiblings = aElementsWithAggregations[1].length > 0;
+				const bIsAvailableForChildren = this.isAvailable(aElementOverlays, false);
+				const bIsAvailableForSibling = this.isAvailable(aElementOverlays, true);
 				if (
 					bIsAvailableForSibling
 					&& (!bIsAvailableForChildren || !bHasChildren)
@@ -574,19 +574,19 @@ sap.ui.define([
 		},
 
 		_buildMenuItem(sPluginId, bOverlayIsSibling, aElementOverlays, aElementsWithAggregations, bHasSubMenu) {
-			var aSubMenuItems;
-			var vHandler;
-			var sAggregationName;
-			var oSelectedOverlay = aElementOverlays[0];
+			let aSubMenuItems;
+			let vHandler;
+			let sAggregationName;
+			const oSelectedOverlay = aElementOverlays[0];
 
 			if (bOverlayIsSibling) {
-				var mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oSelectedOverlay, this);
+				const mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oSelectedOverlay, this);
 				sAggregationName = mParents.responsibleElementOverlay.getParentAggregationOverlay().getAggregationName();
 			} else {
 				// If there are no elements available but the action is still available, the dialog will open just to allow
 				// adding custom fields. When clicking on the parent, the aggregation name is irrelevant for the dialog
 				// since the field will be added to the underlying data model.
-				var bOnlyCustomField = aElementsWithAggregations[0].length === 0 && aElementsWithAggregations[1].length === 0;
+				const bOnlyCustomField = aElementsWithAggregations[0].length === 0 && aElementsWithAggregations[1].length === 0;
 				sAggregationName = bOnlyCustomField ? "$$OnlyChildCustomField$$" :
 					// If only one child aggregation has invisible elements, get its aggregation name
 					aElementsWithAggregations[0] && aElementsWithAggregations[0][0] && aElementsWithAggregations[0][0].aggregation;
@@ -602,7 +602,7 @@ sap.ui.define([
 					return this.showAvailableElements(bOverlayIsSibling, sAggregationName, aElementOverlays);
 				}.bind(this, bOverlayIsSibling);
 			}
-			var oMenuItem = {
+			const oMenuItem = {
 				id: sPluginId,
 				text: this.getContextMenuText.bind(this, bOverlayIsSibling, oSelectedOverlay, sAggregationName, bHasSubMenu),
 				enabled: bHasSubMenu || function(bOverlayIsSibling, aElementOverlays) {
@@ -619,9 +619,9 @@ sap.ui.define([
 		},
 
 		_buildSubmenuItems(bOverlayIsSibling, aElementOverlays, aElementsWithAggregation) {
-			var aSubMenuItems = [];
-			var sPluginId = bOverlayIsSibling ? "CTX_ADD_ELEMENTS_AS_SIBLING" : "CTX_ADD_ELEMENTS_AS_CHILD";
-			var iPosition = 0;
+			const aSubMenuItems = [];
+			const sPluginId = bOverlayIsSibling ? "CTX_ADD_ELEMENTS_AS_SIBLING" : "CTX_ADD_ELEMENTS_AS_CHILD";
+			let iPosition = 0;
 
 			// register BusinessSuite icon font
 			IconPool.registerFont({
@@ -631,16 +631,17 @@ sap.ui.define([
 				lazy: true
 			});
 
-			function getMenuItemText(bOverlayIsSibling, sAggregationName, aElementOverlays) {
-				var oElementOverlay = bOverlayIsSibling ? aElementOverlays[0].getParentElementOverlay() : aElementOverlays[0];
-				var oElementOverlayMetadata = oElementOverlay.getDesignTimeMetadata();
-				var mNames = oElementOverlayMetadata.getAggregationDisplayName(sAggregationName, oElementOverlay.getElement());
-				return mNames ? mNames.singular : sAggregationName;
+			function getMenuItemText(bOverlayIsSibling, sAggregationName, oElementOverlay) {
+				oElementOverlay = bOverlayIsSibling ? oElementOverlay.getParentElementOverlay() : oElementOverlay;
+				const oElementOverlayMetadata = oElementOverlay.getDesignTimeMetadata();
+				const mNames = oElementOverlayMetadata.getAggregationDisplayName(sAggregationName, oElementOverlay.getElement());
+				return mNames ? mNames.plural : sAggregationName;
 			}
 			aElementsWithAggregation.forEach(function(mElementsWithAggregation) {
-				var sAggregationName = mElementsWithAggregation.aggregation;
-				var sDisplayText = getMenuItemText(bOverlayIsSibling, sAggregationName, aElementOverlays);
-				var oItem = {
+				const sAggregationName = mElementsWithAggregation.aggregation;
+				const oResponsibleElementOverlay = this.getResponsibleElementOverlay(aElementOverlays[0]);
+				const sDisplayText = getMenuItemText(bOverlayIsSibling, sAggregationName, oResponsibleElementOverlay);
+				const oItem = {
 					id: `${sPluginId}_${iPosition}`,
 					text: sDisplayText,
 					enabled: function(aElementOverlays) {
@@ -696,14 +697,14 @@ sap.ui.define([
 		// 	}
 		// ]
 		_combineAnalyzerResults(aAllPromises) {
-			var aCollectedPromises = [];
+			const aCollectedPromises = [];
 
 			aAllPromises.forEach(function(aPromisesByAggregation) {
 				aCollectedPromises.push(
 					Promise.all(aPromisesByAggregation.elementPromises).then(function(aAnalyzerValues) {
-						var aRevealElements = aAnalyzerValues[0];
-						var aAddViaDelegateElements = aAnalyzerValues[1];
-						var aAllElements = aRevealElements.concat(aAddViaDelegateElements);
+						const aRevealElements = aAnalyzerValues[0];
+						const aAddViaDelegateElements = aAnalyzerValues[1];
+						const aAllElements = aRevealElements.concat(aAddViaDelegateElements);
 						return {
 							aggregation: aPromisesByAggregation.aggregation,
 							elements: aAllElements
@@ -716,7 +717,7 @@ sap.ui.define([
 			.then(function(aElementsPerAggregation) {
 				// Filter out results without elements
 				return aElementsPerAggregation.filter(function(oElementsPerAggregation) {
-					var aElements = oElementsPerAggregation && oElementsPerAggregation.elements;
+					const aElements = oElementsPerAggregation && oElementsPerAggregation.elements;
 					return aElements.length > 0;
 				});
 			});

package/src/sap/ui/rta/plugin/additionalElements/AdditionalElementsUtils.js

@@ -17,7 +17,7 @@ sap.ui.define([
 	 * Helper object containing methods for the AdditionalElements plugin
 	 *
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @private
 	 * @since 1.94
 	 */

package/src/sap/ui/rta/plugin/additionalElements/CommandBuilder.js

@@ -25,7 +25,7 @@ sap.ui.define([
 	 * Helper object to build the RTA commands resulting from the AdditionalElements Plugin
 	 *
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @private
 	 * @since 1.94
 	 */

package/src/sap/ui/rta/plugin/iframe/AddIFrame.js

@@ -119,7 +119,7 @@ sap.ui.define([
 	 * @class The AddIFrame allows trigger AddIFrame operations on the overlay.
 	 * @extends sap.ui.rta.plugin.BaseCreate
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @constructor
 	 * @private
 	 * @since 1.75
@@ -184,11 +184,12 @@ sap.ui.define([
 		if (this.isAvailable(aElementOverlays, bIsSibling)) {
 			var oAction = this.getCreateAction(bIsSibling, aElementOverlays[0]);
 			if (oAction) {
-				var oSiblingMenuItem = Object.assign({
+				var oSiblingMenuItem = {
 					id: "CTX_CREATE_SIBLING_IFRAME",
 					rank: iBaseRank,
-					action: oAction
-				}, getCommonProperties.call(this));
+					action: oAction,
+					...getCommonProperties.call(this)
+				};
 
 				aMenuItems.push(this.enhanceItemWithResponsibleElement(oSiblingMenuItem, aElementOverlays));
 			}
@@ -198,11 +199,12 @@ sap.ui.define([
 		if (this.isAvailable(aElementOverlays, bIsSibling)) {
 			aMenuItems = aMenuItems.concat(this.getCreateActions(bIsSibling, aElementOverlays[0])
 			.map(function(oAction) {
-				var oParentMenuItem = Object.assign({
+				var oParentMenuItem = {
 					action: oAction,
 					id: `CTX_CREATE_CHILD_IFRAME_${oAction.aggregation.toUpperCase()}`,
-					rank: ++iBaseRank
-				}, getCommonProperties.call(this, oAction.aggregation));
+					rank: ++iBaseRank,
+					...getCommonProperties.call(this, oAction.aggregation)
+				};
 
 				return this.enhanceItemWithResponsibleElement(oParentMenuItem, aElementOverlays);
 			}, this)

package/src/sap/ui/rta/plugin/iframe/AddIFrameDialog.fragment.xml

@@ -420,7 +420,7 @@
 				type="Emphasized"
 				text="{dialogInfo>/text/saveText}"
 				press="onSavePress"
-				enabled="{=!!${dialogInfo>/frameUrl/value} && !${dialogInfo>/frameUrlError/value}}"
+				enabled="{=!!${dialogInfo>/frameUrl/value} && !${dialogInfo>/frameUrlError/value} && ${dialogInfo>/areAllFieldsValid}}"
 			/>
 		</beginButton>
 		<endButton>

package/src/sap/ui/rta/plugin/iframe/AddIFrameDialog.js

@@ -159,7 +159,7 @@ sap.ui.define([
 	 * @class Context - Dialog for IFrame settings in Runtime Authoring
 	 * @extends sap.ui.base.ManagedObject
 	 * @author SAP SE
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @constructor
 	 * @private
 	 * @since 1.78

package/src/sap/ui/rta/plugin/iframe/AddIFrameDialogController.js

@@ -270,8 +270,8 @@ sap.ui.define([
 			if (sValue.trim() === "") {
 				sValueState = "Error";
 				oInput.setValueState(sValueState);
-				bValidationError = true;
-				return bValidationError;
+				this._oJSONModel.setProperty("/areAllFieldsValid", false);
+				return true;
 			}
 
 			try {
@@ -281,8 +281,9 @@ sap.ui.define([
 				bValidationError = true;
 			}
 
+			const bAllFieldsValid = this._areAllTextFieldsValid() && this._areAllValueStateNones() && !bValidationError;
+			this._oJSONModel.setProperty("/areAllFieldsValid", bAllFieldsValid);
 			oInput.setValueState(sValueState);
-
 			return bValidationError;
 		},
 

package/src/sap/ui/rta/service/Action.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * @name sap.ui.rta.service.Action
 	 * @author SAP SE
 	 * @since 1.58
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @private
 	 * @ui5-restricted
 	 *
@@ -78,10 +78,11 @@ sap.ui.define([
 						: aResult;
 				}, [])
 				.map(function(mMenuItem) {
-					return Object.assign({}, mMenuItem, {
+					return {
+						...mMenuItem,
 						enabled: invoke(mMenuItem.enabled, aElementOverlays),
 						text: invoke(mMenuItem.text, aElementOverlays[0])
-					});
+					};
 				});
 			});
 		}

package/src/sap/ui/rta/service/ControllerExtension.js

@@ -32,7 +32,7 @@ sap.ui.define([
 	 * @name sap.ui.rta.service.ControllerExtension
 	 * @author SAP SE
 	 * @since 1.58
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @private
 	 * @ui5-restricted
 	*/

package/src/sap/ui/rta/service/Outline.js

@@ -43,7 +43,7 @@ sap.ui.define([
 	 * @name sap.ui.rta.service.Outline
 	 * @author SAP SE
 	 * @since 1.56
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @private
 	 * @ui5-restricted
 	 */
@@ -211,9 +211,7 @@ sap.ui.define([
 				var sAggregationOverlayId = oChildOverlay.getParent().getId();
 				// Aggregation with root template
 				if (mInnerTemplateData[sAggregationName]) {
-					return Object.assign({
-						templateFor: sAggregationOverlayId
-					}, mInnerTemplateData[sAggregationName]);
+					return { templateFor: sAggregationOverlayId, ...mInnerTemplateData[sAggregationName] };
 				}
 				return ((mTemplateData && mTemplateData.elements) || [])
 				.map(function(oElement) {
@@ -222,9 +220,7 @@ sap.ui.define([
 						oElement.type === "aggregationBindingTemplate"
 							|| oElement.parentAggregationName === sAggregationName
 					) {
-						return Object.assign({
-							templateFor: sAggregationOverlayId
-						}, oElement);
+						return { templateFor: sAggregationOverlayId, ...oElement };
 					}
 					// Regular aggregation
 					return oElement.technicalName === sAggregationName && oElement;
@@ -425,9 +421,9 @@ sap.ui.define([
 			var oData = getDefaultData(oElement, oDtMetadata, mTemplateData);
 
 			if (oOverlay instanceof ElementOverlay) {
-				return Object.assign(oData, getElementOverlayData(oOverlay, oElement, oDtMetadata));
+				return { ...oData, ...getElementOverlayData(oOverlay, oElement, oDtMetadata) };
 			}
-			return Object.assign(oData, getAggregationOverlayData(oOverlay, oParentOverlay, oElement));
+			return { ...oData, ...getAggregationOverlayData(oOverlay, oParentOverlay, oElement) };
 		};
 
 		/**

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

@@ -27,7 +27,7 @@ sap.ui.define([
 	 * @name sap.ui.rta.service.Property
 	 * @author SAP SE
 	 * @since 1.58
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @private
 	 * @ui5-restricted
 	*/
@@ -117,14 +117,13 @@ sap.ui.define([
 				]
 			)
 			.then(function(aPromiseResults) {
-				return Object.assign(
-					{},
-					aPromiseResults[0] && !isEmptyObject(aPromiseResults[0]) && {annotations: aPromiseResults[0]},
-					aPromiseResults[1] && {properties: aPromiseResults[1]},
-					aPromiseResults[2] && {label: validate(aPromiseResults[2])},
-					oDesignTimeMetadataData.name && {name: oDesignTimeMetadata.getName(oElement)},
-					!isEmptyObject(aPromiseResults[3]) && {links: aPromiseResults[3]}
-				);
+				return {
+					...(aPromiseResults[0] && !isEmptyObject(aPromiseResults[0]) && {annotations: aPromiseResults[0]}),
+					...(aPromiseResults[1] && {properties: aPromiseResults[1]}),
+					...(aPromiseResults[2] && {label: validate(aPromiseResults[2])}),
+					...(oDesignTimeMetadataData.name && {name: oDesignTimeMetadata.getName(oElement)}),
+					...(!isEmptyObject(aPromiseResults[3]) && {links: aPromiseResults[3]})
+				};
 			});
 		};
 
@@ -274,7 +273,7 @@ sap.ui.define([
 						// to ensure ignore function is replaced by a boolean value
 						mDtObj[sKey].ignore = bIgnore;
 						if (!bIgnore) {
-							mFiltered[sKey] = Object.assign({}, mDtObj[sKey]);
+							mFiltered[sKey] = { ...mDtObj[sKey] };
 							return oProperty._getResolvedLinks(mFiltered[sKey].links, oElement)
 							.then(function(mLinks) {
 								if (!isEmptyObject(mLinks)) {
@@ -316,7 +315,7 @@ sap.ui.define([
 		 */
 		oProperty._getResolvedLinks = function(mLinks, oElement) {
 			var aTextPromises = [];
-			var mResolvedLinks = Object.assign({}, mLinks);
+			var mResolvedLinks = { ...mLinks };
 
 			Object.keys(mResolvedLinks).forEach(function(sLinkName) {
 				if (Array.isArray(mResolvedLinks[sLinkName])) {

package/src/sap/ui/rta/service/Selection.js

@@ -14,7 +14,7 @@ sap.ui.define(function() {
 	 * @name sap.ui.rta.service.Selection
 	 * @author SAP SE
 	 * @since 1.58
-	 * @version 1.128.0
+	 * @version 1.130.0
 	 * @private
 	 * @ui5-restricted
 	*/