Fixed bug that failed test 'set level on all categories'

Luis Malheiro · Luis Malheiro · commit 39a73586edd1 · 2014-09-01T14:37:25.000+02:00
diff --git a/lib/log4js.js b/lib/log4js.js
@@ -240,13 +240,13 @@ function configureLevels(_levels) {
     var keys = Object.keys(levels.config).sort();
     for (var idx in keys) {
       var category = keys[idx];
+      if(category === ALL_CATEGORIES) {
+        setGlobalLogLevel(_levels[category]);
+      }        
       for(var loggerCategory in loggers) {
         if (doesLevelEntryContainsLogger(category, loggerCategory)) {
           loggers[loggerCategory].setLevel(_levels[category]);
         }
-        if(category === ALL_CATEGORIES) {
-          setGlobalLogLevel(levels[category]);
-        }        
       }
     }
   }

Original file line number	Diff line number	Diff line change
`@@ -240,13 +240,13 @@ function configureLevels(_levels) {`
`240`	`240`	`var keys = Object.keys(levels.config).sort();`
`241`	`241`	`for (var idx in keys) {`
`242`	`242`	`var category = keys[idx];`
	`243`	`+ if(category === ALL_CATEGORIES) {`
	`244`	`+ setGlobalLogLevel(_levels[category]);`
	`245`	`+ }`
`243`	`246`	`for(var loggerCategory in loggers) {`
`244`	`247`	`if (doesLevelEntryContainsLogger(category, loggerCategory)) {`
`245`	`248`	`loggers[loggerCategory].setLevel(_levels[category]);`
`246`	`249`	`}`
`247`		`- if(category === ALL_CATEGORIES) {`
`248`		`- setGlobalLogLevel(levels[category]);`
`249`		`- }`
`250`	`250`	`}`
`251`	`251`	`}`
`252`	`252`	`}`