@openui5/sap.ui.mdc 1.100.0 → 1.102.1

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

@@ -6,20 +6,18 @@
 
 sap.ui.define([
 	"../util/PropertyHelper",
-	"sap/ui/core/Core",
 	"sap/m/table/Util"
 ], function(
 	PropertyHelperBase,
-	Core,
 	TableUtil
 ) {
 	"use strict";
 
 	/**
-	 * @typedef {object} sap.ui.mdc.table.PropertyInfo
+	 * @typedef {Object} sap.ui.mdc.table.PropertyInfo
 	 * @extends sap.ui.mdc.util.PropertyInfo
 	 *
-	 * @property {object} [exportSettings]
+	 * @property {Object} [exportSettings]
 	 *   Object that contains information about the export settings, see {@link sap.ui.export.Spreadsheet}.
 	 * @property {int} [maxConditions]
 	 *   Defines the maximum number of filter conditions for the property. Possible values that can be used:
@@ -43,9 +41,9 @@ sap.ui.define([
 	 *   Name of the text property that is related to this property in a 1:1 relation.
 	 * @property {boolean} [required]
 	 *   Defines whether a filter condition for this property is required.
-	 * @property {object} [visualSettings]
+	 * @property {Object} [visualSettings]
 	 *   This object contains all relevant properties for visual adjustments.
-	 * @property {object} [visualSettings.widthCalculation]
+	 * @property {Object} [visualSettings.widthCalculation]
 	 *   This object contains all properties and their default values for the column width calculation
 	 * @property {integer} [visualSettings.widthCalculation.minWidth]
 	 *   The minimum content width in rem
@@ -71,15 +69,15 @@ sap.ui.define([
 	/**
 	 * Constructor for a new table property helper.
 	 *
-	 * @param {object[]} aProperties
+	 * @param {Object[]} aProperties
 	 *     The properties to process in this helper
-	 * @param {Object<string, object>} [mExtensions]
+	 * @param {Object<string, Object>} [mExtensions]
 	 *     Key-value map, where the key is the name of the property and the value is the extension containing mode-specific information.
 	 *     The extension of a property is stored in a reserved <code>extension</code> attribute, and its attributes must be specified with
 	 *     <code>mExtensionAttributeMetadata</code>.
 	 * @param {sap.ui.base.ManagedObject} [oParent]
 	 *     A reference to an instance that will act as the parent of this helper
-	 * @param {object} [mExtensionAttributeMetadata]
+	 * @param {Object} [mExtensionAttributeMetadata]
 	 *     The attribute metadata for the model-specific property extension
 	 *
 	 * @class
@@ -90,7 +88,7 @@ sap.ui.define([
 	 * @extends sap.ui.mdc.util.PropertyHelper
 	 *
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @private
 	 * @experimental
@@ -134,120 +132,77 @@ sap.ui.define([
 	 *
 	 * @param {sap.ui.mdc.table.Column} oColumn The column for which to get the export settings
 	 * @param {boolean} [bSplitCells=false] Whether the <code>splitCells</code> configuration is enabled
-	 * @returns {null|object} Export setting object for the provided column
+	 * @returns {Object[]} Array of export setting objects for the provided column. Will return more than one object if it is complex property and if <code>splitCells=true</code>
 	 * @public
 	 */
-	PropertyHelper.prototype.getColumnExportSettings = function(oColumn, bSplitCells) {
+	PropertyHelper.prototype.getColumnExportSettings = function(oColumn) {
+		var aColumnExportSettings = [];
+
 		if (!isMdcColumnInstance(oColumn)) {
-			return null;
+			return aColumnExportSettings;
 		}
 
 		var oProperty = this.getProperty(oColumn.getDataProperty());
 
 		if (!oProperty) {
-			return null;
+			return aColumnExportSettings;
 		}
 
-		bSplitCells = bSplitCells === true;
-
-		var	aColumnExportSettings = [];
-		var aPropertiesFromComplexProperty;
 		var oExportSettings = oProperty.exportSettings;
-		var oColumnExportSettings;
+
+		// exportSettings have been set explicitly to null by the application for this column to exclude it from the export
+		if (oExportSettings === null) {
+			return aColumnExportSettings;
+		}
+
 		var aPaths = [];
-		var sAdditionalPath;
-		var oAdditionalProperty;
-		var oAdditionExportSettings;
-		var oAdditionalColumnExportSettings;
+		var oColumnExportSettings;
+		var aPropertiesFromComplexProperty;
 
-		if (oProperty.isComplex()) {
-			aPropertiesFromComplexProperty = oProperty.getReferencedProperties();
-			if (!bSplitCells && oExportSettings) {
-				oColumnExportSettings = getColumnExportSettingsObject(oColumn, oProperty, oExportSettings, bSplitCells);
-				aPropertiesFromComplexProperty.forEach(function(oProperty) {
-					aPaths.push(oProperty.path);
-				});
-				oColumnExportSettings.property = aPaths;
-				aColumnExportSettings.push(oColumnExportSettings);
-			} else {
-				// when there are no exportSettings given for a ComplexProperty or when the splitCells=true
-				aPropertiesFromComplexProperty.forEach(function(oProperty, iIndex) {
-					var oPropertyInfoExportSettings = oProperty.exportSettings,
-						oCurrentColumnExportSettings = getColumnExportSettingsObject(oColumn, oProperty, oPropertyInfoExportSettings, bSplitCells);
-					oCurrentColumnExportSettings.property = oProperty.path;
-					if (iIndex > 0) {
-						oCurrentColumnExportSettings.columnId = oColumn.getId() + "-additionalProperty" + iIndex;
-					}
-					if (oPropertyInfoExportSettings || oCurrentColumnExportSettings.property) {
-						aColumnExportSettings.push(oCurrentColumnExportSettings);
-					}
-				}, this);
-			}
-		} else if (!bSplitCells && oExportSettings) {
-			// called for basic PropertyInfo having exportSettings
-			oColumnExportSettings = getColumnExportSettingsObject(oColumn, oProperty, oExportSettings, bSplitCells);
+		if (!oProperty.isComplex()) {
+			oColumnExportSettings = getColumnExportSettingsObject(oColumn, oProperty, oExportSettings);
 			oColumnExportSettings.property = oProperty.path;
 			aColumnExportSettings.push(oColumnExportSettings);
-		} else {
-			oColumnExportSettings = getColumnExportSettingsObject(oColumn, oProperty, oExportSettings, bSplitCells);
-			oColumnExportSettings.property = oProperty.path;
-			if (oColumnExportSettings.property) {
-				aColumnExportSettings.push(oColumnExportSettings);
-			}
 
-			// get Additional path in case of split cells
-			sAdditionalPath = bSplitCells && oExportSettings && oExportSettings.unitProperty ? oExportSettings.unitProperty : null;
+			return aColumnExportSettings;
+		}
 
-			if (sAdditionalPath) {
-				oAdditionalProperty = getAdditionalProperty(this, sAdditionalPath);
-				oAdditionExportSettings = oAdditionalProperty.exportSettings;
-				oAdditionalColumnExportSettings = getColumnExportSettingsObject(oColumn, oAdditionalProperty, oAdditionExportSettings, bSplitCells);
-				oAdditionalColumnExportSettings.property = oAdditionalProperty.path;
-				oAdditionalColumnExportSettings.columnId = oColumn.getId() + "-additionalProperty";
-				if (oAdditionExportSettings || oAdditionalColumnExportSettings.property) {
-					aColumnExportSettings.push(oAdditionalColumnExportSettings);
+		aPropertiesFromComplexProperty = oProperty.getReferencedProperties();
+		if (Object.keys(oExportSettings).length) {
+			oColumnExportSettings = getColumnExportSettingsObject(oColumn, oProperty, oExportSettings);
+			aPropertiesFromComplexProperty.forEach(function(oProperty) {
+				aPaths.push(oProperty.path);
+			});
+			oColumnExportSettings.property = aPaths;
+			aColumnExportSettings.push(oColumnExportSettings);
+		} else {
+			// when there are no exportSettings given for a ComplexProperty
+			aPropertiesFromComplexProperty.forEach(function(oProperty, iIndex) {
+				var oPropertyInfoExportSettings = oProperty.exportSettings,
+					oCurrentColumnExportSettings = getColumnExportSettingsObject(oColumn, oProperty, oPropertyInfoExportSettings);
+				oCurrentColumnExportSettings.property = oProperty.path;
+				if (iIndex > 0) {
+					oCurrentColumnExportSettings.columnId = oColumn.getId() + "-additionalProperty" + iIndex;
+				}
+				if (oPropertyInfoExportSettings || oCurrentColumnExportSettings.property) {
+					aColumnExportSettings.push(oCurrentColumnExportSettings);
 				}
-			}
+			});
 		}
 
 		return aColumnExportSettings;
 	};
 
-	/**
-	 * Gets the property that is identified by a <code>path</code>.
-	 *
-	 * @param {sap.ui.mdc.table.PropertyHelper} oPropertyHelper Property helper instance
-	 * @param {string} sPath The value of the <code>path</code> attribute of the property
-	 * @returns {object} The property
-	 * @public
-	 */
-	function getAdditionalProperty(oPropertyHelper, sPath) {
-		var oProperty = oPropertyHelper.getProperty(sPath);
-
-		if (!oProperty) {
-			oProperty = oPropertyHelper.getProperties().find(function(oProperty) {
-				return sPath === oProperty.path;
-			});
-		}
-
-		if (oProperty.isComplex()) {
-			throw new Error("The 'unitProperty' points to a complex property");
-		}
-
-		return oProperty;
-	}
-
 	/**
 	 * Sets defaults to export settings and returns a new export settings object.
 	 *
 	 * @param {sap.ui.mdc.table.Column} oColumn The column from which to get default values
-	 * @param {object} oProperty The property from which to get default values
-	 * @param {object} oExportSettings The export settings for which to set defaults
-	 * @param {boolean} bSplitCells Whether the <code>splitCells</code> configuration is enabled
-	 * @returns {object} The new export settings object
+	 * @param {Object} oProperty The property from which to get default values
+	 * @param {Object} oExportSettings The export settings for which to set defaults
+	 * @returns {Object} The new export settings object
 	 * @private
 	 */
-	function getColumnExportSettingsObject(oColumn, oProperty, oExportSettings, bSplitCells) {
+	function getColumnExportSettingsObject(oColumn, oProperty, oExportSettings) {
 	var oExportObj = Object.assign({
 			columnId: oColumn.getId(),
 			label: oProperty.label,
@@ -256,100 +211,67 @@ sap.ui.define([
 			type: "String"
 		}, oExportSettings);
 
-		if (bSplitCells) {
-			oExportObj.displayUnit = false;
-		}
-
 		return oExportObj;
 	}
 
 	/**
-	 * Sets the width of the provided column based on the <code>visualSettings</code> of the relevant <code>PropertyInfo</code>.
+	 * Calculates the width of the provided column based on the <code>visualSettings</code> of the relevant <code>PropertyInfo</code>.
 	 *
 	 * @param {sap.ui.mdc.table.Column} oMDCColumn The <code>Column</code> instance for which to set the width
-	 * @public
-	 * @since 1.95
+	 * @returns {sap.ui.core.CSSSize | null} The calculated width, or <code>null</code> if calculation wasn't possible
 	 */
-	PropertyHelper.prototype.setColumnWidth = function(oMDCColumn) {
+	PropertyHelper.prototype.calculateColumnWidth = function(oMDCColumn) {
 		var sPropertyName = oMDCColumn.getDataProperty();
 		var oProperty = this.getProperty(sPropertyName);
+
 		if (!oProperty) {
-			return;
+			return null;
 		}
 
 		var mPropertyInfoVisualSettings = oProperty.visualSettings;
 		if (mPropertyInfoVisualSettings && mPropertyInfoVisualSettings.widthCalculation === null) {
-			return;
+			return null;
 		}
 
-		var fWidth = this._calcColumnWidth(oProperty) + 1; // add 1rem extra for padding and border
-		oMDCColumn._updateColumnWidth(fWidth + "rem");
+		return this._calcColumnWidth(oProperty);
 	};
 
 	/**
 	 * Calculates the column width based on the provided <code>PropertyInfo</code>.
 	 *
-	 * @param {object} oProperty The properties of <code>PropertyInfo</code> of <code>Column</code> instance for which to set the width
-	 * @param {object} [mWidthCalculation] The configuration object for the width calculation
-	 * @param {int} [mWidthCalculation.minWidth=2] The minimum content width in rem
-	 * @param {int} [mWidthCalculation.maxWidth=19] The maximum content width in rem
-	 * @param {int} [mWidthCalculation.defaultWidth=8] The default column content width when type check fails
-	 * @param {boolean} [mWidthCalculation.includeLabel=true] Whether the label should be taken into account
-	 * @param {float} [mWidthCalculation.gap=0] The additional content width in rem
-	 * @param {boolean} [mWidthCalculation.verticalArrangement=false] Whether the referenced properties are arranged vertically
-	 * @param {string|array[]} [mWidthCalculation.excludeProperties=[]] A list of invisible referenced property names
-	 * @return {float} [fWidth] Calculated width
+	 * @param {Object} oProperty The properties of <code>PropertyInfo</code> of <code>Column</code> instance for which to set the width
+	 * @return {string} The calculated column width
 	 * @since 1.95
 	 * @private
 	 */
-	PropertyHelper.prototype._calcColumnWidth = function (oProperty, mWidthCalculation) {
-		var fWidth = 0;
-		var fLabelWidth = 0;
-		var mPropertyInfoWidthCalculation = oProperty.visualSettings.widthCalculation;
-		mWidthCalculation = Object.assign({}, mPropertyInfoWidthCalculation, mWidthCalculation || {});
-
-		var iMinWidth = Math.max(1, mWidthCalculation.minWidth);
-		var iMaxWidth = Math.max(iMinWidth, mWidthCalculation.maxWidth);
-
+	 PropertyHelper.prototype._calcColumnWidth = function (oProperty) {
+		var mWidthCalculation = Object.assign({
+			gap: 0,
+			includeLabel: true,
+			excludeProperties: []
+		}, oProperty.visualSettings && oProperty.visualSettings.widthCalculation);
+
+		var aTypes = [];
 		if (oProperty.isComplex()) {
-			var aRelevantReferencedProperties = oProperty.getReferencedProperties().filter(function(oProp) {
-				return ![].concat(mWidthCalculation.excludeProperties).includes(oProp.name);
+			// for complex properties generate [<TypeInstance>, <TypeSettings>][] structure
+			aTypes = oProperty.getReferencedProperties().flatMap(function(oProp) {
+				var mPropWidthCalculation = oProp.visualSettings ? oProp.visualSettings.widthCalculation : undefined;
+				return mPropWidthCalculation === null || mWidthCalculation.excludeProperties.includes(oProp.name) ? [] : [
+					[oProp.typeConfig.typeInstance, mPropWidthCalculation]
+				];
 			});
-
-			aRelevantReferencedProperties.forEach(function(oReferencedProperty) {
-				var fReferencedPropertyWidth = this._calcColumnWidth(oReferencedProperty, {
-					includeLabel: false
-				});
-
-				if (mWidthCalculation.verticalArrangement || aRelevantReferencedProperties.length == 1) {
-					fWidth = Math.max(fReferencedPropertyWidth, fWidth);
-				} else {
-					fWidth = fWidth + fReferencedPropertyWidth + 0.5; // add 0.5rem for some extra spacing in h-alignment
-				}
-			}, this);
 		} else {
-			var oTypeConfig = oProperty.typeConfig;
-			var oType = oTypeConfig.typeInstance;
-
-			if (oType) {
-				fWidth = TableUtil.calcTypeWidth(oType, mWidthCalculation);
-			}
-			if (oProperty.unit) {
-				fWidth += 2.5;
-			}
+			// for simple properties generate <TypeInstance>[] structure
+			aTypes.push(oProperty.typeConfig.typeInstance);
 		}
 
-		fWidth += mWidthCalculation.gap;
-
-		if (mWidthCalculation.includeLabel) {
-			fLabelWidth = TableUtil.calcHeaderWidth(oProperty.label, fWidth, iMaxWidth, iMinWidth);
+		if (oProperty.unit) {
+			// @TODO: follow the unit property, like a complex property, instead of adding a fix gap
+			mWidthCalculation.gap += 2.5;
 		}
 
-		fWidth = Math.max(iMinWidth, fWidth, fLabelWidth);
-		fWidth = Math.min(fWidth, iMaxWidth);
-		fWidth = Math.round(fWidth * 100) / 100;
-
-		return fWidth;
+		var sHeader = (mWidthCalculation.includeLabel) ? oProperty.label : "";
+		return TableUtil.calcColumnWidth(aTypes, sHeader, mWidthCalculation);
 	};
 
 	return PropertyHelper;

package/src/sap/ui/mdc/table/ResponsiveTableType.js

@@ -5,8 +5,8 @@
  */
 
 sap.ui.define([
-	"sap/ui/core/Core", "./TableTypeBase", "../library", "sap/m/Button", "sap/ui/Device", "sap/m/plugins/ColumnResizer"
-], function(Core, TableTypeBase, library, Button, Device, ColumnResizer) {
+	"sap/ui/core/Core", "./TableTypeBase", "../library", "sap/m/Button", "sap/ui/Device", "sap/m/plugins/ColumnResizer", "sap/m/SegmentedButton", "sap/m/SegmentedButtonItem"
+], function(Core, TableTypeBase, library, Button, Device, ColumnResizer, SegmentedButton, SegmentedButtonItem) {
 	"use strict";
 
 	var InnerTable, InnerColumn, InnerRow;
@@ -114,7 +114,6 @@ sap.ui.define([
 		// avoid execution of the if and else if block if bValue has not changed
 		if (bValue && !this._oShowDetailsButton) {
 			oTable.getHeaderToolbar().insertEnd(this._getShowDetailsButton(), 0);
-			this._renderShowDetailsButton();
 			oTable.attachEvent("popinChanged", this._onPopinChanged, this);
 			oTable.setHiddenInPopin(this._getImportanceToHide());
 		} else if (!bValue && this._oShowDetailsButton) {
@@ -189,23 +188,15 @@ sap.ui.define([
 			return;
 		}
 
-		var oRowActionsInfo = oRowSettings.getAllActions();
+		var vRowType, bVisibleBound, fnVisibleFormatter, oRowActionsInfo = oRowSettings.getAllActions();
+		// If templateInfo is given, the rowActions are bound
 		if ("templateInfo" in oRowActionsInfo) {
 			var oTemplateInfo = oRowActionsInfo.templateInfo;
 
-			if (oTemplateInfo.visible.formatter) {
-				// Wrap the formatter (which should return a boolean) value and determine the type based on the formatted value.
-				var fnFormatter = oTemplateInfo.visible.formatter;
-				oTemplateInfo.visible.formatter = function (sValue) {
-					var bVisible = fnFormatter(sValue);
-					return bVisible ? RowAction.Navigation : sType;
-				};
-			}
-			if (typeof oTemplateInfo.visible === "object") {
-				oTable._oTemplate.bindProperty("type", oTemplateInfo.visible);
-			} else {
-				oTable._oTemplate.setProperty("type", oTemplateInfo.visible);
-			}
+			fnVisibleFormatter = oTemplateInfo.visible.formatter;
+			// If visible property is of type object, we know for certain the property is bound (see RowSettings.getAllActions)
+			bVisibleBound = typeof oTemplateInfo.visible == "object";
+			vRowType = oTemplateInfo.visible;
 		} else if (oRowActionsInfo && oRowActionsInfo.items){
 			var _oRowActionItem;
 			if (oRowActionsInfo.items.length == 0) {
@@ -213,20 +204,40 @@ sap.ui.define([
 				return;
 			}
 
+			// Check if rowActions are of type Navigation. ResponsiveTable currently only supports RowActionItem<Navigation>
 			_oRowActionItem = oRowActionsInfo.items.find(function (oRowAction) {
 				return oRowAction.getType() == "Navigation";
 			});
-
-			// ResponsiveTable currently only supports RowActionItem<Navigation>
 			if (!_oRowActionItem && oRowActionsInfo.items.length > 0) {
 				throw new Error("No RowAction of type 'Navigation' found. sap.m.Table only accepts RowAction of type 'Navigation'.");
 			}
 
 			// Associate RowActionItem<Navigation> to template for reference
 			oTable._oTemplate.data("rowAction", _oRowActionItem);
+
+			// Check if visible property is bound
+			bVisibleBound = _oRowActionItem.isBound("visible");
+			// Based on whether visible property is bound, either get binding info or the actual property
+			vRowType = bVisibleBound ? _oRowActionItem.getBindingInfo("visible") : _oRowActionItem.getVisible();
+			fnVisibleFormatter = vRowType.formatter;
 		}
 
-		oTable._oTemplate.setType(RowAction.Navigation);
+		// If a custom formatter exists, apply it before converting it to row type, otherwise just convert
+		if (fnVisibleFormatter) {
+			vRowType.formatter = function (sValue) {
+				var bVisible = fnVisibleFormatter(sValue);
+				return bVisible ? RowAction.Navigation : sType;
+			};
+		} else {
+			vRowType = vRowType ? RowAction.Navigation : sType;
+		}
+
+		// Depending on whether the property is bound, either bind or set
+		if (bVisibleBound) {
+			oTable._oTemplate.bindProperty("type", vRowType);
+		} else {
+			oTable._oTemplate.setProperty("type", vRowType);
+		}
 	};
 
 	ResponsiveTableType.disableColumnResizer = function(oTable, oInnerTable) {
@@ -252,26 +263,18 @@ sap.ui.define([
 		}
 	};
 
-	ResponsiveTableType.startColumnResize = function(oInnerTable, oColumn) {
+	ResponsiveTableType.startColumnResize = function(oInnerTable, oColumn, oColumnMenu) {
 		var oColumnResizer = ColumnResizer.getPlugin(oInnerTable);
 
 		if (!oColumnResizer) {
 			return;
 		}
 
-		return oColumnResizer.getColumnResizeButton(oColumn);
-	};
-
-	/**
-	 * Renders the look and feel of the Show / Hide Details button
-	 *
-	 * @private
-	 */
-	ResponsiveTableType.prototype._renderShowDetailsButton = function() {
-		var oRb = Core.getLibraryResourceBundle("sap.ui.mdc"), sText;
-
-		sText = this.bHideDetails ? oRb.getText("table.SHOWDETAILS_TEXT") : oRb.getText("table.HIDEDETAILS_TEXT");
-		this._oShowDetailsButton.setText(sText);
+		if (oColumnMenu && oColumnMenu.isA("sap.m.table.columnmenu.Menu")) {
+			return oColumnResizer.getColumnResizeQuickAction(oColumn, oColumnMenu);
+		} else {
+			return oColumnResizer.getColumnResizeButton(oColumn);
+		}
 	};
 
 	/**
@@ -296,17 +299,37 @@ sap.ui.define([
 		} else {
 			oTable.setHiddenInPopin([]);
 		}
-		this._renderShowDetailsButton();
 	};
 
 	ResponsiveTableType.prototype._getShowDetailsButton = function() {
 		if (!this._oShowDetailsButton) {
+			var oRb = Core.getLibraryResourceBundle("sap.ui.mdc");
 			this.bHideDetails = true;
-			this._oShowDetailsButton = new Button(this.getId() + "-showHideDetails", {
+			this._oShowDetailsButton = new SegmentedButton(this.getId() + "-showHideDetails", {
 				visible: false,
-				press: [function() {
-					this._toggleShowDetails(!this.bHideDetails);
-				}, this]
+				selectedKey: "hideDetails",
+				items: [
+					new SegmentedButtonItem({
+						icon: "sap-icon://detail-more",
+						key: "showDetails",
+						tooltip: oRb.getText("table.SHOWDETAILS_TEXT"),
+						press: [
+							function() {
+								this._toggleShowDetails(false);
+							}, this
+						]
+					}),
+					new SegmentedButtonItem({
+						icon: "sap-icon://detail-less",
+						key: "hideDetails",
+						tooltip: oRb.getText("table.HIDEDETAILS_TEXT"),
+						press: [
+							function() {
+								this._toggleShowDetails(true);
+							}, this
+						]
+					})
+				]
 			});
 		}
 		return this._oShowDetailsButton;
@@ -320,15 +343,12 @@ sap.ui.define([
 	 */
 	ResponsiveTableType.prototype._getImportanceToHide = function() {
 		var aDetailsButtonSetting = this.getDetailsButtonSetting() || [];
-		var aImportanceToHide = [];
 
 		if (aDetailsButtonSetting.length) {
-			aImportanceToHide = aDetailsButtonSetting;
+			return aDetailsButtonSetting;
 		} else {
-			aImportanceToHide = Device.system.phone ? ["Low", "Medium"] : ["Low"];
+			return Device.system.phone ? ["Low", "Medium"] : ["Low"];
 		}
-
-		return aImportanceToHide;
 	};
 
 	/**

package/src/sap/ui/mdc/table/RowActionItem.js

@@ -24,7 +24,7 @@ sap.ui.define([
 	 * The <code>RowActionItem</code> control represents a action for a row.
 	 * This control can only be used in the context of <code>sap.ui.mdc.Table</code> control to define row actions.
 	 * @extends sap.ui.core.Element
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @constructor
 	 * @experimental

package/src/sap/ui/mdc/table/RowSettings.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * The <code>RowSettings</code> control is used to configure a row.
 	 * This control can only be used in the context of the <code>sap.ui.mdc.Table</code> control to define row settings.
 	 * @extends sap.ui.core.Element
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @constructor
 	 * @experimental

package/src/sap/ui/mdc/table/TableSettings.js

@@ -118,7 +118,7 @@ sap.ui.define([
 		},
 
 		showUIColumns: function(oControl, oSource) {
-			oControl.getEngine().uimanager.show(oControl, oControl.getP13nMode(), oSource);
+			oControl.getEngine().uimanager.show(oControl, oControl.getActiveP13nModes(), oSource);
 		},
 
 		showUIFilter: function(oControl, oSource) {
@@ -130,31 +130,19 @@ sap.ui.define([
 		 *
 		 * @param {object} oControl The control for which the sorting is used
 		 * @param {string} sProperty The property for which the sorting is used
-		 * @param {boolean} bDescending Whether to sort in descending or ascending order
+		 * @param {sap.ui.core.SortOrder} sSortOrder Whether to sort in descending or ascending order
 		 * @param {boolean} [bRemoveAllExisting=true] Set to remove the previous sorters
 		 */
-		createSort: function(oControl, sProperty, bDescending, bRemoveAllExisting) {
-
-			var oSorter = {
-				selected: true,
-				name: sProperty,
-				descending: bDescending,
-				sorted: true
-			};
-
-			//check to revert sorting in case the sorter and its type already exists
-			oControl.getCurrentState().sorters.forEach(function(oProp) {
-				if (oProp.name == sProperty && oProp.descending === bDescending) {
-					oSorter.sorted = false;
-				}
-			});
-
-			var aItems = [oSorter];
-
+		createSort: function(oControl, sProperty, sSortOrder, bRemoveAllExisting) {
 			oControl.getEngine().createChanges({
 				control: oControl,
 				key: "Sort",
-				state: aItems,
+				state: [{
+					selected: true,
+					name: sProperty,
+					descending: sSortOrder === CoreLibrary.SortOrder.Descending,
+					sorted: sSortOrder !== CoreLibrary.SortOrder.None
+				}],
 				applyAbsolute: bRemoveAllExisting
 			});
 
@@ -176,7 +164,7 @@ sap.ui.define([
 				control: oControl,
 				key: "Group",
 				state: aGroup,
-				applyAbsolute: false
+				applyAbsolute: oControl._bMobileTable
 			});
 		},
 

package/src/sap/ui/mdc/table/V4AnalyticsPropertyHelper.js

@@ -27,7 +27,7 @@ sap.ui.define([
 	 * @extends sap.ui.mdc.table.PropertyHelper
 	 *
 	 * @author SAP SE
-	 * @version 1.100.0
+	 * @version 1.102.1
 	 *
 	 * @private
 	 * @experimental