Finish change issue’s milestone

diff --git a/models/issue.go b/models/issue.go
index 26380b603398ae09ec38598c0fae9d30ad958ff6..3d3e433a42695c5fb4860fef85c7daec598ddf52 100644 (file)
--- a/models/issue.go
+++ b/models/issue.go
@@ -543,18 +543,20 @@ func ChangeMilestoneAssign(oldMid, mid int64, isIssueClosed bool) (err error) {
                }
        }
 
-       m, err := GetMilestoneById(mid)
-       if err != nil {
-               return err
-       }
-       m.NumIssues++
-       if isIssueClosed {
-               m.NumClosedIssues++
-       }
-       m.Completeness = m.NumClosedIssues * 100 / m.NumIssues
-       if _, err = sess.Id(m.Id).Update(m); err != nil {
-               sess.Rollback()
-               return err
+       if mid > 0 {
+               m, err := GetMilestoneById(mid)
+               if err != nil {
+                       return err
+               }
+               m.NumIssues++
+               if isIssueClosed {
+                       m.NumClosedIssues++
+               }
+               m.Completeness = m.NumClosedIssues * 100 / m.NumIssues
+               if _, err = sess.Id(m.Id).Update(m); err != nil {
+                       sess.Rollback()
+                       return err
+               }
        }
        return sess.Commit()
 }

diff --git a/routers/repo/issue.go b/routers/repo/issue.go
index 2346539d0c470bd817ab19ba5e2b911b8560dcab..7b8c912e69805a75841ce99126c8e5397b0701e8 100644 (file)
--- a/routers/repo/issue.go
+++ b/routers/repo/issue.go
@@ -53,17 +53,21 @@ func Issues(ctx *middleware.Context) {
                filterMode = models.FM_MENTION
        }
 
+       var mid int64
        midx, _ := base.StrTo(ctx.Query("milestone")).Int64()
-       mile, err := models.GetMilestoneByIndex(ctx.Repo.Repository.Id, midx)
-       if err != nil {
-               ctx.Handle(500, "issue.Issues(GetMilestoneByIndex): %v", err)
-               return
+       if midx > 0 {
+               mile, err := models.GetMilestoneByIndex(ctx.Repo.Repository.Id, midx)
+               if err != nil {
+                       ctx.Handle(500, "issue.Issues(GetMilestoneByIndex): %v", err)
+                       return
+               }
+               mid = mile.Id
        }
 
        page, _ := base.StrTo(ctx.Query("page")).Int()
 
        // Get issues.
-       issues, err := models.GetIssues(assigneeId, ctx.Repo.Repository.Id, posterId, mile.Id, page,
+       issues, err := models.GetIssues(assigneeId, ctx.Repo.Repository.Id, posterId, mid, page,
                isShowClosed, ctx.Query("labels"), ctx.Query("sortType"))
        if err != nil {
                ctx.Handle(500, "issue.Issues(GetIssues): %v", err)
@@ -120,6 +124,19 @@ func CreateIssue(ctx *middleware.Context, params martini.Params) {
        ctx.Data["IsRepoToolbarIssues"] = true
        ctx.Data["IsRepoToolbarIssuesList"] = false
 
+       var err error
+       // Get all milestones.
+       ctx.Data["OpenMilestones"], err = models.GetMilestones(ctx.Repo.Repository.Id, false)
+       if err != nil {
+               ctx.Handle(500, "issue.ViewIssue(GetMilestones.1): %v", err)
+               return
+       }
+       ctx.Data["ClosedMilestones"], err = models.GetMilestones(ctx.Repo.Repository.Id, true)
+       if err != nil {
+               ctx.Handle(500, "issue.ViewIssue(GetMilestones.2): %v", err)
+               return
+       }
+
        us, err := models.GetCollaborators(strings.TrimPrefix(ctx.Repo.RepoLink, "/"))
        if err != nil {
                ctx.Handle(500, "issue.CreateIssue(GetCollaborators)", err)
@@ -134,6 +151,19 @@ func CreateIssuePost(ctx *middleware.Context, params martini.Params, form auth.C
        ctx.Data["IsRepoToolbarIssues"] = true
        ctx.Data["IsRepoToolbarIssuesList"] = false
 
+       var err error
+       // Get all milestones.
+       ctx.Data["OpenMilestones"], err = models.GetMilestones(ctx.Repo.Repository.Id, false)
+       if err != nil {
+               ctx.Handle(500, "issue.ViewIssue(GetMilestones.1): %v", err)
+               return
+       }
+       ctx.Data["ClosedMilestones"], err = models.GetMilestones(ctx.Repo.Repository.Id, true)
+       if err != nil {
+               ctx.Handle(500, "issue.ViewIssue(GetMilestones.2): %v", err)
+               return
+       }
+
        us, err := models.GetCollaborators(strings.TrimPrefix(ctx.Repo.RepoLink, "/"))
        if err != nil {
                ctx.Handle(500, "issue.CreateIssue(GetCollaborators)", err)

diff --git a/templates/issue/create.tmpl b/templates/issue/create.tmpl
index 4b43116cca4e7f64b02f194dd952d799c7b2028d..34cecc78bdf3dc06f779119e8281f9f0c63d842a 100644 (file)
--- a/templates/issue/create.tmpl
+++ b/templates/issue/create.tmpl
@@ -48,25 +48,33 @@
                                     </ul>
                                     <div class="tab-content">
                                         <div class="tab-pane active" id="milestone-open">
+                                            {{if not .OpenMilestones}}
                                             <p class="milestone-item">Nothing to show</p>
+                                            {{else}}
                                             <ul class="list-unstyled">
-                                                <li class="milestone-item" data-id="1">
-                                                    <p><strong>Milestone name</strong></p>
-                                                    <p>due to 3 days later</p>
-                                                </li>
-                                                <li class="milestone-item" data-id="1">
-                                                    <p><strong>Milestone name</strong></p>
-                                                    <p>due to 3 days later</p>
+                                                {{range .OpenMilestones}}
+                                                <li class="milestone-item" data-id="{{.Id}}">
+                                                    <p><strong>{{.Name}}</strong></p>
+                                                    <!-- <p>due to 3 days later</p> -->
                                                 </li>
+                                                {{end}}
                                             </ul>
+                                            {{end}}
                                         </div>
+
                                         <div class="tab-pane" id="milestone-close">
+                                            {{if not .ClosedMilestones}}
+                                            <p class="milestone-item">Nothing to show</p>
+                                            {{else}}
                                             <ul class="list-unstyled">
-                                                <li class="milestone-item" data-id="1">
-                                                    <p><strong>Milestone name</strong></p>
-                                                    <p>closed 3 days ago</p>
+                                                {{range .ClosedMilestones}}
+                                                <li class="milestone-item" data-id="{{.Id}}">
+                                                    <p><strong>{{.Name}}</strong></p>
+                                                    <p>Closed {{TimeSince .ClosedDate}}</p>
                                                 </li>
+                                                {{end}}
                                             </ul>
+                                            {{end}}
                                         </div>
                                     </div>
                                 </li>

diff --git a/templates/issue/view.tmpl b/templates/issue/view.tmpl
index 5aeb9409e1329753c5ff9ff51111f2a0fa760029..18ec5faf0e1b73ea7094f1a5a4584a8c8d937485 100644 (file)
--- a/templates/issue/view.tmpl
+++ b/templates/issue/view.tmpl
@@ -100,7 +100,7 @@
             </div>
 
             <div class="issue-bar col-md-2">
-                <div class="milestone" data-milestone="0" data-ajax="{{.Issue.Index}}/milestone">
+                <div class="milestone" data-milestone="{{.Milestone.Id}}" data-ajax="{{.Issue.Index}}/milestone">
                     <div class="pull-right action">
                         <button class="btn btn-default btn-sm" data-toggle="dropdown">
                             <i class="fa fa-check-square-o"></i>
@@ -108,7 +108,7 @@
                         </button>
                         <div class="dropdown-menu dropdown-menu-right">
                             <ul class="list-unstyled">
-                                <li data-id="0" class="clear-milestone milestone-item hidden"><i class="fa fa-times-circle-o"></i> Clear milestone </li>
+                                <li data-id="0" class="clear-milestone milestone-item  hidden"><i class="fa fa-times-circle-o"></i> Clear milestone </li>
                                 <li class="milestone-list">
                                     <ul class="nav nav-tabs" data-init="tabs">
                                         <li class="active"><a href="#milestone-open" data-toggle="tab">Open</a></li>
@@ -138,7 +138,7 @@
                                                 {{range .ClosedMilestones}}
                                                 <li class="milestone-item" data-id="{{.Id}}">
                                                     <p><strong>{{.Name}}</strong></p>
-                                                    <p>{{TimeSince .ClosedDate}}</p>
+                                                    <p>Closed {{TimeSince .ClosedDate}}</p>
                                                 </li>
                                                 {{end}}
                                             </ul>