@openui5/sap.ui.table 1.117.1 → 1.118.0

package/src/sap/ui/table/rowmodes/RowMode.js

@@ -4,13 +4,11 @@
  * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
  */
 sap.ui.define([
-	"../library",
 	"../utils/TableUtils",
 	"sap/ui/core/Element",
 	"sap/ui/thirdparty/jquery",
 	"sap/ui/Device"
 ], function(
-	library,
 	TableUtils,
 	Element,
 	jQuery,
@@ -21,34 +19,27 @@ sap.ui.define([
 	var _private = TableUtils.createWeakMapFacade();
 
 	/**
-	 * Constructor for a new row mode.
+	 * Constructor for a new <code>RowMode</code>.
 	 *
 	 * @param {string} [sId] id for the new control, generated automatically if no id is given
 	 * @param {object} [mSettings] initial settings for the new control
 	 *
 	 * @abstract
 	 * @class
-	 * TODO: Class description
+	 * Base class for row modes.
+	 * Note: Do not create subclasses.
 	 * @extends sap.ui.core.Element
 	 * @constructor
-	 * @alias sap.ui.table.rowmodes.RowMode
+	 * @alias module:sap/ui/table/rowmodes/RowMode
 	 * @private
-	 * @ui5-restricted sap.ui.mdc
 	 *
 	 * @author SAP SE
-	 * @version 1.117.1
+	 * @version 1.118.0
 	 */
 	var RowMode = Element.extend("sap.ui.table.rowmodes.RowMode", /** @lends sap.ui.table.rowmodes.RowMode.prototype */ {
 		metadata: {
 			library: "sap.ui.table",
-			"abstract": true/*,
-			events: {
-				rowCountChange: {
-					parameters: {
-						count: {type: "int"}
-					}
-				}
-			}*/
+			"abstract": true
 		}
 	});
 
@@ -166,7 +157,7 @@ sap.ui.define([
 	 * Gets the number of contexts that should be requested at least from the rows aggregation binding of the table.
 	 *
 	 * @returns {int} The minimum request length
-	 * @protected
+	 * @private
 	 * @abstract
 	 */
 	RowMode.prototype.getMinRequestLength = function() {
@@ -179,7 +170,7 @@ sap.ui.define([
 	 * The sum of <code>scrollable</code>, <code>fixedTop</code> and <code>fixedBottom</code> is equal to <code>count</code>.
 	 *
 	 * @returns {{count: int, scrollable: int, fixedTop: int, fixedBottom: int}} The computed counts
-	 * @protected
+	 * @private
 	 * @abstract
 	 */
 	RowMode.prototype.getComputedRowCounts = function() {
@@ -191,7 +182,7 @@ sap.ui.define([
 	 *
 	 * @returns {{height: sap.ui.core.CSSSize?, minHeight: sap.ui.core.CSSSize?, maxHeight: sap.ui.core.CSSSize?}}
 	 * The styles the table should have
-	 * @protected
+	 * @private
 	 * @abstract
 	 */
 	RowMode.prototype.getTableStyles = function() {
@@ -204,11 +195,11 @@ sap.ui.define([
 	 *
 	 * @returns {{height: sap.ui.core.CSSSize?}|undefined}
 	 * The styles the table's bottom placeholder should have
-	 * @protected
+	 * @private
 	 * @abstract
 	 */
 	RowMode.prototype.getTableBottomPlaceholderStyles = function() {
-		throwNotImplementedError(this, "getTableBottomPlaceholderStyles");
+		return undefined;
 	};
 
 	/**
@@ -216,7 +207,7 @@ sap.ui.define([
 	 *
 	 * @returns {{height: sap.ui.core.CSSSize?, minHeight: sap.ui.core.CSSSize?, maxHeight: sap.ui.core.CSSSize?}}
 	 * The styles the row container should have
-	 * @protected
+	 * @private
 	 * @abstract
 	 */
 	RowMode.prototype.getRowContainerStyles = function() {
@@ -227,7 +218,7 @@ sap.ui.define([
 	 * Gets the parent table.
 	 *
 	 * @returns {sap.ui.table.Table|null} The instance of the table or <code>null</code>.
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.getTable = function() {
 		var oParent = this.getParent();
@@ -237,7 +228,7 @@ sap.ui.define([
 	/**
 	 * Updates the table's rows aggregation according to the current computed row count, and updates the rows binding contexts.
 	 *
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.updateTable = function(sReason /* private parameter */) {
 		var oTable = this.getTable();
@@ -279,7 +270,7 @@ sap.ui.define([
 	 *
 	 * @returns {int} The base row content height in pixels.
 	 * @see {@link sap.ui.table.rowmodes.RowMode#getBaseRowHeightOfTable}
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.getBaseRowContentHeight = function() {
 		return 0;
@@ -292,7 +283,7 @@ sap.ui.define([
 	 *
 	 * @returns {int} The base row height in pixels.
 	 * @see {@link sap.ui.table.rowmodes.RowMode#getBaseRowContentHeight}
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.getBaseRowHeightOfTable = function() {
 		var oTable = this.getTable();
@@ -314,7 +305,7 @@ sap.ui.define([
 	 * Gets total row count of the table. Returns 0 if this mode is not child of a table.
 	 *
 	 * @returns {int} The total row count.
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.getTotalRowCountOfTable = function() {
 		var oTable = this.getTable();
@@ -433,7 +424,7 @@ sap.ui.define([
 	 * @param {int} iFixedTop The fixed top row count.
 	 * @param {int} iFixedBottom The fixed bottom row count.
 	 * @returns {{count: int, scrollable: int, fixedTop: int, fixedBottom: int}} The standardized counts
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.computeStandardizedRowCounts = function(iCount, iFixedTop, iFixedBottom) {
 		var mRowCountConstraints = this.getRowCountConstraints();
@@ -494,7 +485,7 @@ sap.ui.define([
 	 * </ul>
 	 *
 	 * @returns {{fixedTop: (boolean|undefined), fixedBottom: (boolean|undefined)}} The row count constraints
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.getRowCountConstraints = function() {
 		// TODO: Add a type definition for a protected type "rowCountConstraints" in the library file to document the return value
@@ -519,6 +510,26 @@ sap.ui.define([
 	 */
 	RowMode.prototype.renderCellContentStyles = function(oRM) {};
 
+	/**
+	 * Renders something at the bottom of the table.
+	 *
+	 * @param {sap.ui.core.RenderManager} oRM The render manager.
+	 * @private
+	 */
+	RowMode.prototype.renderInTableBottomArea = function(oRM) {
+		var mPlaceholderHeight = this.getTableBottomPlaceholderStyles();
+
+		if (mPlaceholderHeight === undefined) {
+			return;
+		}
+
+		oRM.openStart("div", this.getTable().getId() + "-placeholder-bottom");
+		oRM.class("sapUiTablePlaceholder");
+		this.applyTableBottomPlaceholderStyles(oRM);
+		oRM.openEnd();
+		oRM.close("div");
+	};
+
 	/**
 	 * This method can be used after a binding refresh to asynchronously create rows after the data request was sent by the binding.
 	 * The rows are added to the table's rows aggregation. No binding contexts are set for the rows. If the table already has rows, no action is
@@ -712,7 +723,7 @@ sap.ui.define([
 	 * {@link sap.ui.table.Table#getShowNoData showNoData} is set to <code>true</code>. The text is hidden if it is currently shown. Has no effect for
 	 * the text that is shown when the table has no visible columns.
 	 *
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.disableNoData = function() {
 		if (this.isNoDataDisabled()) {
@@ -731,7 +742,7 @@ sap.ui.define([
 	 * Enables the "NoData" text of the table. Whether the text is shown depends on the state of the table and its
 	 * {@link sap.ui.table.Table#getShowNoData showNoData} property.
 	 *
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.enableNoData = function() {
 		if (!this.isNoDataDisabled()) {
@@ -750,7 +761,7 @@ sap.ui.define([
 	 * Checks whether the "NoData" text of the table is disabled.
 	 *
 	 * @returns {boolean} Whether the "NoData" text is disabled
-	 * @protected
+	 * @private
 	 */
 	RowMode.prototype.isNoDataDisabled = function() {
 		return _private(this).bNoDataDisabled;

package/src/sap/ui/table/rowmodes/Type.js

@@ -0,0 +1,38 @@
+/*!
+ * OpenUI5
+ * (c) Copyright 2009-2023 SAP SE or an SAP affiliate company.
+ * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
+ */
+sap.ui.define(function() {
+	"use strict";
+
+	/**
+	 * Defines the row mode.
+	 *
+	 * @enum {string}
+	 * @alias sap.ui.table.rowmodes.Type
+	 * @private
+	 */
+	var Type = {
+		/**
+		 * Equivalent to the default configuration of {@link module:sap/ui/table/rowmodes/Fixed}
+		 *
+		 * @public
+		 */
+		Fixed: "Fixed",
+		/**
+		 * Equivalent to the default configuration of {@link module:sap/ui/table/rowmodes/Auto}
+		 *
+		 * @public
+		 */
+		Auto: "Auto",
+		/**
+		 * Equivalent to the default configuration of {@link module:sap/ui/table/rowmodes/Interactive}
+		 *
+		 * @public
+		 */
+		Interactive: "Interactive"
+	};
+
+	return Type;
+}, /* bExport= */ true);

package/src/sap/ui/table/rowmodes/{VariableRowMode.js → Variable.js}

@@ -20,17 +20,17 @@ sap.ui.define([
 	 *
 	 * @class
 	 * // TODO: Class description
-	 * @extends sap.ui.table.rowmodes.RowMode
+	 * @extends module:sap/ui/table/rowmodes/RowMode
 	 *
 	 * @author SAP SE
-	 * @version 1.117.1
+	 * @version 1.118.0
 	 *
 	 * @constructor
 	 * @private
 	 * @experimental
-	 * @alias sap.ui.table.rowmodes.VariableRowMode
+	 * @alias module:sap/ui/table/rowmodes/Variable
 	 */
-	var VariableRowMode = RowMode.extend("sap.ui.table.rowmodes.VariableRowMode", {
+	var VariableRowMode = RowMode.extend("sap.ui.table.rowmodes.Variable", {
 		metadata: {
 			library: "sap.ui.table",
 			properties: {
@@ -64,20 +64,6 @@ sap.ui.define([
 		};
 	};
 
-	/**
-	 * @inheritDoc
-	 */
-	VariableRowMode.prototype.getComputedRowCounts = function() {
-		var iTotalRowCount = this.getTotalRowCountOfTable();
-		var iRowCount = this.getHideEmptyRows() ? Math.min(this.getRowCount(), iTotalRowCount) : this.getRowCount();
-
-		return {
-			count: iRowCount,
-			fixedTopCount: this.getFixedTopRowCount(),
-			fixedBottomCount: this.getFixedBottomRowCount()
-		};
-	};
-
 	/**
 	 * @inheritDoc
 	 */

package/src/sap/ui/table/rules/Rows.support.js

@@ -164,12 +164,12 @@ sap.ui.define([
 			}
 
 			function checkConfiguration(oTable, oDynamicPage) {
-				if (oTable._getRowMode().isA("sap.ui.table.rowmodes.AutoRowMode") && !oDynamicPage.getFitContent()) {
+				if (oTable._getRowMode().isA("sap.ui.table.rowmodes.Auto") && !oDynamicPage.getFitContent()) {
 					SupportHelper.reportIssue(oIssueManager,
 						"A table with an auto row mode is placed inside a sap.f.DynamicPage with fitContent=\"false\"",
 						Severity.High, oTable.getId());
-				} else if ((oTable._getRowMode().isA("sap.ui.table.rowmodes.FixedRowMode")
-							|| oTable._getRowMode().isA("sap.ui.table.rowmodes.InteractiveRowMode"))
+				} else if ((oTable._getRowMode().isA("sap.ui.table.rowmodes.Fixed")
+							|| oTable._getRowMode().isA("sap.ui.table.rowmodes.Interactive"))
 						   && oDynamicPage.getFitContent()) {
 					SupportHelper.reportIssue(oIssueManager,
 						"A table with a fixed or interactive row mode is placed inside a sap.f.DynamicPage with fitContent=\"true\"",

package/src/sap/ui/table/utils/TableUtils.js

@@ -16,9 +16,11 @@ sap.ui.define([
 	"sap/ui/core/ResizeHandler",
 	"sap/ui/core/library",
 	"sap/ui/core/theming/Parameters",
+	"sap/ui/core/Icon",
 	"sap/ui/model/ChangeReason",
 	"sap/ui/thirdparty/jquery",
-	"sap/base/util/restricted/_throttle"
+	"sap/base/util/restricted/_throttle",
+	"sap/base/Log"
 ], function(
 	GroupingUtils,
 	ColumnUtils,
@@ -30,9 +32,11 @@ sap.ui.define([
 	ResizeHandler,
 	coreLibrary,
 	ThemeParameters,
+	Icon,
 	ChangeReason,
 	jQuery,
-	throttle
+	throttle,
+	Log
 ) {
 	"use strict";
 
@@ -203,7 +207,7 @@ sap.ui.define([
 	 * Static collection of utility functions related to the sap.ui.table.Table, ...
 	 *
 	 * @author SAP SE
-	 * @version 1.117.1
+	 * @version 1.118.0
 	 * @namespace
 	 * @alias sap.ui.table.utils.TableUtils
 	 * @private
@@ -337,7 +341,7 @@ sap.ui.define([
 		 * @returns {boolean} Whether the no data text is shown.
 		 */
 		isNoDataVisible: function(oTable) {
-			return oTable.getShowNoData() && !oTable._getRowMode().isNoDataDisabled() && !TableUtils.hasData(oTable)
+			return oTable.getShowNoData() && (oTable._getRowMode() && !oTable._getRowMode().isNoDataDisabled()) && !TableUtils.hasData(oTable)
 				   || TableUtils.getVisibleColumnCount(oTable) === 0;
 		},
 
@@ -578,6 +582,138 @@ sap.ui.define([
 			}
 		},
 
+		/**
+		 * Scrolls the table to the <code>iIndex</code>.
+		 * If <code>bReverse</code> is true the <code>firstVisibleRow</code> property of the Table is set to <code>iIndex</code> - 1,
+		 * otherwise to <code>iIndex</code> - row count + 2.
+		 *
+		 * @param {sap.ui.table.Table} oTable Instance of the table.
+		 * @param {int} iIndex The index of the row to which to scroll to.
+		 * @param {boolean} bReverse Whether the row should be displayed at the bottom of the table.
+		 * @returns {Promise} A promise that resolves when the table is scrolled.
+		 */
+		scrollTableToIndex: function(oTable, iIndex, bReverse) {
+			if (!oTable) {
+				return Promise.resolve();
+			}
+
+			var iFirstVisibleRow = oTable.getFirstVisibleRow();
+			var mRowCounts = oTable._getRowCounts();
+			var iLastVisibleRow = iFirstVisibleRow + mRowCounts.scrollable - 1;
+			var bExpectRowsUpdatedEvent = false;
+
+			if (iIndex < iFirstVisibleRow || iIndex > iLastVisibleRow) {
+				var iNewIndex = bReverse ? iIndex - mRowCounts.fixedTop - 1 : iIndex - mRowCounts.scrollable - mRowCounts.fixedTop + 2;
+
+				bExpectRowsUpdatedEvent = oTable._setFirstVisibleRowIndex(Math.max(0, iNewIndex));
+			}
+
+			return new Promise(function(resolve) {
+				if (bExpectRowsUpdatedEvent) {
+					oTable.attachEventOnce("rowsUpdated", resolve);
+				} else {
+					resolve();
+				}
+			});
+		},
+
+		/**
+		 * Displays a notification Popover beside the row selector that indicates a limited selection. The given index
+		 * references the index of the data context in the binding.
+		 *
+		 * @param {sap.ui.table.Table} oTable Instance of the table.
+		 * @param {number} iIndex Index of the data context
+		 * @param {number} iLimit Maximum number of rows that can be selected at once
+		 * @returns {Promise} A Promise that resolves after the notification popover has been opened
+		 */
+		showNotificationPopoverAtIndex: function(oTable, iIndex, iLimit) {
+			var oPopover = oTable._oNotificationPopover;
+			var oRow = oTable.getRows()[iIndex - oTable._getFirstRenderedRowIndex()];
+			var sTitle = TableUtils.getResourceText("TBL_SELECT_LIMIT_TITLE");
+			var sMessage = TableUtils.getResourceText("TBL_SELECT_LIMIT", [iLimit]);
+
+			return new Promise(function(resolve) {
+				sap.ui.require([
+					"sap/m/Popover", "sap/m/Bar", "sap/m/Title", "sap/m/Text", "sap/m/HBox", "sap/ui/core/library", "sap/m/library"
+				], function(Popover, Bar, Title, Text, HBox, coreLib, mLib) {
+					if (!oPopover) {
+						oPopover = new Popover(oTable.getId() + "-notificationPopover", {
+							customHeader: [
+								new Bar({
+									contentMiddle: [
+										new HBox({
+											items: [
+												new Icon({src: "sap-icon://message-warning", color: coreLib.IconColor.Critical})
+													.addStyleClass("sapUiTinyMarginEnd"),
+												new Title({text: sTitle, level: coreLib.TitleLevel.H2})
+											],
+											renderType: mLib.FlexRendertype.Bare,
+											justifyContent: mLib.FlexJustifyContent.Center,
+											alignItems: mLib.FlexAlignItems.Center
+										})
+									]
+								})
+							],
+							content: new Text({text: sMessage})
+						});
+
+						oPopover.addStyleClass("sapUiContentPadding");
+						oTable._oNotificationPopover = oPopover;
+						oTable.addAggregation("_hiddenDependents", oTable._oNotificationPopover);
+					} else {
+						oPopover.getContent()[0].setText(sMessage);
+					}
+
+					oTable.detachFirstVisibleRowChanged(this.onFirstVisibleRowChange, this);
+					oTable.attachFirstVisibleRowChanged(this.onFirstVisibleRowChange, this);
+
+					var oRowSelector = oRow.getDomRefs().rowSelector;
+
+					if (oRowSelector) {
+						oPopover.attachEventOnce("afterOpen", resolve);
+						oPopover.openBy(oRowSelector);
+					} else {
+						resolve();
+					}
+				}.bind(this));
+			}.bind(this));
+		},
+
+		onFirstVisibleRowChange: function(oEvent) {
+			var oTable = oEvent.getSource();
+			if (!oTable._oNotificationPopover) {
+				return;
+			}
+
+			if (oTable) {
+				oTable.detachFirstVisibleRowChanged(this.onFirstVisibleRowChange, oTable);
+			}
+			oTable._oNotificationPopover.close();
+		},
+
+		/**
+		 * Loads <code>iLength</code> binding contexts starting from index <code>iStartIndex</code>.
+		 *
+		 * @param {object} oBinding The binding
+		 * @param {int} iStartIndex The starting index
+		 * @param {int} iLength The length
+		 * @returns {Promise<sap.ui.model.Context[]>} Promise that resolves with the row contexts once they are loaded.
+		 */
+		loadContexts: function(oBinding, iStartIndex, iLength) {
+			var aContexts = oBinding.getContexts(iStartIndex, iLength, 0, true);
+			var bContextsAvailable = aContexts.length === Math.min(iLength, oBinding.getLength()) && !aContexts.includes(undefined);
+
+			if (bContextsAvailable) {
+				return Promise.resolve(aContexts);
+			}
+
+			return new Promise(function(resolve) {
+				oBinding.attachEventOnce("dataReceived", function() {
+					resolve(this.loadContexts(oBinding, iStartIndex, iLength));
+				}.bind(this));
+			}.bind(this));
+		},
+
 		/**
 		 * The following rules apply for the cell information.
 		 * <ul>
@@ -1421,6 +1557,31 @@ sap.ui.define([
 
 				return oWeakMap.get(oKey);
 			};
+		},
+
+		/**
+		 * Returns the active Table helper.
+		 *
+		 * @param {boolean} bCallByAPI Whether the call is initiated by an API setting
+		 * @throws Error when no Table helper can be found.
+		 * @returns {object} Table helper
+		 */
+		_getTableTemplateHelper: function(bCallByAPI) {
+			var sMessage = "An automatic control and template generation for the sap.ui.table.Table is not supported " +
+							"anymore for the aggragations footer and title and the aggregations label and template of " +
+							"the sap.ui.table.Columns. Use concrete controls for those aggregations instead of altType string.";
+
+			/**
+			 * @deprecated As of version 1.118
+			 */
+			if (library.TableHelper) {
+				if (!bCallByAPI) {
+					Log.warning(sMessage);
+				}
+				return library.TableHelper;
+			}
+
+			throw new Error(sMessage);
 		}
 	};
 

package/src/sap/ui/table/utils/_BindingUtils.js

@@ -14,7 +14,7 @@ sap.ui.define([], function() {
 	 * Note: Do not access the functions of this helper directly, but via <code>sap.ui.table.utils.TableUtils.Binding...</code>
 	 *
 	 * @author SAP SE
-	 * @version 1.117.1
+	 * @version 1.118.0
 	 * @namespace
 	 * @alias sap.ui.table.utils._BindingUtils
 	 * @private

package/src/sap/ui/table/utils/_ColumnUtils.js

@@ -18,7 +18,7 @@ sap.ui.define([
 	 * Note: Do not access the functions of this helper directly, but via <code>sap.ui.table.utils.TableUtils.Column...</code>
 	 *
 	 * @author SAP SE
-	 * @version 1.117.1
+	 * @version 1.118.0
 	 * @namespace
 	 * @alias sap.ui.table.utils._ColumnUtils
 	 * @private

package/src/sap/ui/table/utils/_GroupingUtils.js

@@ -23,7 +23,7 @@ sap.ui.define([
 	 * Note: Do not access the functions of this helper directly, but via <code>sap.ui.table.utils.TableUtils.Grouping...</code>
 	 *
 	 * @author SAP SE
-	 * @version 1.117.1
+	 * @version 1.118.0
 	 * @namespace
 	 * @alias sap.ui.table.utils._GroupingUtils
 	 * @private
@@ -425,6 +425,7 @@ sap.ui.define([
 		 * Initializes the experimental grouping for sap.ui.table.Table.
 		 *
 		 * @param {sap.ui.table.Table} oTable Instance of the table.
+		 * @deprecated As of version 1.118.
 		 */
 		setupExperimentalGrouping: function(oTable) {
 			if (!oTable.getEnableGrouping()) {

package/src/sap/ui/table/utils/_HookUtils.js

@@ -22,7 +22,7 @@ sap.ui.define(["sap/ui/base/DataType"], function(DataType) {
 	 * - There is no concept for public or protected hooks. Never expose a hook directly, only indirectly as can be seen in the examples.
 	 *
 	 * @author SAP SE
-	 * @version 1.117.1
+	 * @version 1.118.0
 	 * @namespace
 	 * @alias sap.ui.table.utils._HookUtils
 	 *

package/src/sap/ui/table/utils/_MenuUtils.js

@@ -39,7 +39,7 @@ sap.ui.define([
 	 * Note: Do not access the functions of this helper directly, but via <code>sap.ui.table.utils.TableUtils.Menu...</code>
 	 *
 	 * @author SAP SE
-	 * @version 1.117.1
+	 * @version 1.118.0
 	 * @namespace
 	 * @alias sap.ui.table.utils._MenuUtils
 	 * @private
@@ -103,6 +103,9 @@ sap.ui.define([
 					oRowBindingContext = oRow.getBindingContext(oRowBindingInfo.model);
 				}
 
+				/**
+				 * @deprecated As of version 1.21.0
+				 */
 				if (iColumnIndex >= 0) {
 					bExecuteDefault = oTable.fireCellContextmenu({
 						rowIndex: oRow.getIndex(),
@@ -114,21 +117,26 @@ sap.ui.define([
 					});
 				}
 
-				// fire beforeOpenContextMenu event if the default is not prevented in the cellContextMenu event
-				if (bExecuteDefault) {
-					var oRowContextMenu = oTable.getContextMenu();
+				/**
+				 * @deprecated As of version 1.21.0
+				 */
+				if (!bExecuteDefault) {
+					return true;
+				}
 
-					if (oRowContextMenu && oRowBindingInfo) {
-						oRowContextMenu.setBindingContext(oRowBindingContext, oRowBindingInfo.model);
-					}
+				// fire beforeOpenContextMenu event
+				var oRowContextMenu = oTable.getContextMenu();
 
-					bExecuteDefault = oTable.fireBeforeOpenContextMenu({
-						rowIndex: oRow.getIndex(),
-						columnIndex: oRowColCell.column ? iColumnIndex : null,
-						contextMenu: oRowContextMenu
-					});
+				if (oRowContextMenu && oRowBindingInfo) {
+					oRowContextMenu.setBindingContext(oRowBindingContext, oRowBindingInfo.model);
 				}
 
+				bExecuteDefault = oTable.fireBeforeOpenContextMenu({
+					rowIndex: oRow.getIndex(),
+					columnIndex: oRowColCell.column ? iColumnIndex : null,
+					contextMenu: oRowContextMenu
+				});
+
 				if (bExecuteDefault) {
 					return MenuUtils._openContentCellContextMenu(oTable, oCell, oEvent);
 				} else {
@@ -287,7 +295,7 @@ sap.ui.define([
 		 */
 		_closeCustomContentCellContextMenu: function(oTable) {
 			var oCustomMenu = oTable.getContextMenu();
-			var bCustomContextMenuOpen = oCustomMenu ? oCustomMenu.bOpen : false;
+			var bCustomContextMenuOpen = oCustomMenu ? oCustomMenu.isOpen() : false;
 
 			if (bCustomContextMenuOpen) {
 				oCustomMenu.close();
@@ -302,7 +310,7 @@ sap.ui.define([
 		 */
 		_closeDefaultContentCellContextMenu: function(oTable) {
 			var oDefaultMenu = oTable._oCellContextMenu;
-			var bDefaultMenuOpen = oDefaultMenu ? oDefaultMenu.bOpen : false;
+			var bDefaultMenuOpen = oDefaultMenu ? oDefaultMenu.isOpen() : false;
 
 			if (bDefaultMenuOpen) {
 				oDefaultMenu.close();