From d9ed56f7c2ce3fc3f4c6027f0ae64892b0037ae2 Mon Sep 17 00:00:00 2001
From: CommanderRoot <CommanderRoot@users.noreply.github.com>
Date: Fri, 18 Mar 2022 12:25:46 +0100
Subject: [PATCH] refactor: replace deprecated `String.prototype.substr()`
 (#1227)

---
 src/utils/getFilenameFromUrl.js |  2 +-
 src/utils/setupWriteToDisk.js   |  2 +-
 test/middleware.test.js         | 10 +++++-----
 3 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/utils/getFilenameFromUrl.js b/src/utils/getFilenameFromUrl.js
index 9a555281f..4aa845b9a 100644
--- a/src/utils/getFilenameFromUrl.js
+++ b/src/utils/getFilenameFromUrl.js
@@ -86,7 +86,7 @@ function getFilenameFromUrl(context, url) {
 
       // Strip the `pathname` property from the `publicPath` option from the start of requested url
       // `/complex/foo.js` => `foo.js`
-      const pathname = urlObject.pathname.substr(
+      const pathname = urlObject.pathname.slice(
         publicPathObject.pathname.length
       );
 
diff --git a/src/utils/setupWriteToDisk.js b/src/utils/setupWriteToDisk.js
index ac8958984..4ac3af302 100644
--- a/src/utils/setupWriteToDisk.js
+++ b/src/utils/setupWriteToDisk.js
@@ -53,7 +53,7 @@ function setupWriteToDisk(context) {
               const queryStringIdx = targetFile.indexOf("?");
 
               if (queryStringIdx >= 0) {
-                targetFile = targetFile.substr(0, queryStringIdx);
+                targetFile = targetFile.slice(0, queryStringIdx);
               }
 
               let { outputPath } = compiler;
diff --git a/test/middleware.test.js b/test/middleware.test.js
index 8808cc6bb..646836a7a 100644
--- a/test/middleware.test.js
+++ b/test/middleware.test.js
@@ -281,7 +281,7 @@ describe.each([
           expect(response.headers["content-type"]).toEqual(
             "application/javascript; charset=utf-8"
           );
-          expect(response.text).toBe(codeContent.substr(3000, 501));
+          expect(response.text).toBe(codeContent.slice(3000, 3501));
           expect(response.text.length).toBe(501);
         });
 
@@ -314,7 +314,7 @@ describe.each([
           expect(response.headers["content-type"]).toEqual(
             "application/javascript; charset=utf-8"
           );
-          expect(response.text).toBe(codeContent.substr(3000, 501));
+          expect(response.text).toBe(codeContent.slice(3000, 3501));
           expect(response.text.length).toBe(501);
         });
 
@@ -331,7 +331,7 @@ describe.each([
           expect(response.headers["content-type"]).toEqual(
             "application/javascript; charset=utf-8"
           );
-          expect(response.text).toBe(codeContent.substr(3000, 501));
+          expect(response.text).toBe(codeContent.slice(3000, 3501));
           expect(response.text.length).toBe(501);
         });
 
@@ -348,7 +348,7 @@ describe.each([
           expect(response.headers["content-type"]).toEqual(
             "application/javascript; charset=utf-8"
           );
-          expect(response.text).toBe(codeContent.substr(0, 3501));
+          expect(response.text).toBe(codeContent.slice(0, 3501));
           expect(response.text.length).toBe(3501);
         });
 
@@ -365,7 +365,7 @@ describe.each([
           expect(response.headers["content-type"]).toEqual(
             "application/javascript; charset=utf-8"
           );
-          expect(response.text).toBe(codeContent.substr(0, 801));
+          expect(response.text).toBe(codeContent.slice(0, 801));
           expect(response.text.length).toBe(801);
         });