Fix #303: better options handling

gruntjs · Mar 25, 2015 · 12c7ded · 12c7ded
1 parent 8f9bb92
commit 12c7ded
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 5 deletions.
diff --git a/tasks/lib/uglify.js b/tasks/lib/uglify.js
@@ -83,6 +83,9 @@ exports.init = function(grunt) {
       if (options.compress.warnings !== true) {
         options.compress.warnings = false;
       }
+      if (options.screwIE8) {
+        options.compress.screw_ie8 = true;
+      }
       var compressor = UglifyJS.Compressor(options.compress);
       topLevel = topLevel.transform(compressor);
 
@@ -91,6 +94,12 @@ exports.init = function(grunt) {
     }
 
     if (options.mangle !== false) {
+      if (options.mangle === true) {
+        options.mangle = {};
+      }
+      if (options.screwIE8) {
+        options.mangle.screw_ie8 = true;
+      }
       // disabled due to:
       //   1) preserve stable name mangling
       //   2) it increases gzipped file size, see https://github.com/mishoo/UglifyJS2#mangler-options

diff --git a/tasks/uglify.js b/tasks/uglify.js
@@ -131,11 +131,6 @@ module.exports = function(grunt) {
         options.destToSourceMap = destToSourceMapPath + sourceMapBasename;
       }
 
-      if (options.screwIE8) {
-        if (options.mangle) { options.mangle.screw_ie8 = true; }
-        if (options.compress) { options.compress.screw_ie8 = true; }
-      }
-
       // Minify files, warn and fail on error.
       var result;
       try {