fix: bug where fallback window trigger was not firing if there were no hook listeners attached

... so in essence the backwards compatibility was not working at all.

closes 
v1.18.x
Julian Lam
parent c8b78654d9
commit 1e579428e7

@ -63,12 +63,10 @@ define('hooks', [], () => {
};
const _fireActionHook = (hookName, data) => {
if (!Hooks.hasListeners(hookName)) {
return;
if (Hooks.hasListeners(hookName)) {
Hooks.loaded[hookName].forEach(listener => listener(data));
}
Hooks.loaded[hookName].forEach(listener => listener(data));
// Backwards compatibility (remove this when we eventually remove jQuery from NodeBB core)
$(window).trigger(hookName, data);
};

Loading…
Cancel
Save