@openui5/sap.ui.rta 1.115.0 → 1.116.0

package/src/sap/ui/rta/util/changeVisualization/ChangeIndicatorRegistry.js

@@ -39,7 +39,7 @@ sap.ui.define([
 	 * @alias sap.ui.rta.util.changeVisualization.ChangeIndicatorRegistry
 	 * @author SAP SE
 	 * @since 1.86.0
-	 * @version 1.115.0
+	 * @version 1.116.0
 	 * @private
 	 */
 	var ChangeIndicatorRegistry = ManagedObject.extend("sap.ui.rta.util.changeVisualization.ChangeIndicatorRegistry", {
@@ -60,7 +60,7 @@ sap.ui.define([
 				}
 			}
 		},
-		constructor: function () {
+		constructor: function() {
 			ManagedObject.prototype.constructor.apply(this, arguments);
 
 			// List of entries with indicator data, grouped by Change ID
@@ -71,7 +71,7 @@ sap.ui.define([
 		}
 	});
 
-	ChangeIndicatorRegistry.prototype.exit = function () {
+	ChangeIndicatorRegistry.prototype.exit = function() {
 		this.reset();
 	};
 
@@ -80,8 +80,8 @@ sap.ui.define([
 	 *
 	 * @returns {object[]} Change indicator data for all registered changes
 	 */
-	ChangeIndicatorRegistry.prototype.getAllRegisteredChanges = function () {
-		return values(this._oRegisteredChanges || {}).map(function (oChange) {
+	ChangeIndicatorRegistry.prototype.getAllRegisteredChanges = function() {
+		return values(this._oRegisteredChanges || {}).map(function(oChange) {
 			return Object.assign({}, oChange);
 		});
 	};
@@ -91,7 +91,7 @@ sap.ui.define([
 	 *
 	 * @returns {string[]} Array with both design time and runtime registered changes
 	 */
-	ChangeIndicatorRegistry.prototype.getRegisteredChangeIds = function () {
+	ChangeIndicatorRegistry.prototype.getRegisteredChangeIds = function() {
 		return Object.keys(this._oRegisteredChanges || {});
 	};
 
@@ -101,7 +101,7 @@ sap.ui.define([
 	 * @param {string} sChangeId - ID of the registered change
 	 * @returns {object} Registered change
 	 */
-	ChangeIndicatorRegistry.prototype.getRegisteredChange = function (sChangeId) {
+	ChangeIndicatorRegistry.prototype.getRegisteredChange = function(sChangeId) {
 		return this._oRegisteredChanges[sChangeId] && Object.assign({}, this._oRegisteredChanges[sChangeId]);
 	};
 
@@ -111,11 +111,11 @@ sap.ui.define([
 	 *
 	 * @returns {object} List of selectors with change indicator data.
 	 */
-	ChangeIndicatorRegistry.prototype.getSelectorsWithRegisteredChanges = function () {
+	ChangeIndicatorRegistry.prototype.getSelectorsWithRegisteredChanges = function() {
 		var oChangeIndicators = {};
 		var sPreviousAffectedElementId;
 
-		function addSelector (sSelectorId, sAffectedElementId, oChangeIndicatorData, bDependent) {
+		function addSelector(sSelectorId, sAffectedElementId, oChangeIndicatorData, bDependent) {
 			if (oChangeIndicators[sSelectorId] === undefined) {
 				oChangeIndicators[sSelectorId] = [];
 			}
@@ -133,21 +133,21 @@ sap.ui.define([
 		}
 
 		values(this._oRegisteredChanges)
-			.forEach(function (oChangeIndicatorData) {
-				oChangeIndicatorData.visualizationInfo.displayElementIds
-				.forEach(function (sId, iIndex) {
-					addSelector(sId, oChangeIndicatorData.visualizationInfo.affectedElementIds[iIndex], oChangeIndicatorData, false);
-				});
+		.forEach(function(oChangeIndicatorData) {
+			oChangeIndicatorData.visualizationInfo.displayElementIds
+			.forEach(function(sId, iIndex) {
+				addSelector(sId, oChangeIndicatorData.visualizationInfo.affectedElementIds[iIndex], oChangeIndicatorData, false);
 			});
+		});
 
 		return oChangeIndicators;
 	};
 
-	ChangeIndicatorRegistry.prototype.getRelevantChangesWithSelector = function () {
+	ChangeIndicatorRegistry.prototype.getRelevantChangesWithSelector = function() {
 		var oSelectors = this.getSelectorsWithRegisteredChanges();
 		var aRelevantChanges = [];
 		Object.keys(oSelectors).forEach(function(sSelectorId) {
-			var aRelevantChangesForSelector = oSelectors[sSelectorId].filter(function (oChange) {
+			var aRelevantChangesForSelector = oSelectors[sSelectorId].filter(function(oChange) {
 				return !oChange.dependent;
 			});
 			aRelevantChanges = aRelevantChanges.concat(aRelevantChangesForSelector);
@@ -161,7 +161,7 @@ sap.ui.define([
 	 * @param {string} sSelectorId - ID of the indicator
 	 * @returns {object} Registered change indicator
 	 */
-	ChangeIndicatorRegistry.prototype.getChangeIndicator = function (sSelectorId) {
+	ChangeIndicatorRegistry.prototype.getChangeIndicator = function(sSelectorId) {
 		return this._oChangeIndicators[sSelectorId];
 	};
 
@@ -170,7 +170,7 @@ sap.ui.define([
 	 *
 	 * @returns {object[]} Registered change indicators
 	 */
-	ChangeIndicatorRegistry.prototype.getChangeIndicators = function () {
+	ChangeIndicatorRegistry.prototype.getChangeIndicators = function() {
 		return values(this._oChangeIndicators || {});
 	};
 
@@ -191,7 +191,7 @@ sap.ui.define([
 			if (sCommandName === "settings" && includes(Object.keys(aCategories), mChangeVisualizationInfo.descriptionPayload.category)) {
 				sChangeCategory = mChangeVisualizationInfo.descriptionPayload.category;
 			} else {
-				sChangeCategory = Object.keys(aCategories).find(function (sChangeCategoryName) {
+				sChangeCategory = Object.keys(aCategories).find(function(sChangeCategoryName) {
 					return includes(aCategories[sChangeCategoryName], sCommandName);
 				});
 				if (!sChangeCategory) {
@@ -228,33 +228,33 @@ sap.ui.define([
 				return undefined;
 			}
 			return aSelectorList
-				.map(function(vSelector) {
-					var oElement = typeof vSelector.getId === "function"
-						? vSelector
-						: JsControlTreeModifier.bySelector(vSelector, oAppComponent);
-					return oElement && oElement.getId();
-				})
-				.filter(Boolean);
+			.map(function(vSelector) {
+				var oElement = typeof vSelector.getId === "function"
+					? vSelector
+					: JsControlTreeModifier.bySelector(vSelector, oAppComponent);
+				return oElement && oElement.getId();
+			})
+			.filter(Boolean);
 		}
 
 		return getInfoFromChangeHandler(oAppComponent, oChange)
-			.then(function(oInfoFromChangeHandler) {
-				var mVisualizationInfo = oInfoFromChangeHandler || {};
-				var aChangeSelectors = oChange.getSelector && oChange.getSelector() && [oChange.getSelector()];
-				var aAffectedElementSelectors = mVisualizationInfo.affectedControls || aChangeSelectors || [];
-				// If there is an original selector (e.g. control is inside a template),
-				// the indicator should be displayed on the host control (change selector)
-				var oChangeOriginalSelector = oChange.getOriginalSelector && oChange.getOriginalSelector();
-				var aDisplayElementSelectors = oChangeOriginalSelector ? aChangeSelectors : aAffectedElementSelectors;
+		.then(function(oInfoFromChangeHandler) {
+			var mVisualizationInfo = oInfoFromChangeHandler || {};
+			var aChangeSelectors = oChange.getSelector && oChange.getSelector() && [oChange.getSelector()];
+			var aAffectedElementSelectors = mVisualizationInfo.affectedControls || aChangeSelectors || [];
+			// If there is an original selector (e.g. control is inside a template),
+			// the indicator should be displayed on the host control (change selector)
+			var oChangeOriginalSelector = oChange.getOriginalSelector && oChange.getOriginalSelector();
+			var aDisplayElementSelectors = oChangeOriginalSelector ? aChangeSelectors : aAffectedElementSelectors;
 
-				return {
-					affectedElementIds: getSelectorIds(aAffectedElementSelectors),
-					dependentElementIds: getSelectorIds(mVisualizationInfo.dependentControls) || [],
-					displayElementIds: getSelectorIds(mVisualizationInfo.displayControls || getSelectorIds(aDisplayElementSelectors)),
-					updateRequired: mVisualizationInfo.updateRequired,
-					descriptionPayload: mVisualizationInfo.descriptionPayload || {}
-				};
-			});
+			return {
+				affectedElementIds: getSelectorIds(aAffectedElementSelectors),
+				dependentElementIds: getSelectorIds(mVisualizationInfo.dependentControls) || [],
+				displayElementIds: getSelectorIds(mVisualizationInfo.displayControls || getSelectorIds(aDisplayElementSelectors)),
+				updateRequired: mVisualizationInfo.updateRequired,
+				descriptionPayload: mVisualizationInfo.descriptionPayload || {}
+			};
+		});
 	}
 
 	function getInfoFromChangeHandler(oAppComponent, oChange) {
@@ -270,19 +270,19 @@ sap.ui.define([
 				modifier: JsControlTreeModifier,
 				layer: oChange.getLayer()
 			})
-				.then(function(oChangeHandler) {
-					if (
-						oChangeHandler && typeof oChangeHandler.getChangeVisualizationInfo === "function"
+			.then(function(oChangeHandler) {
+				if (
+					oChangeHandler && typeof oChangeHandler.getChangeVisualizationInfo === "function"
 						&& oChange.isSuccessfullyApplied && oChange.isSuccessfullyApplied()
-					) {
-						return oChangeHandler.getChangeVisualizationInfo(oChange, oAppComponent);
-					}
-					return undefined;
-				})
-				.catch(function(vErr) {
-					Log.error(vErr);
-					return undefined;
-				});
+				) {
+					return oChangeHandler.getChangeVisualizationInfo(oChange, oAppComponent);
+				}
+				return undefined;
+			})
+			.catch(function(vErr) {
+				Log.error(vErr);
+				return undefined;
+			});
 		}
 
 		return Promise.resolve();
@@ -294,19 +294,19 @@ sap.ui.define([
 	 * @param {string} sSelectorId - The ID of the selector for which the change indicator is registered
 	 * @param {object} oChangeIndicator - The change indicator to register
 	 */
-	ChangeIndicatorRegistry.prototype.registerChangeIndicator = function (sSelectorId, oChangeIndicator) {
+	ChangeIndicatorRegistry.prototype.registerChangeIndicator = function(sSelectorId, oChangeIndicator) {
 		this._oChangeIndicators[sSelectorId] = oChangeIndicator;
 	};
 
 	/**
 	 * Resets the change and change indicator registries.
 	 */
-	ChangeIndicatorRegistry.prototype.reset = function () {
-		Object.keys(this._oRegisteredChanges).forEach(function (sKeyToRemove) {
+	ChangeIndicatorRegistry.prototype.reset = function() {
+		Object.keys(this._oRegisteredChanges).forEach(function(sKeyToRemove) {
 			this.removeRegisteredChange(sKeyToRemove);
 		}.bind(this));
 
-		values(this._oChangeIndicators).forEach(function (oIndicator) {
+		values(this._oChangeIndicators).forEach(function(oIndicator) {
 			oIndicator.destroy();
 		});
 		this._oChangeIndicators = {};
@@ -317,7 +317,7 @@ sap.ui.define([
 	 *
 	 * @param {string} sChangeId - ID of the registered change
 	 */
-	ChangeIndicatorRegistry.prototype.removeRegisteredChange = function (sChangeId) {
+	ChangeIndicatorRegistry.prototype.removeRegisteredChange = function(sChangeId) {
 		delete this._oRegisteredChanges[sChangeId];
 	};
 
@@ -325,7 +325,7 @@ sap.ui.define([
 	 * Removes changes with the updateRequired flag from the registry so the change can be re-registered and
 	 * the visualizationInfo is updated => if an element has an unstable id this updates the id information in the registry (e.g simple forms)
 	 */
-	ChangeIndicatorRegistry.prototype.removeOutdatedRegisteredChanges = function () {
+	ChangeIndicatorRegistry.prototype.removeOutdatedRegisteredChanges = function() {
 		this.getAllRegisteredChanges().forEach(function(oChange) {
 			if (oChange.visualizationInfo && oChange.visualizationInfo.updateRequired) {
 				this.removeRegisteredChange(oChange.change.getId());
@@ -337,7 +337,7 @@ sap.ui.define([
 	 * Removes changes without any displayElementIds from the registry so the change can be re-registered and
 	 * the visualizationInfo is updated => if an element is inside a dialog which hasn't been opened yet
 	 */
-	ChangeIndicatorRegistry.prototype.removeRegisteredChangesWithoutVizInfo = function () {
+	ChangeIndicatorRegistry.prototype.removeRegisteredChangesWithoutVizInfo = function() {
 		this.getAllRegisteredChanges().forEach(function(oChange) {
 			if (oChange.visualizationInfo && oChange.visualizationInfo.displayElementIds.length === 0) {
 				this.removeRegisteredChange(oChange.change.getId());

package/src/sap/ui/rta/util/changeVisualization/ChangeVisualization.js

@@ -59,6 +59,21 @@ sap.ui.define([
 		return !oOverlay || !oOverlay.getDomRef() || !oOverlay.isVisible();
 	}
 
+	function _determineElementOverlay(oElementId, oAffectedElementId) {
+		var oOverlay = OverlayRegistry.getOverlay(oElementId);
+		if (!oOverlay) {
+			// When the element has no Overlay, check if there is a relevant container Overlay
+			// e.g. change on a SmartForm group (Element: parent Form; Relevant Container: SmartForm)
+			var oElementOverlay = OverlayRegistry.getOverlay(oAffectedElementId);
+			var oRelevantContainer = oElementOverlay && oElementOverlay.getRelevantContainer();
+			if (oRelevantContainer) {
+				oOverlay = OverlayRegistry.getOverlay(oRelevantContainer);
+			}
+		}
+
+		return oOverlay;
+	}
+
 	/**
 	 * @class
 	 * Root control for RTA change visualization.
@@ -67,7 +82,7 @@ sap.ui.define([
 	 * @alias sap.ui.rta.util.changeVisualization.ChangeVisualization
 	 * @author SAP SE
 	 * @since 1.84.0
-	 * @version 1.115.0
+	 * @version 1.116.0
 	 * @private
 	 */
 	var ChangeVisualization = Control.extend("sap.ui.rta.util.changeVisualization.ChangeVisualization", {
@@ -163,12 +178,12 @@ sap.ui.define([
 		if (this.getProperty("rootControlId")) {
 			this._oChangeIndicatorRegistry.reset();
 			this._updateChangeRegistry()
-				.then(function() {
-					this._selectChangeCategory(this._sSelectedChangeCategory);
-					this._selectChangeState(ChangeStates.ALL);
-					this._updateVisualizationModelMenuData();
-					oToolbar.setModel(this._oChangeVisualizationModel, "visualizationModel");
-				}.bind(this));
+			.then(function() {
+				this._selectChangeCategory(this._sSelectedChangeCategory);
+				this._selectChangeState(ChangeStates.ALL);
+				this._updateVisualizationModelMenuData();
+				oToolbar.setModel(this._oChangeVisualizationModel, "visualizationModel");
+			}.bind(this));
 		}
 	};
 
@@ -176,7 +191,11 @@ sap.ui.define([
 		this._oChangeIndicatorRegistry.reset();
 	};
 
-	ChangeVisualization.prototype._determineChangeVisibility = function(aRegisteredIndependentChanges, aAllRelevantChanges, sVisualizedChangeState) {
+	ChangeVisualization.prototype._determineChangeVisibility = function(
+		aRegisteredIndependentChanges,
+		aAllRelevantChanges,
+		sVisualizedChangeState
+	) {
 		function filterRelevantChanges(aChanges) {
 			return aChanges.filter(function(oChange) {
 				if (
@@ -201,7 +220,7 @@ sap.ui.define([
 			return oChange.id;
 		});
 
-		aRegisteredIndependentChanges.forEach(function (oChange) {
+		aRegisteredIndependentChanges.forEach(function(oChange) {
 			if (oChange.changeStates.includes(ChangeStates.DIRTY)) {
 				bHasDraftChanges = true;
 				bHasDirtyChanges = true;
@@ -209,7 +228,10 @@ sap.ui.define([
 				bHasDraftChanges = true;
 			}
 
-			var oOverlay = OverlayRegistry.getOverlay(oChange.visualizationInfo.displayElementIds[0]);
+			var oOverlay = _determineElementOverlay(
+				oChange.visualizationInfo.displayElementIds[0],
+				oChange.visualizationInfo.affectedElementIds[0]
+			);
 
 			if (!aAllRelevantChangeIds.includes(oChange.change.getId())) {
 				aHiddenChanges.push(oChange);
@@ -266,7 +288,10 @@ sap.ui.define([
 		aCommandData.unshift({
 			key: ChangeCategories.ALL,
 			count: this._getChangesForChangeCategory(ChangeCategories.ALL, oSortedChanges.relevantVisualizedChanges).length,
-			title: this._getChangeCategoryLabel(ChangeCategories.ALL, this._getChangesForChangeCategory(ChangeCategories.ALL, oSortedChanges.relevantVisualizedChanges).length),
+			title: this._getChangeCategoryLabel(ChangeCategories.ALL, this._getChangesForChangeCategory(
+				ChangeCategories.ALL,
+				oSortedChanges.relevantVisualizedChanges
+			).length),
 			icon: ChangeCategories.getIconForCategory(ChangeCategories.ALL)
 		});
 
@@ -313,18 +338,18 @@ sap.ui.define([
 				id: this._getComponent().createId("changeVisualization_changesList"),
 				controller: this
 			})
-				.then(function(oPopover) {
-					this._oToolbarButton.addDependent(oPopover);
-					oPopover.setModel(this._oChangeVisualizationModel, "visualizationModel");
-					oPopover.openBy(this._oToolbarButton);
-					this.setPopover(oPopover);
-					//Currently required because of an binding issue from the control
-					//At the first opening of the popover the controls don't get updated when the bound
-					//model changes. With the reopening this Problem gets fixed
-					//TODO Remove once control owners have fixed the issue
-					oPopover.close();
-					oPopover.openBy(this._oToolbarButton);
-				}.bind(this));
+			.then(function(oPopover) {
+				this._oToolbarButton.addDependent(oPopover);
+				oPopover.setModel(this._oChangeVisualizationModel, "visualizationModel");
+				oPopover.openBy(this._oToolbarButton);
+				this.setPopover(oPopover);
+				// Currently required because of an binding issue from the control
+				// At the first opening of the popover the controls don't get updated when the bound
+				// model changes. With the reopening this Problem gets fixed
+				// TODO Remove once control owners have fixed the issue
+				oPopover.close();
+				oPopover.openBy(this._oToolbarButton);
+			}.bind(this));
 			return;
 		}
 
@@ -448,10 +473,10 @@ sap.ui.define([
 			}
 			var aRegisteredChangeIds = this._oChangeIndicatorRegistry.getRegisteredChangeIds();
 			var oCurrentChanges = aChanges
-				.reduce(function(oChanges, oChange) {
-					oChanges[oChange.getId()] = oChange;
-					return oChanges;
-				}, {});
+			.reduce(function(oChanges, oChange) {
+				oChanges[oChange.getId()] = oChange;
+				return oChanges;
+			}, {});
 			var aCurrentChangeIds = Object.keys(oCurrentChanges);
 
 			// Remove registered changes which no longer exist
@@ -503,20 +528,8 @@ sap.ui.define([
 		Object.keys(oSelectors).forEach(function(sSelectorId) {
 			var aChangesOnIndicator = oSelectors[sSelectorId];
 			var aRelevantChanges = this._filterRelevantChanges(oSelectors[sSelectorId]);
-			var oOverlay = OverlayRegistry.getOverlay(sSelectorId);
-			if (!oOverlay) {
-				// When the selector has no Overlay, check if there is a relevant container Overlay
-				// e.g. change on a SmartForm group (Selector: parent Form; Relevant Container: SmartForm)
-				aChangesOnIndicator.some(function(oChange) {
-					var oElementOverlay = OverlayRegistry.getOverlay(oChange.affectedElementId);
-					var oRelevantContainer = oElementOverlay && oElementOverlay.getRelevantContainer();
-					if (oRelevantContainer) {
-						oOverlay = OverlayRegistry.getOverlay(oRelevantContainer);
-						return true;
-					}
-					return false;
-				});
-			}
+			var oOverlay = _determineElementOverlay(sSelectorId, aChangesOnIndicator[0].affectedElementId);
+
 			if (_isOverlayInvisible(oOverlay)) {
 				// Change is not visible
 				return undefined;
@@ -647,15 +660,15 @@ sap.ui.define([
 		Core.applyChanges();
 
 		var aVisibleIndicators = this._oChangeIndicatorRegistry.getChangeIndicators()
-			.filter(function(oIndicator) {
-				return oIndicator.getVisible();
-			})
-			.sort(function(oIndicator1, oIndicator2) {
-				var iDeltaY = oIndicator1.getPosY() - oIndicator2.getPosY();
-				var iDeltaX = oIndicator1.getPosX() - oIndicator2.getPosX();
-				// Only consider x value if y is the same
-				return iDeltaY || iDeltaX;
-			});
+		.filter(function(oIndicator) {
+			return oIndicator.getVisible();
+		})
+		.sort(function(oIndicator1, oIndicator2) {
+			var iDeltaY = oIndicator1.getPosY() - oIndicator2.getPosY();
+			var iDeltaX = oIndicator1.getPosX() - oIndicator2.getPosX();
+			// Only consider x value if y is the same
+			return iDeltaY || iDeltaX;
+		});
 
 		if (aVisibleIndicators.length === 0) {
 			return;
@@ -678,7 +691,7 @@ sap.ui.define([
 		}
 	};
 
-	ChangeVisualization.prototype._toggleRootOverlayClickHandler = function (bEnable) {
+	ChangeVisualization.prototype._toggleRootOverlayClickHandler = function(bEnable) {
 		var oRootOverlayDomRef = this.oRootOverlay && this.oRootOverlay.getDomRef();
 		if (oRootOverlayDomRef) {
 			if (bEnable) {
@@ -732,15 +745,15 @@ sap.ui.define([
 		this.setIsActive(true);
 		// show all change visualizations at startup
 		this._updateChangeRegistry()
-			.then(function() {
-				this._selectChangeCategory(this._sSelectedChangeCategory);
-				// This is required to avoid flickering of the toolbar when switching
-				// to visualization mode when the mode switcher is displayed as icons
-				oToolbar.adjustToolbarSectionWidths();
-
-				this._updateVisualizationModelMenuData();
-				oToolbar.setModel(this._oChangeVisualizationModel, "visualizationModel");
-			}.bind(this));
+		.then(function() {
+			this._selectChangeCategory(this._sSelectedChangeCategory);
+			// This is required to avoid flickering of the toolbar when switching
+			// to visualization mode when the mode switcher is displayed as icons
+			oToolbar.adjustToolbarSectionWidths();
+
+			this._updateVisualizationModelMenuData();
+			oToolbar.setModel(this._oChangeVisualizationModel, "visualizationModel");
+		}.bind(this));
 	};
 
 	return ChangeVisualization;

package/src/sap/ui/rta/util/changeVisualization/commands/CombineVisualization.js

@@ -32,7 +32,7 @@ sap.ui.define([
 	 * @param {sap.ui.core.Component} mPropertyBag.appComponent - Application component
 	 * @returns {object} Map containing localized description text and tooltip
 	 */
-	CombineVisualization.getDescription = function (mPayload, sLabel, mPropertyBag) {
+	CombineVisualization.getDescription = function(mPayload, sLabel, mPropertyBag) {
 		var iOriginalSelectorCount = (mPayload.originalSelectors || []).length;
 		if (iOriginalSelectorCount < 2) {
 			// Fallback if no payload was provided
@@ -50,7 +50,7 @@ sap.ui.define([
 
 		var oAppComponent = mPropertyBag.appComponent;
 		var aOriginalSelectors = mPayload.originalSelectors;
-		var aLabels = aOriginalSelectors.map(function (oSelector) {
+		var aLabels = aOriginalSelectors.map(function(oSelector) {
 			var sId = JsControlTreeModifier.getControlIdBySelector(oSelector, oAppComponent);
 			var oControl = Core.byId(sId);
 			return oControl ? ElementUtil.getLabelForElement(oControl) : sId;
@@ -76,10 +76,10 @@ sap.ui.define([
 				[aLabels.length]
 			),
 			descriptionTooltip: aLabels
-				.map(function(sLabel) {
-					return "\"" + sLabel + "\"";
-				})
-				.join(",\n")
+			.map(function(sLabel) {
+				return "\"" + sLabel + "\"";
+			})
+			.join(",\n")
 		};
 	};
 

package/src/sap/ui/rta/util/changeVisualization/commands/CreateContainerVisualization.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * @param {string} sLabel - Current element label
 	 * @returns {object} Map containing localized description text and tooltip
 	 */
-	CreateContainerVisualization.getDescription = function (mPayload, sLabel) {
+	CreateContainerVisualization.getDescription = function(mPayload, sLabel) {
 		var oRtaResourceBundle = Core.getLibraryResourceBundle("sap.ui.rta");
 		var sElementLabel = mPayload.originalLabel || sLabel;
 		var sShortenedElementLabel = ChangeVisualizationUtils.shortenString(sElementLabel);

package/src/sap/ui/rta/util/changeVisualization/commands/SplitVisualization.js

@@ -23,7 +23,7 @@ sap.ui.define([
 	 * @param {string} sLabel - Current element label
 	 * @returns {object} Localized description text and button text
 	 */
-	SplitVisualization.getDescription = function (mPayload, sLabel) {
+	SplitVisualization.getDescription = function(mPayload, sLabel) {
 		var oRtaResourceBundle = Core.getLibraryResourceBundle("sap.ui.rta");
 		var sDescriptionText = oRtaResourceBundle.getText("TXT_CHANGEVISUALIZATION_CHANGE_SPLIT", ChangeVisualizationUtils.shortenString(sLabel));
 		var sDescriptionTooltip = oRtaResourceBundle.getText("TXT_CHANGEVISUALIZATION_CHANGE_SPLIT", sLabel);

package/src/sap/ui/rta/util/hasStableId.js

@@ -8,7 +8,7 @@ sap.ui.define([
 	"sap/ui/fl/Utils",
 	"sap/ui/dt/ElementUtil",
 	"sap/base/util/isPlainObject"
-], function (
+], function(
 	FlUtils,
 	ElementUtil,
 	isPlainObject
@@ -52,7 +52,7 @@ sap.ui.define([
 
 			if (aStableElements.length > 0) {
 				if (isCloneFromAggregationBinding(oElementOverlay.getElement())) {
-					bUnstable = aStableElements.some(function (vStableElement) {
+					bUnstable = aStableElements.some(function(vStableElement) {
 						var vControl;
 						var oAppComponent;
 						var bUnstable = false;

package/src/sap/ui/rta/util/showMessageBox.js

@@ -9,7 +9,7 @@ sap.ui.define([
 	"sap/ui/layout/HorizontalLayout",
 	"sap/m/Link",
 	"sap/m/Text"
-], function (
+], function(
 	MessageBox,
 	HorizontalLayout,
 	Link,
@@ -45,7 +45,7 @@ sap.ui.define([
 	function convertIntoControls(aSymbols) {
 		var oLayout = new HorizontalLayout({
 			allowWrapping: true,
-			content: aSymbols.map(function (sSymbol) {
+			content: aSymbols.map(function(sSymbol) {
 				if (isLink(sSymbol)) {
 					var mLink = extractLink(sSymbol);
 					return new Link({
@@ -72,7 +72,7 @@ sap.ui.define([
 		var aSymbols = [sMessage];
 		var aLinks = getLinks(sMessage);
 
-		aLinks.forEach(function (sLink) {
+		aLinks.forEach(function(sLink) {
 			var i = 0;
 			while (i < aSymbols.length) {
 				var sSymbol = aSymbols[i];
@@ -83,7 +83,7 @@ sap.ui.define([
 					var aParts = sSymbol.split(sLink);
 					var aInsert = [];
 
-					aParts.forEach(function (sPart, iIndex, aOriginal) { // eslint-disable-line no-loop-func
+					aParts.forEach(function(sPart, iIndex, aOriginal) { // eslint-disable-line no-loop-func
 						aInsert.push(sPart);
 						if (iIndex !== aOriginal.length - 1) {
 							aInsert.push(sLink);
@@ -99,7 +99,6 @@ sap.ui.define([
 		return aSymbols;
 	}
 
-
 	/**
 	 * Displays sap.m.MessageBox and interprets markdown links in the messages.
 	 *

package/src/sap/ui/rta/util/validateFlexEnabled.js

@@ -13,7 +13,7 @@ sap.ui.define([
 	"sap/base/Log",
 	"sap/ui/rta/Utils",
 	"sap/ui/dt/Util"
-], function (
+], function(
 	FlUtils,
 	MessageBox,
 	ObjectPath,
@@ -24,7 +24,7 @@ sap.ui.define([
 	DtUtil
 ) {
 	"use strict";
-	return function (oRta) {
+	return function(oRta) {
 		var mMessageBoxShow = {};
 		var aPendingOverlaysToValidate = [];
 		var oComponent = FlUtils.getAppComponentForControl(oRta.getRootControlInstance());
@@ -88,7 +88,7 @@ sap.ui.define([
 			var oElementOverlayCreated = oEvent.getParameters().elementOverlay;
 			if (oRta.getMode() === "adaptation") {
 				var oComponent = FlUtils.getAppComponentForControl(oRta.getRootControlInstance());
-				DtUtil.waitForSynced(oRta._oDesignTime, function (oOverlay) {
+				DtUtil.waitForSynced(oRta._oDesignTime, function(oOverlay) {
 					if (_isControlAvailable(oOverlay)) {
 						_handleUnstableIds(oRta, oComponent, [oOverlay]);
 					}
@@ -102,14 +102,14 @@ sap.ui.define([
 			var aUnstableOverlays = validateStableIds(aElementOverlays, oComponent);
 
 			if (aUnstableOverlays.length) {
-				aUnstableOverlays.forEach(function (oElementOverlay) {
+				aUnstableOverlays.forEach(function(oElementOverlay) {
 					Log.error("Control ID was generated dynamically by SAPUI5. To support SAPUI5 flexibility, a stable control ID is needed to assign the changes to.", oElementOverlay.getElement().getId());
 				});
 				_displayMessage(oRta, oComponent, "MSG_UNSTABLE_ID_FOUND", "ERROR", "HEADER_ERROR");
 			}
 		}
 
-		oRta.attachEventOnce("stop", function () {
+		oRta.attachEventOnce("stop", function() {
 			_setMessageBoxShow(oComponent.getId(), false);
 		});