@openui5/sap.ui.rta 1.115.1 → 1.117.0

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

@@ -38,7 +38,7 @@ sap.ui.define([
 	var SINGULAR = true;
 	var PLURAL = false;
 
-	function isThereAnAggregationActionForSameAggregation (mActions, mParents) {
+	function isThereAnAggregationActionForSameAggregation(mActions, mParents) {
 		var sResponsibleElementsParentAggregation = mParents.responsibleElementOverlay.getParentAggregationOverlay().getAggregationName();
 		return Object.keys(mActions).some(function(sAggregationName) {
 			return sAggregationName === sResponsibleElementsParentAggregation;
@@ -75,7 +75,7 @@ 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.115.1
+	 * @version 1.117.0
 	 * @constructor
 	 * @private
 	 * @since 1.44
@@ -104,7 +104,7 @@ sap.ui.define([
 
 		// For add elements plugin, include other aggregations which are potentially valid targets for hidden elements
 		_getRelevantOverlays: function(oOverlay) {
-			var aRelevantOverlays = OverlayUtil.findAllOverlaysInContainer(oOverlay, /*bIncludeOtherAggregations=*/true);
+			var aRelevantOverlays = OverlayUtil.findAllOverlaysInContainer(oOverlay, /* bIncludeOtherAggregations= */true);
 
 			oOverlay.setRelevantOverlays(aRelevantOverlays);
 			return aRelevantOverlays;
@@ -145,7 +145,7 @@ sap.ui.define([
 		},
 
 		isAvailable: function(aElementOverlays, bOverlayIsSibling) {
-			return aElementOverlays.every(function (oElementOverlay) {
+			return aElementOverlays.every(function(oElementOverlay) {
 				return this._isEditableByPlugin(oElementOverlay, bOverlayIsSibling);
 			}, this);
 		},
@@ -235,72 +235,72 @@ sap.ui.define([
 			var aAllElements = [];
 
 			return ActionExtractor.getActions(bOverlayIsSibling, oResponsibleElementOverlay, this, undefined, this.getDesignTime())
-				.then(function(mRetrievedActions) {
-					if (sAggregationName === "$$OnlyChildCustomField$$") {
-						return [];
-					}
-					mActions = mRetrievedActions[sAggregationName];
-					return this.getAllElements(bOverlayIsSibling, [mParents.responsibleElementOverlay], sControlName, sDisplayText);
-				}.bind(this))
+			.then(function(mRetrievedActions) {
+				if (sAggregationName === "$$OnlyChildCustomField$$") {
+					return [];
+				}
+				mActions = mRetrievedActions[sAggregationName];
+				return this.getAllElements(bOverlayIsSibling, [mParents.responsibleElementOverlay], sControlName, sDisplayText);
+			}.bind(this))
+
+			.then(function(aCollectedElements) {
+				aAllElements = aCollectedElements;
+				// getAllElements() also sets the extensibility info
+				var oExtensibilityInfo = this.getExtensibilityInfo(bOverlayIsSibling);
+				this.getDialog().setCustomFieldEnabled(!!oExtensibilityInfo);
+				if (oExtensibilityInfo) {
+					this.getDialog().detachEvent("openCustomField", this._onOpenCustomField, this);
+					this.getDialog().attachEvent("openCustomField", bOverlayIsSibling, this._onOpenCustomField, this);
+					this.getDialog().setCustomFieldButtonVisible(true);
+					return this.getDialog().addExtensionData(oExtensibilityInfo.extensionData);
+				}
+				return this.getDialog().setCustomFieldButtonVisible(false);
+			}.bind(this))
+
+			.then(function() {
+				var oAggregationWithElements = aAllElements.filter(function(mElementsPerAggregation) {
+					return mElementsPerAggregation.aggregation === sAggregationName;
+				})[0];
+				var aElementsPerAggregation = oAggregationWithElements ? oAggregationWithElements.elements : [];
+
+				this.getDialog().setElements(aElementsPerAggregation);
+				if (sDisplayText) {
+					// Aggregation is part of title
+					var oTextResources = sap.ui.getCore().getLibraryResourceBundle("sap.ui.rta");
+					var sDialogTitle = oTextResources.getText("HEADER_ADDITIONAL_ELEMENTS_WITH_AGGREGATION", sDisplayText);
+					this.getDialog().setTitle(sDialogTitle);
+				} else if (sAggregationName || sControlName) {
+					// Only one aggregation, no aggregation in title
+					this._setDialogTitle(mActions || {}, mParents.parent, sControlName);
+				}
 
-				.then(function(aCollectedElements) {
-					aAllElements = aCollectedElements;
-					// getAllElements() also sets the extensibility info
-					var oExtensibilityInfo = this.getExtensibilityInfo(bOverlayIsSibling);
-					this.getDialog().setCustomFieldEnabled(!!oExtensibilityInfo);
-					if (oExtensibilityInfo) {
-						this.getDialog().detachEvent("openCustomField", this._onOpenCustomField, this);
-						this.getDialog().attachEvent("openCustomField", bOverlayIsSibling, this._onOpenCustomField, this);
-						this.getDialog().setCustomFieldButtonVisible(true);
-						return this.getDialog().addExtensionData(oExtensibilityInfo.extensionData);
-					}
-					return this.getDialog().setCustomFieldButtonVisible(false);
-				}.bind(this))
+				return this.getDialog().open()
 
 				.then(function() {
-					var oAggregationWithElements = aAllElements.filter(function(mElementsPerAggregation) {
-						return mElementsPerAggregation.aggregation === sAggregationName;
-					})[0];
-					var aElementsPerAggregation = oAggregationWithElements ? oAggregationWithElements.elements : [];
-
-					this.getDialog().setElements(aElementsPerAggregation);
-					if (sDisplayText) {
-						//Aggregation is part of title
-						var oTextResources = sap.ui.getCore().getLibraryResourceBundle("sap.ui.rta");
-						var sDialogTitle = oTextResources.getText("HEADER_ADDITIONAL_ELEMENTS_WITH_AGGREGATION", sDisplayText);
-						this.getDialog().setTitle(sDialogTitle);
-					} else if (sAggregationName || sControlName) {
-						//Only one aggregation, no aggregation in title
-						this._setDialogTitle(mActions || {}, mParents.parent, sControlName);
-					}
-
-					return this.getDialog().open()
-
-						.then(function() {
-							var aSelectedElements = this.getDialog().getSelectedElements();
-							return CommandBuilder.createCommands(mParents, vSiblingElement, mActions, iIndex, aSelectedElements, sAggregationName, this);
-						}.bind(this))
-
-						.then(function() {
-							var oOverlayToFocus = OverlayRegistry.getOverlay(vSiblingElement) || oResponsibleElementOverlay;
-							oOverlayToFocus.focus();
-						})
-
-						.catch(function(oError) {
-							//no error means canceled dialog
-							if (oError instanceof Error) {
-								throw oError;
-							}
-						});
+					var aSelectedElements = this.getDialog().getSelectedElements();
+					return CommandBuilder.createCommands(mParents, vSiblingElement, mActions, iIndex, aSelectedElements, sAggregationName, this);
 				}.bind(this))
 
+				.then(function() {
+					var oOverlayToFocus = OverlayRegistry.getOverlay(vSiblingElement) || oResponsibleElementOverlay;
+					oOverlayToFocus.focus();
+				})
+
 				.catch(function(oError) {
+					// no error means canceled dialog
 					if (oError instanceof Error) {
 						throw oError;
-					} else {
-						Log.info("Service not up to date, skipping add dialog", "sap.ui.rta");
 					}
 				});
+			}.bind(this))
+
+			.catch(function(oError) {
+				if (oError instanceof Error) {
+					throw oError;
+				} else {
+					Log.info("Service not up to date, skipping add dialog", "sap.ui.rta");
+				}
+			});
 		},
 
 		_setDialogTitle: function(mActions, oParentElement, sControlName) {
@@ -308,8 +308,8 @@ sap.ui.define([
 			this.getDialog().setTitle(sDialogTitle);
 		},
 
-		//Function called when custom field button was pressed
-		_onOpenCustomField: function (oEvent, bOverlayIsSibling) {
+		// Function called when custom field button was pressed
+		_onOpenCustomField: function(oEvent, bOverlayIsSibling) {
 			var sRtaStyleClassName = Utils.getRtaStyleClassName();
 			return FieldExtensibility.onTriggerCreateExtensionData(this.getExtensibilityInfo(bOverlayIsSibling), sRtaStyleClassName);
 		},
@@ -323,57 +323,57 @@ sap.ui.define([
 		 */
 		_isEditable: function(oOverlay, mPropertyBag) {
 			return Promise.all([this._isEditableCheck(mPropertyBag.sourceElementOverlay, true), this._isEditableCheck(mPropertyBag.sourceElementOverlay, false)])
-				.then(function(aPromiseValues) {
-					return {
-						asSibling: aPromiseValues[0],
-						asChild: aPromiseValues[1]
-					};
-				})
-				.catch(function (vError) {
-					Log.error(vError);
-				});
+			.then(function(aPromiseValues) {
+				return {
+					asSibling: aPromiseValues[0],
+					asChild: aPromiseValues[1]
+				};
+			})
+			.catch(function(vError) {
+				Log.error(vError);
+			});
 		},
 
 		_isEditableCheck: function(oOverlay, bOverlayIsSibling) {
 			return Promise.resolve()
-				.then(function() {
-					var mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oOverlay, this);
+			.then(function() {
+				var mParents = AdditionalElementsUtils.getParents(bOverlayIsSibling, oOverlay, this);
 
-					if (!mParents.relevantContainerOverlay) {
-						return false;
-					}
+				if (!mParents.relevantContainerOverlay) {
+					return false;
+				}
 
-					return ActionExtractor.getActions(bOverlayIsSibling, oOverlay, this, true, this.getDesignTime())
-						.then(function (mActions) {
-							this.clearCachedElements();
-							return Utils.doIfAllControlsAreAvailable([oOverlay, mParents.parentOverlay], function () {
-								var bEditable = false;
-								// For the sibling case, check if anything is available for the same aggregation
-								if (bOverlayIsSibling) {
-									bEditable = isThereAnAggregationActionForSameAggregation(mActions, mParents);
-								} else {
-									bEditable = Object.keys(mActions).some(function(sAggregationName) {
-										if (mActions[sAggregationName].addViaDelegate) {
-											bEditable = this.checkAggregationsOnSelf(mParents.parentOverlay, "add", undefined, "delegate");
-										}
-										if (!bEditable && mActions[sAggregationName].reveal) {
-											return true;
-										}
-										return bEditable;
-									}.bind(this));
+				return ActionExtractor.getActions(bOverlayIsSibling, oOverlay, this, true, this.getDesignTime())
+				.then(function(mActions) {
+					this.clearCachedElements();
+					return Utils.doIfAllControlsAreAvailable([oOverlay, mParents.parentOverlay], function() {
+						var bEditable = false;
+						// For the sibling case, check if anything is available for the same aggregation
+						if (bOverlayIsSibling) {
+							bEditable = isThereAnAggregationActionForSameAggregation(mActions, mParents);
+						} else {
+							bEditable = Object.keys(mActions).some(function(sAggregationName) {
+								if (mActions[sAggregationName].addViaDelegate) {
+									bEditable = this.checkAggregationsOnSelf(mParents.parentOverlay, "add", undefined, "delegate");
+								}
+								if (!bEditable && mActions[sAggregationName].reveal) {
+									return true;
 								}
 								return bEditable;
 							}.bind(this));
-						}.bind(this))
-						.then(function (bEditable) {
-							if (bEditable) {
-								bEditable =
-									this.hasStableId(oOverlay) //don't confuse the user/Web IDE by an editable overlay without stable ID
+						}
+						return bEditable;
+					}.bind(this));
+				}.bind(this))
+				.then(function(bEditable) {
+					if (bEditable) {
+						bEditable =
+									this.hasStableId(oOverlay) // don't confuse the user/Web IDE by an editable overlay without stable ID
 									&& this.hasStableId(mParents.parentOverlay);
-							}
-							return bEditable;
-						}.bind(this));
+					}
+					return bEditable;
 				}.bind(this));
+			}.bind(this));
 		},
 
 		/**
@@ -398,41 +398,41 @@ sap.ui.define([
 			this.clearExtensibilityInfo(bOverlayIsSibling);
 
 			return ActionExtractor.getActions(bOverlayIsSibling, oElementOverlay, this, undefined, this.getDesignTime())
-				.then(function(mAllActions) {
-					each(mAllActions, function(sAggregationName) {
-						mActions = mAllActions[sAggregationName];
-						mActions.aggregation = sAggregationName;
-						if (mActions.addViaDelegate) {
-							bCheckExtensibility = true;
-						}
-						aPromises.push({
-							aggregation: sAggregationName,
-							elementPromises: [
-								mActions.reveal ? AdditionalElementsAnalyzer.enhanceInvisibleElements(mParents.parent, mActions) : Promise.resolve([]),
-								mActions.addViaDelegate ? AdditionalElementsAnalyzer.getUnrepresentedDelegateProperties(mParents.parent, mActions.addViaDelegate) : Promise.resolve([])
-							]
-						});
-					});
-					if (bCheckExtensibility) {
-						return handleExtensibility(mParents.parent);
+			.then(function(mAllActions) {
+				each(mAllActions, function(sAggregationName) {
+					mActions = mAllActions[sAggregationName];
+					mActions.aggregation = sAggregationName;
+					if (mActions.addViaDelegate) {
+						bCheckExtensibility = true;
 					}
-					return undefined;
-				})
+					aPromises.push({
+						aggregation: sAggregationName,
+						elementPromises: [
+							mActions.reveal ? AdditionalElementsAnalyzer.enhanceInvisibleElements(mParents.parent, mActions) : Promise.resolve([]),
+							mActions.addViaDelegate ? AdditionalElementsAnalyzer.getUnrepresentedDelegateProperties(mParents.parent, mActions.addViaDelegate) : Promise.resolve([])
+						]
+					});
+				});
+				if (bCheckExtensibility) {
+					return handleExtensibility(mParents.parent);
+				}
+				return undefined;
+			})
 
-				.then(function(oExtensibilityInfo) {
-					this.setExtensibilityInfo(bOverlayIsSibling, oExtensibilityInfo);
-				}.bind(this))
+			.then(function(oExtensibilityInfo) {
+				this.setExtensibilityInfo(bOverlayIsSibling, oExtensibilityInfo);
+			}.bind(this))
 
-				.then(this._combineAnalyzerResults.bind(this, aPromises))
+			.then(this._combineAnalyzerResults.bind(this, aPromises))
 
-				.then(function(aAllElements) {
-					this.setCachedElements(aAllElements, bOverlayIsSibling);
-					return aAllElements;
-				}.bind(this))
+			.then(function(aAllElements) {
+				this.setCachedElements(aAllElements, bOverlayIsSibling);
+				return aAllElements;
+			}.bind(this))
 
-				.catch(function(oError) {
-					throw oError;
-				});
+			.catch(function(oError) {
+				throw oError;
+			});
 		},
 
 		/**
@@ -442,7 +442,7 @@ sap.ui.define([
 		 * @param  {sap.ui.dt.ElementOverlay} aElementOverlays - List of overlays for which the context menu was opened
 		 * @return {object[]} Array containing the items with required data
 		 */
-		getMenuItems: function (aElementOverlays) {
+		getMenuItems: function(aElementOverlays) {
 			var aMenuItems = [];
 			var oMenuItem;
 			this.clearCachedElements();
@@ -499,7 +499,7 @@ sap.ui.define([
 					aSubMenuItems = aSubMenuItems.concat(this._buildSubmenuItems(true, aElementOverlays, aElementsWithAggregations[1]));
 				}
 			} else {
-				vHandler = function (bOverlayIsSibling, aElementOverlays) {
+				vHandler = function(bOverlayIsSibling, aElementOverlays) {
 					return this.showAvailableElements(bOverlayIsSibling, sAggregationName, aElementOverlays);
 				}.bind(this, bOverlayIsSibling);
 			}
@@ -542,12 +542,12 @@ sap.ui.define([
 				var sAggregationName = mElementsWithAggregation.aggregation;
 				var sDisplayText = getMenuItemText(bOverlayIsSibling, sAggregationName, aElementOverlays);
 				var oItem = {
-					id: sPluginId + '_' + iPosition,
+					id: sPluginId + "_" + iPosition,
 					text: sDisplayText,
 					enabled: function(aElementOverlays) {
 						return this.isEnabled(aElementOverlays, bOverlayIsSibling, sAggregationName);
 					}.bind(this),
-					handler: function (bOverlayIsSibling, aElementOverlays) {
+					handler: function(bOverlayIsSibling, aElementOverlays) {
 						// showAvailableElements has optional parameters
 						return this.showAvailableElements(bOverlayIsSibling, sAggregationName, aElementOverlays, undefined, undefined, sDisplayText);
 					}.bind(this, bOverlayIsSibling),
@@ -607,25 +607,25 @@ sap.ui.define([
 			});
 
 			return Promise.all(aCollectedPromises)
-				.then(function(aElementsPerAggregation) {
-					// Filter out results without elements
-					return aElementsPerAggregation.filter(function(oElementsPerAggregation) {
-						var aElements = oElementsPerAggregation && oElementsPerAggregation.elements;
-						return aElements.length > 0;
-					});
+			.then(function(aElementsPerAggregation) {
+				// Filter out results without elements
+				return aElementsPerAggregation.filter(function(oElementsPerAggregation) {
+					var aElements = oElementsPerAggregation && oElementsPerAggregation.elements;
+					return aElements.length > 0;
 				});
+			});
 		},
 
 		clearCachedElements: function() {
 			this._oCachedElements = undefined;
 		},
 
-		setCachedElements: function (aElements, bOverlayIsSibling) {
+		setCachedElements: function(aElements, bOverlayIsSibling) {
 			this._oCachedElements = this._oCachedElements || {};
 			this._oCachedElements[bOverlayIsSibling ? "asSibling" : "asChild"] = aElements;
 		},
 
-		getCachedElements: function (bOverlayIsSibling) {
+		getCachedElements: function(bOverlayIsSibling) {
 			if (this._oCachedElements) {
 				return this._oCachedElements[bOverlayIsSibling ? "asSibling" : "asChild"];
 			}

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

@@ -15,7 +15,7 @@ sap.ui.define([
 	 * Helper object containing methods for the AdditionalElements plugin
 	 *
 	 * @author SAP SE
-	 * @version 1.115.1
+	 * @version 1.117.0
 	 * @private
 	 * @since 1.94
 	 * @experimental Since 1.94. This class is experimental and provides only limited functionality. Also the API might be changed in future.
@@ -28,7 +28,7 @@ sap.ui.define([
 	* @returns {sap.ui.dt.ElementDesignTimeMetadata} Design time metadata
 	*/
 	function getDTMetadataFromActions(mActions) {
-		return mActions["addViaDelegate"] && mActions["addViaDelegate"].designTimeMetadata;
+		return mActions.addViaDelegate && mActions.addViaDelegate.designTimeMetadata;
 	}
 
 	/**
@@ -59,7 +59,7 @@ sap.ui.define([
 			relevantContainerOverlay: oRelevantContainerOverlay,
 			parentOverlay: oParentOverlay,
 			relevantContainer: oRelevantContainer,
-			parent: oParentOverlay && oParentOverlay.getElement() //root overlay has no parent
+			parent: oParentOverlay && oParentOverlay.getElement() // root overlay has no parent
 		};
 	};
 
@@ -99,7 +99,6 @@ sap.ui.define([
 			return _aNames;
 		}, []);
 
-
 		var oTextResources = sap.ui.getCore().getLibraryResourceBundle("sap.ui.rta");
 
 		if (aNonDuplicateNames.length === 1) {

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.115.1
+	 * @version 1.117.0
 	 * @private
 	 * @since 1.94
 	 * @experimental Since 1.94. This class is experimental and provides only limited functionality. Also the API might be changed in future.
@@ -55,20 +55,20 @@ sap.ui.define([
 		var sTargetAggregation = mPropertyBag.targetAggregation;
 		var oPlugin = mPropertyBag.plugin;
 		return createRevealCommandForInvisible(oSelectedElement, mActions, mParents, oPlugin)
-			.then(function(oRevealCommandForInvisible) {
-				oCompositeCommand.addCommand(oRevealCommandForInvisible);
-				return createMoveCommandForInvisible(oSelectedElement, mParents, oSiblingElement, iIndex, sTargetAggregation, oPlugin);
-			})
-			.then(function(oMoveCommandForInvisible) {
-				if (oMoveCommandForInvisible) {
-					oCompositeCommand.addCommand(oMoveCommandForInvisible);
-				} else {
-					Log.warning("No move action configured for "
+		.then(function(oRevealCommandForInvisible) {
+			oCompositeCommand.addCommand(oRevealCommandForInvisible);
+			return createMoveCommandForInvisible(oSelectedElement, mParents, oSiblingElement, iIndex, sTargetAggregation, oPlugin);
+		})
+		.then(function(oMoveCommandForInvisible) {
+			if (oMoveCommandForInvisible) {
+				oCompositeCommand.addCommand(oMoveCommandForInvisible);
+			} else {
+				Log.warning("No move action configured for "
 						+ mParents.parent.getMetadata().getName()
 						+ ", aggregation: " + oSelectedElement.aggregation, "sap.ui.rta");
-				}
-				return oCompositeCommand;
-			});
+			}
+			return oCompositeCommand;
+		});
 	}
 
 	function createRevealCommandForInvisible(mSelectedElement, mActions, mParents, oPlugin) {
@@ -102,7 +102,7 @@ sap.ui.define([
 	function createMoveCommandForInvisible(oSelectedElement, mParents, oSiblingElement, iIndex, sTargetAggregationName, oPlugin) {
 		var oRevealedElement = ElementUtil.getElementInstance(oSelectedElement.elementId);
 		var oRevealedElementOverlay = OverlayRegistry.getOverlay(oRevealedElement);
-		//Elements can also be moved between aggregations inside the parent
+		// Elements can also be moved between aggregations inside the parent
 		var sSourceAggregationName = oSelectedElement.sourceAggregation;
 		var oSourceParent = oRevealedElementOverlay.getParentElementOverlay().getElement() || mParents.parent;
 		var oTargetParent = mParents.parent;
@@ -171,18 +171,18 @@ sap.ui.define([
 		var oParentAggregationOverlay = mPropertyBag.parents.parentOverlay.getAggregationOverlay(mPropertyBag.actions.aggregation);
 		var oParentAggregationDTMetadata = oParentAggregationOverlay.getDesignTimeMetadata();
 		return createCommandForAddLibrary(mPropertyBag.parents, mRequiredLibraries, oParentAggregationDTMetadata, mPropertyBag.plugin)
-			.then(function(oCommandForAddLibrary) {
-				if (oCommandForAddLibrary) {
-					oCompositeCommand.addCommand(oCommandForAddLibrary);
-				}
-				return createAddViaDelegateCommand(mPropertyBag, oParentAggregationDTMetadata);
-			})
-			.then(function(oAddViaDelegateCommand) {
-				if (oAddViaDelegateCommand) {
-					oCompositeCommand.addCommand(oAddViaDelegateCommand);
-				}
-				return oCompositeCommand;
-			});
+		.then(function(oCommandForAddLibrary) {
+			if (oCommandForAddLibrary) {
+				oCompositeCommand.addCommand(oCommandForAddLibrary);
+			}
+			return createAddViaDelegateCommand(mPropertyBag, oParentAggregationDTMetadata);
+		})
+		.then(function(oAddViaDelegateCommand) {
+			if (oAddViaDelegateCommand) {
+				oCompositeCommand.addCommand(oAddViaDelegateCommand);
+			}
+			return oCompositeCommand;
+		});
 	}
 
 	function getODataServiceUriFromManifest(oManifest) {
@@ -256,49 +256,49 @@ sap.ui.define([
 		if (aSelectedElements.length > 0) {
 			return oPlugin.getCommandFactory().getCommandFor(mParents.parent, "composite")
 
-					.then(function(oCompositeCommand) {
-						var oPromise = Promise.resolve();
-						aSelectedElements.forEach(function(oSelectedElement) {
-							var mPropertyBag = {
-								compositeCommand: oCompositeCommand,
-								selectedElement: oSelectedElement,
-								parents: mParents,
-								siblingElement: oSiblingElement,
-								actions: mActions,
-								index: iIndex,
-								targetAggregation: sTargetAggregation,
-								plugin: oPlugin
-							};
-							switch (oSelectedElement.type) {
-								case "invisible":
-									oPromise = oPromise.then(
-										createCommandsForInvisibleElement.bind(this, mPropertyBag));
-									break;
-								case "delegate":
-									oPromise = oPromise.then(
-										createCommandsForAddViaDelegate.bind(this, mPropertyBag));
-									break;
-								default:
-									Log.error("Can't create command for untreated element.type " + oSelectedElement.type);
-							}
-						}, this);
-						return oPromise.then(function() { return oCompositeCommand; });
-					}.bind(this))
+			.then(function(oCompositeCommand) {
+				var oPromise = Promise.resolve();
+				aSelectedElements.forEach(function(oSelectedElement) {
+					var mPropertyBag = {
+						compositeCommand: oCompositeCommand,
+						selectedElement: oSelectedElement,
+						parents: mParents,
+						siblingElement: oSiblingElement,
+						actions: mActions,
+						index: iIndex,
+						targetAggregation: sTargetAggregation,
+						plugin: oPlugin
+					};
+					switch (oSelectedElement.type) {
+						case "invisible":
+							oPromise = oPromise.then(
+								createCommandsForInvisibleElement.bind(this, mPropertyBag));
+							break;
+						case "delegate":
+							oPromise = oPromise.then(
+								createCommandsForAddViaDelegate.bind(this, mPropertyBag));
+							break;
+						default:
+							Log.error("Can't create command for untreated element.type " + oSelectedElement.type);
+					}
+				}, this);
+				return oPromise.then(function() { return oCompositeCommand; });
+			}.bind(this))
 
-					.then(function(oCompositeCommand) {
-						oPlugin.fireElementModified({
-							command: oCompositeCommand
-						});
-					})
+			.then(function(oCompositeCommand) {
+				oPlugin.fireElementModified({
+					command: oCompositeCommand
+				});
+			})
 
-					.catch(function(vMessage) {
-						throw DtUtils.propagateError(
-							vMessage,
-							"AdditionalElementsPlugin#_createCommands",
-							"Error occurred during _createCommands execution",
-							"sap.ui.rta.plugin"
-						);
-					});
+			.catch(function(vMessage) {
+				throw DtUtils.propagateError(
+					vMessage,
+					"AdditionalElementsPlugin#_createCommands",
+					"Error occurred during _createCommands execution",
+					"sap.ui.rta.plugin"
+				);
+			});
 		}
 		return Promise.resolve();
 	};