changed appenders to register themselves

Pollenware · Aug 15, 2012 · a4e3ca5 · a4e3ca5
1 parent 9a16742
commit a4e3ca5
Show file tree

Hide file tree

Showing 11 changed files with 257 additions and 214 deletions.
diff --git a/browser/comb.js b/browser/comb.js
diff --git a/examples/browser/define.html b/examples/browser/define.html
diff --git a/lib/logging/appenders/appender.js b/lib/logging/appenders/appender.js
@@ -1,5 +1,8 @@
 var define = require("../../define.js").define, base = require("../../base"), Level = require("../level");
 
+
+var APPENDER_TYPES = {};
+
 /**
  * @class Base class for all appenders
  *
@@ -105,6 +108,25 @@ define(null, {
                 return this.__pattern;
             }
         }
+    },
+
+    "static":{
+        registerType:function (type) {
+            if (base.isString(type)) {
+                APPENDER_TYPES[type.toLowerCase()] = this;
+            }
+            return this;
+        },
+
+        createAppender:function (type, options) {
+            var caseType = type.toLowerCase();
+            if (caseType in APPENDER_TYPES) {
+                return new APPENDER_TYPES[caseType](options);
+            } else {
+                throw new Error(type + " appender is not registered!");
+            }
+
+        }
     }
 
 }).as(module);
diff --git a/lib/logging/appenders/consoleAppender.js b/lib/logging/appenders/consoleAppender.js
@@ -7,31 +7,31 @@ var define = require("../../define.js").define, base = require("../../base"), st
  * @augments comb.logging.appenders.Appender
  * @memberOf comb.logging.appenders
  */
-exports = module.exports = define(Appender, {
-			instance : {
+define(Appender, {
+    instance:{
 
-				constructor : function(options){
-					options = options || {};
-					!options.name && (options.name = "consoleAppender");
-					this._super(arguments, [options]);
-				},
+        constructor:function (options) {
+            options = options || {};
+            !options.name && (options.name = "consoleAppender");
+            this._super(arguments, [options]);
+        },
 
-				append : function(event) {
-					if (this._canAppend(event)) {
-						var message = format(this.__pattern, event);
-						var level = event.level;
-						if (Level.ERROR.equals(level) || Level.FATAL.equals(level)) {
-							console.log(style(message, "red"));
-						} else if(Level.WARN.equals(level)){
-							 console.log(style(message, "yellow"));
-						}else if (Level.DEBUG.equals(level)) {
-							console.log(style(message, "magenta"));
-						} else if (Level.TRACE.equals(level)) {
-							console.log(style(message, "cyan"));
-						} else {
-							console.log(message);
-						}
-					}
-				}
-			}
-		});
+        append:function (event) {
+            if (this._canAppend(event)) {
+                var message = format(this.__pattern, event);
+                var level = event.level;
+                if (Level.ERROR.equals(level) || Level.FATAL.equals(level)) {
+                    console.log(style(message, "red"));
+                } else if (Level.WARN.equals(level)) {
+                    console.log(style(message, "yellow"));
+                } else if (Level.DEBUG.equals(level)) {
+                    console.log(style(message, "magenta"));
+                } else if (Level.TRACE.equals(level)) {
+                    console.log(style(message, "cyan"));
+                } else {
+                    console.log(message);
+                }
+            }
+        }
+    }
+}).registerType("ConsoleAppender").as(module);
diff --git a/lib/logging/appenders/fileAppender.js b/lib/logging/appenders/fileAppender.js
@@ -11,7 +11,6 @@ var define = require("../../define.js").define,
     fs = require("fs");
 
 
-
 /**
  * @class Appends messages to a file.
  *
@@ -47,7 +46,7 @@ var define = require("../../define.js").define,
  * @param {Boolean} [options.overwrite=false] if true the log file is overwritten otherwise it is appended to.
  * @ignoreCode
  */
-exports = module.exports = define(Appender, {
+define(Appender, {
     instance:{
 
         constructor:function (options) {
@@ -64,7 +63,6 @@ exports = module.exports = define(Appender, {
         },
 
 
-
         __onExit:function () {
             var ret = new Promise();
             var ws = this.__writeStream;
@@ -83,4 +81,4 @@ exports = module.exports = define(Appender, {
             }
         }
     }
-});
+}).registerType("FileAppender").as(module);
diff --git a/lib/logging/appenders/index.js b/lib/logging/appenders/index.js
@@ -1,11 +1,11 @@
 /**@ignore*/
-exports.Appender = require("./appender"),
+exports.Appender = require("./appender");
 /**@ignore*/
-exports.ConsoleAppender = require("./consoleAppender"),
+exports.ConsoleAppender = require("./consoleAppender");
 /**@ignore*/
-exports.FileAppender = require("./fileAppender"),
+exports.FileAppender = require("./fileAppender");
 /**@ignore*/
-exports.JSONAppender = require("./jsonAppender"),
+exports.JSONAppender = require("./jsonAppender");
 /**@ignore*/
 exports.RollingFileAppender = require("./rollingFileAppender");
 
diff --git a/lib/logging/appenders/jsonAppender.js b/lib/logging/appenders/jsonAppender.js
@@ -82,4 +82,4 @@ define(FileAppender, {
             this._super(arguments);
         }
     }
-}).as(module);
+}).registerType("JSONAppender").as(module);