@openui5/sap.ui.layout 1.91.0 → 1.93.2

package/src/sap/ui/layout/changeHandler/AddFormField.js

@@ -19,7 +19,7 @@ sap.ui.define([
 	 *
 	 * @author SAP SE
 	 *
-	 * @version 1.91.0
+	 * @version 1.93.2
 	 *
 	 * @experimental Since 1.50.0 This class is experimental and provides only limited functionality. Also the API might be
 	 *               changed in future.
@@ -37,37 +37,47 @@ sap.ui.define([
 			var mFieldSelector = mChangeContent.newFieldSelector;
 
 			var oCreatedFormElement;
+			var oParentFormContainer;
 
-			// "layoutControl" property is present only when the control is returned from Delegate.createLayout()
-			if (!mInnerControls.layoutControl) {
-				oCreatedFormElement = oModifier.createControl(
-					"sap.ui.layout.form.FormElement",
-					oAppComponent,
-					oView,
-					mFieldSelector
-				);
-				oModifier.insertAggregation(oCreatedFormElement, "label", mInnerControls.label, 0, oView);
-				oModifier.insertAggregation(oCreatedFormElement, "fields", mInnerControls.control, 0, oView);
-			} else {
-				oCreatedFormElement = mInnerControls.control;
-			}
+			return Promise.resolve()
+				.then(function(){
+					// "layoutControl" property is present only when the control is returned from Delegate.createLayout()
+					if (!mInnerControls.layoutControl) {
+						return Promise.resolve()
+							.then(oModifier.createControl.bind(oModifier, "sap.ui.layout.form.FormElement", oAppComponent, oView, mFieldSelector))
+							.then(function(oCreatedControl) {
+								oCreatedFormElement = oCreatedControl;
+								return Promise.all([
+									oModifier.insertAggregation(oCreatedFormElement, "label", mInnerControls.label, 0, oView),
+									oModifier.insertAggregation(oCreatedFormElement, "fields", mInnerControls.control, 0, oView)
+								]);
+							});
+					}
+					oCreatedFormElement = mInnerControls.control;
+					return undefined;
+				})
+				.then(function(){
+					oParentFormContainer = oChange.getDependentControl("parentFormContainer", mPropertyBag);
+					return oModifier.insertAggregation(oParentFormContainer,
+						"formElements",
+						oCreatedFormElement,
+						iIndex,
+						oView
+					);
+				})
+				.then(function(){
+					if (mInnerControls.valueHelp) {
+						return oModifier.insertAggregation(
+							oParentFormContainer,
+							"dependents",
+							mInnerControls.valueHelp,
+							0,
+							oView
+						);
+					}
+					return undefined;
+				});
 
-			var oParentFormContainer = oChange.getDependentControl("parentFormContainer", mPropertyBag);
-			oModifier.insertAggregation(oParentFormContainer,
-				"formElements",
-				oCreatedFormElement,
-				iIndex,
-				oView
-			);
-			if (mInnerControls.valueHelp) {
-				oModifier.insertAggregation(
-					oParentFormContainer,
-					"dependents",
-					mInnerControls.valueHelp,
-					0,
-					oView
-				);
-			}
 		},
 		aggregationName: "formElements",
 		parentAlias: "parentFormContainer",

package/src/sap/ui/layout/changeHandler/AddSimpleFormField.js

@@ -67,15 +67,17 @@ sap.ui.define([
 	}
 
 	function recreateContentAggregation(oSimpleForm, aContentClone, oModifier, mPropertyBag) {
-		oModifier.removeAllAggregation(oSimpleForm, "content");
-			for (var i = 0; i < aContentClone.length; ++i) {
-				oModifier.insertAggregation(oSimpleForm,
-					"content",
-					aContentClone[i],
-					i,
-					mPropertyBag.view
-				);
-			}
+		return aContentClone.reduce(function(oPreviousPromise, oContentClone, iIndex) {
+			return oPreviousPromise
+				.then(function() {
+					return oModifier.insertAggregation(oSimpleForm,
+						"content",
+						oContentClone,
+						iIndex,
+						mPropertyBag.view
+					);
+				});
+		}, Promise.resolve());
 	}
 
 	/**
@@ -87,7 +89,7 @@ sap.ui.define([
 	 *
 	 * @author SAP SE
 	 *
-	 * @version 1.91.0
+	 * @version 1.93.2
 	 *
 	 * @experimental Since 1.49.0 This class is experimental and provides only limited functionality. Also the API might be
 	 *               changed in future.
@@ -99,27 +101,38 @@ sap.ui.define([
 			var mInnerControls = mPropertyBag.innerControls;
 			var oModifier = mPropertyBag.modifier;
 			var oAppComponent = mPropertyBag.appComponent;
+			var aContent;
+			var iNewIndex;
+			var aContentClone;
 
 			var oChange = mPropertyBag.change;
 			// as the label is stored independent of the field and will not be destroyed by destroying the field, is needs to be remembered
 			var oRevertData = oChange.getRevertData();
 			oRevertData.labelSelector = oModifier.getSelector(mInnerControls.label, oAppComponent);
 
-			var aContent = oModifier.getAggregation(oSimpleForm, "content");
-			var iNewIndex = getIndex(aContent, mPropertyBag);
-			var aContentClone = insertLabelAndField(aContent, iNewIndex, mInnerControls);
-
-			recreateContentAggregation(oSimpleForm, aContentClone, oModifier, mPropertyBag);
-
-			if (mInnerControls.valueHelp) {
-				oModifier.insertAggregation(
-					oSimpleForm,
-					"dependents",
-					mInnerControls.valueHelp,
-					0,
-					mPropertyBag.view
-				);
-			}
+			return Promise.resolve()
+				.then(oModifier.getAggregation.bind(oModifier, oSimpleForm, "content"))
+				.then(function(aAggregationContent) {
+					aContent = aAggregationContent;
+					iNewIndex = getIndex(aContent, mPropertyBag);
+					aContentClone = insertLabelAndField(aContent, iNewIndex, mInnerControls);
+					return oModifier.removeAllAggregation(oSimpleForm, "content");
+				})
+				.then(function() {
+					return recreateContentAggregation(oSimpleForm, aContentClone, oModifier, mPropertyBag);
+				})
+				.then(function() {
+					if (mInnerControls.valueHelp) {
+						return oModifier.insertAggregation(
+							oSimpleForm,
+							"dependents",
+							mInnerControls.valueHelp,
+							0,
+							mPropertyBag.view
+						);
+					}
+					return undefined;
+				});
 		},
 		revertAdditionalControls : function(mPropertyBag) {
 			var oSimpleForm = mPropertyBag.control;
@@ -130,9 +143,11 @@ sap.ui.define([
 			var mLabelSelector = oChange.getRevertData().labelSelector;
 			if (mLabelSelector) {
 				var oLabel = oModifier.bySelector(mLabelSelector, oAppComponent);
-				oModifier.removeAggregation(oSimpleForm, "content", oLabel);
-				oModifier.destroy(oLabel);
+				return Promise.resolve()
+					.then(oModifier.removeAggregation.bind(oModifier, oSimpleForm, "content", oLabel))
+					.then(oModifier.destroy.bind(oModifier, oLabel));
 			}
+			return Promise.resolve();
 		},
 		aggregationName: "content",
 		mapParentIdIntoChange: function (oChange, mSpecificChangeInfo, mPropertyBag) {

package/src/sap/ui/layout/changeHandler/AddSimpleFormGroup.js

@@ -6,7 +6,7 @@
 
 sap.ui.define([
 	"sap/ui/fl/changeHandler/Base",
-	"sap/ui/fl/changeHandler/JsControlTreeModifier",
+	"sap/ui/core/util/reflection/JsControlTreeModifier",
 	"sap/base/Log"
 ], function (
 	Base,
@@ -19,7 +19,7 @@ sap.ui.define([
 		* Change handler for adding a simple form group.
 		* @alias sap.ui.layout.changeHandler.AddSimpleFormGroup
 		* @author SAP SE
-		* @version 1.91.0
+		* @version 1.93.2
 		* @experimental Since 1.27.0
 		*/
 	var AddSimpleFormGroup = {};
@@ -27,16 +27,25 @@ sap.ui.define([
 	AddSimpleFormGroup.CONTENT_AGGREGATION = "content";
 
 	var fnFirstGroupWithoutTitle = function(oModifier, aStopToken, aContent) {
-		for (var i = 0; i < aContent.length; i++) {
-			var sType = oModifier.getControlType(aContent[i]);
-			if (aStopToken.indexOf(sType) === -1) {
-				if (oModifier.getVisible(aContent[i])) {
-					return true;
-				}
-			} else {
-				return false;
-			}
-		}
+		return aContent.reduce(function(oPreviousPromise, oContent){
+			return oPreviousPromise
+				.then(function(bReturnValue){
+					if (bReturnValue !== undefined) {
+						return bReturnValue;
+					}
+					var sType = oModifier.getControlType(oContent);
+					if (aStopToken.indexOf(sType) === -1) {
+						return Promise.resolve()
+							.then(oModifier.getVisible.bind(oModifier, oContent))
+							.then(function(bVisible){
+								return bVisible || undefined;
+							});
+					} else {
+						return false;
+					}
+				});
+		}, Promise.resolve());
+
 	};
 
 	var fnMapGroupIndexToContentAggregationIndex = function(oModifier, aStopToken, aContent, iGroupIndex) {
@@ -47,22 +56,24 @@ sap.ui.define([
 		if (iGroupIndex === 0) {
 			return iGroupIndex;
 		}
+		return fnFirstGroupWithoutTitle(oModifier, aStopToken, aContent)
+			.then(function(bFirstGroupWithoutName){
+				if (bFirstGroupWithoutName) {
+					iCurrentGroupIndex++;
+				}
 
-		if (fnFirstGroupWithoutTitle(oModifier, aStopToken, aContent)) {
-			iCurrentGroupIndex++;
-		}
-
-		for (var i = 0; i < aContent.length; i++) {
-			var sType = oModifier.getControlType(aContent[i]);
-			if (aStopToken.indexOf(sType) > -1) {
-				iCurrentGroupIndex++;
-				if (iCurrentGroupIndex === iGroupIndex) {
-					oResult = aContent[i];
-					return aContent.indexOf(oResult);
+				for (var i = 0; i < aContent.length; i++) {
+					var sType = oModifier.getControlType(aContent[i]);
+					if (aStopToken.indexOf(sType) > -1) {
+						iCurrentGroupIndex++;
+						if (iCurrentGroupIndex === iGroupIndex) {
+							oResult = aContent[i];
+							return aContent.indexOf(oResult);
+						}
+					}
 				}
-			}
-		}
-		return aContent.length;
+				return aContent.length;
+			});
 	};
 
 	/**
@@ -74,14 +85,14 @@ sap.ui.define([
 	 * @param {object} mPropertyBag.modifier Modifier for the controls
 	 * @param {object} mPropertyBag.appComponent Component in which the change should be applied
 	 * @param {object} mPropertyBag.view Application view
-	 * @return {boolean} True if successful
+	 * @return {Promise} Promise resolving when change is applied successfully
 	 * @public
 	 */
 	AddSimpleFormGroup.applyChange = function (oChangeWrapper, oForm, mPropertyBag) {
 		var oModifier = mPropertyBag.modifier;
 		var oView = mPropertyBag.view;
-		var oAppComponent = mPropertyBag.appComponent,
-			oTitle;
+		var oAppComponent = mPropertyBag.appComponent;
+		var iInsertIndex;
 
 		var oChange = oChangeWrapper.getDefinition();
 		if (oChange.texts && oChange.texts.groupLabel && oChange.texts.groupLabel.value &&
@@ -100,37 +111,39 @@ sap.ui.define([
 			}
 			oChangeWrapper.setRevertData({groupId: sGroupId});
 			var sLabelText = oChange.texts.groupLabel.value;
-
-			var aContent = oModifier.getAggregation(oForm, AddSimpleFormGroup.CONTENT_AGGREGATION);
-
-			var iInsertIndex;
 			var iRelativeIndex;
-
-			if (typeof oChange.content.group.index === "number") {
-				// The old code support
-				iInsertIndex = oChange.content.group.index;
-			} else {
-				iRelativeIndex = oChange.content.group.relativeIndex;
-				iInsertIndex = fnMapGroupIndexToContentAggregationIndex(oModifier,
-					["sap.ui.core.Title", "sap.m.Title", "sap.m.Toolbar", "sap.m.OverflowToolbar"],
-					aContent, iRelativeIndex);
-			}
-
-			// Check if the change is applicable
-			if (oModifier.bySelector(sGroupId, oAppComponent)) {
-				return Base.markAsNotApplicable("Control to be created already exists:" + sGroupId);
-			}
-			oTitle = oModifier.createControl("sap.ui.core.Title", oAppComponent, oView, sGroupId);
-
-			oModifier.setProperty(oTitle, "text", sLabelText);
-			oModifier.insertAggregation(oForm, "content", oTitle, iInsertIndex, oView);
+			return Promise.resolve()
+				.then(function(){
+					return oModifier.getAggregation(oForm, AddSimpleFormGroup.CONTENT_AGGREGATION);
+				})
+				.then(function(aContent){
+					if (typeof oChange.content.group.index === "number") {
+						// The old code support
+						return oChange.content.group.index;
+					} else {
+						iRelativeIndex = oChange.content.group.relativeIndex;
+						return fnMapGroupIndexToContentAggregationIndex(oModifier,
+							["sap.ui.core.Title", "sap.m.Title", "sap.m.Toolbar", "sap.m.OverflowToolbar"],
+							aContent, iRelativeIndex);
+					}
+				})
+				.then(function(iReturnedInsertIndex) {
+					iInsertIndex = iReturnedInsertIndex;
+					// Check if the change is applicable
+					if (oModifier.bySelector(sGroupId, oAppComponent)) {
+						return Base.markAsNotApplicable("Control to be created already exists:" + sGroupId);
+					}
+					return oModifier.createControl("sap.ui.core.Title", oAppComponent, oView, sGroupId);
+				})
+				.then(function(oTitle) {
+					oModifier.setProperty(oTitle, "text", sLabelText);
+					return oModifier.insertAggregation(oForm, "content", oTitle, iInsertIndex, oView);
+				});
 
 		} else {
 			Log.error("Change does not contain sufficient information to be applied: [" + oChange.layer + "]" + oChange.namespace + "/" + oChange.fileName + "." + oChange.fileType);
 			//however subsequent changes should be applied
 		}
-
-		return true;
 	};
 
 	/**
@@ -198,7 +211,7 @@ sap.ui.define([
 	 * @param {object} mPropertyBag.modifier Modifier for the controls
 	 * @param {object} mPropertyBag.appComponent Component in which the change should be applied
 	 * @param {object} mPropertyBag.view Application view
-	 * @return {boolean} True if successful
+	 * @return {Promise} Promise resolving when change is successfully reverted
 	 * @public
 	 */
 	AddSimpleFormGroup.revertChange = function (oChangeWrapper, oForm, mPropertyBag) {
@@ -209,11 +222,14 @@ sap.ui.define([
 
 		var oGroupSelector = oModifier.getSelector(sGroupId, oAppComponent);
 		var oGroup = oModifier.bySelector(oGroupSelector, oAppComponent, oView);
-		oModifier.removeAggregation(oForm, AddSimpleFormGroup.CONTENT_AGGREGATION, oGroup);
-		oModifier.destroy(oGroup);
-		oChangeWrapper.resetRevertData();
-
-		return true;
+		return Promise.resolve()
+			.then(function() {
+				return oModifier.removeAggregation(oForm, AddSimpleFormGroup.CONTENT_AGGREGATION, oGroup);
+			})
+			.then(function() {
+				oModifier.destroy(oGroup);
+				oChangeWrapper.resetRevertData();
+			});
 	};
 
 	AddSimpleFormGroup.getChangeVisualizationInfo = function(oChange, oAppComponent) {