fix: copying same file to multiple targets (#165)

webpack-contrib · Nov 17, 2017 · 43a9870 · 43a9870
1 parent 1ceee6c
commit 43a9870
Show file tree

Hide file tree

Showing 3 changed files with 44 additions and 22 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/src/writeFile.js b/src/writeFile.js
@@ -67,13 +67,17 @@ export default function writeFile(globalRef, pattern, file) {
             }
 
             if (!copyUnmodified &&
-                written[file.absoluteFrom] && written[file.absoluteFrom][hash]) {
+                written[file.absoluteFrom] &&
+                written[file.absoluteFrom]['hash'] === hash &&
+                written[file.absoluteFrom]['webpackTo'] === file.webpackTo
+            ) {
                 info(`skipping '${file.webpackTo}', because it hasn't changed`);
                 return;
             } else {
                 debug(`added ${hash} to written tracking for '${file.absoluteFrom}'`);
                 written[file.absoluteFrom] = {
-                    [hash]: true
+                    hash: hash,
+                    webpackTo: file.webpackTo
                 };
             }
 

diff --git a/tests/index.js b/tests/index.js
@@ -820,6 +820,24 @@ describe('apply function', () => {
             .then(done)
             .catch(done);
         });
+
+        it('same file to multiple targets', (done) => {
+            runEmit({
+                expectedAssetKeys: [
+                    'first/file.txt',
+                    'second/file.txt'
+                ],
+                patterns: [{
+                    from: 'file.txt',
+                    to: 'first/file.txt'
+                }, {
+                    from: 'file.txt',
+                    to: 'second/file.txt'
+                }]
+            })
+              .then(done)
+              .catch(done);
+        });
     });
 
     describe('with directory in from', () => {