Remove spurious ConcatStatement check from template-no-element-event-actions

johanrd · johanrd · commit e986769da350 · 2026-03-12T23:25:06.000+01:00
diff --git a/lib/rules/template-no-element-event-actions.js b/lib/rules/template-no-element-event-actions.js
@@ -58,10 +58,7 @@ module.exports = {
             }
           } else {
             // Flag any mustache value on event attributes
-            if (
-              attr.value?.type === 'GlimmerMustacheStatement' ||
-              attr.value?.type === 'GlimmerConcatStatement'
-            ) {
+            if (attr.value?.type === 'GlimmerMustacheStatement') {
               context.report({ node: attr, messageId: 'noElementEventActions' });
             }
           }

Original file line number	Diff line number	Diff line change
`@@ -58,10 +58,7 @@ module.exports = {`
`58`	`58`	`}`
`59`	`59`	`} else {`
`60`	`60`	`// Flag any mustache value on event attributes`
`61`		`- if (`
`62`		`- attr.value?.type === 'GlimmerMustacheStatement' \|\|`
`63`		`- attr.value?.type === 'GlimmerConcatStatement'`
`64`		`- ) {`
	`61`	`+ if (attr.value?.type === 'GlimmerMustacheStatement') {`
`65`	`62`	`context.report({ node: attr, messageId: 'noElementEventActions' });`
`66`	`63`	`}`
`67`	`64`	`}`