Merge pull request #781 from andreynering/notifications-step-5

Notifications step 5
go-gitea · Jan 28, 2017 · cc31a21 · cc31a21
2 parents 31c717f + aa59131
commit cc31a21
Show file tree

Hide file tree

Showing 6 changed files with 49 additions and 25 deletions.
diff --git a/models/issue.go b/models/issue.go
@@ -93,6 +93,16 @@ func (issue *Issue) loadRepo(e Engine) (err error) {
 	return nil
 }
 
+// GetPullRequest returns the issue pull request
+func (issue *Issue) GetPullRequest() (pr *PullRequest, err error) {
+	if !issue.IsPull {
+		return nil, fmt.Errorf("Issue is not a pull request")
+	}
+
+	pr, err = getPullRequestByIssueID(x, issue.ID)
+	return
+}
+
 func (issue *Issue) loadAttributes(e Engine) (err error) {
 	if err := issue.loadRepo(e); err != nil {
 		return err
@@ -953,9 +963,9 @@ func Issues(opts *IssuesOptions) ([]*Issue, error) {
 
 	switch opts.IsPull {
 	case util.OptionalBoolTrue:
-		sess.And("issue.is_pull=?",true)
+		sess.And("issue.is_pull=?", true)
 	case util.OptionalBoolFalse:
-		sess.And("issue.is_pull=?",false)
+		sess.And("issue.is_pull=?", false)
 	}
 
 	sortIssuesSession(sess, opts.SortType)
@@ -1780,4 +1790,3 @@ func DeleteMilestoneByRepoID(repoID, id int64) error {
 	}
 	return sess.Commit()
 }
-
diff --git a/public/css/index.css b/public/css/index.css
@@ -2726,17 +2726,16 @@ footer .ui.language .menu {
 .user.notification table tr {
   cursor: pointer;
 }
-.user.notification .octicon-issue-opened,
-.user.notification .octicon-git-pull-request {
+.user.notification .octicon.green {
   color: #21ba45;
 }
-.user.notification .octicon-issue-closed {
+.user.notification .octicon.red {
   color: #d01919;
 }
-.user.notification .octicon-git-merge {
+.user.notification .octicon.purple {
   color: #a333c8;
 }
-.user.notification .octicon-pin {
+.user.notification .octicon.blue {
   color: #2185d0;
 }
 .dashboard {

diff --git a/public/less/_user.less b/public/less/_user.less
@@ -99,17 +99,19 @@
             }
         }
 
-        .octicon-issue-opened, .octicon-git-pull-request {
-            color: #21ba45;
-        }
-        .octicon-issue-closed {
-            color: #d01919;
-        }
-        .octicon-git-merge {
-            color: #a333c8;
-        }
-        .octicon-pin {
-            color: #2185d0;
+        .octicon {
+            &.green {
+                color: #21ba45;
+            }
+            &.red {
+                color: #d01919;
+            }
+            &.purple {
+                color: #a333c8;
+            }
+            &.blue {
+                color: #2185d0;
+            }
         }
     }
 }
diff --git a/routers/repo/issue.go b/routers/repo/issue.go
@@ -875,6 +875,8 @@ func NewComment(ctx *context.Context, form auth.CreateCommentForm) {
 					log.Error(4, "ChangeStatus: %v", err)
 				} else {
 					log.Trace("Issue [%d] status changed to closed: %v", issue.ID, issue.IsClosed)
+
+					notification.Service.NotifyIssue(issue, ctx.User.ID)
 				}
 			}
 		}

diff --git a/routers/repo/pull.go b/routers/repo/pull.go
@@ -16,7 +16,9 @@ import (
 	"code.gitea.io/gitea/modules/base"
 	"code.gitea.io/gitea/modules/context"
 	"code.gitea.io/gitea/modules/log"
+	"code.gitea.io/gitea/modules/notification"
 	"code.gitea.io/gitea/modules/setting"
+
 	"github.com/Unknwon/com"
 )
 
@@ -415,6 +417,8 @@ func MergePullRequest(ctx *context.Context) {
 		return
 	}
 
+	notification.Service.NotifyIssue(pr.Issue, ctx.User.ID)
+
 	log.Trace("Pull request merged: %d", pr.ID)
 	ctx.Redirect(ctx.Repo.RepoLink + "/pulls/" + com.ToStr(pr.Index))
 }
@@ -707,6 +711,8 @@ func CompareAndPullRequestPost(ctx *context.Context, form auth.CreateIssueForm)
 		return
 	}
 
+	notification.Service.NotifyIssue(pullIssue, ctx.User.ID)
+
 	log.Trace("Pull request created: %d/%d", repo.ID, pullIssue.ID)
 	ctx.Redirect(ctx.Repo.RepoLink + "/pulls/" + com.ToStr(pullIssue.Index))
 }

diff --git a/templates/user/notification/notification.tmpl b/templates/user/notification/notification.tmpl
@@ -40,25 +40,31 @@
 										<i class="blue octicon octicon-pin"></i>
 									{{else if $issue.IsPull}}
 										{{if $issue.IsClosed}}
-											<i class="octicon octicon-git-merge"></i>
+											{{if $issue.GetPullRequest.HasMerged}}
+												<i class="purple octicon octicon-git-merge"></i>
+											{{else}}
+												<i class="red octicon octicon-git-pull-request"></i>
+											{{end}}
 										{{else}}
-											<i class="octicon octicon-git-pull-request"></i>
+											<i class="green octicon octicon-git-pull-request"></i>
 										{{end}}
 									{{else}}
 										{{if $issue.IsClosed}}
-											<i class="octicon octicon-issue-closed"></i>
+											<i class="red octicon octicon-issue-closed"></i>
 										{{else}}
-											<i class="octicon octicon-issue-opened"></i>
+											<i class="green octicon octicon-issue-opened"></i>
 										{{end}}
 									{{end}}
 								</td>
-								<td class="twelve wide">
+								<td class="eleven wide">
 									<a class="item" href="{{AppSubUrl}}/{{$repoOwner.Name}}/{{$repo.Name}}/issues/{{$issue.Index}}">
 										#{{$issue.Index}} - {{$issue.Title}}
 									</a>
 								</td>
 								<td>
-									{{$repoOwner.Name}}/{{$repo.Name}}
+									<a class="item" href="{{AppSubUrl}}/{{$repoOwner.Name}}/{{$repo.Name}}">
+										{{$repoOwner.Name}}/{{$repo.Name}}
+									</a>
 								</td>
 								<td class="collapsing">
 									{{if ne $notification.Status 3}}