From 34bb9dddb53e546899de352f1ad3b7532cc57722 Mon Sep 17 00:00:00 2001
From: Julian Lam <julian.lam@gmail.com>
Date: Sun, 28 Jul 2013 02:24:41 -0400
Subject: [PATCH] sticking to style guide for method names (camelcase)

---
 src/plugins.js   | 21 ++++++++++++---------
 src/postTools.js |  8 ++++++--
 src/posts.js     |  6 ++++--
 3 files changed, 22 insertions(+), 13 deletions(-)

diff --git a/src/plugins.js b/src/plugins.js
index ae7274c2fb..d6c5a6db79 100644
--- a/src/plugins.js
+++ b/src/plugins.js
@@ -1,9 +1,10 @@
 var	fs = require('fs'),
 	path = require('path'),
 	RDB = require('./redis.js'),
+	async = require('async'),
 	plugins = {
 		libraries: [],
-		loaded_hooks: {},
+		loadedHooks: {},
 		init: function() {
 			if (this.initialized) return;
 			if (global.env === 'development') console.log('Info: [plugins] Initializing plugins system');
@@ -28,7 +29,7 @@ var	fs = require('fs'),
 									_self.libraries[pluginData.id] = require(path.join(pluginPath, pluginData.library));
 									if (pluginData.hooks) {
 										for(var x=0,numHooks=pluginData.hooks.length;x<numHooks;x++) {
-											_self.register_hook(pluginData.id, pluginData.hooks[x]);
+											_self.registerHook(pluginData.id, pluginData.hooks[x]);
 										}
 									}
 									if (global.env === 'development') console.log('Info: [plugins] Loaded plugin: ' + pluginData.id);
@@ -52,26 +53,26 @@ var	fs = require('fs'),
 			});
 		},
 		initialized: false,
-		register_hook: function(id, data) {
+		registerHook: function(id, data) {
 			/*
 				`data` is an object consisting of (* is required):
 					`data.hook`*, the name of the NodeBB hook
 					`data.method`*, the method called in that plugin
-					`data.callbacked`, whether or not the hook expects a callback (true), or a return (false). (Default: false)
+					`data.callbacked`, whether or not the hook expects a callback (true), or a return (false). Only used for filters. (Default: false)
 					(Not implemented) `data.priority`, the relative priority of the method when it is eventually called (default: 10)
 			*/
 			var	_self = this;
 
 			if (data.hook && data.method) {
-				_self.loaded_hooks[data.hook] = _self.loaded_hooks[data.hook] || [];
-				_self.loaded_hooks[data.hook].push([id, data.method]);
+				_self.loadedHooks[data.hook] = _self.loadedHooks[data.hook] || [];
+				_self.loadedHooks[data.hook].push([id, data.method]);
 				if (global.env === 'development') console.log('Info: [plugins] Hook registered: ' + data.hook + ' will call ' + id);
 			} else return;
 		},
-		fire_hook: function(hook, args, callback) {
+		fireHook: function(hook, args, callback) {
 			// TODO: Implement priority hook firing
 			var	_self = this
-				hookList = this.loaded_hooks[hook];
+				hookList = this.loadedHooks[hook];
 
 			if(hookList && Array.isArray(hookList)) {
 				if (global.env === 'development') console.log('Info: [plugins] Firing hook: \'' + hook + '\'');
@@ -100,7 +101,9 @@ var	fs = require('fs'),
 						});
 					break;
 					case 'action':
-
+						async.each(hookList, function(hookObj) {
+							_self.libraries[hookObj[0]][hookObj[1]].apply(_self.libraries[hookObj[0]], args);
+						});
 					break;
 					default:
 						// Do nothing...
diff --git a/src/postTools.js b/src/postTools.js
index 1b2c588a55..1afa47c10c 100644
--- a/src/postTools.js
+++ b/src/postTools.js
@@ -5,7 +5,8 @@ var	RDB = require('./redis.js'),
 	user = require('./user.js'),
 	async = require('async'),
 	
-	utils = require('../public/src/utils');
+	utils = require('../public/src/utils'),
+	plugins = require('./plugins');
 
 (function(PostTools) {
 	PostTools.isMain = function(pid, tid, callback) {
@@ -71,7 +72,10 @@ var	RDB = require('./redis.js'),
 
 		PostTools.privileges(pid, uid, function(privileges) {
 			if (privileges.editable) {
-				success();
+				plugins.fireHook('filter:save_post_content', content, function(parsedContent) {
+					content = parsedContent;
+					success();
+				});
 			}
 		});
 	}
diff --git a/src/posts.js b/src/posts.js
index 80d745178e..bb10a854ec 100644
--- a/src/posts.js
+++ b/src/posts.js
@@ -280,7 +280,7 @@ var	RDB = require('./redis.js'),
 				RDB.incr('global:next_post_id', function(err, pid) {
 					RDB.handle(err);
 
-					plugins.fire_hook('filter:save_post_content', content, function(content) {
+					plugins.fireHook('filter:save_post_content', content, function(content) {
 						var timestamp = Date.now(),
 							postData = {
 								'pid': pid,
@@ -341,7 +341,9 @@ var	RDB = require('./redis.js'),
 									}
 								}
 							});			
-						}				
+						}
+
+						plugins.fireHook('action:save_post_content', [content])
 						
 						if(!images) {
 							postData.uploadedImages = JSON.stringify(uploadedImages);