improved getAndRemoveConfig util function

Author: luwol03

src/core/render/utils.js

@@ -23,8 +23,8 @@ export function getAndRemoveConfig(str = '') {
 
   if (str) {
     str = str
-      .replace(/^('|")/, '')
-      .replace(/('|")$/, '')
+      .replace(/^"(:.*)"$/, '$1')
+      .replace(/^'(:.*)'$/, '$1')
       .replace(/(?:^|\s):([\w-]+:?)=?([\w-%]+)?/g, (m, key, value) => {
         if (key.indexOf(':') === -1) {
           config[key] = (value && value.replace(/"/g, '')) || true;

test/unit/render-util.test.js

@@ -58,6 +58,37 @@ describe('core/render/utils', () => {
         str: `[filename](_media/example.md ":include")`,
       });
     });
+
+    test('dont parse quotes without colon', () => {
+      const result = getAndRemoveConfig(
+        `[filename](_media/example.md 'include')`
+      );
+
+      expect(result).toMatchObject({
+        config: {},
+        str: `[filename](_media/example.md 'include')`,
+      });
+    });
+
+    test('dont parse with different quotes', () => {
+      const result = getAndRemoveConfig(
+        `[filename](_media/example.md ":include')`
+      );
+
+      expect(result).toMatchObject({
+        config: {},
+        str: `[filename](_media/example.md ":include')`,
+      });
+    });
+
+    test('parse headings without quotes', () => {
+      const result = getAndRemoveConfig('### Hello, world! :id=hello-world');
+
+      expect(result).toMatchObject({
+        config: { id: 'hello-world' },
+        str: '### Hello, world!',
+      });
+    });
   });
 });