Bugfix multirow delete

Author: ewowi

interface/src/lib/components/moonbase/MultiRow.svelte

@@ -63,7 +63,7 @@
 		for (let i = 0; i < definition.length; i++) {
 			let property = definition[i];
 			if (property.name == propertyName) {
-				console.log('addItem def', propertyName, property);
+				console.log('addItem def', propertyName);
 				for (let i = 0; i < property.n.length; i++) {
 					let propertyN = property.n[i];
 					// console.log("propertyN", propertyN)
@@ -73,57 +73,40 @@
 		}
 	}
 
-	function handleEdit(propertyName: string, index: number) {
-		console.log('handleEdit', propertyName, index, data[propertyName][index]);
-
+	function handleEdit(propertyName: string, itemToEdit: any) {
+		console.log('handleEdit', propertyName);
 		modals.open(EditObject as any, {
-			property: property,
-			localDefinition: localDefinition,
-			title: initCap(propertyName) + ' #' + (index + 1),
-			dataEditable: data[propertyName][index], // By reference (bindable)
+			property,
+			localDefinition,
+			title: initCap(propertyName),
+			dataEditable: itemToEdit, // direct reference
 			onChange,
 			changeOnInput
 		});
 	}
 
-	function deleteItem(propertyName: string, index: number) {
-		// Remove item from array
-		console.log(propertyName, index, data[propertyName]);
-		data[propertyName].splice(index, 1);
-		data[propertyName] = [...data[propertyName]]; //Trigger reactivity
+	function deleteItem(propertyName: string, itemToDelete: any) {
+		console.log('deleteItem', propertyName);
+		data[propertyName] = data[propertyName].filter((item: any) => item !== itemToDelete);
 		onChange();
 	}
 
 	// Filter items based on filter query - returns array of {item, originalIndex}
 	let filteredItems = $derived.by(() => {
-		if (!data[property.name + '_filter']) {
-			return data[property.name].map((item: any, index: number) => ({
-				item,
-				originalIndex: index
-			}));
-		}
+		const filterValue = data[property.name + '_filter']?.trim() || '';
+		const isNegated = filterValue.startsWith('!');
+		const query = (isNegated ? filterValue.slice(1) : filterValue).toLowerCase();
 
-		const isNegated = data[property.name + '_filter'].startsWith('!');
-		const query = (
-			isNegated ? data[property.name + '_filter'].slice(1) : data[property.name + '_filter']
-		)
-			.toLowerCase()
-			.trim();
-
-		if (!query) {
-			return data[property.name].map((item: any, index: number) => ({
-				item,
-				originalIndex: index
-			}));
-		}
+		// No filter or empty query → return items directly
+		if (!query) return data[property.name].map((item:any) => ({ item }));
 
-		const filtered = data[property.name]
-			.map((item: any, index: number) => ({ item, originalIndex: index }))
+		// Filtered items
+		return data[property.name]
+			.map((item:any) => ({ item }))
 			.filter(({ item }: { item: any }) => {
-				const matchFound = property.n.slice(0, 3).some((propertyN: any) => {
+				const matchFound = property.n.slice(0, 3).some((propertyN:any) => {
 					let valueStr;
 
-					// Check dropdown - only check the SELECTED option's label, not all options
 					if (
 						propertyN.values &&
 						Array.isArray(propertyN.values) &&
@@ -137,11 +120,8 @@
 					return String(valueStr).toLowerCase().includes(query);
 				});
 
-				// If negated (!), return items that DON'T match
 				return isNegated ? !matchFound : matchFound;
 			});
-
-		return filtered;
 	});
 
 	const findItemInDefinition = $derived(definition.find((obj: any) => obj.name === property.name));
@@ -199,7 +179,7 @@
 
 <div class="overflow-x-auto space-y-1" transition:slide|local={{ duration: 300, easing: cubicOut }}>
 	<DraggableList items={filteredItems} onReorder={handleReorder} class="space-y-2">
-		{#snippet children({ item: itemWrapper, index }: { item: any; index: number })}
+		{#snippet children({ item: itemWrapper }: { item: any })}
 			<!-- svelte-ignore a11y_click_events_have_key_events -->
 			<div class="rounded-box bg-base-100 flex items-center space-x-3 px-4 py-2">
 				<Grip class="h-6 w-6 text-base-content/30 cursor-grab flex-shrink-0" />
@@ -208,7 +188,7 @@
 					{#if propertyN.type != 'array' && propertyN.type != 'controls' && propertyN.type != 'password'}
 						<MultiInput
 							property={propertyN}
-							bind:value={data[property.name][itemWrapper.originalIndex][propertyN.name]}
+							bind:value={itemWrapper.item[propertyN.name]}
 							noPrompts={true}
 							onChange={(event) => {
 								onChange(event);
@@ -232,15 +212,15 @@
 						<button
 							class="btn btn-ghost btn-sm"
 							onclick={() => {
-								handleEdit(property.name, itemWrapper.originalIndex);
+								handleEdit(property.name, itemWrapper.item);
 							}}
 						>
 							<SearchIcon class="h-6 w-6" /></button
 						>
 						<button
 							class="btn btn-ghost btn-sm"
 							onclick={() => {
-								deleteItem(property.name, itemWrapper.originalIndex);
+								deleteItem(property.name, itemWrapper.item);
 							}}
 						>
 							<Delete class="text-error h-6 w-6" />