Browse Source

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

pull/426/head
Vyacheslav Bakhmutov 10 years ago
parent
commit
2a7a03e5b3
  1. 2
      public/js/app.js
  2. 4
      routers/repo/issue.go

2
public/js/app.js

@ -870,7 +870,7 @@ function initIssue() {
$.post($m.data("ajax"), { $.post($m.data("ajax"), {
issue: $('#issue').data("id"), issue: $('#issue').data("id"),
milestone: id milestoneid: id
}, function (json) { }, function (json) {
if (json.ok) { if (json.ok) {
//window.location.reload(); //window.location.reload();

4
routers/repo/issue.go

@ -582,7 +582,7 @@ func UpdateIssueMilestone(ctx *middleware.Context) {
} }
oldMid := issue.MilestoneId oldMid := issue.MilestoneId
mid := com.StrTo(ctx.Params(":milestone")).MustInt64() mid := com.StrTo(ctx.Query("milestoneid")).MustInt64()
if oldMid == mid { if oldMid == mid {
ctx.JSON(200, map[string]interface{}{ ctx.JSON(200, map[string]interface{}{
"ok": true, "ok": true,
@ -627,7 +627,7 @@ func UpdateAssignee(ctx *middleware.Context) {
return 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. // Not check for invalid assignne id and give responsibility to owners.
issue.AssigneeId = aid issue.AssigneeId = aid
if err = models.UpdateIssueUserPairByAssignee(aid, issue.Id); err != nil { if err = models.UpdateIssueUserPairByAssignee(aid, issue.Id); err != nil {

Loading…
Cancel
Save