@openui5/sap.ui.mdc 1.97.1 → 1.98.0

package/src/sap/ui/mdc/p13n/FlexUtil.js

@@ -10,101 +10,6 @@ sap.ui.define([
 
 	var FlexUtil = {
 
-		/**
-		* Generates a set of changes based on the given arrays for a specified control
-		*
-		* @public
-		*
-		* @param {object} mDeltaInfo Map containing the necessary information to calculate the diff as change objects
-		* @param {array} mDeltaInfo.existingState An array describing the control state before a adaptation
-		* @param {array} mDeltaInfo.changedState An array describing the control state after a certain adaptation
-		* @param {object} mDeltaInfo.control Control instance which is being used to generate the changes
-		* @param {object} mDeltaInfo.changeOperations Map containing the changeOperations for the given Control instance
-		* @param {string} mDeltaInfo.changeOperations.add Name of the control specific 'add' changehandler
-		* @param {string} mDeltaInfo.changeOperations.remove Name of the control specific 'remove' changehandler
-		* @param {string} [mDeltaInfo.changeOperations.move] Name of the control specific 'move' changehandler
-		* @param {string} [mDeltaInfo.generator] Name of the change generator (E.g. the namespace of the UI creating the change object)
-		*
-		* @returns {array} Array containing the delta based created changes
-		*/
-		getArrayDeltaChanges: function (mDeltaInfo) {
-
-			var aExistingArray = mDeltaInfo.existingState;
-			var aChangedArray = mDeltaInfo.changedState;
-			var oControl = mDeltaInfo.control;
-			var sInsertOperation = mDeltaInfo.changeOperations.add;
-			var sRemoveOperation = mDeltaInfo.changeOperations.remove;
-			var sMoveOperation = mDeltaInfo.changeOperations.move;
-			var sGenerator = mDeltaInfo.generator;
-
-			var aDeltaAttributes = mDeltaInfo.deltaAttributes || [];
-
-			var fnSymbol = function(o) {
-				var sDiff = "";
-				aDeltaAttributes.forEach(function(sAttribute){
-					sDiff = sDiff + o[sAttribute];
-				});
-				return sDiff;
-			};
-
-			var aResults = diff(aExistingArray, aChangedArray, fnSymbol);
-			// Function to match field with exising field in the given array
-			var fMatch = function (oField, aArray) {
-				return aArray.filter(function (oExistingField) {
-					return oExistingField && (oExistingField.name === oField.name);
-				})[0];
-			};
-
-			var aChanges = [];
-			var aProcessedArray = aExistingArray.slice(0);
-
-			aResults.forEach(function (oResult) {
-				// Begin --> hack for handling result returned by diff
-				if (oResult.type === "delete" && aProcessedArray[oResult.index] === undefined) {
-					aProcessedArray.splice(oResult.index, 1);
-					return;
-				}
-
-				var oProp, oExistingProp, iLength;
-				if (oResult.type === "insert") {
-					oExistingProp = fMatch(aChangedArray[oResult.index], aProcessedArray);
-					if (oExistingProp) {
-						oExistingProp.index = aProcessedArray.indexOf(oExistingProp);
-						aProcessedArray.splice(oExistingProp.index, 1, undefined);
-						aChanges.push(FlexUtil.createAddRemoveChange(oControl, sRemoveOperation, FlexUtil._getChangeContent(oExistingProp, aDeltaAttributes), sGenerator));
-					}
-				}
-				// End hack
-				oProp = oResult.type === "delete" ? aProcessedArray[oResult.index] : aChangedArray[oResult.index];
-				oProp.index = oResult.index;
-				if (oResult.type === "delete") {
-					aProcessedArray.splice(oProp.index, 1);
-				} else {
-					aProcessedArray.splice(oProp.index, 0, oProp);
-				}
-				// Move operation shows up as insert followed by delete OR delete followed by insert
-				if (sMoveOperation) {
-					iLength = aChanges.length;
-					// Get the last added change
-					if (iLength) {
-						oExistingProp = aChanges[iLength - 1];
-						oExistingProp = oExistingProp ? oExistingProp.changeSpecificData.content : undefined;
-					}
-					// Matching property exists with a different index --> then this is a move operation
-					if (oExistingProp && oExistingProp.name === oProp.name && oResult.index != oExistingProp.index) {
-						// remove the last insert/delete operation
-						aChanges.pop();
-						aChanges.push(FlexUtil.createMoveChange(oExistingProp.id, oExistingProp.name, oResult.index, sMoveOperation, oControl, sMoveOperation !== "moveSort", sGenerator));
-						return;
-					}
-				}
-
-				aChanges.push(FlexUtil.createAddRemoveChange(oControl, oResult.type === "delete" ? sRemoveOperation : sInsertOperation, FlexUtil._getChangeContent(oProp, aDeltaAttributes), sGenerator));
-
-			});
-			return aChanges;
-		},
-
 		getPropertySetterChanges: function(mDeltaInfo) {
 			var oControl = mDeltaInfo.control;
 			var aExistingState = mDeltaInfo.existingState;
@@ -124,7 +29,7 @@ sap.ui.define([
 					var vNewValue = oItem[sSetAttribute];
 					var bValueChanged = vOldValue !== vNewValue;
 					if (bValueChanged) {
-						aSetterChanges.push(this.createAddRemoveChange(oControl, sOperation, {
+						aSetterChanges.push(this.createChange(oControl, sOperation, {
 							name: oItem.name,
 							value: oItem[sSetAttribute]
 						}));
@@ -193,7 +98,14 @@ sap.ui.define([
 					Log.warning("property '" + sFieldPath + "' not supported");
 					continue;
 				}
-				aConditionChanges = aConditionChanges.concat(FlexUtil._diffConditionPath(sFieldPath, mNewConditionState[sFieldPath], mPreviousConditionState[sFieldPath], oAdaptationControl));
+
+				var aFilterConditionChanges = FlexUtil._diffConditionPath(sFieldPath, mNewConditionState[sFieldPath], mPreviousConditionState[sFieldPath], oAdaptationControl);
+
+				if (oAdaptationControl && oAdaptationControl.createPropertyInfoChanges && (aFilterConditionChanges.length > 0)) {
+					aFilterConditionChanges = oAdaptationControl.createPropertyInfoChanges(sFieldPath).concat(aFilterConditionChanges);
+				}
+
+				aConditionChanges = aConditionChanges.concat(aFilterConditionChanges);
 			}
 
 			return aConditionChanges;
@@ -274,7 +186,8 @@ sap.ui.define([
 
 			return aChanges;
 		},
-		createAddRemoveChange: function(oControl, sOperation, oContent){
+
+		createChange: function(oControl, sOperation, oContent){
 			var oAddRemoveChange = {
 				selectorElement: oControl,
 				changeSpecificData: {
@@ -285,26 +198,6 @@ sap.ui.define([
 			return oAddRemoveChange;
 		},
 
-		createMoveChange: function (sId, sPropertyName, iNewIndex, sMoveOperation, oControl, bPersistId) {
-			var oMoveChange =  {
-				selectorElement: oControl,
-				changeSpecificData: {
-					changeType: sMoveOperation,
-					content: {
-						id: sId,
-						name: sPropertyName,
-						index: iNewIndex
-					}
-				}
-			};
-
-			if (!bPersistId) {
-				delete oMoveChange.changeSpecificData.content.id;
-			}
-
-			return oMoveChange;
-		},
-
 		createConditionChange: function(sChangeType, oControl, sFieldPath, oCondition) {
 			var oConditionChange = {
 				selectorElement: oControl,

package/src/sap/ui/mdc/p13n/P13nBuilder.js

@@ -196,7 +196,7 @@ sap.ui.define([
         prepareAdaptationData: function(vProperties, fnEnhace, bGroupData) {
 
 			var oPropertyHelper =
-				vProperties && vProperties.getProperties instanceof Function ?
+                vProperties && vProperties.getProperties instanceof Function ?
 				vProperties : new P13nPropertyHelper(vProperties);
 
             var aItems = [];

package/src/sap/ui/mdc/p13n/PropertyHelper.js

@@ -29,7 +29,7 @@ sap.ui.define([
 	 * @extends sap.ui.mdc.util.PropertyHelper
 	 *
 	 * @author SAP SE
-	 * @version 1.97.1
+	 * @version 1.98.0
 	 *
 	 * @private
 	 * @experimental

package/src/sap/ui/mdc/p13n/StateUtil.js

@@ -79,6 +79,16 @@ sap.ui.define([
 		*			Country: {
 		*				aggregated: false
 		*			}
+		*		},
+		*		//The supplementaryConfig can be used to modify control specific attributes, for example the column width in the Table
+		*		supplementaryConfig: {
+		*			aggregations: {
+		*				columns: {
+		*					Category: {
+		*						width: "150px"
+		*					}
+		*				}
+		*			}
 		*		}
 		* }
 		*
@@ -112,6 +122,9 @@ sap.ui.define([
 		 * Attaches an event handler to the <code>StateUtil</code>.
 		 * The event handler may be fired every time a user triggers a personalization change for a control instance during runtime.
 		 *
+		 * @private
+		 * @ui5-restricted sap.fe
+		 * @MDC_PUBLIC_CANDIDATE
 		 * @param {function} fnListener fnFunction The handler function to call when the event occurs
 		 */
 		attachStateChange: function(fnListener) {
@@ -122,6 +135,9 @@ sap.ui.define([
 		 * Removes a previously attached state change event handler from the <code>StateUtil</code> class.
 		 * The passed parameters must match those used for registration with {@link StateUtil#attachChange} beforehand.
 		 *
+		 * @private
+		 * @ui5-restricted sap.fe
+		 * @MDC_PUBLIC_CANDIDATE
 		 * @param {function} fnListener fnFunction The handler function to detach from the event
 		 */
 		detachStateChange: function(fnListener) {
@@ -135,7 +151,8 @@ sap.ui.define([
 				Aggregate: "aggregations",
 				Filter: "filter",
 				Item: "items",
-				Column: "items"
+				Column: "items",
+				ColumnWidth: "supplementaryConfig"
 			};
 			var oTransformedState = {};
 
@@ -144,10 +161,6 @@ sap.ui.define([
 				var sTransformedKey = sExternalKey || sProvidedEngineKey;//no external key --> provide internal key
 				oTransformedState[sTransformedKey] = oInternalState[sProvidedEngineKey];
 			});
-			//TODO: Remove once the stateUtil for columnWidth is introduced
-			if (oTransformedState.ColumnWidth) {
-				delete oTransformedState.ColumnWidth;
-			}
 			return oTransformedState;
 		},
 
@@ -157,7 +170,8 @@ sap.ui.define([
 				groupLevels: ["Group"],
 				aggregations: ["Aggregate"],
 				filter: ["Filter"],
-				items: ["Item", "Column"]
+				items: ["Item", "Column"],
+				supplementaryConfig: ["ColumnWidth"]
 			};
 
 			var oTransformedState = {};

package/src/sap/ui/mdc/p13n/UIManager.js

@@ -32,7 +32,7 @@ sap.ui.define([
 	 * @extends sap.ui.base.Object
 	 *
 	 * @author SAP SE
-	 * @version 1.97.1
+	 * @version 1.98.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.mdc
@@ -364,7 +364,7 @@ sap.ui.define([
 
 		var mSettings = Object.assign({
 			verticalScrolling: true,
-			reset: mUISettings.reset,
+			reset: mUISettings.reset || {},
 			afterClose: function(oEvt) {
 				var oDialog = oEvt.getSource();
 				if (oDialog) {
@@ -375,11 +375,9 @@ sap.ui.define([
 		}, mUISettings.containerSettings);
 
 		if (mUISettings.resetEnabled){
-			mSettings.reset = {
-				onExecute: function() {
-					this.oAdaptationProvider.reset(oControl, aKeys);
-				}.bind(this)
-			};
+			mSettings.reset.onExecute = function() {
+				this.oAdaptationProvider.reset(oControl, aKeys);
+			}.bind(this);
 		}
 
 		mSettings.confirm = {

package/src/sap/ui/mdc/p13n/modification/FlexModificationHandler.js

@@ -54,8 +54,11 @@ sap.ui.define([
 
     FlexModificationHandler.prototype.reset = function(mPropertyBag, oModificationPayload){
         var sPersistenceMode = oModificationPayload.mode;
+
         var bIsGlobal = sPersistenceMode === mode.Global;
-        return bIsGlobal ? FlexUtil.reset.call(this, mPropertyBag) : FlexUtil.restore.call(this, mPropertyBag);
+        var bIsAutoGlobal = !oModificationPayload.hasVM && oModificationPayload.hasPP && sPersistenceMode === mode.Auto;
+
+        return (bIsGlobal || bIsAutoGlobal) ? FlexUtil.reset.call(this, mPropertyBag) : FlexUtil.restore.call(this, mPropertyBag);
     };
 
     FlexModificationHandler.prototype.isModificationSupported = function(mPropertyBag, oModificationPayload){

package/src/sap/ui/mdc/p13n/modification/ModificationHandler.js

@@ -160,15 +160,18 @@ sap.ui.define([
 
 				var oAppComponent = mPropertyBag ? mPropertyBag.appComponent : undefined;
 
-				if (!oControl._bHasXConfig) {
-					oControl._bHasXConfig = true;
-					return oModifier.createAndAddCustomData(oControl, "xConfig", oConfig, oAppComponent)
-					.then(function() {
+				if (!oControl._pXConfigCreation) {
+					oControl._pXConfigCreation = oModifier.createAndAddCustomData(oControl, "xConfig", oConfig, oAppComponent);
+					return oControl._pXConfigCreation
+					.then(function(){
 						return oConfig;
 					});
 				} else {
-					oModifier.setProperty(oXConfig, "value", oConfig);
-					return oConfig;
+					oControl._pXConfigCreation
+					.then(function(oCustomData){
+						oModifier.setProperty(oCustomData, "value", oConfig);
+						return oConfig;
+					});
 				}
 			});
 	};

package/src/sap/ui/mdc/p13n/modules/DefaultProviderRegistry.js

@@ -25,7 +25,7 @@ sap.ui.define([
 	 * @extends sap.ui.base.Object
 	 *
 	 * @author SAP SE
-	 * @version 1.97.1
+	 * @version 1.98.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.mdc