Merge pull request sveltejs#1679 from sveltejs/sveltejsgh-1676

Allows actions to use any expression type

src/parse/read/directives.ts

@@ -77,7 +77,7 @@ const DIRECTIVES: Record<string, {
 		attribute(start, end, type, name, expression) {
 			return { start, end, type, name, expression };
 		},
-		allowedExpressionTypes: ['Identifier', 'MemberExpression', 'ObjectExpression', 'Literal', 'CallExpression'],
+		allowedExpressionTypes: ['*'],
 		error: 'Data passed to actions must be an identifier (e.g. `foo`), a member expression ' +
 			'(e.g. `foo.bar` or `foo[baz]`), a method call (e.g. `foo()`), or a literal (e.g. `true` or `\'a string\'`'
 	},
@@ -163,7 +163,8 @@ export function readDirective(
 
 		try {
 			expression = readExpression(parser, expressionStart, quoteMark);
-			if (directive.allowedExpressionTypes.indexOf(expression.type) === -1) {
+			const allowed = directive.allowedExpressionTypes;
+			if (allowed[0] !== '*' && allowed.indexOf(expression.type) === -1) {
 				parser.error({
 					code: `invalid-directive-value`,
 					message: directive.error

test/runtime/samples/action-ternary-template/_config.js

@@ -0,0 +1,20 @@
+export default {
+	data: {
+		target: 'World!',
+		display: true,
+	},
+
+	html: `
+		<h1></h1>
+	`,
+
+	test ( assert, component, target, window ) {
+		const header = target.querySelector( 'h1' );
+		const eventClick = new window.MouseEvent( 'click' );
+
+		header.dispatchEvent( eventClick );
+		assert.htmlEqual( target.innerHTML, `
+			<h1>Hello World!</h1>
+		` );
+	}
+};

test/runtime/samples/action-ternary-template/main.html

@@ -0,0 +1,21 @@
+<h1 use:insert="display ? `Hello ${target}` : ''"></h1>
+
+<script>
+	export default {
+		actions: {
+			insert(node, text) {
+
+				function onClick() {
+					node.textContent = text;
+				}
+				node.addEventListener('click', onClick);
+
+				return {
+					destroy() {
+						node.removeEventListener('click', onClick);
+					}
+				}
+			}
+		}
+	}
+</script>