Merge pull request #52 from Va1/develop

merge develop into master

Author: Va1

README.md

@@ -94,6 +94,31 @@ module.exports = {
 }
 ```
 
+### Callback replacement
+
+You can specify a callback function to have dynamic replacement values.
+
+In your `webpack.config.js`:
+
+```javascript
+module.exports = {
+  // ...
+  module: {
+    rules: [
+      {
+        test: /\.js$/,
+        loader: 'string-replace-loader',
+        options: {
+          search: '^Hello, (.*)!$',
+          replace: (match, p1, offset, string) => `Bonjour, ${p1.toUpperCase()}!`,
+          flags: 'g'
+        }
+      }
+    ]
+  }
+}
+``` 
+
 ### Strict mode replacement:
 
 You can enable strict mode to ensure that the replacement was performed.

lib/getOptionsArray.js

@@ -10,7 +10,14 @@ const optionsSchema = {
       type: 'string'
     },
     replace: {
-      type: 'string'
+      anyOf: [
+        {
+          typeof: 'function'
+        },
+        {
+          type: 'string'
+        }
+      ]
     },
     flags: {
       type: 'string',

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "string-replace-loader",
-  "version": "2.1.1",
+  "version": "2.2.0",
   "description": "Replace loader for Webpack",
   "keywords": [
     "webpack",

package.json

@@ -55,9 +55,9 @@ describe('Webpack replace loader ...', () => {
         test: /\.js$/,
         loader: '__this-loader',
         options: {
-          search: 'var VALUE = \'\.*\'',
-          replace: 'var a = \'\'',
-          flags: 'i'
+          search: `var VALUE = '.*'`,
+          replace: `var a = ''`,
+          flags: 'ig'
         }
       }),
       (error, stats) => {
@@ -67,7 +67,7 @@ describe('Webpack replace loader ...', () => {
           expect(error).to.equal(null)
           expect(contents).to.be.a('string')
           expect(contents).to.not.include('var value')
-          expect(contents).to.include('var a = \'\'')
+          expect(contents).to.include(`var a = ''`)
           done()
         })
       }
@@ -107,6 +107,30 @@ describe('Webpack replace loader ...', () => {
     )
   })
 
+  it('should replace with function replace', done => {
+    webpack(getTestWebPackConfig(
+      {
+        test: /\.js$/,
+        loader: '__this-loader',
+        options: {
+          search: `var value = '(baz)'`,
+          replace: (match, p1, offset, string) => `var a = '${p1.toUpperCase()}'`,
+          flags: 'g'
+        }
+      }),
+      (error, stats) => {
+        expect(error).to.equal(null)
+
+        fs.readFile(outputFilePath, 'utf8', (error, contents) => {
+          expect(error).to.equal(null)
+          expect(contents).to.be.a('string')
+          expect(contents).to.include(`var a = 'BAZ'`)
+          done()
+        })
+      }
+    )
+  })
+
   it('should replace using multiple queries', done => {
     webpack(getTestWebPackConfig(
       {