logs can be disabled

neekware · Jun 20, 2018 · f7c6271 · f7c6271
1 parent 922e02c
commit f7c6271
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 13 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+## 1.0.4
+
+Bugfix:
+
+  - Disable logs on level == none
+
 ## 1.0.3
 
 Enhancements:

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "@nwx/logger",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "repository": {
     "type": "git",
     "url": "git+https://github.com/neekware/nwx-logger.git"

diff --git a/pkgs/logger/src/logger.service.ts b/pkgs/logger/src/logger.service.ts
@@ -100,18 +100,25 @@ export class LogService {
    * @param extras extra message
    */
   private doLog(level: LogLevels, message: any, extras: any[] = []) {
-    if (message && level !== LogLevels.none && level <= this._options.log.level) {
-      if (this.isPlatformIE) {
-        this.handleIE(level, message, extras);
-      } else {
-        const color = LogColors[level];
-        console.log(
-          `%c${this.time} [${LogNames[level]}]`,
-          `color:${color}`,
-          message,
-          ...extras
-        );
-      }
+    if (
+      !message ||
+      level === LogLevels.none ||
+      level > this._options.log.level ||
+      this._options.log.level === LogLevels.none
+    ) {
+      return;
+    }
+
+    if (this.isPlatformIE) {
+      this.handleIE(level, message, extras);
+    } else {
+      const color = LogColors[level];
+      console.log(
+        `%c${this.time} [${LogNames[level]}]`,
+        `color:${color}`,
+        message,
+        ...extras
+      );
     }
   }