@openui5/sap.ui.mdc 1.141.2 → 1.143.0

package/src/sap/ui/mdc/valuehelp/base/DefineConditionPanel.js

@@ -15,6 +15,7 @@ sap.ui.define([
 	'sap/ui/mdc/condition/Condition',
 	'sap/ui/mdc/condition/FilterOperatorUtil',
 	'sap/ui/mdc/field/ConditionsType',
+	'sap/ui/mdc/field/ConditionType',
 	'sap/ui/mdc/field/splitValue',
 	'sap/ui/mdc/enums/FieldEditMode',
 	'sap/ui/mdc/enums/FieldDisplay',
@@ -58,6 +59,7 @@ sap.ui.define([
 	Condition,
 	FilterOperatorUtil,
 	ConditionsType,
+	ConditionType,
 	splitValue,
 	FieldEditMode,
 	FieldDisplay,
@@ -121,7 +123,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.Control
 	 *
 	 * @author SAP SE
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 *
 	 * @constructor
 	 * @alias sap.ui.mdc.valuehelp.base.DefineConditionPanel
@@ -304,6 +306,8 @@ sap.ui.define([
 				onpaste: this.onPaste
 			};
 
+			this._oConditionType = new ConditionType();
+
 			_createInnerControls.call(this);
 			this.setModel(this._oManagedObjectModel, "$this");
 			this.setModel(this._oManagedObjectModel, "$condition"); // TODO: better solution to have 2 bindingContexts on one control
@@ -326,6 +330,9 @@ sap.ui.define([
 
 			this._oManagedObjectModel.destroy();
 			delete this._oManagedObjectModel;
+
+			this._oConditionType.destroy();
+			delete this._oConditionType;
 		},
 
 		byId: function(sId) {
@@ -479,7 +486,7 @@ sap.ui.define([
 		onChange: function(oEvent) {
 			const oPromise = oEvent && oEvent.getParameter("promise");
 			const oSourceControl = oEvent && oEvent.getSource();
-			const fnHandleChange = function(oEvent) {
+			const fnHandleChange = function(oEvent, oControl) {
 				const aOperators = _getOperators.call(this);
 				const aConditions = this.getConditions();
 				FilterOperatorUtil.checkConditionsEmpty(aConditions, aOperators);
@@ -496,28 +503,39 @@ sap.ui.define([
 				}
 
 				this.setProperty("conditions", aConditions, true); // do not invalidate whole DefineConditionPanel
+				if (deepEqual(aConditions, this.getConditions())) {
+					const oInvisibleText = _getInvisibleTextForField(oControl);
+					if (oInvisibleText) {
+						oInvisibleText.getBinding("text").checkUpdate(true);
+					}
+				}
 			}.bind(this);
 
 			if (oPromise) {
 				oPromise.then((vResult) => {
 					this._bPendingChange = false;
-					fnHandleChange({ mParameters: { value: vResult } }); // TODO: use a real event?
+					fnHandleChange({ mParameters: { value: vResult } }, oSourceControl); // TODO: use a real event?
 					if (this._bPendingValidateCondition) {
 						_validateCondition.call(this, oSourceControl);
 						delete this._bPendingValidateCondition;
 					}
 				}).catch((oError) => { // cleanup pending stuff
 					this._bPendingChange = false;
+					_setInvalidInvisibleText(oSourceControl);
 					if (this._bPendingValidateCondition) {
 						_validateCondition.call(this, oSourceControl);
 						delete this._bPendingValidateCondition;
 					}
 				});
 
+				if (oSourceControl) {
+					_resetInvisibleRemoveText(oSourceControl);
+				}
+
 				this._bPendingChange = true; // TODO: handle multiple changes
 				return;
 			} else {
-				fnHandleChange();
+				fnHandleChange(undefined, oSourceControl);
 			}
 
 		},
@@ -576,8 +594,7 @@ sap.ui.define([
 								bInvalid = true;
 								oCondition.invalid = true;
 								bUpdate = true;
-								oField.setValueState(ValueState.Error);
-								oField.setValueStateText(oMessageBundle.getText("field.CONDITION_ALREADY_EXIST", [sKey]));
+								_updateErrorValueState(oField, oMessageBundle.getText("field.CONDITION_ALREADY_EXIST", [sKey]));
 								break;
 							}
 						}
@@ -666,12 +683,8 @@ sap.ui.define([
 				const sConditionPath = oSource.getBindingContext("$condition").getPath(); // Path to condition of the active control
 				const iIndex = parseInt(sConditionPath.split("/")[2]); // index of current condition - to remove before adding new ones
 				const aConditions = this.getConditions();
-				const oFormatOptions = merge({}, this.getConfig());
-				oFormatOptions.display = FieldDisplay.Value;
-				oFormatOptions.getConditions = function() { return aConditions; }; // as condition where inserted will be removed
-				oFormatOptions.defaultOperatorName = aConditions[iIndex].operator; // use current operator as default
-				oFormatOptions.valueType = oFormatOptions.dataType;
-				delete oFormatOptions.dataType;
+
+				const oFormatOptions = _adjustFormatOptions(this.getConfig(), aConditions, iIndex);
 				const oConditionsType = new ConditionsType(oFormatOptions);
 
 				aConditions.splice(iIndex, 1); // remove old condition that is overwitten by pasting
@@ -745,7 +758,11 @@ sap.ui.define([
 			return oCtrl;
 		},
 
-		// TODO: remove this function and replace by getValueHelp once FieldHelp association is completetly removed.
+		/**
+		 * @deprecated As of 1.114.0, together with `fieldHelp` association
+		 * @ui5-transform-hint replace-call getValueHelp
+		 * @private
+		 */
 		_getValueHelp: function() {
 
 			return this.getValueHelp() || (this.getFieldHelp && this.getFieldHelp()); // as getFieldHelp not exist in legacy-free UI5
@@ -753,6 +770,40 @@ sap.ui.define([
 		}
 	});
 
+	function _adjustFormatOptions(oOrigFormatOptions, aConditions, iCurrentCondition) {
+		const oCurrentCondition = aConditions[iCurrentCondition];
+
+		const oFormatOptions = merge({}, oOrigFormatOptions);
+		oFormatOptions.display = FieldDisplay.Value;
+		oFormatOptions.getConditions = function() { return aConditions; }; // as condition where inserted will be removed
+		oFormatOptions.defaultOperatorName = oCurrentCondition?.operator; // use current operator as default
+		oFormatOptions.valueType = oFormatOptions.dataType;
+		delete oFormatOptions.dataType;
+
+		return oFormatOptions;
+	}
+	// resetting the InvisibleText is necessary because conditions are updated async.
+	// the InvisibleText will be updated after the value is changed.
+	// and reading the old value again
+	function _resetInvisibleRemoveText(oField) {
+		const oInvisibleText = _getInvisibleTextForField(oField);
+		if (oInvisibleText) {
+			oInvisibleText.setText("");
+		}
+	}
+
+
+	function _getInvisibleTextForField(oField) {
+		if (!oField) {
+			return null;
+		}
+
+		const oGrid = oField.getParent();
+		const oFieldIndex = oGrid.getContent().indexOf(oField);
+		const oInvisibleText = oGrid.getContent().find((oControl, iIndex) => oControl.isA("sap.ui.core.InvisibleText") && iIndex > oFieldIndex);
+		return oInvisibleText;
+	}
+
 	function _observeChanges(oChanges) {
 
 		if (oChanges.name === "value") {
@@ -780,6 +831,10 @@ sap.ui.define([
 			const oFormatOptionsOld = oTypeOld && oTypeOld.getFormatOptions();
 			const oConstraints = oType && oType.getConstraints();
 			const oConstraintsOld = oTypeOld && oTypeOld.getConstraints();
+
+			const oCopiedFormatOptions = _adjustFormatOptions(oChanges.current, aConditions, -1);
+			this._oConditionType.setFormatOptions(oCopiedFormatOptions);
+
 			if (sType !== sTypeOld || !deepEqual(oFormatOptions, oFormatOptionsOld) || !deepEqual(oConstraints, oConstraintsOld)) {
 				// operators might be changed if type changed
 				// Field binding needs to be updated if type changed
@@ -822,6 +877,9 @@ sap.ui.define([
 		// return aValueHelpSupportedOperators.length === 0 || aValueHelpSupportedOperators.indexOf(sKey) >= 0;
 	}
 
+	/**
+	 * @this {sap.ui.mdc.valuehelp.base.DefineConditionPanel}
+	 */
 	function _operatorChanged(oField, sKey, sOldKey) {
 		oField._sOldKey = sOldKey; // to know in change event
 
@@ -908,9 +966,12 @@ sap.ui.define([
 
 		oField.setAdditionalValue(sDescription);
 
-		this.onChange();
+		this.onChange(undefined);
 	}
 
+	/**
+	 * @this {sap.ui.mdc.valuehelp.base.DefineConditionPanel}
+	 */
 	function _createControl(oCondition, iIndex, sId, oBindingContext) {
 
 		const oOperator = FilterOperatorUtil.getOperator(oCondition.operator);
@@ -1545,11 +1606,26 @@ sap.ui.define([
 		oGrid.insertContent(oOperatorField, iIndex); // insert as add-Button is already at the end
 		iIndex++;
 
+		const oInvisibleRemoveText = new InvisibleText(sIdPrefix + "--ivtRemove", {
+			text: {
+				parts: [{ path: "$this>" }],
+				formatter: _getRemoveTextForCondition.bind(this)
+			}
+		}).setLayoutData(new GridData({
+			span: "XL1 L1 M1 S1",
+			visibleS: false,
+			visibleM: false,
+			visibleL: false,
+			visibleXL: false
+		}))
+		.setBindingContext(oBindingContext, "$this"); // to find condition on remove
+
 		const oRemoveButton = new Button(sIdPrefix + "--removeBtnSmall", {
 				press: this.removeCondition.bind(this),
 				type: ButtonType.Transparent,
 				icon: "sap-icon://decline",
-				tooltip: "{$i18n>valuehelp.DEFINECONDITIONS_REMOVECONDITION}"
+				tooltip: "{$i18n>valuehelp.DEFINECONDITIONS_REMOVECONDITION}",
+				ariaLabelledBy: oInvisibleRemoveText
 			})
 			.setLayoutData(new GridData({
 				span: "XL1 L1 M1 S2",
@@ -1560,6 +1636,7 @@ sap.ui.define([
 				visibleXL: false
 			}))
 			.setBindingContext(oBindingContext, "$this"); // to find condition on remove
+
 		if (oBindingContext) {
 			// as Button is between Operatot and Value don't trigger validation on tabbing between
 			oRemoveButton.setFieldGroupIds([oBindingContext.getPath()]); // use path to have a ID for every condition
@@ -1582,7 +1659,8 @@ sap.ui.define([
 				press: this.removeCondition.bind(this),
 				type: ButtonType.Transparent,
 				icon: "sap-icon://decline",
-				tooltip: "{$i18n>valuehelp.DEFINECONDITIONS_REMOVECONDITION}"
+				tooltip: "{$i18n>valuehelp.DEFINECONDITIONS_REMOVECONDITION}",
+				ariaLabelledBy: oInvisibleRemoveText
 			})
 			.setLayoutData(new GridData({
 				span: "XL1 L1 M1 S1",
@@ -1594,13 +1672,30 @@ sap.ui.define([
 			}))
 			.setBindingContext(oBindingContext, "$this"); // to find condition on remove
 
-		oGrid.insertContent(oRemoveButton2, iIndex);
-		iIndex++;
+			oGrid.insertContent(oRemoveButton2, iIndex);
+			iIndex++;
+			oGrid.insertContent(oInvisibleRemoveText, iIndex);
+			iIndex++;
 
 		return iIndex;
 
 	}
 
+	function _getRemoveTextForCondition(oCondition) {
+		const sRemoveUndefinedConditionText = oMessageBundle.getText("valuehelp.DEFINECONDITIONS_REMOVECONDITION_ARIALABEL_UNDEFINED");
+		const sFormatedValue = this._oConditionType.formatValue(oCondition);
+		if (!oCondition) { // can only happen on delete
+			return "";
+		}
+
+		if (oCondition.isEmpty || !sFormatedValue) {
+			return sRemoveUndefinedConditionText;
+		}
+
+		const sRemoveConditionText = oMessageBundle.getText("valuehelp.DEFINECONDITIONS_REMOVECONDITION_ARIALABEL", [sFormatedValue]);
+		return sRemoveConditionText;
+	}
+
 	function _getEditModeFromOperator(sOperator, bInvalid) {
 
 		if (!sOperator) {
@@ -1689,6 +1784,9 @@ sap.ui.define([
 		}
 	}
 
+	/**
+	 * @this {sap.ui.mdc.valuehelp.base.DefineConditionPanel}
+	 */
 	function _updateRow(oCondition, oGrid, iIndex, oBindingContext, iRow) {
 
 		const sIdPrefix = this.getId() + "--" + iRow;
@@ -1778,6 +1876,10 @@ sap.ui.define([
 		oRemoveButton2.setBindingContext(oBindingContext, "$this");
 		iIndex++;
 
+		const oInvisibleText = aGridContent[iIndex];
+		oInvisibleText.setBindingContext(oBindingContext, "$this");
+		iIndex++;
+
 		return iIndex;
 
 	}
@@ -1800,6 +1902,22 @@ sap.ui.define([
 
 	}
 
+	function _updateErrorValueState(oField, sMessage) {
+		oField.setValueState(ValueState.Error);
+		oField.setValueStateText(sMessage);
+		// update aria-label text
+		_setInvalidInvisibleText(oField);
+	}
+
+	function _setInvalidInvisibleText(oField) {
+		const sInvalidMessage = oMessageBundle.getText("valuehelp.DEFINECONDITIONS_REMOVECONDITION_ARIALABEL_INVALID");
+		const oInvisibleText = _getInvisibleTextForField(oField);
+		if (oInvisibleText) {
+			oInvisibleText.setText(sInvalidMessage);
+		}
+	}
+
+
 	function _validateCondition(oField) {
 
 		const oGrid = oField.getParent();
@@ -1849,11 +1967,9 @@ sap.ui.define([
 					}
 				} catch (oException) {
 					bInvalid = true;
-					oField.setValueState(ValueState.Error);
-					oField.setValueStateText(oException.message);
+					_updateErrorValueState(oField, oException.message);
 					if (oField2 && oField2.getMetadata().getAllProperties().valueState) {
-						oField2.setValueState(ValueState.Error);
-						oField2.setValueStateText(oException.message);
+						_updateErrorValueState(oField2, oException.message);
 					}
 				}
 			}
@@ -1866,11 +1982,9 @@ sap.ui.define([
 				for (let i = 0; i < aConditions.length; i++) {
 					if (i !== iIndex && !oCondition.isEmpty && FilterOperatorUtil.compareConditions(oCondition, aConditions[i])) {
 						bInvalid = true;
-						oField.setValueState(ValueState.Error);
-						oField.setValueStateText(oMessageBundle.getText("field.CONDITION_ALREADY_EXIST", [oCondition.values[0]]));
+						_updateErrorValueState(oField, oMessageBundle.getText("field.CONDITION_ALREADY_EXIST", [oCondition.values[0]]));
 						if (oField2 && oField2.getMetadata().getAllProperties().valueState) {
-							oField2.setValueState(ValueState.Error);
-							oField2.setValueStateText(oMessageBundle.getText("field.CONDITION_ALREADY_EXIST", [oCondition.values[1]]));
+							_updateErrorValueState(oField2, oMessageBundle.getText("field.CONDITION_ALREADY_EXIST", [oCondition.values[1]]));
 						}
 						break;
 					}

package/src/sap/ui/mdc/valuehelp/base/DialogTab.js

@@ -20,7 +20,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new control
 	 * @class Content for the <code>sap.ui.mdc.valuehelp.content.Dialog</code> element.
 	 * @extends sap.ui.core.Control
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 * @constructor
 	 * @private
 	 * @ui5-restricted sap.ui.mdc

package/src/sap/ui/mdc/valuehelp/base/FilterableListContent.js

@@ -44,7 +44,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new element
 	 * @class Content for the {@link sap.ui.mdc.valuehelp.base.Container Container} element.
 	 * @extends sap.ui.mdc.valuehelp.base.ListContent
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 * @constructor
 	 * @abstract
 	 * @public
@@ -227,7 +227,9 @@ sap.ui.define([
 	FilterableListContent.prototype._findConditionsForContext = function(oContext, aConditions) {
 		const oDelegate = this.isValueHelpDelegateInitialized() && this.getValueHelpDelegate();
 		if (oContext && oDelegate) {
-			// <!-- Support for deprecated delegate method isFilterableListItemSelected
+			/**
+			 *  @deprecated since 1.118.0
+			 */
 			if (oDelegate.isFilterableListItemSelected) {
 				Log.warning("MDC.ValueHelp", "Delegate method 'isFilterableListItemSelected' is deprecated, please implement 'findConditionsForContext' instead.");
 
@@ -247,7 +249,6 @@ sap.ui.define([
 				}
 				return [];
 			}
-			// -->
 			return oDelegate.findConditionsForContext(this.getValueHelpInstance(), this, oContext, aConditions);
 		}
 		return [];

package/src/sap/ui/mdc/valuehelp/base/ListContent.js

@@ -21,7 +21,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new element
 	 * @class Content for the {@link sap.ui.mdc.valuehelp.base.Container Container} element.
 	 * @extends sap.ui.mdc.valuehelp.base.Content
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 * @constructor
 	 * @abstract
 	 * @public

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

@@ -20,7 +20,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new element
 	 * @class Content for the {@link sap.ui.mdc.valuehelp.base.Container Container} element to provide a value help for boolean fields.
 	 * @extends sap.ui.mdc.valuehelp.content.FixedList
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 * @constructor
 	 * @private
 	 * @ui5-restricted sap.ui.mdc

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

@@ -27,7 +27,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new element
 	 * @class Content for the {@link sap.ui.mdc.valuehelp.base.Container Container} element showing a condition panel.
 	 * @extends sap.ui.mdc.valuehelp.base.Content
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 * @constructor
 	 * @public
 	 * @since 1.95.0

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

@@ -34,7 +34,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new element
 	 * @class Content for the {@link sap.ui.mdc.valuehelp.base.Container Container} element showing a list with fixed values.
 	 * @extends sap.ui.mdc.valuehelp.base.ListContent
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 * @constructor
 	 * @public
 	 * @since 1.95.0

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

@@ -23,7 +23,7 @@ sap.ui.define([
 	 * @extends sap.ui.core.ListItem
 	 *
 	 * @author SAP SE
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 *
 	 * @public
 	 * @since 1.114.0

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

@@ -42,7 +42,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new element
 	 * @class Content for the {@link sap.ui.mdc.valuehelp.base.Container Container} element using a {@link sap.ui.mdc.Table}.
 	 * @extends sap.ui.mdc.valuehelp.base.FilterableListContent
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 * @constructor
 	 * @public
 	 * @since 1.95.0

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

@@ -22,7 +22,8 @@ sap.ui.define([
 	'sap/ui/core/Element',
 	'sap/ui/Device',
 	'sap/ui/dom/containsOrEquals',
-	"sap/m/table/Util"
+	"sap/m/table/Util",
+	"sap/ui/events/KeyCodes"
 ], (
 	Library,
 	FilterableListContent,
@@ -41,7 +42,8 @@ sap.ui.define([
 	Element,
 	Device,
 	containsOrEquals,
-	MTableUtil
+	MTableUtil,
+	KeyCodes
 ) => {
 	"use strict";
 
@@ -55,7 +57,7 @@ sap.ui.define([
 	 * @param {object} [mSettings] Initial settings for the new element
 	 * @class Content for the {@link sap.ui.mdc.valuehelp.base.Container Container} element using a {@link sap.m.Table}.
 	 * @extends sap.ui.mdc.valuehelp.base.FilterableListContent
-	 * @version 1.141.2
+	 * @version 1.143.0
 	 * @constructor
 	 * @public
 	 * @since 1.95.0
@@ -1187,7 +1189,7 @@ sap.ui.define([
 
 		switch (oEvent.type) {
 			case "sapprevious":
-				if (oItem.isA("sap.m.ListItemBase")) {
+				if (oEvent.which !== KeyCodes.ARROW_LEFT && oItem.isA("sap.m.ListItemBase")) {
 					if (oTable.indexOfItem(oItem) === 0) {
 						this.fireNavigated({ condition: undefined, itemId: undefined, leaveFocus: true });
 						oEvent.preventDefault();
@@ -1197,7 +1199,7 @@ sap.ui.define([
 				}
 				break;
 			case "sapnext":
-				if (oItem.isA("sap.m.ListItemBase") && this._oShowAllItemsButton) {
+				if (oEvent.which !== KeyCodes.ARROW_RIGHT && oItem.isA("sap.m.ListItemBase") && this._oShowAllItemsButton) {
 					const aItems = oTable.getItems();
 					if (aItems.indexOf(oItem) === aItems.length - 1) { // end reached, focus show-all to behave similar to single-select
 						oEvent.preventDefault();

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

@@ -533,6 +533,42 @@ sap.ui.define([
 				}
 			});
 		},
+		iChangeFilterFieldSelectionInAdaptFiltersPanel: function(sName, bSelect) {
+			return this.waitFor({
+				controlType: "sap.ui.mdc.p13n.panels.AdaptFiltersPanel",
+				success:function(aAdaptFiltersPanel) {
+					Opa5.assert.ok(true, "Adapt Filters Panel found");
+						return this.waitFor({
+							controlType: "sap.m.CustomListItem",
+							matchers: {
+								ancestor: {
+									controlType: "sap.m.Panel",
+									visible: true
+								}
+							},
+							success: function(aItems) {
+								let bFound = false;
+								let oItemToSelect;
+								aItems.forEach(function(oItem) {
+									if (oItem.getContent()[0].getItems()[0].getText() === sName) {
+										bFound = true;
+										oItemToSelect = oItem;
+									}
+								});
+								if (bFound) {
+									//here we only select/deselect the Checkbox - not testing the CheckBox in the item
+									oItemToSelect.setSelected(bSelect);
+									Opa5.assert.ok(bFound, "FilterField " + sName + " selected" );
+								} else {
+									Opa5.assert.notOk(!bFound, "No FilterField with label" + sName + " found" );
+								}
+							}
+						});
+				},
+				errorMessage: "No Adapt Filters Panel found"
+			});
+
+		},
 
 		iPressOnTheAdaptFiltersButton: function() {
 			return waitForAdaptFiltersButton.call(this, {

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

@@ -8,14 +8,18 @@ sap.ui.define([
 	"./waitForFilterField",
 	"sap/ui/events/KeyCodes",
 	"../Utils",
-	"../actions/TriggerEvent"
+	"../actions/TriggerEvent",
+	"sap/ui/test/matchers/PropertyStrictEquals",
+	"sap/ui/test/actions/Press"
 ], function(
 	Opa5,
 	EnterText,
 	waitForFilterField,
 	KeyCodes,
 	Utils,
-	TriggerEvent
+	TriggerEvent,
+	PropertyStrictEquals,
+	Press
 ) {
     "use strict";
 
@@ -78,7 +82,20 @@ sap.ui.define([
 					}));
 				}
 			}));
-        }
+        },
+		iSelectDisplayListItem: function(sValue) {
+			return this.waitFor({
+				controlType: "sap.m.DisplayListItem",
+				matchers: new PropertyStrictEquals({
+				name: "label",
+				value: sValue
+				}),
+				success: function(oDisplayListItems) {
+					new Press().executeOn(oDisplayListItems[0]);
+				},
+				errorMessage: "The value [" + sValue + "] was not found in the ValueHelp"
+			});
+		}
 	};
 
 	return oActions;

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

@@ -79,7 +79,13 @@ sap.ui.define([
                     var oMatcher = new Matcher();
                     oMatcher.isMatching = function(oListItem) {
                         return !!oListItem.getCells().find(function (oCell) {
-                            return sText === oCell.mProperties["text"];
+                            const oMetadata = oCell.getMetadata();
+                            if (oMetadata.hasProperty("value")) {
+                                return sText === oCell.getProperty("value");
+                            } else if (oMetadata.hasProperty("text")) {
+                                return sText === oCell.getProperty("text");
+                            }
+                            return false;
                         });
                     };
 
@@ -316,6 +322,32 @@ sap.ui.define([
                 },
                 errorMessage: "Expand/Hide Filters Button not found."
             });
+        },
+        iSelectSearchTemplate: function (sTemplateName, sValueHelpId) {
+            return doWait(this).forValueHelpDialog({
+                success: function(oDialog) {
+                    return this.waitFor({
+                        controlType: "sap.ui.mdc.valuehelp.CollectiveSearchSelect",
+                        success: function (aSelects) {
+                            new Press().executeOn(aSelects[0]);
+                            return this.waitFor({
+                                controlType: "sap.ui.core.Item",
+                                matchers: [
+                                    new Ancestor(aSelects[0]),
+                                    new PropertyStrictEquals({
+                                        name: "text",
+                                        value: sTemplateName
+                                    })
+                                ],
+                                success: function (aItems) {
+                                    new Press().executeOn(aItems[0]);
+                                    Opa5.assert.ok(aItems[0], "Selected Search Template " + sTemplateName);
+                                }
+                            });
+                        }
+                    });
+                }
+            }, sValueHelpId);
         }
     };