yv01p/Umbraco-CMS
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

split into method

Browse Source
This commit is contained in:
Mads Rasmussen
2024-08-13 14:14:47 +02:00
parent d4be18e62f
commit cd7231fc2b
1 changed files with 36 additions and 32 deletions
Download Patch File Download Diff File Expand all files Collapse all files

68
src/Umbraco.Web.UI.Client/src/packages/core/tree/tree-picker/tree-picker-modal.element.ts
View File

@@ -57,38 +57,7 @@ export class UmbTreePickerModalElement<TreeItemType extends UmbTreeItemModelBase
// Same here [NL]
this._selectionConfiguration.multiple = this.data?.multiple ?? false;
// TODO: If data.enableCreate is true, we should add a button to create a new item. [NL]
// Does the tree know enough about this, for us to be able to create a new item? [NL]
// I think we need to be able to get entityType and a parentId?, or do we only allow creation in the root? and then create via entity actions? [NL]
// To remove the hardcoded URLs for workspaces of entity types, we could make an create event from the tree, which either this or the sidebar impl. will pick up and react to. [NL]
// Or maybe the tree item context base can handle this? [NL]
// Maybe its a general item context problem to be solved. [NL]
const createAction = this.data?.createAction;
if (createAction) {
this._createLabel = createAction.label;
new UmbModalRouteRegistrationController(
this,
(createAction.modalToken as typeof UMB_WORKSPACE_MODAL) ?? UMB_WORKSPACE_MODAL,
)
.onSetup(() => {
return { data: createAction.modalData };
})
.onSubmit((value) => {
if (value) {
this.value = { selection: [value.unique] };
this._submitModal();
} else {
this._rejectModal();
}
})
.observeRouteBuilder((routeBuilder) => {
const oldPath = this._createPath;
this._createPath =
routeBuilder({}) + createAction.extendWithPathPattern.generateLocal(createAction.extendWithPathParams);
this.requestUpdate('_createPath', oldPath);
});
}
this.#initCreateAction();
this.#initSearch();
}
@@ -155,6 +124,41 @@ export class UmbTreePickerModalElement<TreeItemType extends UmbTreeItemModelBase
this._searchResult = [];
}
// Create action
#initCreateAction() {
// TODO: If data.enableCreate is true, we should add a button to create a new item. [NL]
// Does the tree know enough about this, for us to be able to create a new item? [NL]
// I think we need to be able to get entityType and a parentId?, or do we only allow creation in the root? and then create via entity actions? [NL]
// To remove the hardcoded URLs for workspaces of entity types, we could make an create event from the tree, which either this or the sidebar impl. will pick up and react to. [NL]
// Or maybe the tree item context base can handle this? [NL]
// Maybe its a general item context problem to be solved. [NL]
const createAction = this.data?.createAction;
if (createAction) {
this._createLabel = createAction.label;
new UmbModalRouteRegistrationController(
this,
(createAction.modalToken as typeof UMB_WORKSPACE_MODAL) ?? UMB_WORKSPACE_MODAL,
)
.onSetup(() => {
return { data: createAction.modalData };
})
.onSubmit((value) => {
if (value) {
this.value = { selection: [value.unique] };
this._submitModal();
} else {
this._rejectModal();
}
})
.observeRouteBuilder((routeBuilder) => {
const oldPath = this._createPath;
this._createPath =
routeBuilder({}) + createAction.extendWithPathPattern.generateLocal(createAction.extendWithPathParams);
this.requestUpdate('_createPath', oldPath);
});
}
}
override render() {
return html`
<umb-body-layout headline="Select">