From 4fb393e98f8fde16302b3b0792749b196274a436 Mon Sep 17 00:00:00 2001
From: Patrik Oldsberg <poldsberg@gmail.com>
Date: Wed, 10 Mar 2021 22:35:31 +0100
Subject: [PATCH] todo-backend: use integrations to resolve line numbers

Signed-off-by: Patrik Oldsberg <poldsberg@gmail.com>
---
 .../todo-backend/src/lib/TodoReader/TodoScmReader.ts  | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/plugins/todo-backend/src/lib/TodoReader/TodoScmReader.ts b/plugins/todo-backend/src/lib/TodoReader/TodoScmReader.ts
index 7979b8f18e..316ac7840f 100644
--- a/plugins/todo-backend/src/lib/TodoReader/TodoScmReader.ts
+++ b/plugins/todo-backend/src/lib/TodoReader/TodoScmReader.ts
@@ -114,10 +114,6 @@ export class TodoScmReader implements TodoReader {
           path: file.path,
           content: content.toString('utf8'),
         });
-        const viewUrl = this.integrations.resolveUrl({
-          url: file.path,
-          base: url,
-        });
 
         todos.push(
           ...items.map(({ lineNumber, text, author }) => ({
@@ -125,8 +121,11 @@ export class TodoScmReader implements TodoReader {
             author,
             lineNumber,
             repoFilePath: file.path,
-            viewUrl:
-              lineNumber === undefined ? viewUrl : `${viewUrl}#L${lineNumber}`,
+            viewUrl: this.integrations.resolveUrl({
+              url: file.path,
+              base: url,
+              lineNumber,
+            }),
           })),
         );
       } catch (error) {