fix handleEvent this

Author: 7nik

packages/svelte/src/internal/client/dom/elements/events.js

@@ -57,12 +57,7 @@ export function call_event_handler(handler, current_target, event, data = []) {
 	if (typeof handler === 'function') {
 		handler.call(current_target, event, ...data);
 	} else {
-		if (handler && handler.handleEvent == null) {
-			// @ts-expect-error - do so to get a nicer "handler.handleEvent is not a function" error instead of "Cannot read properties of null (reading 'call')"
-			handler.handleEvent();
-		} else {
-			handler?.handleEvent.call(current_target, event);
-		}
+		handler?.handleEvent(event);
 	}
 }
 

packages/svelte/tests/runtime-runes/samples/event-handler-object-dev/main.svelte

@@ -2,8 +2,8 @@
 	let count = $state(0);
 
 	let onclick = $state({
-		handleEvent() {
-			count += +this.dataset.step;
+		handleEvent(ev) {
+			count += +ev.currentTarget.dataset.step;
 		}
 	});
 </script>

packages/svelte/tests/runtime-runes/samples/event-handler-object/_config.js

@@ -13,7 +13,20 @@ export default test({
 		flushSync();
 		buttons.forEach((b) => b.click());
 		flushSync();
-		assert.deepEqual(logs, [true, true, 'mutated', 'mutated', 'assigned', 'assigned']);
+		assert.deepEqual(logs, [
+			'click',
+			true,
+			'click',
+			true,
+			'mutated',
+			true,
+			'mutated',
+			true,
+			'assigned',
+			true,
+			'assigned',
+			true
+		]);
 		assert.htmlEqual(target.innerHTML, '<button>a</button><button>b</button><button>next</button>');
 	}
 });

packages/svelte/tests/runtime-runes/samples/event-handler-object/main.svelte

@@ -3,16 +3,16 @@
 
 	let onclick = $state.raw({
 		handleEvent(ev) {
-			console.log(this === ev.currentTarget);
+			console.log(ev.type, this === onclick);
 		}
 	});
 
 	function click2() {
-		console.log("mutated");
+		console.log("mutated", this === onclick);
 	}
 
 	function click3() {
-		console.log("assigned");
+		console.log("assigned", this === onclick);
 	}
 
 	let btn;