allow core to tap into NodeBB's hook system, closes #989

src/plugins.js

@@ -269,11 +269,13 @@ var fs = require('fs'),
 
 		var method;
 
-		if (data.hook && data.method && typeof data.method === 'string' && data.method.length > 0) {
+		if (data.hook && data.method) {
 			data.id = id;
 			if (!data.priority) {
 				data.priority = 10;
 			}
+
+			if (typeof data.method === 'string' && data.method.length > 0) {
 				method = data.method.split('.').reduce(function(memo, prop) {
 					if (memo !== null && memo[prop]) {
 						return memo[prop];
@@ -283,14 +285,18 @@ var fs = require('fs'),
 					}
 				}, Plugins.libraries[data.id]);
 
-			if (method === null) {
-				winston.warn('[plugins/' + id + '] Hook method mismatch: ' + data.hook + ' => ' + data.method);
-				return callback();
-			}
-
 				// Write the actual method reference to the hookObj
 				data.method = method;
 
+				register();
+			} else if (typeof data.method === 'function') {
+				register();
+			} else {
+				winston.warn('[plugins/' + id + '] Hook method mismatch: ' + data.hook + ' => ' + data.method);
+			}
+		}
+
+		function register() {
 			Plugins.loadedHooks[data.hook] = Plugins.loadedHooks[data.hook] || [];
 			Plugins.loadedHooks[data.hook].push(data);
 
@@ -680,4 +686,5 @@ var fs = require('fs'),
 			callback(err, plugins);
 		});
 	};
+
 }(exports));