jlog-rails 0.1.0 → 0.1.1

data/README.md

@@ -23,6 +23,12 @@ On the client-side:
 Changelog:
 ----------
 
+* 0.1.1
+  * Cleaned JLog.JS code
+* 0.1.0
+  * JLog rails engine can now be mounted several times
+  * Added ability to configure the ruby logger(s) and the output format(s)
+  * Fixed JSHint warnings
 * 0.0.3
   * Adding bulk flush in the ajax appender
 * 0.0.2

data/lib/jlog/version.rb

@@ -1,3 +1,3 @@
 module Jlog
-  VERSION = '0.1.0'
+  VERSION = '0.1.1'
 end

data/vendor/assets/javascripts/jlog.js

@@ -26,155 +26,158 @@
 
 /**
 *
-* @remixer Helios Technologies Ltd. mailto:contact@heliostech.fr
+* @remixer Helios Technologies mailto:contact@heliostech.fr
 *
 **/
 
 function JLog(name) {
-  var _currentLevel = JLog.ALL,
-      _appenders = [new JLog.ConsoleAppender()],
-      _name = null,
-      _enabled = true;
+  this._appenders.push(new JLog.ConsoleAppender());
+  this.setName(name);
+}
 
-  this.setName = function(name) {
-    _name = name || null;
-  };
+JLog.ALL   = 0;
+JLog.DEBUG = 1;
+JLog.INFO  = 2;
+JLog.WARN  = 3;
+JLog.ERROR = 4;
+JLog.FATAL = 5;
+JLog.NONE  = 6;
+
+JLog.prototype = {
+  _currentLevel: JLog.ALL,
+  _appenders: [],
+  _enabled: true,
+  _name: null,
+
+  debug: function() {
+    if (this.getLevel() <= JLog.DEBUG) {
+      this._log("DEBUG", arguments);
+    }
+  },
 
-  if (name) {
-    this.setName(name);
-  }
+  info: function() {
+    if (this.getLevel() <= JLog.INFO) {
+      this._log("INFO", arguments);
+    }
+  },
+
+  warn: function() {
+    if (this.getLevel() <= JLog.WARN) {
+      this._log("WARN", arguments);
+    }
+  },
 
-  this.addAppender = function(appender) {
+  error: function() {
+    if (this.getLevel() <= JLog.ERROR) {
+      this._log("ERROR", arguments);
+    }
+  },
+
+  fatal: function() {
+    if (this.getLevel() <= JLog.FATAL) {
+      this._log("FATAL", arguments);
+    }
+  },
+
+  _log: function() {
+    if (this.isOn()) {
+      var level = arguments[0],
+      args = Array.prototype.slice.call(arguments[1] || []),
+      namePrefix = this.getName() ? '[' + this.getName() + ']' : '',
+      msgString = level + namePrefix + ': ',
+      appenders = this.getAppenders();
+
+      for (var i in args) {
+        if (typeof args[i] === 'object') {
+          args[i] = JSON.stringify(args[i]);
+        }
+      }
+
+      msgString += args.join(', ');
+      for(i = 0; i < appenders.length; ++i) {
+        appenders[i].log(msgString);
+      }
+    }
+  },
+
+  getName: function() {
+    return _name;
+  },
+
+  setName: function(name) {
+    _name = name || null;
+  },
+
+  addAppender: function(appender) {
     if (appender) {
-      _appenders.push(appender);
+      this._appenders.push(appender);
     }
-  };
+  },
 
-  this.removeAppender = function(name) {
+  removeAppender: function(name) {
     for (var i in _appenders) {
-      if (_appenders[i].name === name) {
-        _appenders.splice(i, 1);
+      if (this._appenders[i].name === name) {
+        this._appenders.splice(i, 1);
       }
     }
     return null;
-  };
+  },
 
-  this.turnOn = function() {
-    _enabled = true;
-  };
+  turnOn: function() {
+    this._enabled = true;
+  },
 
-  this.turnOff = function() {
-    _enabled = false;
-  };
+  turnOff: function() {
+    this._enabled = false;
+  },
 
-  this.isOn = function() {
-    return _enabled;
-  };
+  isOn: function() {
+    return this._enabled;
+  },
 
-  // Sets the current threshold log level for this Log instance.  Only events that have a priority of this level or greater are logged.
-  this.setLevel = function(level) {
+  //
+  // Sets the current threshold log level for this Log instance.
+  // Only events that have a priority of this level or greater are logged.
+  //
+  setLevel: function(level) {
     if (typeof level === 'number') {
       if (level >= JLog.ALL && level <= JLog.NONE) {
-        _currentLevel = level;
+        this._currentLevel = level;
       } else {
-        _currentLevel = JLog.NONE;
+        this._currentLevel = JLog.NONE;
       }
     } else if (level) {
       switch(level) {
-        case 'all': _currentLevel = JLog.ALL; break;
-        case 'debug': _currentLevel = JLog.DEBUG; break;
-        case 'info': _currentLevel = JLog.INFO; break;
-        case 'warn': _currentLevel = JLog.WARN; break;
-        case 'error': _currentLevel = JLog.ERROR; break;
-        case 'fatal': _currentLevel = JLog.FATAL; break;
-        default: _currentLevel = JLog.NONE;
+        case 'all': this._currentLevel = JLog.ALL; break;
+        case 'debug': this._currentLevel = JLog.DEBUG; break;
+        case 'info': this._currentLevel = JLog.INFO; break;
+        case 'warn': this._currentLevel = JLog.WARN; break;
+        case 'error': this._currentLevel = JLog.ERROR; break;
+        case 'fatal': this._currentLevel = JLog.FATAL; break;
+        default: this._currentLevel = JLog.NONE;
       }
     } else {
-      _currentLevel = JLog.NONE;
+      this._currentLevel = JLog.NONE;
     }
-  };
-
-  this.getName = function() {
-    return _name;
-  };
+  },
 
-  this.getAppender = function(name) {
-    for (var i in _appenders) {
-      if (_appenders[i].name === name) {
-        return _appenders[i];
+  getAppender: function(name) {
+    for (var i in this._appenders) {
+      if (this._appenders[i].name === name) {
+        return this._appenders[i];
       }
     }
     return null;
-  };
-
-  this.getAppenders = function() {
-    return _appenders;
-  };
-
-  this.getLevel = function() {
-    return _currentLevel;
-  };
-}
-
-JLog.ALL    = 0;
-JLog.DEBUG  = 1;
-JLog.INFO   = 2;
-JLog.WARN   = 3;
-JLog.ERROR  = 4;
-JLog.FATAL  = 5;
-JLog.NONE   = 6;
+  },
 
+  getAppenders: function() {
+    return this._appenders;
+  },
 
-JLog.prototype.debug = function() {
-  if (this.getLevel() <= JLog.DEBUG) {
-    this._log("DEBUG", arguments);
+  getLevel: function() {
+    return this._currentLevel;
   }
-};
-
-JLog.prototype.info = function() {
-  if (this.getLevel() <= JLog.INFO) {
-    this._log("INFO", arguments);
-  }
-};
-
-JLog.prototype.warn = function() {
-  if (this.getLevel() <= JLog.WARN) {
-    this._log("WARN", arguments);
-  }
-};
-
-JLog.prototype.error = function() {
-  if (this.getLevel() <= JLog.ERROR) {
-    this._log("ERROR", arguments);
-  }
-};
-
-JLog.prototype.fatal = function() {
-  if (this.getLevel() <= JLog.FATAL) {
-    this._log("FATAL", arguments);
-  }
-};
-
-JLog.prototype._log = function() {
-  if (this.isOn()) {
-    var level = arguments[0],
-        args = Array.prototype.slice.call(arguments[1]),
-        namePrefix = this.getName() ? '[' + this.getName() + ']' : '',
-        msgString = level + namePrefix + ': ',
-        appenders = this.getAppenders();
-
-    for (var i in args) {
-      if (typeof args[i] === 'object') {
-        args[i] = JSON.stringify(args[i]);
-      }
-    }
-
-    msgString += args.join(', ');
-    for(i = 0; i < appenders.length; ++i) {
-      appenders[i].log(msgString);
-    }
-  }
-};
+}
 
 JLog.ConsoleAppender = function() {
   this.name = 'ConsoleAppender';

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jlog-rails
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.1.1
   prerelease: 
 platform: ruby
 authors: