@openui5/sap.ui.mdc 1.101.0 → 1.103.0

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

@@ -41,7 +41,7 @@ sap.ui.define([
 	 * @extends sap.ui.mdc.p13n.AdaptationProvider
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.103.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.mdc
@@ -61,7 +61,7 @@ sap.ui.define([
 			this._aStateHandlers = [];
 
 			//Default Provider Registry to be used for internal PersistenceProvider functionality access
-			this.defaultProviderRegistry = DefaultProviderRegistry.getInstance();
+			this.defaultProviderRegistry = DefaultProviderRegistry.getInstance(this);
 
 			//UIManager to be used for p13n UI creation
 			this.uimanager = UIManager.getInstance(this);
@@ -989,6 +989,8 @@ sap.ui.define([
 	 * @param {sap.ui.mdc.Control} vControl The registered control instance.
 	 * @param {string} sKey The registerd key to get the corresponding Controller.
 	 * @param {sap.ui.core.Control} oP13nUI The adaptation UI displayed in the container (e.g. BasePanel derivation).
+	 *
+	 * @returns {object} Object defining the state validation result
 	 */
 	Engine.prototype.validateP13n = function(vControl, sKey, oP13nUI) {
 		var oController = this.getController(vControl, sKey);
@@ -1003,7 +1005,7 @@ sap.ui.define([
 		});
 
 		//Only execeute validation for controllers that support 'model2State'
-		if (oController.model2State instanceof Function) {
+		if (oController && oController.model2State instanceof Function) {
 			oTheoreticalState[sKey] = oController.model2State();
 
 			var mInfoState = oControl.validateState(this.externalizeKeys(oControl, oTheoreticalState), sKey);
@@ -1022,7 +1024,9 @@ sap.ui.define([
 			} else {
 				Log.warning("message strip could not be provided - the adaptation UI needs to implement 'setMessageStrip'");
 			}
-
+			return mInfoState;
+		} else {
+			return undefined;
 		}
 
 	};

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

@@ -83,6 +83,7 @@ sap.ui.define([
 				oVM.setModel(oModel, Utils.VARIANT_MODEL_NAME);
 			});
 			this._oWrapper = new VerticalLayout(this.getId() + "--accWrapper", {
+				visible: false,
 				content: [
 					oVM
 				]

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.101.0
+	 * @version 1.103.0
 	 *
 	 * @private
 	 * @experimental

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

@@ -33,7 +33,7 @@ sap.ui.define([
 	 * @extends sap.ui.base.Object
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.103.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.mdc

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.101.0
+	 * @version 1.103.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.mdc
@@ -34,7 +34,7 @@ sap.ui.define([
 	 * @alias sap.ui.mdc.p13n.modules.DefaultProviderRegistry
 	 */
 	var DefaultProviderRegistry = BaseObject.extend("sap.ui.mdc.p13n.modules.DefaultProviderRegistry", {
-		constructor: function() {
+		constructor: function(oEngine) {
 
 			if (oDefaultProviderRegistry) {
 				throw Error(ERROR_INSTANCING);
@@ -42,6 +42,7 @@ sap.ui.define([
 
 			BaseObject.call(this);
 			this._mDefaultProviders = {};
+			this._oEngine = oEngine;
 		}
 	});
 
@@ -54,27 +55,29 @@ sap.ui.define([
 			this._mDefaultProviders[sProviderName].destroy();
 			delete this._mDefaultProviders[sProviderName];
 		}.bind(this));
+		this._oEngine = null;
 		BaseObject.prototype.destroy.apply(this, arguments);
+		oDefaultProviderRegistry = null;
 	};
 
 	/**
 	 * @private
 	 * @ui5-restricted sap.ui.mdc
 	 *
-	 * Attaches a control to a default persistence provider held inside the DefaultProviderRegistry for the given <code>PersistenceMode</code>
+	 * Attaches an element to a default persistence provider held inside the DefaultProviderRegistry for the given <code>PersistenceMode</code>
 	 *
-	 * @param {sap.ui.mdc.Control|string} vControl The control instance or a control id.
+	 * @param {sap.ui.core.Element|string} vElement The control instance or an element id.
 	 * @param {sap.ui.mdc.enum.PersistenceMode} sPersistenceMode Desired persistence mode for the retrieved persistence provider
 	 * @returns {sap.ui.mdc.p13n.PersistenceProvider} Returns a persistence provider instance, if possible
 	 */
-	DefaultProviderRegistry.prototype.attach = function (vControl, sPersistenceMode) {
-		if (vControl.getEngine().isRegisteredForModification(vControl)) { // Modification settings for a registered control are only determined once in the Engine
-			throw new Error("DefaultProviderRegistry: You must not change the modificationSettings for an already registered control");
+	DefaultProviderRegistry.prototype.attach = function (vElement, sPersistenceMode) {
+		if (this._oEngine.isRegisteredForModification(vElement)) { // Modification settings for a registered element are only determined once in the Engine
+			throw new Error("DefaultProviderRegistry: You must not change the modificationSettings for an already registered element");
 		}
 		var oDefaultProvider = this._retrieveDefaultProvider(sPersistenceMode);
-		var sControlId = typeof vControl === "string" ? vControl : vControl.getId();
+		var sControlId = typeof vElement === "string" ? vElement : vElement.getId();
 		if (oDefaultProvider.getFor().indexOf(sControlId) === -1) {
-			oDefaultProvider.addFor(vControl);
+			oDefaultProvider.addFor(vElement);
 		}
 
 		return oDefaultProvider;
@@ -115,9 +118,9 @@ sap.ui.define([
 	 *
 	 * This method is the central point of access to the DefaultProviderRegistry Singleton.
 	 */
-	 DefaultProviderRegistry.getInstance = function() {
+	 DefaultProviderRegistry.getInstance = function(oEngine) {
 		if (!oDefaultProviderRegistry) {
-			oDefaultProviderRegistry = new DefaultProviderRegistry();
+			oDefaultProviderRegistry = new DefaultProviderRegistry(oEngine);
 		}
 		return oDefaultProviderRegistry;
 	};

package/src/sap/ui/mdc/p13n/panels/ActionToolbarPanel.js

@@ -54,6 +54,139 @@ sap.ui.define([
 		}
     };
 
+	ActionToolbarPanel.prototype._removeFactoryControl = function() {
+		this._oListControl.getItems().filter(function(oItem) {
+			return !oItem._bGroupHeader;
+		}).forEach(function(oItem){
+			var oFirstCell = oItem.getCells()[0];
+			if (oFirstCell.getItems().length > 1){
+				oFirstCell.removeItem(oFirstCell.getItems()[1]);
+			}
+		});
+		this.removeStyleClass("sapUiMDCAFLabelMarkingList");
+		return this._aInitializedFields;
+	};
+
+	ActionToolbarPanel.prototype._moveTableItem = function(oItem, iNewIndex) {
+		var aItems = this._oListControl.getItems();
+		var aFields = this._getP13nModel().getProperty("/items");
+
+		// index of the item in the model not the index in the aggregation
+		var iOlModelIndex = aFields.indexOf(this._getModelEntry(oItem));
+
+		// limit the minumum and maximum index
+		iNewIndex = (iNewIndex <= 0) ? 0 : Math.min(iNewIndex, aItems.length - 1);
+
+		// new index of the item in the model
+		var iNewModelIndex = aFields.indexOf(aItems[iNewIndex].getBindingContext(this.P13N_MODEL).getObject());
+		if (iNewModelIndex == iOlModelIndex) {
+			return;
+		}
+
+		// remove data from old position and insert it into new position
+		aFields.splice(iNewModelIndex, 0, aFields.splice(iOlModelIndex, 1)[0]);
+		this._getP13nModel().setProperty("/items", aFields);
+
+		// store the moved item again due to binding
+		this._oSelectedItem = this._oListControl.getItems()[iNewIndex];
+
+		this._updateEnableOfMoveButtons(this._oSelectedItem, true);
+
+		this._handleActivated(this._oSelectedItem);
+
+		this.fireChange({
+			reason: "Move",
+			item: this._getModelEntry(this._oSelectedItem)
+		});
+	};
+
+	ActionToolbarPanel.prototype._onPressButtonMoveToTop = function() {
+		var iIndex = this._oListControl.getItems().indexOf(this._oHoveredItem);
+
+		while (!this._oListControl.getItems()[iIndex - 1]._bGroupHeader) {
+			iIndex--;
+		}
+
+		this._moveSelectedItem(iIndex);
+	};
+
+	ActionToolbarPanel.prototype._onPressButtonMoveToBottom = function() {
+		var iIndex = this._oListControl.getItems().indexOf(this._oHoveredItem);
+
+		while (iIndex < this._oListControl.getItems().length - 1 && !this._oListControl.getItems()[iIndex + 1]._bGroupHeader) {
+			iIndex++;
+		}
+
+		this._moveSelectedItem(iIndex);
+	};
+
+	ActionToolbarPanel.prototype._onItemPressed = function(oEvent) {
+		var oSourceControl = oEvent.getParameter('srcControl');
+		if (!this._isControlPartOfMoveButtons(oSourceControl)) {
+			var oTableItem = oEvent.getParameter('listItem');
+			this._oSelectedItem = oTableItem;
+
+			var oContext = oTableItem.getBindingContext(this.P13N_MODEL);
+			if (this.getEnableReorder() && oContext && oContext.getProperty(this.PRESENCE_ATTRIBUTE)){
+				this._handleActivated(oTableItem);
+				this._updateEnableOfMoveButtons(oTableItem, true);
+			}
+		}
+	};
+
+	ActionToolbarPanel.prototype._isControlPartOfMoveButtons = function(oControl) {
+		var aSelectedItemActions = this._oSelectedItem ? this._oSelectedItem.getCells()[1].getItems() : [];
+		var bIsControlPartOfMoveButtons = false;
+
+		// Starting at index 2 as 0 and 1 are occupied by the active information
+		if (aSelectedItemActions.length > 2) {
+			for (var iIndex = 2; iIndex < aSelectedItemActions.length; iIndex++) {
+				if (aSelectedItemActions[iIndex] == oControl ||
+					aSelectedItemActions[iIndex] == oControl.getParent()) {
+						bIsControlPartOfMoveButtons = true;
+				}
+			}
+		}
+
+		return bIsControlPartOfMoveButtons;
+	};
+
+	ActionToolbarPanel.prototype._updateEnableOfMoveButtons = function(oTableItem, bFocus) {
+		var iTableItemPos = this._oListControl.getItems().indexOf(oTableItem);
+		var iLastItemPos =  this._oListControl.getItems().length - 1;
+		var bUpEnabled = true, bDownEnabled = true;
+		if (iTableItemPos == 0) {
+			// disable move buttons upwards, if the item is at the top
+			bUpEnabled = false;
+		}
+		if (iTableItemPos == iLastItemPos) {
+			// disable move buttons downwards, if the item is at the bottom
+			bDownEnabled = false;
+		}
+		// Check if list is grouped
+		var bListIsGrouped = this._oListControl.getItems().some(function(oItem) {
+			return oItem._bGroupHeader;
+		});
+		if (bListIsGrouped) {
+
+			if (iTableItemPos > 0 && this._oListControl.getItems()[iTableItemPos - 1]._bGroupHeader) {
+				// disable move buttons upwards, if there is a group header item over it
+				bUpEnabled = false;
+			}
+			if (iTableItemPos < iLastItemPos && this._oListControl.getItems()[iTableItemPos + 1]._bGroupHeader) {
+				// disable move buttons upwards, if there is a group header item below it
+				bDownEnabled = false;
+			}
+		}
+		this._getMoveTopButton().setEnabled(bUpEnabled);
+		this._getMoveUpButton().setEnabled(bUpEnabled);
+		this._getMoveDownButton().setEnabled(bDownEnabled);
+		this._getMoveBottomButton().setEnabled(bDownEnabled);
+		if (bFocus) {
+			oTableItem.focus();
+		}
+	};
+
     return ActionToolbarPanel;
 
 });

package/src/sap/ui/mdc/p13n/panels/FilterPanel.js

@@ -5,11 +5,13 @@
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
-    "sap/m/p13n/QueryPanel", "sap/m/VBox", "sap/m/Text", "sap/ui/layout/Grid", "sap/ui/layout/GridData", "sap/m/Button", "sap/m/ComboBox", 'sap/base/util/merge'
+    "sap/m/p13n/QueryPanel", "sap/m/VBox", "sap/m/Text", "sap/ui/layout/Grid", "sap/ui/layout/GridData", "sap/m/Button", "sap/m/ComboBox", "sap/ui/core/library"
 
-], function (QueryPanel, VBox, Text, Grid, GridData, Button, ComboBox, merge) {
+], function (QueryPanel, VBox, Text, Grid, GridData, Button, ComboBox, coreLibrary) {
     "use strict";
 
+    var ValueState = coreLibrary.ValueState;
+
     var FilterPanel = QueryPanel.extend("sap.ui.mdc.p13n.panels.FilterPanel", {
         metadata: {
             properties: {
@@ -31,45 +33,45 @@ sap.ui.define([
             ]
         }) : this._createKeySelect(oItem.name);
 
-
-        var oAddButton;
-        if (!oItem.name) {
-            oAddButton = new Button({
-                text: "Add",
-                press: this._addPressed.bind(this),
-                enabled: false
-            });
-            oAddButton.setLayoutData(new GridData({
-                indent: "XL7 L7 M7 S7",
-                span: "XL1 L1 M1 S1"
-            }));
+        var aContent = [oSelect];
+        if (oItem.name) {
+            aContent.push(this._createFactoryControl(oItem));
         }
-
         return new Grid({
             containerQuery: true,
             defaultSpan: "XL4 L4 M4 S4",
-            content: [
-                oSelect, oItem.name ? this._createFactoryControl(oItem) : oAddButton
-            ]
+            content: aContent
         }).addStyleClass("sapUiTinyMargin");
     };
 
+    FilterPanel.prototype._getPlaceholderText = function () {
+        return this._getResourceText("p13n.FILTER_PLACEHOLDER");
+	};
+
+	FilterPanel.prototype._getRemoveButtonTooltipText = function () {
+		return this._getResourceText("p13n.FILTER_REMOVEICONTOOLTIP");
+	};
+
 	FilterPanel.prototype._createKeySelect = function (sKey) {
 
-        var oKeySelect = new ComboBox({
+        var oComboBox = new ComboBox({
 			width: "100%",
 			items: this._getAvailableItems(),
-			placeholder: "Select Filter",
-			selectionChange: this._selectKey.bind(this)
+			placeholder: this._getPlaceholderText(),
+			selectionChange: function(oEvt) {
+                var oComboBox = oEvt.getSource();
+                this._selectKey(oComboBox);
+            }.bind(this),
+            change: function(oEvt) {
+				var oComboBox = oEvt.getSource();
+				var newValue = oEvt.getParameter("newValue");
+				oComboBox.setValueState( newValue && !oComboBox.getSelectedItem() ? ValueState.Error : ValueState.None);
+				this._selectKey();
+			}.bind(this)
 		});
 
-        return oKeySelect;
-	};
 
-	FilterPanel.prototype._getAvailableItems = function (sKey) {
-        var aItems = QueryPanel.prototype._getAvailableItems.apply(this, arguments);
-        aItems.shift();
-        return aItems;
+        return oComboBox;
 	};
 
     FilterPanel.prototype._createRemoveButton = function (bVisible) {
@@ -84,27 +86,24 @@ sap.ui.define([
         this._selectKey();
     };
 
-    FilterPanel.prototype._selectKey = function(oEvt) {
+    FilterPanel.prototype._selectKey = function(oComboBox) {
         var oQueryRowGrid, sKey;
-        if (oEvt) {
-            this._oEvt = merge({}, oEvt);
-            oQueryRowGrid = oEvt.oSource.getParent();
-            sKey = oEvt.oSource.getSelectedKey();
-            oQueryRowGrid.getContent()[1].setEnabled(!!sKey);
-        } else if (this._oEvt) {
-            oEvt = this._oEvt;
-            oQueryRowGrid = oEvt.oSource.getParent();
-            sKey = oEvt.oSource.getSelectedKey();
+        if (oComboBox) {
+            this._oComboBox = oComboBox;
+            oQueryRowGrid = oComboBox.getParent();
+            sKey = oComboBox.getSelectedKey();
+        } else if (this._oComboBox) {
+            oComboBox = this._oComboBox;
+            oQueryRowGrid = oComboBox.getParent();
+            sKey = oComboBox.getSelectedKey();
 
             if (sKey) {
-                QueryPanel.prototype._selectKey.call(this, oEvt);
+                QueryPanel.prototype._selectKey.call(this, oComboBox);
 
-                var sText = sKey ? oEvt.getParameter("selectedItem").getText() : "";
+                var sText = sKey ? oComboBox.getSelectedItem().getText() : "";
 
                 var oSelect = oQueryRowGrid.getContent()[0];
                 oQueryRowGrid.removeContent(oSelect);
-                var oAddBtn = oQueryRowGrid.getContent()[0];
-                oQueryRowGrid.removeContent(oAddBtn);
 
                 // var sKey = oSelect._key;
                 var oFieldBox = new VBox({
@@ -120,7 +119,7 @@ sap.ui.define([
                 var oFilterField = this._createFactoryControl({name: sKey});
                 oQueryRowGrid.insertContent(oFilterField, 1);
             }
-            delete this._oEvt;
+            delete this._oComboBox;
         }
     };
 

package/src/sap/ui/mdc/p13n/panels/GroupView.js

@@ -232,6 +232,7 @@ sap.ui.define([
 			return new CustomListItem({
 				selected: "{" + this.P13N_MODEL + ">visible}",
 				tooltip: "{" + this.P13N_MODEL + ">tooltip}",
+				accDescription: "{" + this.P13N_MODEL + ">label}",
 				content: aInnerListItemContent
 			});
 

package/src/sap/ui/mdc/p13n/panels/LinkSelectionPanel.js

@@ -14,8 +14,9 @@ sap.ui.define([
     "sap/ui/core/Icon",
     "sap/m/library",
     "sap/m/OverflowToolbar",
-    "sap/ui/model/Filter"
-], function (SelectionPanel, ColumnListItem, HBox, VBox, Link, Text, Icon, mLibrary, OverflowToolbar, Filter) {
+    "sap/ui/model/Filter",
+    "sap/ui/core/CustomData"
+], function (SelectionPanel, ColumnListItem, HBox, VBox, Link, Text, Icon, mLibrary, OverflowToolbar, Filter, CustomData) {
     "use strict";
 
     // shortcut for sap.m.ListType
@@ -50,7 +51,11 @@ sap.ui.define([
                                     text: "{" + this.P13N_MODEL + ">text}",
                                     href: "{" + this.P13N_MODEL + ">href}",
                                     target: "{" + this.P13N_MODEL + ">target}",
-                                    press: this._onLinkPressed.bind(this)
+                                    press: this._onLinkPressed.bind(this),
+                                    customData: new CustomData({
+                                        key: "internalHref",
+                                        value: "{" + this.P13N_MODEL + ">internalHref}"
+                                    })
                                 }),
                                 new Text({
                                     text: "{" + this.P13N_MODEL + ">description}",
@@ -85,7 +90,10 @@ sap.ui.define([
 	};
 
     LinkSelectionPanel.prototype._onLinkPressed = function(oEvent) {
-        this.fireLinkPressed(oEvent);
+        if (oEvent.getSource().getTarget() !== "_blank") {
+            oEvent.preventDefault();
+            this.fireLinkPressed(oEvent);
+        }
     };
 
     LinkSelectionPanel.prototype.setMultiSelectMode = function(sMultiSelectMode) {

package/src/sap/ui/mdc/p13n/subcontroller/ActionToolbarController.js

@@ -26,7 +26,7 @@ sap.ui.define([
         var oSelectionPanel = new ActionToolbarPanel({
             showHeader: true
         });
-        oSelectionPanel.setEnableReorder(false);
+        //oSelectionPanel.setEnableReorder(false);
         oSelectionPanel.setFieldColumn(oResourceBundle.getText("actiontoolbar.RTA_COLUMN_HEADER"));
 
         var oAdaptationData = this.mixInfoAndState(oPropertyHelper);

package/src/sap/ui/mdc/p13n/subcontroller/BaseController.js

@@ -28,7 +28,7 @@ sap.ui.define([
 	 * @extends sap.ui.base.Object
 	 *
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.103.0
 	 *
 	 * @private
 	 * @ui5-restricted sap.ui.mdc

package/src/sap/ui/mdc/p13n/subcontroller/LinkPanelController.js

@@ -55,35 +55,29 @@ sap.ui.define([
 
     LinkPanelController.prototype._onLinkPressed = function(oEvent) {
         var oSource = oEvent.getParameter("oSource");
-        var sTarget = oSource.getTarget();
+        var oPanel = this.getAdaptationControl();
+        var sHref = oSource && oSource.getCustomData() ? oSource.getCustomData()[0].getValue() : oSource.getHref();
 
-        if (sTarget !== "_blank") {
-            oEvent.preventDefault();
-
-            var oPanel = this.getAdaptationControl();
-            var sHref = oSource.getHref();
-
-            if (oPanel.getBeforeNavigationCallback) {
-                oPanel.getBeforeNavigationCallback()(oEvent).then(function (bNavigate) {
-                    if (bNavigate) {
-                        oPanel.navigate(sHref);
+        if (oPanel.getBeforeNavigationCallback) {
+            oPanel.getBeforeNavigationCallback()(oEvent).then(function (bNavigate) {
+                if (bNavigate) {
+                    oPanel.getMetadata()._oClass.navigate(sHref);
+                }
+            });
+        } else {
+            MessageBox.show(sap.ui.getCore().getLibraryResourceBundle("sap.ui.mdc").getText("info.SELECTION_DIALOG_LINK_VALIDATION_QUESTION"), {
+                icon: MessageBox.Icon.WARNING,
+                title: sap.ui.getCore().getLibraryResourceBundle("sap.ui.mdc").getText("info.SELECTION_DIALOG_LINK_VALIDATION_TITLE"),
+                actions: [
+                    MessageBox.Action.YES, MessageBox.Action.NO
+                ],
+                onClose: function (oAction) {
+                    if (oAction === MessageBox.Action.YES) {
+                        oPanel.getMetadata()._oClass.navigate(sHref);
                     }
-                });
-            } else {
-                MessageBox.show(sap.ui.getCore().getLibraryResourceBundle("sap.ui.mdc").getText("info.SELECTION_DIALOG_LINK_VALIDATION_QUESTION"), {
-                    icon: MessageBox.Icon.WARNING,
-                    title: sap.ui.getCore().getLibraryResourceBundle("sap.ui.mdc").getText("info.SELECTION_DIALOG_LINK_VALIDATION_TITLE"),
-                    actions: [
-                        MessageBox.Action.YES, MessageBox.Action.NO
-                    ],
-                    onClose: function (oAction) {
-                        if (oAction === MessageBox.Action.YES) {
-                            oPanel.navigate(sHref);
-                        }
-                    },
-                    styleClass: this.$().closest(".sapUiSizeCompact").length ? "sapUiSizeCompact" : ""
-                });
-            }
+                },
+                styleClass: this.$().closest(".sapUiSizeCompact").length ? "sapUiSizeCompact" : ""
+            });
         }
     };
 
@@ -128,6 +122,7 @@ sap.ui.define([
             mItem.visible = oExistingLinkItem ? true : false;
             mItem.position = oExistingLinkItem ? oExistingLinkItem.position : -1;
             mItem.href = oProperty.href;
+            mItem.internalHref = oProperty.internalHref;
             mItem.description = oProperty.description;
             mItem.target = oProperty.target;
             mItem.text = oProperty.text;

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

@@ -375,7 +375,10 @@ sap.ui.define([
 
 		if (this._oInnerColumn) {
 			this._oInnerColumn.destroy("KeepDom");
-			oTable.invalidate();
+
+			if (!oTable.isInvalidateSuppressed()) {
+				oTable.invalidate();
+			}
 		}
 	};
 

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

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * {@link sap.ui.mdc.table.Column#getCreationTemplate creationTemplate} aggregation of the {@link sap.ui.mdc.table.Column}.
 	 * @extends sap.ui.core.Element
 	 * @author SAP SE
-	 * @version 1.101.0
+	 * @version 1.103.0
 	 * @constructor
 	 * @experimental
 	 * @private
@@ -201,6 +201,8 @@ sap.ui.define([
 				]
 			});
 
+			this._getTable()._oTable.getRowMode().setHideEmptyRows(this.getVisible());
+
 			for (var sModelName in this._mBindingContexts) {
 				var mBindingContext = this._mBindingContexts[sModelName];
 				this._oInnerCreationRow.setBindingContext(mBindingContext.context, mBindingContext.modelName);

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

@@ -280,11 +280,6 @@ sap.ui.define([
 		this._oRowActionItem.firePress({
 			bindingContext: oRow.getBindingContext()
 		});
-		if (this._oTableRowAction) {
-			this.fireRowPress({
-				bindingContext: oRow.getBindingContext()
-			});
-		}
 	};
 
 	return GridTableType;