From 1e0b046054dfa9089e5d91a6dff2df3a45b1a152 Mon Sep 17 00:00:00 2001
From: yp05327 <576951401@qq.com>
Date: Sun, 19 Feb 2023 15:23:16 +0900
Subject: [PATCH 1/2] improve GetBoards

---
 models/project/board.go      | 6 +++---
 routers/web/org/projects.go  | 2 +-
 routers/web/repo/projects.go | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/models/project/board.go b/models/project/board.go
index dc4e2e6882a55..b8d9f5e093ab9 100644
--- a/models/project/board.go
+++ b/models/project/board.go
@@ -229,14 +229,14 @@ func UpdateBoard(ctx context.Context, board *Board) error {
 
 // GetBoards fetches all boards related to a project
 // if no default board set, first board is a temporary "Uncategorized" board
-func GetBoards(ctx context.Context, projectID int64) (BoardList, error) {
+func (p *Project) GetBoards(ctx context.Context) (BoardList, error) {
 	boards := make([]*Board, 0, 5)
 
-	if err := db.GetEngine(ctx).Where("project_id=? AND `default`=?", projectID, false).OrderBy("Sorting").Find(&boards); err != nil {
+	if err := db.GetEngine(ctx).Where("project_id=? AND `default`=?", p.ID, false).OrderBy("Sorting").Find(&boards); err != nil {
 		return nil, err
 	}
 
-	defaultB, err := getDefaultBoard(ctx, projectID)
+	defaultB, err := getDefaultBoard(ctx, p.ID)
 	if err != nil {
 		return nil, err
 	}
diff --git a/routers/web/org/projects.go b/routers/web/org/projects.go
index 6449d12de105a..03272088b7128 100644
--- a/routers/web/org/projects.go
+++ b/routers/web/org/projects.go
@@ -292,7 +292,7 @@ func ViewProject(ctx *context.Context) {
 		return
 	}
 
-	boards, err := project_model.GetBoards(ctx, project.ID)
+	boards, err := project.GetBoards(ctx)
 	if err != nil {
 		ctx.ServerError("GetProjectBoards", err)
 		return
diff --git a/routers/web/repo/projects.go b/routers/web/repo/projects.go
index 967b81c608516..a8c94ad2c3960 100644
--- a/routers/web/repo/projects.go
+++ b/routers/web/repo/projects.go
@@ -294,7 +294,7 @@ func ViewProject(ctx *context.Context) {
 		return
 	}
 
-	boards, err := project_model.GetBoards(ctx, project.ID)
+	boards, err := project.GetBoards(ctx)
 	if err != nil {
 		ctx.ServerError("GetProjectBoards", err)
 		return

From f47a201117014972d7c1df377861c7db5f2caf98 Mon Sep 17 00:00:00 2001
From: yp05327 <576951401@qq.com>
Date: Sun, 19 Feb 2023 15:27:49 +0900
Subject: [PATCH 2/2] improve getDefaultBoard

---
 models/project/board.go | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/models/project/board.go b/models/project/board.go
index b8d9f5e093ab9..2ab1d5c7b5136 100644
--- a/models/project/board.go
+++ b/models/project/board.go
@@ -236,7 +236,7 @@ func (p *Project) GetBoards(ctx context.Context) (BoardList, error) {
 		return nil, err
 	}
 
-	defaultB, err := getDefaultBoard(ctx, p.ID)
+	defaultB, err := p.getDefaultBoard(ctx)
 	if err != nil {
 		return nil, err
 	}
@@ -245,9 +245,9 @@ func (p *Project) GetBoards(ctx context.Context) (BoardList, error) {
 }
 
 // getDefaultBoard return default board and create a dummy if none exist
-func getDefaultBoard(ctx context.Context, projectID int64) (*Board, error) {
+func (p *Project) getDefaultBoard(ctx context.Context) (*Board, error) {
 	var board Board
-	exist, err := db.GetEngine(ctx).Where("project_id=? AND `default`=?", projectID, true).Get(&board)
+	exist, err := db.GetEngine(ctx).Where("project_id=? AND `default`=?", p.ID, true).Get(&board)
 	if err != nil {
 		return nil, err
 	}
@@ -257,7 +257,7 @@ func getDefaultBoard(ctx context.Context, projectID int64) (*Board, error) {
 
 	// represents a board for issues not assigned to one
 	return &Board{
-		ProjectID: projectID,
+		ProjectID: p.ID,
 		Title:     "Uncategorized",
 		Default:   true,
 	}, nil