From cb59a7ad40c86f3ce3ce8fd7b838e9e09634a361 Mon Sep 17 00:00:00 2001
From: Runxi Yu <me@runxiyu.org>
Date: Wed, 02 Apr 2025 02:44:45 +0800
Subject: [PATCH] Git: Return fsPath from OpenRepo

---
 git_misc.go    | 4 +---
 http_server.go | 2 +-

diff --git a/git_misc.go b/git_misc.go
index a8d7c30e7205d53808502a817a9ded4de3aef340..3ff6f81ab228d88a5a84ec0116f4be42524cd48a 100644
--- a/git_misc.go
+++ b/git_misc.go
@@ -22,9 +22,7 @@ //
 // TODO: This should be deprecated in favor of doing it in the relevant
 // request/router context in the future, as it cannot cover the nuance of
 // fields needed.
-func openRepo(ctx context.Context, groupPath []string, repoName string) (repo *git.Repository, description string, repoID int, err error) {
-	var fsPath string
-
+func openRepo(ctx context.Context, groupPath []string, repoName string) (repo *git.Repository, description string, repoID int, fsPath string, err error) {
 	err = database.QueryRow(ctx, `
 WITH RECURSIVE group_path_cte AS (
 	-- Start: match the first name in the path where parent_group IS NULL
diff --git a/http_server.go b/http_server.go
index 8931853052ef46f4fd3ae08822ca79f6cf892e4b..739c0aaa9b63696bf8b63f0de621b8ed6d0cc5fc 100644
--- a/http_server.go
+++ b/http_server.go
@@ -181,7 +181,7 @@ 			}
 
 			// TODO: subgroups
 
-			if params["repo"], params["repo_description"], params["repo_id"], err = openRepo(request.Context(), groupPath, moduleName); err != nil {
+			if params["repo"], params["repo_description"], params["repo_id"], _, err = openRepo(request.Context(), groupPath, moduleName); err != nil {
 				errorPage500(writer, params, "Error opening repo: "+err.Error())
 				return
 			}

-- 
2.48.1