GRAILS-8746 removed environment check to determine whether to write s…

…tacktrace.log in the target directory and switched to seeing if BuildSettingsHolder has a BuildSettings; this way when running a script in any env it will work (e.g. war, test) and still works when deployed since the holder will be empty
grails · Feb 7, 2012 · c49d606 · c49d606
1 parent 469b68b
commit c49d606
Showing 1 changed file with 6 additions and 9 deletions.
diff --git a/...-plugin-log4j/src/main/groovy/org/codehaus/groovy/grails/plugins/log4j/Log4jConfig.groovy b/...-plugin-log4j/src/main/groovy/org/codehaus/groovy/grails/plugins/log4j/Log4jConfig.groovy
@@ -225,15 +225,12 @@ class Log4jConfig {
         }
 
         def fileAppender = new FileAppender(layout:DEFAULT_PATTERN_LAYOUT, name:"stacktraceLog")
-        if (Environment.current == Environment.DEVELOPMENT) {
-            BuildSettings settings = BuildSettingsHolder.getSettings()
-            def targetDir = settings?.getProjectTargetDir()
-            if (targetDir) targetDir.mkdirs()
-            fileAppender.file = targetDir ? "${targetDir.absolutePath}/stacktrace.log" : "stacktrace.log"
-        }
-        else {
-            fileAppender.file = "stacktrace.log"
-        }
+
+        BuildSettings settings = BuildSettingsHolder.getSettings()
+        def targetDir = settings?.getProjectTargetDir()
+        targetDir?.mkdirs()
+        fileAppender.file = targetDir ? "${targetDir.absolutePath}/stacktrace.log" : "stacktrace.log"
+
         fileAppender.activateOptions()
         appenders.stacktrace = fileAppender
         return fileAppender