Kanban board

.eslintrc

@@ -20,6 +20,7 @@ globals:
   Dropzone: false
   u2fApi: false
   hljs: false
+  Sortable: false
 
 rules:
   no-unused-vars: [error, {args: all, argsIgnorePattern: ^_, varsIgnorePattern: ^_, ignoreRestSiblings: true}]

custom/conf/app.ini.sample

@@ -311,6 +311,8 @@ REPO_INDEXER_EXCLUDE =
 [admin]
 ; Disallow regular (non-admin) users from creating organizations.
 DISABLE_REGULAR_ORG_CREATION = false
+; Enable the kanban board feature system wide.
+ENABLE_KANBAN_BOARD = true
 ; Default configuration for email notifications for users (user configurable). Options: enabled, onmention, disabled
 DEFAULT_EMAIL_NOTIFICATIONS = enabled
 
@@ -415,7 +417,7 @@ DEFAULT_ALLOW_CREATE_ORGANIZATION = true
 ; Public is for everyone
 DEFAULT_ORG_VISIBILITY = public
 ; Default value for DefaultOrgMemberVisible
-; True will make the membership of the users visible when added to the organisation  
+; True will make the membership of the users visible when added to the organisation
 DEFAULT_ORG_MEMBER_VISIBLE = false
 ; Default value for EnableDependencies
 ; Repositories will use dependencies by default depending on this setting

models/error.go

@@ -1264,6 +1264,45 @@ func (err ErrLabelNotExist) Error() string {
 	return fmt.Sprintf("label does not exist [label_id: %d, repo_id: %d]", err.LabelID, err.RepoID)
 }
 
+// __________                   __               __
+// \______   \_______  ____    |__| ____   _____/  |_  ______
+//  |     ___/\_  __ \/  _ \   |  |/ __ \_/ ___\   __\/  ___/
+//  |    |     |  | \(  <_> )  |  \  ___/\  \___|  |  \___ \
+//  |____|     |__|   \____/\__|  |\___  >\___  >__| /____  >
+//                         \______|    \/     \/          \/
+
+// ErrProjectNotExist represents a "ProjectNotExist" kind of error.
+type ErrProjectNotExist struct {
+	ID     int64
+	RepoID int64
+}
+
+// IsErrProjectNotExist checks if an error is a ErrProjectNotExist
+func IsErrProjectNotExist(err error) bool {
+	_, ok := err.(ErrProjectNotExist)
+	return ok
+}
+
+func (err ErrProjectNotExist) Error() string {
+	return fmt.Sprintf("projects does not exist [id: %d, repo_id: %d]", err.ID, err.RepoID)
+}
+
+type ErrProjectBoardNotExist struct {
+	BoardID   int64
+	RepoID    int64
+	ProjectID int64
+}
+
+// IsErrProjectBoardNotExist checks if an error is a ErrProjectBoardNotExist
+func IsErrProjectBoardNotExist(err error) bool {
+	_, ok := err.(ErrProjectBoardNotExist)
+	return ok
+}
+
+func (err ErrProjectBoardNotExist) Error() string {
+	return fmt.Sprintf("project board does not exist [board_id: %d, repo_id: %d, project_id: %d]", err.BoardID, err.RepoID, err.ProjectID)
+}
+
 //    _____  .__.__                   __
 //   /     \ |__|  |   ____   _______/  |_  ____   ____   ____
 //  /  \ /  \|  |  | _/ __ \ /  ___/\   __\/  _ \ /    \_/ __ \

models/issue.go

@@ -39,15 +39,19 @@ type Issue struct {
 	Labels           []*Label   `xorm:"-"`
 	MilestoneID      int64      `xorm:"INDEX"`
 	Milestone        *Milestone `xorm:"-"`
-	Priority         int
-	AssigneeID       int64        `xorm:"-"`
-	Assignee         *User        `xorm:"-"`
-	IsClosed         bool         `xorm:"INDEX"`
-	IsRead           bool         `xorm:"-"`
-	IsPull           bool         `xorm:"INDEX"` // Indicates whether is a pull request or not.
-	PullRequest      *PullRequest `xorm:"-"`
-	NumComments      int
-	Ref              string
+	ProjectID        int64      `xorm:"INDEX"`
+	// If 0, then it has not been added to a specific board in the project
+	ProjectBoardID int64    `xorm:"INDEX"`
+	Project        *Project `xorm:"-"`
+	Priority       int
+	AssigneeID     int64        `xorm:"-"`
+	Assignee       *User        `xorm:"-"`
+	IsClosed       bool         `xorm:"INDEX"`
+	IsRead         bool         `xorm:"-"`
+	IsPull         bool         `xorm:"INDEX"` // Indicates whether is a pull request or not.
+	PullRequest    *PullRequest `xorm:"-"`
+	NumComments    int
+	Ref            string
 
 	DeadlineUnix timeutil.TimeStamp `xorm:"INDEX"`
 
@@ -257,6 +261,13 @@ func (issue *Issue) loadAttributes(e Engine) (err error) {
 		}
 	}
 
+	if issue.ProjectID > 0 && issue.Project == nil {
+		issue.Project, err = getProjectByRepoID(e, issue.RepoID, issue.ProjectID)
+		if err != nil && !IsErrProjectNotExist(err) {
+			return fmt.Errorf("getProjectByRepoID [repo_id: %d, milestone_id: %d]: %v", issue.RepoID, issue.ProjectID, err)
+		}
+	}
+
 	if err = issue.loadAssignees(e); err != nil {
 		return
 	}
@@ -1305,6 +1316,7 @@ type IssuesOptions struct {
 	PosterID    int64
 	MentionedID int64
 	MilestoneID int64
+	ProjectID   int64
 	Page        int
 	PageSize    int
 	IsClosed    util.OptionalBool
@@ -1385,6 +1397,10 @@ func (opts *IssuesOptions) setupSession(sess *xorm.Session) {
 		sess.And("issue.milestone_id=?", opts.MilestoneID)
 	}
 
+	if opts.ProjectID > 0 {
+		sess.And("issue.project_id=?", opts.ProjectID)
+	}
+
 	switch opts.IsPull {
 	case util.OptionalBoolTrue:
 		sess.And("issue.is_pull=?", true)

models/issue_comment.go

@@ -80,6 +80,10 @@ const (
 	CommentTypeLock
 	// Unlocks a previously locked issue
 	CommentTypeUnlock
+	// Project changed
+	CommentTypeProject
+	// Project board changed
+	CommentTypeProjectBoard
 )
 
 // CommentTag defines comment tag type
@@ -105,6 +109,10 @@ type Comment struct {
 	Issue            *Issue `xorm:"-"`
 	LabelID          int64
 	Label            *Label `xorm:"-"`
+	OldProjectID     int64
+	ProjectID        int64
+	OldProject       *Project `xorm:"-"`
+	Project          *Project `xorm:"-"`
 	OldMilestoneID   int64
 	MilestoneID      int64
 	OldMilestone     *Milestone `xorm:"-"`
@@ -312,6 +320,32 @@ func (c *Comment) LoadLabel() error {
 	return nil
 }
 
+// LoadProject if comment.Type is CommentTypeProject, then load project.
+func (c *Comment) LoadProject() error {
+
+	if c.OldProjectID > 0 {
+		var oldProject Project
+		has, err := x.ID(c.OldProjectID).Get(&oldProject)
+		if err != nil {
+			return err
+		} else if has {
+			c.OldProject = &oldProject
+		}
+	}
+
+	if c.ProjectID > 0 {
+		var project Project
+		has, err := x.ID(c.ProjectID).Get(&project)
+		if err != nil {
+			return err
+		} else if has {
+			c.Project = &project
+		}
+	}
+
+	return nil
+}
+
 // LoadMilestone if comment.Type is CommentTypeMilestone, then load milestone
 func (c *Comment) LoadMilestone() error {
 	if c.OldMilestoneID > 0 {
@@ -482,6 +516,8 @@ func createComment(e *xorm.Session, opts *CreateCommentOptions) (_ *Comment, err
 		LabelID:          LabelID,
 		OldMilestoneID:   opts.OldMilestoneID,
 		MilestoneID:      opts.MilestoneID,
+		OldProjectID:     opts.OldProjectID,
+		ProjectID:        opts.ProjectID,
 		RemovedAssignee:  opts.RemovedAssignee,
 		AssigneeID:       opts.AssigneeID,
 		CommitID:         opts.CommitID,
@@ -636,6 +672,17 @@ func createLabelComment(e *xorm.Session, doer *User, repo *Repository, issue *Is
 	})
 }
 
+func createProjectComment(e *xorm.Session, doer *User, repo *Repository, issue *Issue, oldProjectID, projectID int64) (*Comment, error) {
+	return createComment(e, &CreateCommentOptions{
+		Type:         CommentTypeProject,
+		Doer:         doer,
+		Repo:         repo,
+		Issue:        issue,
+		OldProjectID: oldProjectID,
+		ProjectID:    projectID,
+	})
+}
+
 func createMilestoneComment(e *xorm.Session, doer *User, repo *Repository, issue *Issue, oldMilestoneID, milestoneID int64) (*Comment, error) {
 	return createComment(e, &CreateCommentOptions{
 		Type:           CommentTypeMilestone,
@@ -758,6 +805,8 @@ type CreateCommentOptions struct {
 	DependentIssueID int64
 	OldMilestoneID   int64
 	MilestoneID      int64
+	OldProjectID     int64
+	ProjectID        int64
 	AssigneeID       int64
 	RemovedAssignee  bool
 	OldTitle         string

models/migrations/migrations.go

@@ -250,6 +250,16 @@ var migrations = []Migration{
 	NewMigration("delete orphaned attachments", deleteOrphanedAttachments),
 	// v97 -> v98
 	NewMigration("add repo_admin_change_team_access to user", addRepoAdminChangeTeamAccessColumnForUser),
+	// v98 -> v99
+	NewMigration("add projects database table", addProjectsTable),
+	// v99 -> v100
+	NewMigration("add projects info to repository table", addProjectsInfo),
+	// v100 -> v101
+	NewMigration("add project ID to comments table", addProjectIDToCommentsTable),
+	// v101 -> v102
+	NewMigration("add project ID to issue table", addProjectIDToIssueTable),
+	// v102 -> v103
+	NewMigration("add project board table", addProjectBoardTable),
 }
 
 // Migrate database to current version

models/migrations/v100.go

@@ -0,0 +1,26 @@
+// Copyright 2019 The Gitea Authors. All rights reserved.
+// Use of this source code is governed by a MIT-style
+// license that can be found in the LICENSE file.
+
+package migrations
+
+import (
+	"github.com/go-xorm/xorm"
+)
+
+func addProjectIDToCommentsTable(x *xorm.Engine) error {
+
+	sess := x.NewSession()
+	defer sess.Close()
+
+	type Comment struct {
+		OldProjectID int64
+		ProjectID    int64
+	}
+
+	if err := sess.Sync2(new(Comment)); err != nil {
+		return err
+	}
+
+	return sess.Commit()
+}

models/migrations/v101.go

@@ -0,0 +1,28 @@
+// Copyright 2019 The Gitea Authors. All rights reserved.
+// Use of this source code is governed by a MIT-style
+// license that can be found in the LICENSE file.
+
+package migrations
+
+import (
+	"code.gitea.io/gitea/modules/timeutil"
+
+	"github.com/go-xorm/xorm"
+)
+
+func addProjectBoardTable(x *xorm.Engine) error {
+	type ProjectBoard struct {
+		ID        int64 `xorm:"pk autoincr"`
+		ProjectID int64 `xorm:"INDEX NOT NULL"`
+		Title     string
+		RepoID    int64 `xorm:"INDEX NOT NULL"`
+
+		// Not really needed but helpful
+		CreatorID int64 `xorm:"NOT NULL"`
+
+		CreatedUnix timeutil.TimeStamp `xorm:"INDEX created"`
+		UpdatedUnix timeutil.TimeStamp `xorm:"INDEX updated"`
+	}
+
+	return x.Sync(new(ProjectBoard))
+}

models/migrations/v102.go

@@ -0,0 +1,19 @@
+// Copyright 2019 The Gitea Authors. All rights reserved.
+// Use of this source code is governed by a MIT-style
+// license that can be found in the LICENSE file.
+
+package migrations
+
+import (
+	"github.com/go-xorm/xorm"
+)
+
+func addProjectIDToIssueTable(x *xorm.Engine) error {
+
+	type Issue struct {
+		ProjectID      int64 `xorm:"INDEX"`
+		ProjectBoardID int64 `xorm:"INDEX"`
+	}
+
+	return x.Sync2(new(Issue))
+}

models/migrations/v96.go

@@ -23,10 +23,7 @@ func deleteOrphanedAttachments(x *xorm.Engine) error {
 		CommentID int64
 	}
 
-	sess := x.NewSession()
-	defer sess.Close()
-
-	err := sess.BufferSize(setting.Database.IterateBufferSize).
+	err := x.BufferSize(setting.Database.IterateBufferSize).
 		Where("`comment_id` = 0 and (`release_id` = 0 or `release_id` not in (select `id` from `release`))").Cols("uuid").
 		Iterate(new(Attachment),
 			func(idx int, bean interface{}) error {
@@ -36,13 +33,8 @@ func deleteOrphanedAttachments(x *xorm.Engine) error {
 					return err
 				}
 
-				_, err := sess.ID(attachment.ID).NoAutoCondition().Delete(attachment)
+				_, err := x.ID(attachment.ID).NoAutoCondition().Delete(attachment)
 				return err
 			})
-
-	if err != nil {
-		return err
-	}
-
-	return sess.Commit()
+	return err
 }

models/migrations/v98.go

@@ -0,0 +1,35 @@
+// Copyright 2019 The Gitea Authors. All rights reserved.
+// Use of this source code is governed by a MIT-style
+// license that can be found in the LICENSE file.
+
+package migrations
+
+import (
+	"code.gitea.io/gitea/modules/timeutil"
+
+	"github.com/go-xorm/xorm"
+)
+
+func addProjectsTable(x *xorm.Engine) error {
+
+	type ProjectType uint8
+
+	type Project struct {
+		ID              int64  `xorm:"pk autoincr"`
+		Title           string `xorm:"INDEX NOT NULL"`
+		Description     string `xorm:"TEXT"`
+		RepoID          int64  `xorm:"NOT NULL"`
+		CreatorID       int64  `xorm:"NOT NULL"`
+		IsClosed        bool   `xorm:"INDEX"`
+		NumIssues       int
+		NumClosedIssues int
+
+		Type ProjectType
+
+		ClosedDateUnix timeutil.TimeStamp
+		CreatedUnix    timeutil.TimeStamp `xorm:"INDEX created"`
+		UpdatedUnix    timeutil.TimeStamp `xorm:"INDEX updated"`
+	}
+
+	return x.Sync(new(Project))
+}