Add warning message about deprecated hooks

2 files changed, 17 insertions, 4 deletions

diff --git a/lib/plugin.js b/lib/plugin.js
index f558bf8..3d60321 100644
--- a/lib/plugin.js
+++ b/lib/plugin.js
@@ -29,6 +29,9 @@ var Plugin = function(book, name) {
 
 // Type of plugins resources
 Plugin.RESOURCES = ["js", "css"];
+Plugin.HOOKS = [
+    "init", "finish"
+]
 
 // Load from a name
 Plugin.prototype.load = function(name, baseDir) {
@@ -100,13 +103,21 @@ Plugin.prototype.getResources = function(base) {
 
 // Test if it's a valid plugin
 Plugin.prototype.isValid = function() {
-    return (
+    var that = this;
+    var isValid = (
         this.packageInfos &&
         this.packageInfos.name &&
         this.packageInfos.engines &&
         this.packageInfos.engines.gitbook &&
         semver.satisfies(pkg.version, this.packageInfos.engines.gitbook)
     );
+
+    // Valid hooks
+    _.each(this.infos.hooks, function(hook, hookName) {
+        that.book.log.warn.ln("Hook '"+hookName+" 'used by plugin '"+that.packageInfos.name+"' has been removed or is deprecated");
+    });
+
+    return isValid;
 };
 
 // Resolve file path
diff --git a/lib/utils/logger.js b/lib/utils/logger.js
index 927048b..9ef9ccf 100644
--- a/lib/utils/logger.js
+++ b/lib/utils/logger.js
@@ -5,14 +5,14 @@ var color = require('bash-color');
 var LEVELS = {
 	DEBUG: 0,
 	INFO: 1,
-	WARNING: 2,
+	WARN: 2,
 	ERROR: 3
 };
 
 var COLORS = {
 	DEBUG: color.purple,
 	INFO: color.cyan,
-	WARNING: color.yellow,
+	WARN: color.yellow,
 	ERROR: color.red
 };
 
@@ -53,6 +53,8 @@ module.exports = function(_write, logLevel) {
 		return logger.write(msg);
 	};
 	logger.logLn = function() {
+		if (lastChar != '\n') logger.write("\n");
+
 		var args = Array.prototype.slice.apply(arguments);
 		args.push("\n");
 		logger.log.apply(logger, args);
@@ -65,7 +67,7 @@ module.exports = function(_write, logLevel) {
 		if (arguments.length > 1) {
 			logger.logLn(level, color.green('>> ') + msg.trim().replace(/\n/g, color.green('\n>> ')));
 		} else {
-			logger.logLn(level, color.green("OK"));
+			logger.log(level, color.green("OK"), "\n");
 		}
 	};