bugfix

Author: yhzz

src/functions/Common/Parser.js

@@ -13,13 +13,19 @@ export const removeFirstSharp = (id_str) => {
 };
 
 export const removeLastSlash = (url) => {
+  if(!isValidVariable(url)){
+    return null;
+  }
   if (url.length > 1 && /\/$/.test(url)) {
     url = url.slice(0, -1);
   }
   return url;
 };
 
 export const removeLastSpace = (url) => {
+  if(!isValidVariable(url)){
+    return null;
+  }
   if (url.length > 1 && / +$/.test(url)) {
     url = url.slice(0, -1);
   }

src/functions/Common/Parser.test.js

@@ -24,6 +24,10 @@ describe('removeFirstSharp', () => {
   test('#1234', () => {
     expect(removeFirstSharp('#1234')).toBe('1234');
   });
+
+  test('null', () => {
+    expect(removeFirstSharp(null)).toBe(null);
+  });
 });
 
 describe('removeLastSlash', () => {
@@ -34,6 +38,10 @@ describe('removeLastSlash', () => {
   test('a/a', () => {
     expect(removeLastSlash('a/a')).toBe('a/a');
   });
+
+  test('null', () => {
+    expect(removeLastSlash(null)).toBe(null);
+  });
 });
 
 describe('removeLastSpace', () => {
@@ -44,6 +52,10 @@ describe('removeLastSpace', () => {
   test('a a', () => {
     expect(removeLastSpace('a a')).toBe('a a');
   });
+
+  test('null', () => {
+    expect(removeLastSpace(null)).toBe(null);
+  });
 });
 
 const failed_yaml_description_part = `\`\`\`yaml