Fix 'redundant character escape' warning using inspection

dilame · Apr 4, 2019 · c14ad5d · c14ad5d
1 parent af8b735
commit c14ad5d
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 13 deletions.
diff --git a/src/finder.ts b/src/finder.ts
@@ -22,7 +22,7 @@ export function main (usernameToFind, commentToFind, postUrl) {
 
     let mediaId;
 
-    const re = /https?:\/\/(www\.)?[-a-zA-Z0-9@:%._\+~#=]{2,256}\.[a-z]{2,6}\b([-a-zA-Z0-9@:%_\+.~#?&//=]*)/;
+    const re = /https?:\/\/(www\.)?[-a-zA-Z0-9@:%._+~#=]{2,256}\.[a-z]{2,6}\b([-a-zA-Z0-9@:%_+.~#?&//=]*)/;
 
     if (postUrl.match(re)) {
       mediaId = postUrl.split('/')[4];

diff --git a/src/v1/json-pruned.ts b/src/v1/json-pruned.ts
@@ -7,7 +7,7 @@ let seen; // Same variable used for all stringifications
 (Date as any).prototype.toPrunedJSON = Date.prototype.toJSON;
 (String as any).prototype.toPrunedJSON = (String as any).prototype.toJSON;
 
-const escapable = /[\\\"\x00-\x1f\x7f-\x9f\u00ad\u0600-\u0604\u070f\u17b4\u17b5\u200c-\u200f\u2028-\u202f\u2060-\u206f\ufeff\ufff0-\uffff]/g;
+const escapable = /[\\"\x00-\x1f\x7f-\x9f\u00ad\u0600-\u0604\u070f\u17b4\u17b5\u200c-\u200f\u2028-\u202f\u2060-\u206f\ufeff\ufff0-\uffff]/g;
 
 const meta = {
   // table of character substitutions

diff --git a/src/v1/media.ts b/src/v1/media.ts
@@ -126,11 +126,11 @@ export class Media extends Resource {
             source_height: height,
           },
         });
-        payload = payload.replace(/\"\$width\"/gi, width.toFixed(1));
-        payload = payload.replace(/\"\$height\"/gi, height.toFixed(1));
-        payload = payload.replace(/\"\$zero\"/gi, (0).toFixed(1));
-        payload = payload.replace(/\"\$negativeZero\"/gi, `-${(0).toFixed(1)}`);
-        payload = payload.replace(/\"\$crop\"/gi, CROP.toFixed(1));
+        payload = payload.replace(/"\$width"/gi, width.toFixed(1));
+        payload = payload.replace(/"\$height"/gi, height.toFixed(1));
+        payload = payload.replace(/"\$zero"/gi, (0).toFixed(1));
+        payload = payload.replace(/"\$negativeZero"/gi, `-${(0).toFixed(1)}`);
+        payload = payload.replace(/"\$crop"/gi, CROP.toFixed(1));
 
         return new Request(session)
           .setMethod('POST')
@@ -167,11 +167,11 @@ export class Media extends Resource {
             source_height: height,
           },
         });
-        payload = payload.replace(/\"\$width\"/gi, width.toFixed(1));
-        payload = payload.replace(/\"\$height\"/gi, height.toFixed(1));
-        payload = payload.replace(/\"\$zero\"/gi, (0).toFixed(1));
-        payload = payload.replace(/\"\$negativeZero\"/gi, `-${(0).toFixed(1)}`);
-        payload = payload.replace(/\"\$crop\"/gi, CROP.toFixed(1));
+        payload = payload.replace(/"\$width"/gi, width.toFixed(1));
+        payload = payload.replace(/"\$height"/gi, height.toFixed(1));
+        payload = payload.replace(/"\$zero"/gi, (0).toFixed(1));
+        payload = payload.replace(/"\$negativeZero"/gi, `-${(0).toFixed(1)}`);
+        payload = payload.replace(/"\$crop"/gi, CROP.toFixed(1));
 
         return new Request(session)
           .setMethod('POST')

diff --git a/tests/cases/device.js b/tests/cases/device.js
@@ -102,7 +102,7 @@ describe('`Device` class', () => {
   });
 
   it('should not be problem to get userAgent', () => {
-    const yellowstone = /^Instagram\s[0-9\.]*\sAndroid\s\(18\/4.0.4\;\s424dpi\;\s1080x1920\;\sGoogle\;\sYellowstone\;\syellowstone\;\sen_US\)$/;
+    const yellowstone = /^Instagram\s[0-9.]*\sAndroid\s\(18\/4.0.4;\s424dpi;\s1080x1920;\sGoogle;\sYellowstone;\syellowstone;\sen_US\)$/;
     device1.userAgent().should.match(yellowstone);
   });
 });