Merge pull request #293 from wood1986/fix-mutating-issue

fix: call stats.toJson on each time to avoid the stats mutation
webpack-contrib · Jul 29, 2019 · fa2ac7b · fa2ac7b
2 parents 44bd8d0 + 1cf518d
commit fa2ac7b
Showing 1 changed file with 3 additions and 4 deletions.
diff --git a/src/BundleAnalyzerPlugin.js b/src/BundleAnalyzerPlugin.js
@@ -35,12 +35,11 @@ class BundleAnalyzerPlugin {
 
     const done = (stats, callback) => {
       callback = callback || (() => {});
-      stats = stats.toJson(this.opts.statsOptions);
 
       const actions = [];
 
       if (this.opts.generateStatsFile) {
-        actions.push(() => this.generateStatsFile(stats));
+        actions.push(() => this.generateStatsFile(stats.toJson(this.opts.statsOptions)));
       }
 
       // Handling deprecated `startAnalyzer` flag
@@ -49,9 +48,9 @@ class BundleAnalyzerPlugin {
       }
 
       if (this.opts.analyzerMode === 'server') {
-        actions.push(() => this.startAnalyzerServer(stats));
+        actions.push(() => this.startAnalyzerServer(stats.toJson()));
       } else if (this.opts.analyzerMode === 'static') {
-        actions.push(() => this.generateStaticReport(stats));
+        actions.push(() => this.generateStaticReport(stats.toJson()));
       }
 
       if (actions.length) {