Commit 1b83af6eed3a4069a8f66a6785632e6f950e900f
1 parent
f7894762
Exists in
master
and in
4 other branches
IssueController#new refactored
Showing
2 changed files
with
3 additions
and
5 deletions
Show diff stats
app/assets/javascripts/issues.js
| @@ -137,10 +137,10 @@ function updateNewIssueURL(){ | @@ -137,10 +137,10 @@ function updateNewIssueURL(){ | ||
| 137 | var assignee_id = $("#assignee_id").val(); | 137 | var assignee_id = $("#assignee_id").val(); |
| 138 | var new_href = ""; | 138 | var new_href = ""; |
| 139 | if(milestone_id){ | 139 | if(milestone_id){ |
| 140 | - new_href = "milestone_id=" + milestone_id + "&"; | 140 | + new_href = "issue[milestone_id]=" + milestone_id + "&"; |
| 141 | } | 141 | } |
| 142 | if(assignee_id){ | 142 | if(assignee_id){ |
| 143 | - new_href = new_href + "assignee_id=" + assignee_id; | 143 | + new_href = new_href + "issue[assignee_id]=" + assignee_id; |
| 144 | } | 144 | } |
| 145 | if(new_href.length){ | 145 | if(new_href.length){ |
| 146 | new_href = new_issue_link.attr("href") + "?" + new_href; | 146 | new_href = new_issue_link.attr("href") + "?" + new_href; |
app/controllers/issues_controller.rb
| @@ -37,9 +37,7 @@ class IssuesController < ApplicationController | @@ -37,9 +37,7 @@ class IssuesController < ApplicationController | ||
| 37 | end | 37 | end |
| 38 | 38 | ||
| 39 | def new | 39 | def new |
| 40 | - @issue = @project.issues.new | ||
| 41 | - @issue.milestone_id = params[:milestone_id] if params[:milestone_id].present? | ||
| 42 | - @issue.assignee_id = params[:assignee_id] if params[:assignee_id].present? | 40 | + @issue = @project.issues.new(params[:issue]) |
| 43 | respond_with(@issue) | 41 | respond_with(@issue) |
| 44 | end | 42 | end |
| 45 | 43 |