fix: Fix replacing multiple links at once

svsool · Oct 31, 2020 · 2d70222 · 2d70222
1 parent 8bd7863
commit 2d70222
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/src/utils/utils.spec.ts b/src/utils/utils.spec.ts
@@ -766,6 +766,21 @@ describe('replaceRefs()', () => {
  }),
  ).toBe(initialContent);
  });
+
+ it('should replace multiple links at once', async () => {
+ expect(
+ replaceRefs({
+ refs: [
+ { old: 'test-ref', new: 'folder2/new-test-ref' },
+ { old: 'folder1/test-ref', new: 'folder2/new-test-ref' },
+ ],
+ document: await workspace.openTextDocument({
+ language: 'markdown',
+ content: '[[test-ref]] [[folder1/test-ref|Test Label]]',
+ }),
+ }),
+ ).toBe('[[folder2/new-test-ref]] [[folder2/new-test-ref|Test Label]]');
+ });
 });
 
 describe('isLongRef()', () => {

diff --git a/src/utils/utils.ts b/src/utils/utils.ts
@@ -471,7 +471,7 @@ export const replaceRefs = ({
  if (new RegExp(pattern, 'i').exec(content)) {
  let replacedOnce = false;
 
- const nextContent = content.replace(new RegExp(pattern, 'gi'), ($0, $1, offset) => {
+ const content = nextContent.replace(new RegExp(pattern, 'gi'), ($0, $1, offset) => {
  const pos = document.positionAt(offset);
 
  if (
@@ -494,7 +494,7 @@ export const replaceRefs = ({
 
  return {
  updatedOnce: true,
- nextContent,
+ nextContent: content,
  };
  }