take params for milestone and assignee from query instead of path args

diff --git a/public/js/app.js b/public/js/app.js
index a5bb9569ef0a8e15a0047f36a38675bc56c1d9fc..2a7cf08bae46092baf6562f275a44be96a3fd140 100644 (file)
--- a/public/js/app.js
+++ b/public/js/app.js
@@ -870,7 +870,7 @@ function initIssue() {
 
                 $.post($m.data("ajax"), {
                     issue: $('#issue').data("id"),
-                    milestone: id
+                    milestoneid: id
                 }, function (json) {
                     if (json.ok) {
                         //window.location.reload();

diff --git a/routers/repo/issue.go b/routers/repo/issue.go
index 412d03c6e6a9a115bad8aabd501392a9bc66b1d4..59921d551552dc39ca8175f25aa34ea5c1424a7d 100644 (file)
--- a/routers/repo/issue.go
+++ b/routers/repo/issue.go
@@ -582,7 +582,7 @@ func UpdateIssueMilestone(ctx *middleware.Context) {
        }
 
        oldMid := issue.MilestoneId
-       mid := com.StrTo(ctx.Params(":milestone")).MustInt64()
+       mid := com.StrTo(ctx.Query("milestoneid")).MustInt64()
        if oldMid == mid {
                ctx.JSON(200, map[string]interface{}{
                        "ok": true,
@@ -627,7 +627,7 @@ func UpdateAssignee(ctx *middleware.Context) {
                return
        }
 
-       aid := com.StrTo(ctx.Params(":assigneeid")).MustInt64()
+       aid := com.StrTo(ctx.Query("assigneeid")).MustInt64()
        // Not check for invalid assignne id and give responsibility to owners.
        issue.AssigneeId = aid
        if err = models.UpdateIssueUserPairByAssignee(aid, issue.Id); err != nil {