From 48ac1c19c5719c1a69e5dc8b1ec8df82c46196e7 Mon Sep 17 00:00:00 2001
From: leekelleher <leekelleher@gmail.com>
Date: Fri, 24 May 2024 21:05:04 +0100
Subject: [PATCH] MultiUrlPicker: ensures min/max is a positive number

---
 ...operty-editor-ui-multi-url-picker.element.ts | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

diff --git a/src/Umbraco.Web.UI.Client/src/packages/multi-url-picker/property-editor/property-editor-ui-multi-url-picker.element.ts b/src/Umbraco.Web.UI.Client/src/packages/multi-url-picker/property-editor/property-editor-ui-multi-url-picker.element.ts
index 69ded9db4f..f3c3fc1853 100644
--- a/src/Umbraco.Web.UI.Client/src/packages/multi-url-picker/property-editor/property-editor-ui-multi-url-picker.element.ts
+++ b/src/Umbraco.Web.UI.Client/src/packages/multi-url-picker/property-editor/property-editor-ui-multi-url-picker.element.ts
@@ -23,11 +23,16 @@ export class UmbPropertyEditorUIMultiUrlPickerElement extends UmbLitElement impl
 		if (!config) return;
 
 		this._hideAnchor = Boolean(config.getValueByAlias('hideAnchor')) ?? false;
-		this._minNumber = Number(config.getValueByAlias('minNumber')) || 0;
-		this._maxNumber = Number(config.getValueByAlias('maxNumber')) || Infinity;
+		this._min = this.#parseInt(config.getValueByAlias('minNumber'), 0);
+		this._max = this.#parseInt(config.getValueByAlias('maxNumber'), Infinity);
 		this._overlaySize = config.getValueByAlias<UUIModalSidebarSize>('overlaySize') ?? 'small';
 	}
 
+	#parseInt(value: unknown, fallback: number): number {
+		const num = Number(value);
+		return num > 0 ? num : fallback;
+	}
+
 	@state()
 	private _overlaySize?: UUIModalSidebarSize;
 
@@ -35,10 +40,10 @@ export class UmbPropertyEditorUIMultiUrlPickerElement extends UmbLitElement impl
 	private _hideAnchor?: boolean;
 
 	@state()
-	private _minNumber? = 0;
+	private _min = 0;
 
 	@state()
-	private _maxNumber? = Infinity;
+	private _max = Infinity;
 
 	@state()
 	private _alias?: string;
@@ -64,8 +69,8 @@ export class UmbPropertyEditorUIMultiUrlPickerElement extends UmbLitElement impl
 		return html`
 			<umb-multi-url-picker
 				.alias=${this._alias}
-				.max=${this._maxNumber}
-				.min=${this._minNumber}
+				.max=${this._max}
+				.min=${this._min}
 				.overlaySize=${this._overlaySize}
 				.urls=${this.value ?? []}
 				.variantId=${this._variantId}