RTE Link Picker: Fix media selection to allow items not permitted at root.

src/Umbraco.Web.UI.Client/src/packages/content/content-type/global-components/input-content-type-collection-configuration/input-content-type-collection-configuration.element.ts

@@ -12,7 +12,8 @@ import { UmbFormControlMixin } from '@umbraco-cms/backoffice/validation';
 @customElement('umb-input-content-type-collection-configuration')
 export class UmbInputContentTypeCollectionConfigurationElement extends UmbFormControlMixin<
 	string,
-	typeof UmbLitElement
+	typeof UmbLitElement,
+	undefined
 >(UmbLitElement) {
 	protected override getFormElement() {
 		return undefined;

src/Umbraco.Web.UI.Client/src/packages/media/media-types/workspace/views/structure/media-type-workspace-view-structure.element.ts

@@ -47,7 +47,7 @@ export class UmbMediaTypeWorkspaceViewStructureElement extends UmbLitElement imp
 				const oldValue = this._allowedContentTypeIDs;
 				this._allowedContentTypeIDs = allowedContentTypes
 					?.map((x) => x.contentType.unique)
-					.filter((x) => x !== undefined) as Array<string>;
+					.filter((x) => x !== undefined);
 				this.requestUpdate('_allowedContentTypeIDs', oldValue);
 			},
 			'_allowedContentTypesObserver',

src/Umbraco.Web.UI.Client/src/packages/media/media/components/input-media/input-media.context.ts

@@ -7,7 +7,7 @@ import type { UmbMediaTreeItemModel } from '../../tree/types.js';
 import { isMediaTreeItem } from '../../tree/utils.js';
 import { UmbPickerInputContext } from '@umbraco-cms/backoffice/picker-input';
 import type { UmbControllerHost } from '@umbraco-cms/backoffice/controller-api';
-import type { UmbMediaTypeEntityType } from '@umbraco-cms/backoffice/media-type';
+import { isUmbracoFolder, type UmbMediaTypeEntityType } from '@umbraco-cms/backoffice/media-type';
 import { UMB_VARIANT_CONTEXT } from '@umbraco-cms/backoffice/variant';
 
 interface UmbMediaPickerInputContextOpenArgs {
@@ -63,6 +63,10 @@ export class UmbMediaPickerInputContext extends UmbPickerInputContext<
 		if (isMediaTreeItem(item) && item.noAccess) {
 			return false;
 		}
+		// Exclude folders - they don't have URLs
+		if (isUmbracoFolder(item.mediaType.unique)) {
+			return false;
+		}
 		if (allowedContentTypes && allowedContentTypes.length > 0) {
 			return allowedContentTypes
 				.map((contentTypeReference) => contentTypeReference.unique)

src/Umbraco.Web.UI.Client/src/packages/media/media/components/input-media/input-media.element.ts

@@ -111,7 +111,7 @@ export class UmbInputMediaElement extends UmbFormControlMixin<string | undefined
 	}
 
 	@property({ type: Array })
-	allowedContentTypeIds?: Array<string> | undefined;
+	allowedContentTypeIds?: Array<string>;
 
 	@property({ type: Boolean, attribute: 'include-trashed' })
 	includeTrashed = false;

src/Umbraco.Web.UI.Client/src/packages/media/media/components/input-rich-media/input-rich-media.element.ts

@@ -116,7 +116,7 @@ export class UmbInputRichMediaElement extends UmbFormControlMixin<
 	}
 
 	@property({ type: Array })
-	allowedContentTypeIds?: string[] | undefined;
+	allowedContentTypeIds?: string[];
 
 	@property({ type: Object, attribute: false })
 	startNode?: UmbTreeStartNode;

src/Umbraco.Web.UI.Client/src/packages/media/media/property-editors/media-picker/property-editor-ui-media-picker.element.ts

@@ -29,7 +29,7 @@ export class UmbPropertyEditorUIMediaPickerElement
 
 		if (!config) return;
 
-		this._allowedMediaTypes = config.getValueByAlias<string>('filter')?.split(',') ?? [];
+		this._allowedMediaTypes = config.getValueByAlias<string>('filter')?.split(',') ?? undefined;
 		this._focalPointEnabled = Boolean(config.getValueByAlias('enableLocalFocalPoint'));
 		this._multiple = Boolean(config.getValueByAlias('multiple'));
 		this._preselectedCrops = config?.getValueByAlias<Array<UmbCropModel>>('crops') ?? [];
@@ -71,7 +71,7 @@ export class UmbPropertyEditorUIMediaPickerElement
 	private _preselectedCrops: Array<UmbCropModel> = [];
 
 	@state()
-	private _allowedMediaTypes: Array<string> = [];
+	private _allowedMediaTypes?: Array<string>;
 
 	@state()
 	private _multiple: boolean = false;

src/Umbraco.Web.UI.Client/src/packages/members/member-group/components/input-member-group/input-member-group.element.ts

@@ -86,9 +86,6 @@ export class UmbInputMemberGroupElement extends UmbFormControlMixin<string, type
 		return this.#pickerContext.getSelection();
 	}
 
-	@property({ type: Array })
-	allowedContentTypeIds?: string[] | undefined;
-
 	@property({ type: String })
 	public override set value(selectionString: string | undefined) {
 		this.selection = splitStringToArray(selectionString);

src/Umbraco.Web.UI.Client/src/packages/members/member/components/input-member/input-member.element.ts

@@ -85,7 +85,7 @@ export class UmbInputMemberElement extends UmbFormControlMixin<string, typeof Um
 	}
 
 	@property({ type: Array })
-	allowedContentTypeIds?: string[] | undefined;
+	allowedContentTypeIds?: string[];
 
 	@property({ type: String })
 	public override set value(selectionString: string | undefined) {

src/Umbraco.Web.UI.Client/src/packages/multi-url-picker/link-picker-modal/link-picker-modal.element.ts

@@ -5,7 +5,6 @@ import type {
 	UmbLinkPickerModalValue,
 } from './link-picker-modal.token.js';
 import { css, customElement, html, nothing, query, state, when } from '@umbraco-cms/backoffice/external/lit';
-import { isUmbracoFolder, UmbMediaTypeStructureRepository } from '@umbraco-cms/backoffice/media-type';
 import {
 	umbBindToValidation,
 	UmbObserveValidationStateController,
@@ -32,9 +31,6 @@ export class UmbLinkPickerModalElement extends UmbModalBaseElement<UmbLinkPicker
 
 	#validationContext = new UmbValidationContext(this);
 
-	@state()
-	private _allowedMediaTypeUniques?: Array<string>;
-
 	@state()
 	private _config: UmbLinkPickerConfig = {
 		hideAnchor: false,
@@ -73,18 +69,9 @@ export class UmbLinkPickerModalElement extends UmbModalBaseElement<UmbLinkPicker
 			});
 		}
 
-		this.#getMediaTypes();
 		this.populateLinkUrl();
 	}
 
-	async #getMediaTypes() {
-		// Get all the media types, excluding the folders, so that files are selectable media items.
-		const mediaTypeStructureRepository = new UmbMediaTypeStructureRepository(this);
-		const { data: mediaTypes } = await mediaTypeStructureRepository.requestAllowedChildrenOf(null, null);
-		this._allowedMediaTypeUniques =
-			(mediaTypes?.items.map((x) => x.unique).filter((x) => x && !isUmbracoFolder(x)) as Array<string>) ?? [];
-	}
-
 	async populateLinkUrl() {
 		// Documents and media have URLs saved in the local link format. Display the actual URL to align with what
 		// the user sees when they selected it initially.
@@ -353,7 +340,6 @@ export class UmbLinkPickerModalElement extends UmbModalBaseElement<UmbLinkPicker
 				label=${this.localize.term('general_media')}>
 				<umb-input-media
 					slot="editor"
-					.allowedContentTypeIds=${this._allowedMediaTypeUniques}
 					.max=${1}
 					.value=${this.value.link.unique && this.value.link.type === 'media' ? this.value.link.unique : ''}
 					@change=${(e: UmbInputPickerEvent) => this.#onPickerSelection(e, 'media')}></umb-input-media>

src/Umbraco.Web.UI.Client/src/packages/property-editors/content-picker/components/input-content/input-content.element.ts

@@ -42,13 +42,13 @@ export class UmbInputContentElement extends UmbFormControlMixin<string | undefin
 	startNode?: UmbTreeStartNode;
 
 	@property()
-	public set allowedContentTypeIds(value: string) {
-		this.#allowedContentTypeIds = value ? value.split(',') : [];
+	public set allowedContentTypeIds(value: string | undefined) {
+		this.#allowedContentTypeIds = value ? value.split(',') : undefined;
 	}
-	public get allowedContentTypeIds(): string {
-		return this.#allowedContentTypeIds.join(',');
+	public get allowedContentTypeIds(): string | undefined {
+		return this.#allowedContentTypeIds?.join(',');
 	}
-	#allowedContentTypeIds: Array<string> = [];
+	#allowedContentTypeIds?: Array<string>;
 
 	@property({ type: Array })
 	public set selection(values: Array<UmbReferenceByUniqueAndType>) {

src/Umbraco.Web.UI.Client/src/packages/property-editors/content-picker/property-editor-ui-content-picker.element.ts

@@ -61,7 +61,7 @@ export class UmbPropertyEditorUIContentPickerElement
 	private _maxMessage = '';
 
 	@state()
-	private _allowedContentTypeUniques?: string | null;
+	private _allowedContentTypeUniques?: string;
 
 	@state()
 	private _rootUnique?: string | null;
@@ -208,7 +208,7 @@ export class UmbPropertyEditorUIContentPickerElement
 				.max=${this._max}
 				.maxMessage=${this._maxMessage}
 				.startNode=${startNode}
-				.allowedContentTypeIds=${this._allowedContentTypeUniques ?? ''}
+				.allowedContentTypeIds=${this._allowedContentTypeUniques}
 				?readonly=${this.readonly}
 				?required=${this.mandatory}
 				.requiredMessage=${this.mandatoryMessage}