DGNum/demarches-normaliennes
15
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Merge pull request #8520 from colinux/fix-stimulus-on-handler

Browse source 
fix(stimulus): "on" handler having target
This commit is contained in:
Colin Darie 2023-01-30 11:52:06 +01:00 committed by GitHub
commit 17141a26d5
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
app/javascript/controllers/application_controller.ts
View file

@ -65,7 +65,10 @@ export class ApplicationController extends Controller {
FOCUS_EVENTS.includes(targetOrEventName) FOCUS_EVENTS.includes(targetOrEventName)
); );
} else { } else {
invariant(eventNameOrHandler == 'string', 'event name is required'); invariant(
typeof eventNameOrHandler == 'string',
'event name is required'
);
invariant(handler, 'handler is required'); invariant(handler, 'handler is required');
this.onTarget(targetOrEventName, eventNameOrHandler, handler); this.onTarget(targetOrEventName, eventNameOrHandler, handler);
} }