routers

Author: 6543

routers/repo/projects.go

@@ -410,21 +410,19 @@ func AddBoardToProjectPost(ctx *context.Context, form auth.EditProjectBoardTitle
 	})
 }
 
-// EditProjectBoardTitle allows a project board's title to be updated
-func EditProjectBoardTitle(ctx *context.Context, form auth.EditProjectBoardTitleForm) {
-
+func checkProjectBoardChangePermissions(ctx *context.Context) (*models.Project, *models.ProjectBoard) {
 	if ctx.User == nil {
 		ctx.JSON(403, map[string]string{
 			"message": "Only signed in users are allowed to perform this action.",
 		})
-		return
+		return nil, nil
 	}
 
 	if !ctx.Repo.IsOwner() && !ctx.Repo.IsAdmin() && !ctx.Repo.CanAccess(models.AccessModeWrite, models.UnitTypeProjects) {
 		ctx.JSON(403, map[string]string{
 			"message": "Only authorized users are allowed to perform this action.",
 		})
-		return
+		return nil, nil
 	}
 
 	project, err := models.GetProjectByID(ctx.ParamsInt64(":id"))
@@ -434,33 +432,43 @@ func EditProjectBoardTitle(ctx *context.Context, form auth.EditProjectBoardTitle
 		} else {
 			ctx.ServerError("GetProjectByID", err)
 		}
-		return
+		return nil, nil
 	}
 
 	board, err := models.GetProjectBoard(ctx.ParamsInt64(":boardID"))
 	if err != nil {
 		ctx.InternalServerError(err)
-		return
+		return nil, nil
 	}
 	if board.ProjectID != ctx.ParamsInt64(":id") {
 		ctx.JSON(422, map[string]string{
 			"message": fmt.Sprintf("ProjectBoard[%d] is not in Project[%d] as expected", board.ID, project.ID),
 		})
-		return
+		return nil, nil
 	}
 
 	if project.RepoID != ctx.Repo.Repository.ID {
 		ctx.JSON(422, map[string]string{
 			"message": fmt.Sprintf("ProjectBoard[%d] is not in Repository[%d] as expected", board.ID, ctx.Repo.Repository.ID),
 		})
+		return nil, nil
+	}
+	return project, board
+}
+
+// EditProjectBoardTitle allows a project board's title to be updated
+func EditProjectBoardTitle(ctx *context.Context, form auth.EditProjectBoardTitleForm) {
+
+	_, board := checkProjectBoardChangePermissions(ctx)
+	if ctx.Written() {
 		return
 	}
 
 	if form.Title != "" {
 		board.Title = form.Title
 	}
 
-	if err = models.UpdateProjectBoard(board); err != nil {
+	if err := models.UpdateProjectBoard(board); err != nil {
 		ctx.ServerError("UpdateProjectBoard", err)
 		return
 	}
@@ -470,6 +478,24 @@ func EditProjectBoardTitle(ctx *context.Context, form auth.EditProjectBoardTitle
 	})
 }
 
+// SetDefaultProjectBoard set default board for uncategorized issues/pulls
+func SetDefaultProjectBoard(ctx *context.Context) {
+
+	project, board := checkProjectBoardChangePermissions(ctx)
+	if ctx.Written() {
+		return
+	}
+
+	if err := models.SetDefaultBoard(project.ID, board.ID); err != nil {
+		ctx.InternalServerError(err)
+		return
+	}
+
+	ctx.JSON(200, map[string]interface{}{
+		"ok": true,
+	})
+}
+
 // MoveIssueAcrossBoards move a card from one board to another in a project
 func MoveIssueAcrossBoards(ctx *context.Context) {
 

routers/routes/macaron.go

@@ -790,6 +790,7 @@ func RegisterMacaronRoutes(m *macaron.Macaron) {
 					m.Group("/:boardID", func() {
 						m.Put("", bindIgnErr(auth.EditProjectBoardTitleForm{}), repo.EditProjectBoardTitle)
 						m.Delete("", repo.DeleteProjectBoard)
+						m.Post("/default", repo.SetDefaultProjectBoard)
 
 						m.Post("/:index", repo.MoveIssueAcrossBoards)
 					})