Ensure picker search config query params are always included in the server request (#19194)

* ensure picker search config query params are always included

* fix type error
This commit is contained in:
Mads Rasmussen
2025-04-30 09:58:17 +02:00
committed by GitHub
parent 0a7d00182b
commit 2feef9c5b7
4 changed files with 7 additions and 11 deletions

View File

@@ -10,9 +10,9 @@ export interface UmbPickerModalData<ItemType> {
search?: UmbPickerModalSearchConfig;
}
export interface UmbPickerModalSearchConfig {
export interface UmbPickerModalSearchConfig<QueryParamsType = Record<string, unknown>> {
providerAlias: string;
queryParams?: object;
queryParams?: QueryParamsType;
}
export interface UmbPickerModalValue {

View File

@@ -183,8 +183,10 @@ export class UmbPickerSearchManager<
}
const args = {
searchFrom: this.#config?.searchFrom,
...query,
// ensure that config params are always included
...this.#config?.queryParams,
searchFrom: this.#config?.searchFrom,
};
const { data } = await this.#searchProvider.search(args);

View File

@@ -1,6 +1,7 @@
import type { UmbEntityModel } from '@umbraco-cms/backoffice/entity';
export interface UmbPickerSearchManagerConfig {
export interface UmbPickerSearchManagerConfig<QueryParamsType = Record<string, unknown>> {
providerAlias: string;
searchFrom?: UmbEntityModel;
queryParams?: QueryParamsType;
}

View File

@@ -58,13 +58,6 @@ export class UmbTreePickerModalElement<TreeItemType extends UmbTreeItemModelBase
...this.data.search,
searchFrom: this.data.startNode,
});
const searchQueryParams = this.data.search.queryParams;
if (searchQueryParams) {
// eslint-disable-next-line @typescript-eslint/ban-ts-comment
//@ts-ignore - TODO wire up types
this.#pickerContext.search.setQuery(searchQueryParams);
}
}
const multiple = this.data?.multiple ?? false;