Merge pull request #42 from DickSmith/master

UglifyES options were never used when defined
gdborton · Dec 8, 2017 · 4e3d31a · 4e3d31a
2 parents 00064c7 + cc69fce
commit 4e3d31a
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/lib/worker.js b/lib/worker.js
@@ -53,8 +53,10 @@ function processMessage(msgLocation, callback) {
     const cacheKey = cache.createCacheKey(source + !!map, message.options);
     // We do not check the cache here because we already determined that this asset yields a cache
     // miss in the parent process.
-    const uglifier = message.options.uglifyES ? require('uglify-es') : require('uglify-js'); // eslint-disable-line global-require, max-len
-    const minifiedContent = minify(source, map, message.options.uglifyJS, uglifier);
+    const uglifyES = message.options.uglifyES;
+    const uglifyJS = message.options.uglifyJS;
+    const uglifier = uglifyES ? require('uglify-es') : require('uglify-js'); // eslint-disable-line global-require, max-len
+    const minifiedContent = minify(source, map, uglifyES || uglifyJS, uglifier);
     cache.saveToCache(cacheKey, JSON.stringify({
       source: minifiedContent.code,
       map: minifiedContent.map,