Merge pull request #1557 from PaulMaly/1553-return-from-custom-event-handler

Fix for #1553

Author: Rich-Harris

src/compile/nodes/EventHandler.ts

@@ -67,12 +67,12 @@ export default class EventHandler extends Node {
 				compiler.code.overwrite(
 					this.insertionPoint,
 					this.insertionPoint + 1,
-					`${component}.store.`
+					`return ${component}.store.`
 				);
 			} else {
 				compiler.code.prependRight(
 					this.insertionPoint,
-					`${component}.`
+					`return ${component}.`
 				);
 			}
 		}

test/js/samples/event-handlers-custom/expected-bundle.js

@@ -146,7 +146,7 @@ function create_main_fragment(component, ctx) {
 			button = createElement("button");
 			button.textContent = "foo";
 			foo_handler = foo.call(component, button, function(event) {
-				component.foo( ctx.bar );
+				return component.foo( ctx.bar );
 			});
 		},
 

test/js/samples/event-handlers-custom/expected.js

@@ -19,7 +19,7 @@ function create_main_fragment(component, ctx) {
 			button = createElement("button");
 			button.textContent = "foo";
 			foo_handler = foo.call(component, button, function(event) {
-				component.foo( ctx.bar );
+				return component.foo( ctx.bar );
 			});
 		},