@sapui5/sap.fe.macros 1.142.0 → 1.143.0

package/src/sap/fe/macros/filterBar/FilterBarDelegate.ts

@@ -39,6 +39,7 @@ import DelegateUtil from "sap/fe/macros/DelegateUtil";
 import FieldHelper from "sap/fe/macros/field/FieldHelper";
 import type { IFilterControl } from "sap/fe/macros/filter/FilterUtils";
 import FilterUtils from "sap/fe/macros/filter/FilterUtils";
+import UOMValidationDelegate from "sap/fe/macros/filterBar/UOMValidationDelegate";
 import type { PropertyInfo } from "sap/fe/macros/internal/PropertyInfo";
 import { getValueHelpTemplate } from "sap/fe/macros/internal/valuehelp/ValueHelpTemplating";
 import type { ControlPropertyInfo } from "sap/fe/macros/mdc/adapter/StateHelper";
@@ -60,61 +61,62 @@ import type MacroAPI from "../MacroAPI";
 
 type ModifierElement = UI5Element | UI5Element[] | Element | Element[];
 type NullableModifierElement = void | ModifierElement | null;
-const ODataFilterBarDelegate = Object.assign({}, FilterBarDelegate) as typeof FilterBarDelegate & {
-	apiVersion: number;
-	_isFilterInSelectionFields(selectionFields: SelectionFields, filterFieldInfo: FilterField): boolean;
-	_findSelectionField(aSelectionFields: FilterField[], sFlexName: string): FilterField | undefined;
-	clearFilters(oFilterControl: FilterBar): Promise<void>;
-	getTypeMap(): typeof TypeMap;
-	fetchFilterProperties(filterBar: IFilterControl): Promise<ControlPropertyInfo[] | PropertyInfoExternal[]>;
-	fetchProperties(filterBar: FilterBar): Promise<PropertyInfoExternal[]>;
-	_isFilterAdaptable(
-		filterFieldInfo: FilterField,
-		propertyAnnotations: Property,
-		selectionFields?: SelectionFields,
-		filterFacets?: FilterFacets
-	): boolean;
-	fetchPropertiesForEntity(sEntityTypePath: string, oMetaModel: ODataMetaModel, oFilterControl: IFilterControl): PropertyInfo[];
-	_templateCustomFilter(
-		oFilterBar: IFilterControl,
-		sIdPrefix: string,
-		oSelectionFieldInfo: FilterField,
-		oMetaModel: ODataMetaModel,
-		oModifier: BaseTreeModifier
-	): Promise<ModifierElement>;
-	_addP13nItem(sPropertyInfoName: string, oParentControl: FilterBar): Promise<NullableModifierElement>;
-	_addFlexItem(
-		sFlexPropertyName: string,
-		oParentControl: FilterBar,
-		oMetaModel: ODataMetaModel,
-		oModifier: BaseTreeModifier | undefined,
-		oAppComponent?: AppComponent,
-		view?: View
-	): Promise<NullableModifierElement>;
-	addItem(
-		oParentControl: FilterBar,
-		sPropertyInfoName: string,
-		mPropertyBag?: { modifier: BaseTreeModifier; appComponent: AppComponent }
-	): Promise<NullableModifierElement>;
-	addCondition(
-		oParentControl: FilterBar,
-		sPropertyInfoName: string,
-		mPropertyBag: { appComponent: AppComponent; modifier: BaseTreeModifier }
-	): Promise<void | null>;
-	removeItem(
-		oParentControl: FilterBar,
-		oFilterFieldProperty: UI5Element | Element,
-		mPropertyBag: { appComponent: AppComponent; modifier: BaseTreeModifier }
-	): Promise<boolean | null>;
-	removeCondition(
-		oParentControl: FilterBar,
-		sPropertyInfoName: string,
-		mPropertyBag: {
-			appComponent: AppComponent;
-			modifier: BaseTreeModifier;
-		}
-	): Promise<void | null>;
-};
+const ODataFilterBarDelegate = Object.assign({}, FilterBarDelegate, UOMValidationDelegate) as typeof FilterBarDelegate &
+	typeof UOMValidationDelegate & {
+		apiVersion: number;
+		_isFilterInSelectionFields(selectionFields: SelectionFields, filterFieldInfo: FilterField): boolean;
+		_findSelectionField(aSelectionFields: FilterField[], sFlexName: string): FilterField | undefined;
+		clearFilters(oFilterControl: FilterBar): Promise<void>;
+		getTypeMap(): typeof TypeMap;
+		fetchFilterProperties(filterBar: IFilterControl): Promise<ControlPropertyInfo[] | PropertyInfoExternal[]>;
+		fetchProperties(filterBar: FilterBar): Promise<PropertyInfoExternal[]>;
+		_isFilterAdaptable(
+			filterFieldInfo: FilterField,
+			propertyAnnotations: Property,
+			selectionFields?: SelectionFields,
+			filterFacets?: FilterFacets
+		): boolean;
+		fetchPropertiesForEntity(sEntityTypePath: string, oMetaModel: ODataMetaModel, oFilterControl: IFilterControl): PropertyInfo[];
+		_templateCustomFilter(
+			oFilterBar: IFilterControl,
+			sIdPrefix: string,
+			oSelectionFieldInfo: FilterField,
+			oMetaModel: ODataMetaModel,
+			oModifier: BaseTreeModifier
+		): Promise<ModifierElement>;
+		_addP13nItem(sPropertyInfoName: string, oParentControl: FilterBar): Promise<NullableModifierElement>;
+		_addFlexItem(
+			sFlexPropertyName: string,
+			oParentControl: FilterBar,
+			oMetaModel: ODataMetaModel,
+			oModifier: BaseTreeModifier | undefined,
+			oAppComponent?: AppComponent,
+			view?: View
+		): Promise<NullableModifierElement>;
+		addItem(
+			oParentControl: FilterBar,
+			sPropertyInfoName: string,
+			mPropertyBag?: { modifier: BaseTreeModifier; appComponent: AppComponent }
+		): Promise<NullableModifierElement>;
+		addCondition(
+			oParentControl: FilterBar,
+			sPropertyInfoName: string,
+			mPropertyBag: { appComponent: AppComponent; modifier: BaseTreeModifier }
+		): Promise<void | null>;
+		removeItem(
+			oParentControl: FilterBar,
+			oFilterFieldProperty: UI5Element | Element,
+			mPropertyBag: { appComponent: AppComponent; modifier: BaseTreeModifier }
+		): Promise<boolean | null>;
+		removeCondition(
+			oParentControl: FilterBar,
+			sPropertyInfoName: string,
+			mPropertyBag: {
+				appComponent: AppComponent;
+				modifier: BaseTreeModifier;
+			}
+		): Promise<void | null>;
+	};
 ODataFilterBarDelegate.apiVersion = 2;
 const EDIT_STATE_PROPERTY_NAME = "$editState",
 	SEARCH_PROPERTY_NAME = "$search",
@@ -420,8 +422,9 @@ async function _addPropertyInfo(
 
 		const delegate = await mPropertyBag.modifier.getProperty<FilterBarDelegate>(oParentControl, "delegate");
 		const aPropertyInfo = await mPropertyBag.modifier.getProperty<FEPropertyInfo[]>(oParentControl, "propertyInfo");
+		const propertyInfoExists = await FilterUtils._checkIfPropertyInfoExists(oParentControl, mPropertyBag.modifier, sPropertyInfoName);
 		//We do not get propertyInfo in case of table filters
-		if (aPropertyInfo) {
+		if (aPropertyInfo && !propertyInfoExists) {
 			const hasPropertyInfo = aPropertyInfo.some(function (prop: FEPropertyInfo) {
 				return prop.key === sPropertyInfoKey || prop.name === sPropertyInfoKey;
 			});
@@ -479,12 +482,13 @@ async function _updatePropertyInfo(
 	propertyBag.modifier.setProperty(parentControl, "propertyInfo", _propertyInfo);
 }
 /**
- * Method responsible for creating filter field in standalone mode / in the personalization settings of the filter bar.
+ * Method responsible for creating the filter field in standalone mode and in the personalization settings of the filter bar.
  * @param oParentControl Parent control instance to which the filter field is added
  * @param sPropertyInfoName Name of the property being added as the filter field
  * @param mPropertyBag Instance of the property bag from Flex API
  * @param mPropertyBag.appComponent AppComponent
  * @param mPropertyBag.modifier Modifier from Flex API
+ * @param mPropertyBag.view Instance of the view
  * @returns Once resolved, a filter field definition is returned
  */
 ODataFilterBarDelegate.addItem = async function (
@@ -503,7 +507,8 @@ ODataFilterBarDelegate.addItem = async function (
 		return Promise.resolve(null);
 	}
 	const isXML = modifier && modifier.targets === "xmlTree";
-	if (isXML) {
+	const propertyInfoExists = isXML ? await FilterUtils._checkIfPropertyInfoExists(oParentControl, modifier, sPropertyInfoName) : true;
+	if (isXML && !propertyInfoExists) {
 		await _addPropertyInfo(oParentControl, mPropertyBag, oMetaModel, sPropertyInfoName);
 	}
 	return ODataFilterBarDelegate._addFlexItem(
@@ -540,6 +545,7 @@ ODataFilterBarDelegate.removeItem = async function (
 			return Promise.resolve(null);
 		}
 		const filterFieldProperty = await mPropertyBag.modifier.getProperty(oFilterFieldProperty, "propertyKey");
+
 		await _addPropertyInfo(oParentControl, mPropertyBag, oMetaModel, filterFieldProperty as string);
 	}
 	if (
@@ -864,6 +870,14 @@ ODataFilterBarDelegate._addFlexItem = async function (
 	oAppComponent: AppComponent | undefined,
 	view?: View | undefined
 ): Promise<NullableModifierElement> {
+	let propertyInfosFromFilterBar: PropertyInfo[] = [];
+	const bIsXML = !!oModifier && oModifier.targets === "xmlTree";
+	if (bIsXML) {
+		const customDataValue = await DelegateUtil.getCustomDataWithModifier<string>(oParentControl, "feFilterInfo", oModifier);
+		if (customDataValue) {
+			propertyInfosFromFilterBar = JSON.parse(customDataValue) as PropertyInfo[]; // retrieve pre-fetched filter info from custom data, required during templating
+		}
+	}
 	const sFilterBarId = oModifier ? oModifier.getId(oParentControl) : oParentControl.getId(),
 		sIdPrefix = oModifier ? "" : "Adaptation",
 		aSelectionFields = FilterUtils.getConvertedFilterFields(
@@ -873,11 +887,12 @@ ODataFilterBarDelegate._addFlexItem = async function (
 			oMetaModel,
 			oAppComponent,
 			oModifier,
-			oModifier ? undefined : getLineItemQualifierFromTable(oParentControl.getParent()!, oMetaModel)
+			oModifier ? undefined : getLineItemQualifierFromTable(oParentControl.getParent()!, oMetaModel),
+			propertyInfosFromFilterBar
 		),
 		oSelectionField = ODataFilterBarDelegate._findSelectionField(aSelectionFields, sFlexPropertyName),
-		sPropertyPath = _getPropertyPath(sFlexPropertyName),
-		bIsXML = !!oModifier && oModifier.targets === "xmlTree";
+		sPropertyPath = _getPropertyPath(sFlexPropertyName);
+
 	if (sFlexPropertyName === EDIT_STATE_PROPERTY_NAME) {
 		return _templateEditState(_generateIdPrefix(sFilterBarId, `${sIdPrefix}FilterField`), oMetaModel, oModifier);
 	} else if (sFlexPropertyName === SEARCH_PROPERTY_NAME) {