From 857f2900bb389341540434a45635c4b66f00d71c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Niels=20Lyngs=C3=B8?= Date: Tue, 2 Dec 2025 15:00:25 +0100 Subject: [PATCH] Segments: Fix for processing data for Segments-variants (#21018) * refactor to load segments before processing incoming data * clean up * remove unused segment promise --- .../content-type-workspace-context-base.ts | 1 - .../content-detail-workspace-base.ts | 45 +++++++++++-------- .../entity-detail-workspace-base.ts | 5 +-- .../workspace/document-workspace.context.ts | 26 +++++------ 4 files changed, 38 insertions(+), 39 deletions(-) diff --git a/src/Umbraco.Web.UI.Client/src/packages/content/content-type/workspace/content-type-workspace-context-base.ts b/src/Umbraco.Web.UI.Client/src/packages/content/content-type/workspace/content-type-workspace-context-base.ts index 67873bffb9..2d23f998dd 100644 --- a/src/Umbraco.Web.UI.Client/src/packages/content/content-type/workspace/content-type-workspace-context-base.ts +++ b/src/Umbraco.Web.UI.Client/src/packages/content/content-type/workspace/content-type-workspace-context-base.ts @@ -92,7 +92,6 @@ export abstract class UmbContentTypeWorkspaceContextBase< let { data } = await request; if (data) { - data = await this._processIncomingData(data); data = await this._scaffoldProcessData(data); if (this.modalContext) { diff --git a/src/Umbraco.Web.UI.Client/src/packages/content/content/workspace/content-detail-workspace-base.ts b/src/Umbraco.Web.UI.Client/src/packages/content/content/workspace/content-detail-workspace-base.ts index caa326e926..5abb4d4f7f 100644 --- a/src/Umbraco.Web.UI.Client/src/packages/content/content/workspace/content-detail-workspace-base.ts +++ b/src/Umbraco.Web.UI.Client/src/packages/content/content/workspace/content-detail-workspace-base.ts @@ -368,11 +368,6 @@ export abstract class UmbContentDetailWorkspaceContextBase< (varies) => { this._data.setVariesBySegment(varies); this.#variesBySegment = varies; - if (varies) { - this.loadSegments(); - } else { - this._segments.setValue([]); - } }, null, ); @@ -393,20 +388,28 @@ export abstract class UmbContentDetailWorkspaceContextBase< this.#languages.setValue(data?.items ?? []); } + /** + * @deprecated Call `_loadSegmentsFor` instead. `loadSegments` will be removed in v.18. + * (note this was introduced in v.17, and deprecated in v.17.0.1) + */ protected async loadSegments() { + console.warn('Stop using loadSegments, call _loadSegmentsFor instead. loadSegments will be removed in v.18.'); + const unique = await firstValueFrom(this.unique); + if (!unique) { + this._segments.setValue([]); + return; + } + this._loadSegmentsFor(unique); + } + + // eslint-disable-next-line @typescript-eslint/no-unused-vars + protected async _loadSegmentsFor(unique: string): Promise { console.warn( `UmbContentDetailWorkspaceContextBase: Segments are not implemented in the workspace context for "${this.getEntityType()}" types.`, ); this._segments.setValue([]); } - /** - * @deprecated Call `_processIncomingData` instead. `_scaffoldProcessData` will be removed in v.18. - */ - protected override _scaffoldProcessData(data: DetailModelType): Promise { - return this._processIncomingData(data); - } - protected override async _processIncomingData(data: DetailModelType): Promise { const contentTypeUnique: string | undefined = (data as any)[this.#contentTypePropertyName].unique; if (!contentTypeUnique) { @@ -415,24 +418,28 @@ export abstract class UmbContentDetailWorkspaceContextBase< // Load the content type structure, usually this comes from the data, but in this case we are making the data, and we need this to be able to complete the data. [NL] await this.structure.loadType(contentTypeUnique); + // Load segments if varying by segment, or reset to empty array: + if (this.#variesBySegment) { + await this._loadSegmentsFor(data.unique); + } else { + this._segments.setValue([]); + } + // Set culture and segment for all values: const cultures = this.#languages.getValue().map((x) => x.unique); - if (this.structure.variesBySegment) { - // TODO: v.17 Engage please note we have not implemented support for segments yet. [NL] - console.warn('Segments are not yet implemented for preset'); + let segments: Array | undefined; + if (this.#variesBySegment) { + segments = this._segments.getValue().map((s) => s.alias); } - // TODO: Add Segments for Presets: - const segments: Array | undefined = this.structure.variesBySegment ? [] : undefined; const repo = new UmbDataTypeDetailRepository(this); const propertyTypes = await this.structure.getContentTypeProperties(); const contentTypeVariesByCulture = this.structure.getVariesByCulture(); - const contentTypeVariesBySegment = this.structure.getVariesByCulture(); + const contentTypeVariesBySegment = this.structure.getVariesBySegment(); const valueDefinitions = await Promise.all( propertyTypes.map(async (property) => { - // TODO: Implement caching for data-type requests. [NL] const dataType = (await repo.requestByUnique(property.dataType.unique)).data; // This means if its not loaded this will never resolve and the error below will never happen. if (!dataType) { diff --git a/src/Umbraco.Web.UI.Client/src/packages/core/workspace/entity-detail/entity-detail-workspace-base.ts b/src/Umbraco.Web.UI.Client/src/packages/core/workspace/entity-detail/entity-detail-workspace-base.ts index c1e0912a3f..b9e95b34ef 100644 --- a/src/Umbraco.Web.UI.Client/src/packages/core/workspace/entity-detail/entity-detail-workspace-base.ts +++ b/src/Umbraco.Web.UI.Client/src/packages/core/workspace/entity-detail/entity-detail-workspace-base.ts @@ -253,7 +253,7 @@ export abstract class UmbEntityDetailWorkspaceContextBase< } } } else if (data) { - const processedData = await this._processIncomingData(data); + const processedData = await this._scaffoldProcessData(data); this._data.setPersisted(processedData); this._data.setCurrent(processedData); @@ -311,7 +311,6 @@ export abstract class UmbEntityDetailWorkspaceContextBase< let { data } = await request; if (data) { - data = await this._processIncomingData(data); data = await this._scaffoldProcessData(data); if (this.modalContext) { @@ -336,7 +335,7 @@ export abstract class UmbEntityDetailWorkspaceContextBase< * @returns {Promise} The processed data. */ protected async _scaffoldProcessData(data: DetailModelType): Promise { - return data; + return await this._processIncomingData(data); } protected async _processIncomingData(data: DetailModelType): Promise { return data; diff --git a/src/Umbraco.Web.UI.Client/src/packages/documents/documents/workspace/document-workspace.context.ts b/src/Umbraco.Web.UI.Client/src/packages/documents/documents/workspace/document-workspace.context.ts index 0d199d874c..7ddd97262f 100644 --- a/src/Umbraco.Web.UI.Client/src/packages/documents/documents/workspace/document-workspace.context.ts +++ b/src/Umbraco.Web.UI.Client/src/packages/documents/documents/workspace/document-workspace.context.ts @@ -227,22 +227,16 @@ export class UmbDocumentWorkspaceContext this.#isTrashedContext.setIsTrashed(false); } - protected override async loadSegments(): Promise { - this.observe( - this.unique, - async (unique) => { - if (!unique) { - this._segments.setValue([]); - return; - } - const { data } = await this.#documentSegmentRepository.getDocumentByIdSegmentOptions(unique, { - skip: 0, - take: 9999, - }); - this._segments.setValue(data?.items ?? []); - }, - '_loadSegmentsUnique', - ); + protected override async _loadSegmentsFor(unique: string): Promise { + if (!unique) { + this._segments.setValue([]); + return; + } + const { data } = await this.#documentSegmentRepository.getDocumentByIdSegmentOptions(unique, { + skip: 0, + take: 9999, + }); + this._segments.setValue(data?.items ?? []); } async create(parent: UmbEntityModel, documentTypeUnique: string, blueprintUnique?: string) {