@openui5/sap.m 1.120.7 → 1.121.0

package/src/sap/m/p13n/SortPanel.js

@@ -4,8 +4,22 @@
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
-	"./QueryPanel", "sap/m/Text", "sap/m/SegmentedButton", "sap/m/SegmentedButtonItem", "sap/ui/layout/Grid", "sap/ui/layout/GridData"
-], function (QueryPanel, Text, SegmentedButton, SegmentedButtonItem, Grid, GridData) {
+	"./QueryPanel",
+	"sap/m/Text",
+	"sap/m/SegmentedButton",
+	"sap/m/SegmentedButtonItem",
+	"sap/ui/core/Lib",
+	"sap/ui/layout/Grid",
+	"sap/ui/layout/GridData"
+], (
+	QueryPanel,
+	Text,
+	SegmentedButton,
+	SegmentedButtonItem,
+	Library,
+	Grid,
+	GridData
+) => {
 	"use strict";
 
 	/**
@@ -21,23 +35,23 @@ sap.ui.define([
 	 * @extends sap.m.p13n.QueryPanel
 	 *
 	 * @author SAP SE
-	 * @version 1.120.7
+	 * @version 1.121.0
 	 *
 	 * @public
 	 * @since 1.96
 	 * @alias sap.m.p13n.SortPanel
 	 */
 
-	var SortPanel = QueryPanel.extend("sap.m.p13n.SortPanel", {
+	const SortPanel = QueryPanel.extend("sap.m.p13n.SortPanel", {
 		metadata: {
 			properties: {
 				/**
 				 * A short text describing the panel.
 				 * <b>Note:</b> This text will only be displayed if the panel is being used in a <code>sap.m.p13n.Popup</code>.
 				 */
-				 title: {
+				title: {
 					type: "string",
-					defaultValue: sap.ui.getCore().getLibraryResourceBundle("sap.m").getText("p13n.DEFAULT_TITLE_SORT")
+					defaultValue: Library.getResourceBundleFor("sap.m").getText("p13n.DEFAULT_TITLE_SORT")
 				}
 			},
 			library: "sap.m"
@@ -66,25 +80,26 @@ sap.ui.define([
 
 	/**
 	 * Sets the personalization state of the panel instance.
+	 *
 	 * @name sap.m.p13n.SortPanel.setP13nData
 	 * @public
 	 * @function
 	 *
-	 * @param {sap.m.p13n.SortItem} aP13nData An array containing the personalization state
+	 * @param {sap.m.p13n.SortItem[]} aP13nData An array containing the personalization state
 	 * @returns {sap.m.p13n.SortPanel} The SortPanel instance
 	 *
 	 */
 
-	SortPanel.prototype._createRemoveButton = function () {
-		var oRemoveBtn = QueryPanel.prototype._createRemoveButton.apply(this, arguments);
+	SortPanel.prototype._createRemoveButton = function() {
+		const oRemoveBtn = QueryPanel.prototype._createRemoveButton.apply(this, arguments);
 		oRemoveBtn.setLayoutData(new GridData({
-			span: "XL3 L3 M3 S4"//on "S" the Asc/Desc text is invisible, we need to increase the size the
+			span: "XL3 L3 M3 S4" //on "S" the Asc/Desc text is invisible, we need to increase the size the
 		}));
 		return oRemoveBtn;
 	};
 
-	SortPanel.prototype._createOrderSwitch = function (sKey, bDesc) {
-		var oSortOrderSwitch = new SegmentedButton({
+	SortPanel.prototype._createOrderSwitch = function(sKey, bDesc) {
+		const oSortOrderSwitch = new SegmentedButton({
 			enabled: sKey ? true : false,
 			layoutData: new GridData({
 				span: "XL2 L2 M2 S3" //on "S" the Asc/Desc text is invisible, we need to increase the size then
@@ -99,15 +114,15 @@ sap.ui.define([
 					icon: "sap-icon://sort-descending"
 				})
 			],
-			selectionChange: function (oEvt) {
-				var oItem = oEvt.getParameter("item");
-				var sSortOrder = oItem.getKey();
-				var oText = oEvt.getSource().getParent().getContent()[2];
+			selectionChange: (oEvt) => {
+				const oItem = oEvt.getParameter("item");
+				const sSortOrder = oItem.getKey();
+				const oText = oEvt.getSource().getParent().getContent()[2];
 				oText.setText(this._getSortOrderText(sSortOrder === "desc"));
-				var sKey = oEvt.oSource.getParent().getContent()[0].getSelectedItem().getKey();
+				const sKey = oEvt.oSource.getParent().getContent()[0].getSelectedItem().getKey();
 
 				this._changeOrder(sKey, sSortOrder == "desc");
-			}.bind(this)
+			}
 		});
 
 		oSortOrderSwitch.setSelectedItem(bDesc ? oSortOrderSwitch.getItems()[1] : oSortOrderSwitch.getItems()[0]);
@@ -115,7 +130,7 @@ sap.ui.define([
 		return oSortOrderSwitch;
 	};
 
-	SortPanel.prototype._createSortOrderText = function (sKey, bDesc) {
+	SortPanel.prototype._createSortOrderText = function(sKey, bDesc) {
 		return new Text({
 			layoutData: new GridData({
 				span: "XL3 L3 M3 S3",
@@ -127,9 +142,9 @@ sap.ui.define([
 
 	SortPanel.prototype._createQueryRowGrid = function(oItem) {
 		//Enhance row with sort specific controls (Segmented Button + sort order text)
-		var oSelect = this._createKeySelect(oItem.name);
-		var oSortOrderSwitch = this._createOrderSwitch(oItem.name, oItem.descending);
-		var oSortOrderText = this._createSortOrderText(oItem.name, oItem.descending);
+		const oSelect = this._createKeySelect(oItem.name);
+		const oSortOrderSwitch = this._createOrderSwitch(oItem.name, oItem.descending);
+		const oSortOrderText = this._createSortOrderText(oItem.name, oItem.descending);
 
 		return new Grid({
 			containerQuery: true,
@@ -142,15 +157,15 @@ sap.ui.define([
 		}).addStyleClass("sapUiTinyMargin");
 	};
 
-	SortPanel.prototype._getPlaceholderText = function () {
+	SortPanel.prototype._getPlaceholderText = function() {
 		return this._getResourceText("p13n.SORT_PLACEHOLDER");
 	};
 
-	SortPanel.prototype._getRemoveButtonTooltipText = function () {
+	SortPanel.prototype._getRemoveButtonTooltipText = function() {
 		return this._getResourceText("p13n.SORT_REMOVEICONTOOLTIP");
 	};
 
-	SortPanel.prototype._getRemoveButtonAnnouncementText = function () {
+	SortPanel.prototype._getRemoveButtonAnnouncementText = function() {
 		return this._getResourceText("p13n.SORT_REMOVEICONANNOUNCE");
 	};
 
@@ -158,13 +173,13 @@ sap.ui.define([
 		QueryPanel.prototype._selectKey.apply(this, arguments);
 
 		//Enable SegmentedButton
-		var oListItem = oComboBox.getParent().getParent();
-		var sNewKey = oComboBox.getSelectedKey();
-		var aContent = oListItem.getContent()[0].getContent();
+		const oListItem = oComboBox.getParent().getParent();
+		const sNewKey = oComboBox.getSelectedKey();
+		const aContent = oListItem.getContent()[0].getContent();
 		aContent[1].setEnabled(!!sNewKey);
 
 		//keep existing 'sortorder' selection
-		var bDescending = aContent[1].getSelectedKey() === "desc";
+		const bDescending = aContent[1].getSelectedKey() === "desc";
 		this._changeOrder(sNewKey, bDescending);
 	};
 
@@ -172,8 +187,8 @@ sap.ui.define([
 		return bDesc ? this._getResourceText("p13n.SORT_DESCENDING") : this._getResourceText("p13n.SORT_ASCENDING");
 	};
 
-	SortPanel.prototype._changeOrder = function (sKey, bDesc) {
-		var aItems = this._getP13nModel().getProperty("/items").filter(function (oItem) {
+	SortPanel.prototype._changeOrder = function(sKey, bDesc) {
+		const aItems = this._getP13nModel().getProperty("/items").filter((oItem) => {
 			return oItem.name === sKey;
 		});
 

package/src/sap/m/p13n/enums/PersistenceMode.js

@@ -0,0 +1,57 @@
+/*
+ * OpenUI5
+ * (c) Copyright 2009-2024 SAP SE or an SAP affiliate company.
+ * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
+ */
+sap.ui.define(["sap/ui/base/DataType"], (DataType) => {
+	"use strict";
+
+	/**
+	 * Enumerations for p13n enum artifacts
+	 * @namespace
+	 * @name sap.m.p13n.enums
+	 * @private
+	 * @ui5-restricted sap.m.p13n
+	 */
+
+	/**
+	 * Enumeration of the preferred persistence mode for personalization changes.
+	 *
+	 * @enum {string}
+	 * @since 1.104
+	 * @alias sap.m.p13n.enums.PersistenceMode
+	 * @private
+	 * @ui5-restricted sap.ui.mdc, sap.fe
+	 */
+	const PersistenceMode = {
+		/**
+		 * Personalization changes are created in the flexibility layer using <code>ignoreVariantManagement: true</code>
+		 *
+		 * @private
+		 * @ui5-restricted sap.ui.mdc
+		 */
+		Global: "Global",
+
+		/**
+		 * Personalization changes are created and implicitly persisted only in case no additional <code>VariantManagement</code>
+		 * control reference could be found. If a <code>sap.ui.fl.variants.VariantManagement</code> has been found,
+		 * it will be used instead.
+		 *
+		 * @private
+		 * @ui5-restricted sap.fe
+		 */
+		Auto: "Auto",
+
+		/**
+		 * Personalization changes are not persisted
+		 *
+		 * @private
+		 * @ui5-restricted sap.ui.mdc
+		 */
+		Transient: "Transient"
+	};
+
+	DataType.registerEnum("sap.m.p13n.enums.PersistenceMode", PersistenceMode);
+
+	return PersistenceMode;
+}, /* bExport= */ true);

package/src/sap/m/p13n/{enum → enums}/ProcessingStrategy.js

@@ -5,19 +5,19 @@
  */
 
 // Provides enumeration sap.m.p13n.ProcessingStrategy
-sap.ui.define(function() {
+sap.ui.define(["sap/ui/base/DataType"], (DataType) => {
 	"use strict";
 
-   /**
+	/**
 	 * Defines the delta types of the <code>sap.m.p13n.p13n.Engine</code>.
 	 *
 	 * @enum {string}
-     * @private
-     * @ui5-restricted sap.m.p13n
+	 * @private
+	 * @ui5-restricted sap.m.p13n
 	 * @since 1.108
-	 * @alias sap.m.p13n.enum.ProcessingStrategy
+	 * @alias sap.m.p13n.enums.ProcessingStrategy
 	 */
-	var ProcessingStrategy = {
+	const ProcessingStrategy = {
 		/**
 		 * The delta will only calculate changes that will be added in addition to the existing state
 		 *
@@ -30,7 +30,7 @@ sap.ui.define(function() {
 		 * <b>Note</b>: For example for array based state, entries added/removed will be respected.
 		 * For example for object/path based state, the diff will not include paths that have not been explicitly provided.
 		 *
- 		 * @private
+		 * @private
 		 * @ui5-restricted sap.m.p13n
 		 */
 		PartialReplace: true,
@@ -38,12 +38,14 @@ sap.ui.define(function() {
 		 * The delta will be calculated as complete/absolute state difference, in case the new state does not
 		 * provide a path to an existing state, this is going to be removed in addition to the PartialReplace processing strategy.
 		 *
- 		 * @private
+		 * @private
 		 * @ui5-restricted sap.m.p13n
 		 */
 		FullReplace: "FullReplace"
 	};
 
+	DataType.registerEnum("sap.m.p13n.enums.ProcessingStrategy", ProcessingStrategy);
+
 	return ProcessingStrategy;
 
 });

package/src/sap/m/p13n/handler/xConfigHandler.js

@@ -5,26 +5,26 @@
  */
 
 sap.ui.define([
-    'sap/m/p13n/Engine',
-    'sap/m/p13n/modules/xConfigAPI'
-], function(Engine, xConfigAPI) {
+	'sap/m/p13n/Engine',
+	'sap/m/p13n/modules/xConfigAPI'
+], (Engine, xConfigAPI) => {
 	"use strict";
 
 	/**
 	 * @class Utility class for generic mdc xConfig handling by utilizing the
-     * <code>sap.m.p13n.Engine</code> and its <code>Modificationhandler</code>.
-     * This class should be used to handle property changes that should be persisted
-     * as flex changes for MDC control while enabling preprocessing via customdata.
+	 * <code>sap.m.p13n.Engine</code> and its <code>Modificationhandler</code>.
+	 * This class should be used to handle property changes that should be persisted
+	 * as flex changes for MDC control while enabling preprocessing via customdata.
 	 *
 	 * @author SAP SE
 	 * @private
 	 * @alias sap.m.p13n.handler.xConfigHandler
 	 */
-    var xConfigHandler = {};
+	const xConfigHandler = {};
 
-    var fnQueueChange = function(oControl, fTask) {
-		var fCleanupPromiseQueue = function(pOriginalPromise) {
-			if (oControl._pQueue === pOriginalPromise){
+	const fnQueueChange = (oControl, fTask) => {
+		const fCleanupPromiseQueue = (pOriginalPromise) => {
+			if (oControl._pQueue === pOriginalPromise) {
 				delete oControl._pQueue;
 			}
 		};
@@ -47,8 +47,8 @@ sap.ui.define([
 			});
 
 			if (!oControl._pPendingModification) {
-				oControl._pPendingModification = Engine.getInstance().waitForChanges(oControl).then(function() {
-                    Engine.getInstance().fireStateChange(oControl);
+				oControl._pPendingModification = Engine.getInstance().waitForChanges(oControl).then(() => {
+					Engine.getInstance().fireStateChange(oControl);
 					Engine.getInstance().clearTrace(oControl);
 					delete oControl._pPendingModification;
 				});
@@ -56,163 +56,177 @@ sap.ui.define([
 		}
 	}
 
-    function getOperationType(sChangeType) {
-        const aChangeTypes = ["add", "remove", "move", "set"];
+	function getOperationType(sChangeType) {
+		const aChangeTypes = ["add", "remove", "move", "set"];
 
-        return aChangeTypes.find((sType) => {
-            return sChangeType.indexOf(sType) === 0;
-        });
+		return aChangeTypes.find((sType) => {
+			return sChangeType.indexOf(sType) === 0;
+		});
 
-    }
+	}
 
 	function getRevertOperationType(sChangeType) {
-        const mOppositeType = {
-            add: "remove",
-            remove: "add",
-            move: "move",
-            set: "set"
-        };
+		const mOppositeType = {
+			add: "remove",
+			remove: "add",
+			move: "move",
+			set: "set"
+		};
 		const sType = getOperationType(sChangeType);
-        return mOppositeType[sType];
+		return mOppositeType[sType];
 	}
 
-    /**
-     * Creates a changehandler specific to the provided aggregation and property name,
-     * to enhance the xConfig object for a given mdc control instance.
-     *
-     * The enhanced object can be accesed using <code>Engine#readAggregationConfig</code>.
-     *
-     * @param {object} mMetaConfig A map describing the metadata structure that is affected by this changehandler
-     * @param {boolean} mMetaConfig.aggregationBased Defines whether the aggregation space or the property space should be used in the xConfig object
-     * @param {string} mMetaConfig.property The property name (such as <code>width</code> or <code>label</code>)
-     * @param {string} mMetaConfig.operation The operation to be executed by the handler (add, remove, move, set)
-     *
-     * @returns {object} The created changehandler object
-     */
-    xConfigHandler.createHandler = function(mMetaConfig) {
-
-        if (!mMetaConfig || !mMetaConfig.hasOwnProperty("property")) {
-            throw new Error("Please provide a map containing the affected aggregation and property name!");
-        }
-
-        var sAffectedProperty = mMetaConfig.property;
-        var sAffectedAggregation;
-
-        return {
-            "changeHandler": {
-                applyChange: function (oChange, oControl, mPropertyBag) {
-
-                    return fnQueueChange(oControl, function(){
-                        return Engine.getInstance().readXConfig(oControl, {
-                            propertyBag: mPropertyBag
-                        })
-                        .then(function(oPriorAggregationConfig) {
-
-                            var sOperation = getOperationType(oChange.getChangeType());
-
-                            sAffectedAggregation = oChange.getContent().targetAggregation;
-
-                            var oRevertData = {
-                                key: oChange.getContent().key
-                            };
-
-                            if (sOperation !== "set") {
-                                //In case there is a add/remove operation, flag the revert data as the opposite of the current action (add will be removed as revert and vice versa)
-                                oRevertData.value = sOperation !== "add";
-                            } else {
-                                oRevertData.value = null;
-                            }
-
-							var oController = Engine.getInstance().getController(oControl, oChange.getChangeType());
-							var aCurrentState = oController?.getCurrentState(), oStateItem;
-							if (aCurrentState && aCurrentState instanceof Array) {
-								oStateItem = aCurrentState.find(function(oItem, iIndex){
-									return oItem.key === oChange.getContent().key;
-								});
-							}
+	/**
+	 * Creates a changehandler specific to the provided aggregation and property name,
+	 * to enhance the xConfig object for a given mdc control instance.
+	 *
+	 * The enhanced object can be accessed using <code>Engine#readAggregationConfig</code>.
+	 *
+	 * @param {object} mMetaConfig A map describing the metadata structure that is affected by this changehandler
+	 * @param {boolean} mMetaConfig.aggregationBased Defines whether the aggregation space or the property space should be used in the xConfig object
+	 * @param {string} mMetaConfig.property The property name (such as <code>width</code> or <code>label</code>)
+	 * @param {string} mMetaConfig.operation The operation to be executed by the handler (add, remove, move, set)
+	 *
+	 * @returns {object} The created changehandler object
+	 */
+	xConfigHandler.createHandler = (mMetaConfig) => {
 
-							oRevertData.targetAggregation = oChange.getContent().targetAggregation;
+		if (!mMetaConfig || !mMetaConfig.hasOwnProperty("property")) {
+			throw new Error("Please provide a map containing the affected aggregation and property name!");
+		}
 
-							if (oStateItem) {
-								oRevertData.index = aCurrentState.indexOf(oStateItem);
+		const sAffectedProperty = mMetaConfig.property;
+		let sAffectedAggregation;
+
+		return {
+			"changeHandler": {
+				applyChange: function(oChange, oControl, mPropertyBag) {
+					const sChangePersistenceIdentifier = oChange.getContent().persistenceIdentifier;
+					const oController = Engine.getInstance().getController(oControl, oChange.getChangeType(), sChangePersistenceIdentifier);
+					if (sChangePersistenceIdentifier && oController.getPersistenceIdentifier() !== sChangePersistenceIdentifier) {
+						return Promise.resolve(false);
+					}
+
+					return fnQueueChange(oControl, () => {
+						return Engine.getInstance().readXConfig(oControl, {
+								propertyBag: mPropertyBag
+							})
+							.then((oPriorAggregationConfig) => {
+								const sOperation = getOperationType(oChange.getChangeType());
+								sAffectedAggregation = oChange.getContent().targetAggregation;
+
+								const oRevertData = {
+									key: oChange.getContent().key
+								};
+
+								if (sChangePersistenceIdentifier) {
+									oRevertData.persistenceIdentifier = sChangePersistenceIdentifier;
+								}
+
+								if (sOperation !== "set") {
+									//In case there is a add/remove operation, flag the revert data as the opposite of the current action (add will be removed as revert and vice versa)
+									oRevertData.value = sOperation !== "add";
+								} else {
+									oRevertData.value = null;
+								}
+
+								const oController = Engine.getInstance().getController(oControl, oChange.getChangeType());
+								const aCurrentState = oController?.getCurrentState();
+								let oStateItem;
+								if (aCurrentState && aCurrentState instanceof Array) {
+									oStateItem = aCurrentState.find((oItem, iIndex) => {
+										return oItem.key === oChange.getContent().key;
+									});
+								}
+
+								oRevertData.targetAggregation = oChange.getContent().targetAggregation;
+
+								if (oStateItem) {
+									oRevertData.index = aCurrentState.indexOf(oStateItem);
+								}
+
+								if (oPriorAggregationConfig &&
+									oPriorAggregationConfig.aggregations &&
+									oPriorAggregationConfig.aggregations[sAffectedAggregation] &&
+									oPriorAggregationConfig.aggregations[sAffectedAggregation][oChange.getContent().key] &&
+									oPriorAggregationConfig.aggregations[sAffectedAggregation][oChange.getContent().key][sAffectedProperty]
+								) {
+									oRevertData.value = oPriorAggregationConfig.aggregations[sAffectedAggregation][oChange.getContent().key][sAffectedProperty];
+								}
+
+								oChange.setRevertData(oRevertData);
+
+								const oConfig = {
+									property: sAffectedProperty,
+									key: oChange.getContent().key,
+									value: oChange.getContent(),
+									operation: sOperation,
+									changeType: oChange.getChangeType(),
+									propertyBag: mPropertyBag,
+									markAsModified: true
+								};
+
+								if (mMetaConfig.aggregationBased) {
+									oConfig.controlMeta = {
+										aggregation: sAffectedAggregation
+									};
+								}
+
+								return Engine.getInstance().enhanceXConfig(oControl, oConfig);
+							})
+							.then((bConfigModified) => {
+								if (!bConfigModified) {
+									return;
+								}
+								fConfigModified(oControl, oChange);
+							});
+					});
+
+				},
+				completeChangeContent: function(oChange, mChangeSpecificInfo, mPropertyBag) {
+					// Not used, but needs to be there
+				},
+				revertChange: function(oChange, oControl, mPropertyBag) {
+
+					const sOperation = getRevertOperationType(oChange.getChangeType());
+
+					sAffectedAggregation = oChange.getContent().targetAggregation;
+
+					const oConfig = {
+						controlMeta: {
+							aggregation: sAffectedAggregation,
+							property: sAffectedProperty
+						},
+						property: sAffectedProperty,
+						operation: sOperation,
+						changeType: oChange.getChangeType(),
+						key: oChange.getRevertData().key,
+						value: oChange.getRevertData(),
+						propertyBag: mPropertyBag
+					};
+
+					if (mMetaConfig.aggregationBased) {
+						oConfig.controlMeta = {
+							aggregation: sAffectedAggregation
+						};
+					}
+
+					return Engine.getInstance().enhanceXConfig(oControl, oConfig)
+						.then((bConfigModified) => {
+							if (!bConfigModified) {
+								return;
 							}
+							oChange.resetRevertData();
+							fConfigModified(oControl, oChange);
+						});
+				}
+			},
+			"layers": {
+				"USER": true
+			}
+		};
+	};
+
+	return xConfigHandler;
 
-                            if (oPriorAggregationConfig
-                                && oPriorAggregationConfig.aggregations
-                                && oPriorAggregationConfig.aggregations[sAffectedAggregation]
-                                && oPriorAggregationConfig.aggregations[sAffectedAggregation][oChange.getContent().key]
-                                && oPriorAggregationConfig.aggregations[sAffectedAggregation][oChange.getContent().key][sAffectedProperty]
-                                ){
-                                    oRevertData.value = oPriorAggregationConfig.aggregations[sAffectedAggregation][oChange.getContent().key][sAffectedProperty];
-                            }
-
-                            oChange.setRevertData(oRevertData);
-
-                            var oConfig = {
-                                property: sAffectedProperty,
-                                key: oChange.getContent().key,
-                                value: oChange.getContent(),
-                                operation: sOperation,
-                                changeType: oChange.getChangeType(),
-                                propertyBag: mPropertyBag,
-                                markAsModified: true
-                            };
-
-                            if (mMetaConfig.aggregationBased) {
-                                oConfig.controlMeta = {
-                                    aggregation: sAffectedAggregation
-                                };
-                            }
-
-                            return Engine.getInstance().enhanceXConfig(oControl, oConfig);
-                        })
-                        .then(function(){
-                            fConfigModified(oControl, oChange);
-                        });
-                    });
-
-                },
-                completeChangeContent: function (oChange, mChangeSpecificInfo, mPropertyBag) {
-                    // Not used, but needs to be there
-                },
-                revertChange: function (oChange, oControl, mPropertyBag) {
-
-                    var sOperation = getRevertOperationType(oChange.getChangeType());
-
-                    sAffectedAggregation = oChange.getContent().targetAggregation;
-
-                    var oConfig = {
-                        controlMeta: {
-                            aggregation: sAffectedAggregation,
-                            property: sAffectedProperty
-                        },
-                        property: sAffectedProperty,
-                        operation: sOperation,
-                        changeType: oChange.getChangeType(),
-                        key: oChange.getRevertData().key,
-                        value: oChange.getRevertData(),
-                        propertyBag: mPropertyBag
-                    };
-
-                    if (mMetaConfig.aggregationBased) {
-                        oConfig.controlMeta = {
-                            aggregation: sAffectedAggregation
-                        };
-                    }
-
-                    return Engine.getInstance().enhanceXConfig(oControl, oConfig)
-                    .then(function() {
-                        oChange.resetRevertData();
-                        fConfigModified(oControl, oChange);
-                    });
-                }
-            },
-            "layers": {
-                "USER": true
-            }
-        };
-    };
-
-    return xConfigHandler;
-
-});
+});