fix(interpolateName): don't interpolated hashType without hash or…

… `contenthash` (#140)
webpack · Dec 27, 2018 · 3528fd9 · 3528fd9
1 parent 809b690
commit 3528fd9
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/lib/interpolateName.js b/lib/interpolateName.js
@@ -91,7 +91,7 @@ function interpolateName(loaderContext, name, options) {
       // `hash` and `contenthash` are same in `loader-utils` context
       // let's keep `hash` for backward compatibility
       .replace(
-        /\[(?:([^:\]]+):)?(?:hash||contenthash)(?::([a-z]+\d*))?(?::(\d+))?\]/gi,
+        /\[(?:([^:\]]+):)?(?:hash|contenthash)(?::([a-z]+\d*))?(?::(\d+))?\]/gi,
         (all, hashType, digestType, maxLength) =>
           getHashDigest(content, hashType, digestType, parseInt(maxLength, 10))
       )

diff --git a/test/interpolateName.test.js b/test/interpolateName.test.js
@@ -124,6 +124,13 @@ describe('interpolateName()', () => {
       'test content',
       'modal.1n8osQznuT8jOAwdzg_n.css',
     ],
+    // Should not interpret without `hash` or `contenthash`
+    [
+      '/lib/components/modal/modal.css',
+      '[name].[md5::base64:20].[ext]',
+      'test content',
+      'modal.[md5::base64:20].css',
+    ],
   ].forEach((test) => {
     it('should interpolate ' + test[0] + ' ' + test[1], () => {
       const interpolatedName = loaderUtils.interpolateName(