@openui5/sap.ui.mdc 1.99.0 → 1.100.1

package/src/sap/ui/mdc/valuehelp/content/MDCTable.js

@@ -12,8 +12,7 @@ sap.ui.define([
 	'sap/ui/mdc/library',
 	'sap/m/library',
 	"sap/ui/table/library",
-	"sap/ui/mdc/p13n/Engine",
-	"sap/ui/mdc/enum/PersistenceMode"
+	"sap/ui/thirdparty/jquery"
 
 ], function(
 	FilterableListContent,
@@ -23,8 +22,7 @@ sap.ui.define([
 	library,
 	mLibrary,
 	uiTableLibrary,
-	Engine,
-	PersistenceMode
+	jQuery
 ) {
 	"use strict";
 
@@ -224,9 +222,7 @@ sap.ui.define([
 					return false;
 				},
 				getContexts: function () {
-					return oListBinding && (oListBinding.aContexts || (oListBinding.aIndices && oListBinding.aIndices.map(function (iIndex) {
-						return oInnerTable.getContextByIndex(iIndex);
-					})) || oListBinding.getContexts());
+					return oListBinding && oListBinding.getAllCurrentContexts();
 				},
 				handleListBinding: function () {
 					oListBinding.attachEvent("change", this._handleUpdateFinished.bind(this));
@@ -261,7 +257,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new control
 	 * @class Content for the <code>sap.ui.mdc.valuehelp.base.Container</code> element using a sap.ui.mdc.Table.
 	 * @extends sap.ui.mdc.valuehelp.base.FilterableListContent
-	 * @version 1.99.0
+	 * @version 1.100.1
 	 * @constructor
 	 * @abstract
 	 * @private
@@ -435,14 +431,12 @@ sap.ui.define([
 		if (oChanges.name === "table") { // outer table
 			var oTable = oChanges.child;
 			if (oChanges.mutation === "remove") {
-				Engine.getInstance().defaultProviderRegistry.detach(oTable);
 				this._oObserver.unobserve(oTable);
 				this._oTable = null;
 				this._oTableHelper = null;
 				this._addPromise("listBinding");
 			} else {
 				this._oTable = oTable;
-				Engine.getInstance().defaultProviderRegistry.attach(oTable, PersistenceMode.Transient);
 				this._oObserver.observe(oTable, {aggregations: ["_content"]});
 				this._sTableType = _getMDCTableType(oTable);
 				oTable.addDelegate({ onmouseover: function (oEvent) {	// Fix m.Table itemPress

package/src/sap/ui/mdc/valuehelp/content/MTable.js

@@ -19,7 +19,8 @@ sap.ui.define([
 	'sap/base/strings/formatMessage',
 	'sap/base/util/merge',
 	'sap/ui/mdc/enum/SelectType',
-	'sap/base/Log'
+	'sap/base/Log',
+	'sap/ui/thirdparty/jquery'
 ], function(
 	FilterableListContent,
 	Condition,
@@ -35,7 +36,8 @@ sap.ui.define([
 	formatMessage,
 	merge,
 	SelectType,
-	Log
+	Log,
+	jQuery
 ) {
 	"use strict";
 
@@ -49,7 +51,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new control
 	 * @class Content for the <code>sap.ui.mdc.valuehelp.base.Container</code> element using a sap.m.Table.
 	 * @extends sap.ui.mdc.valuehelp.base.FilterableListContent
-	 * @version 1.99.0
+	 * @version 1.100.1
 	 * @constructor
 	 * @abstract
 	 * @private
@@ -188,7 +190,7 @@ sap.ui.define([
 			bUseFilter = false;
 		}
 
-		if (sFilterFields === "$search" && oDelegate && oDelegate.isSearchSupported(oDelegatePayload, oListBinding)){
+		if (sFilterFields === "$search" && oDelegate && oDelegate.isSearchSupported(oDelegatePayload, this, oListBinding)){
 			if (!oListBinding.isSuspended() && bUseFilter) {
 				// as we trigger two changes this would result to two requests therefore we suspend the binding
 				oListBinding.suspend();

package/test/sap/ui/mdc/testutils/opa/chart/TestObjects.js

@@ -268,6 +268,7 @@ sap.ui.define([
                 },
                 /**
                  * Assertion to check visible dimensions on the MDC Chart
+                 * @param {array} aDimensions
                  * @param {string} sId Id of the MDC Chart
                  * @returns {Promise} OPA waitFor
                  */
@@ -276,6 +277,7 @@ sap.ui.define([
                 },
                 /**
                  * Assertion to check visible measures on the MDC Chart
+                 * @param {array} aMeasures
                  * @param {string} sId Id of the MDC Chart
                  * @returns {Promise} OPA waitFor
                 */

package/test/sap/ui/mdc/testutils/opa/filterbar/TestObjects.js

@@ -88,7 +88,6 @@ sap.ui.define([
 				 * 	</li>
 				 * </ol>
 				 * @param {sap.ui.core.Control | string} oFilterBar Instance / ID of the <code>FilterBar</code>
-				 * @param {string} sFilterLabel Label of the <code>FilterField</code>
 				 * @param {Object} mSettings Map containing the settings for the filter values. Key is the label of the given group in the <code>FilterBar</code> personalization dialog, and value is an object containing the label of the <code>FilterField</code> and the values that are entered
 				 * @returns
 				 */

package/test/sap/ui/mdc/testutils/opa/p13n/Actions.js

@@ -17,7 +17,8 @@ sap.ui.define([
 	"./Util",
 	"sap/base/Log",
 	"sap/base/util/UriParameters",
-	"sap/ui/core/Core"
+	"sap/ui/core/Core",
+	"sap/ui/Device"
 ], function(
 	Opa5,
 	Matcher,
@@ -33,7 +34,8 @@ sap.ui.define([
 	Util,
 	Log,
 	UriParameters,
-	oCore
+	oCore,
+	Device
 ) {
 	"use strict";
 
@@ -495,7 +497,7 @@ sap.ui.define([
 
 	var iPersonalizeListViewItems = function(oP13nDialog, aItems) {
 		this.waitFor({
-			controlType: oP13nDialog.getContent()[0].getView("columns") && oP13nDialog.getContent()[0].getView("columns").getContent().isA("sap.ui.mdc.p13n.panels.ListView") ? "sap.ui.mdc.p13n.panels.ListView" : "sap.m.p13n.SelectionPanel",
+			controlType: oP13nDialog.getContent()[0].getView("columns") && oP13nDialog.getContent()[0].getView("columns").getContent().isA("sap.m.p13n.SelectionPanel") ? "sap.m.p13n.SelectionPanel" : "sap.m.p13n.SelectionPanel",
 			matchers: new Ancestor(oP13nDialog, false),
 			success: function(aListViews) {
 				var oListView = aListViews[0];
@@ -740,16 +742,16 @@ sap.ui.define([
 		 * 2. Selects a chart type given by <code>sChartType</code>.
 		 * 3. Executes the given <code>ChartPersonalizationConfiguration</code>.
 		 * 4. Closes the personalization dialog.
-		 * @param {sap.ui.core.Control | string} oChart Instance / ID of the <code>SmartChart</code> that is personalized
+		 * @param {sap.ui.core.Control | string} oControl Instance / ID of the <code>SmartChart</code> that is personalized
 		 * @param {string} sChartType String containing the type of chart that is displayed
-		 * @param {ChartPersonalizationConfiguration[]} aConfigurations Array containing the chart personalization configuration objects
+		 * @param {ChartPersonalizationConfiguration[]} aItems Array containing the chart personalization configuration objects
 		 * @returns {Promise} OPA waitFor
 		 */
 		iPersonalizeChart: function(oControl, sChartType, aItems) {
 			return iPersonalize.call(this, oControl, Util.texts.chart, {
 				success: function(oP13nDialog) {
 
-					//oP13nDialog.getContent()[0].getView("item") && oP13nDialog.getContent()[0].getView("item").getContent().isA("sap.ui.mdc.p13n.panels.ListView")
+					//oP13nDialog.getContent()[0].getView("item") && oP13nDialog.getContent()[0].getView("item").getContent().isA("sap.m.p13n.SelectionPanel")
 					if (oP13nDialog.getContent()[0].getView("dimeasure").getContent().isA("sap.m.P13nDimMeasurePanel")){
 						iPersonalizeOldChartP13n.call(this, oControl, sChartType, aItems, oP13nDialog);
 					} else {

package/src/sap/ui/mdc/filterbar/vh/GenericFilterBarDelegate.js

@@ -1,150 +0,0 @@
-/*
- * ! OpenUI5
- * (c) Copyright 2009-2022 SAP SE or an SAP affiliate company.
- * Licensed under the Apache License, Version 2.0 - see LICENSE.txt.
- */
-sap.ui.define([
-	"sap/ui/mdc/odata/v4/FilterBarDelegate",
-	'sap/ui/base/ManagedObjectObserver'
-], function(FilterBarDelegate, ManagedObjectObserver) {
-	"use strict";
-
-	/**
-	 * Helper class for sap.ui.mdc.filterbar.vh.GenericFilterBarDelegate.
-	*  This GenericFilterbarDelegate creates the propertyInfo based on the FilterItems of the Filterbar and does NOT load and analyse any metadata!
-	 * <h3><b>Note:</b></h3>
-	 * The class is experimental and the API/behaviour is not finalised and hence this should not be used for productive usage.
-	 *
-	 * @author SAP SE
-	 * @private
-	 * @experimental
-	 * @since 1.86
-	 * @alias sap.ui.mdc.filterbar.vh.GenericFilterBarDelegate
-	 */
-	var GenericFilterBarDelegate = Object.assign({}, FilterBarDelegate);
-
-	/**
-	 * Fetches the relevant metadata (from the FilterItems of the FiterBar) for the FilterBar and returns property info array
-	 *
-	 * @param {Object} oFilterBar - instance of the valuehelp FilterBar
-	 * @returns {Array} array of property info
-	 */
-	GenericFilterBarDelegate.fetchProperties = function(oFilterBar) {
-		if (!oFilterBar.__oObserver) {
-			oFilterBar.__oObserver = new ManagedObjectObserver(_observeChanges.bind(this));
-			oFilterBar.__oObserver.observe(oFilterBar, {
-				aggregations: ["filterItems"]
-			});
-		}
-
-		return new Promise(function(fResolve) {
-			var aFilterItems = oFilterBar.getFilterItems();
-			oFilterBar.__aProperties = [];
-			aFilterItems.forEach(function(oFF){
-				addFilterField.call(this, oFF, oFilterBar.__aProperties);
-			}.bind(this));
-
-			fResolve(oFilterBar.__aProperties);
-		}.bind(this));
-
-	};
-
-	function addFilterField(oFF, aProperties) {
-		var sPath = oFF.getBindingPath("conditions");
-		if (!sPath) {
-			return;
-		}
-		var aPathParts = sPath.split("/");
-		var sFieldPath = aPathParts[aPathParts.length - 1];
-
-		aProperties.push({
-			name: sFieldPath,
-			label: oFF.getLabel() || sFieldPath,
-			type: oFF.getDataType(),
-			formatOptions: oFF.getDataTypeFormatOptions(),
-			constraints: oFF.getDataTypeConstraints(),
-			typeConfig: this.getTypeUtil().getTypeConfig(oFF.getDataType(), oFF.getDataTypeFormatOptions(), oFF.getDataTypeConstraints()),
-			required: oFF.getRequired(),
-			hiddenFilter: false,
-			visible: oFF.getVisible(),
-			maxConditions : oFF.getMaxConditions(),
-			fieldHelp: oFF.getFieldHelp()
-		});
-	}
-
-	function removeFilterField(oFF, aProperties) {
-		var sPath = oFF.getBindingPath("conditions");
-		if (!sPath) {
-			return;
-		}
-
-		function _getProperty(sName) {
-			var oNamedProperty = null;
-			aProperties.some(function(oProperty) {
-				if (oProperty.name === sName) {
-					oNamedProperty = oProperty;
-				}
-				return oNamedProperty !== null;
-			});
-
-			return oNamedProperty;
-		}
-
-		function _removeProperty(sName) {
-			var nIdx = -1;
-			aProperties.some(function(oProperty, index) {
-				if (oProperty.name === sName) {
-					nIdx = index;
-				}
-				return nIdx !== -1;
-			});
-
-			if (nIdx >= 0) {
-				aProperties.splice(nIdx, 1);
-			}
-
-			return nIdx;
-		}
-
-		var aPathParts = sPath.split("/");
-		var sFieldPath = aPathParts[aPathParts.length - 1];
-		if (_getProperty(sFieldPath)) {
-			_removeProperty(sFieldPath);
-		}
-	}
-
-
-
-	function _observeChanges(oChanges) {
-		var oFilterBar, aProperties;
-
-		if (oChanges.name === "filterItems") {
-			if (oChanges.mutation === "insert") {
-				var oNewFF = oChanges.child;
-				oFilterBar = oNewFF.getParent();
-				aProperties = oFilterBar.__aProperties;
-				addFilterField.call(this, oNewFF, aProperties);
-				return;
-			}
-			if (oChanges.mutation === "remove") {
-				var oRemoveFF = oChanges.child;
-				oFilterBar = oRemoveFF.getParent();
-				aProperties = oFilterBar.__aProperties;
-				removeFilterField(oRemoveFF, aProperties);
-				return;
-			}
-		}
-	}
-
-	GenericFilterBarDelegate.cleanup = function(oFilterBar) {
-
-		if (oFilterBar.__oObserver) {
-			oFilterBar.__oObserver.disconnect();
-			delete oFilterBar.__oObserver;
-			delete oFilterBar.__aProperties;
-		}
-
-	};
-
-	return GenericFilterBarDelegate;
-});