Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: pass module name to static:script.init, +comments

Browse Source 
I feel it is difficult to know exactly what the original use-cases were for these hooks, so some comments may be helpful.
This commit is contained in:
Julian Lam
2021-01-28 16:14:52 -05:00
parent 4bb3b0323d
commit f8bf9e99c4
1 changed files with 9 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
public/src/ajaxify.js
View File

@@ -324,6 +324,7 @@ ajaxify = window.ajaxify || {};
scripts: [location + tpl_url],
};
// Hint: useful if you want to load a module on a specific page (append module name to `scripts`)
$(window).trigger('action:script.load', data);
// Require and parse modules
@@ -338,12 +339,14 @@ ajaxify = window.ajaxify || {};
}
if (typeof script === 'string') {
return function (next) {
require([script], function (script) {
$(window).trigger('static:script.init', { tpl_url, module: script });
if (script && script.init) {
script.init();
}
next();
require(['hooks', script], function (hooks, module) {
// Hint: useful if you want to override a loaded library (e.g. replace core client-side logic), or call a method other than .init()
hooks.fire('static:script.init', { tpl_url, name: script, module }).then(() => {
if (module && module.init) {
module.init();
}
next();
});
}, function () {
// ignore 404 error
next();