diff options
| author | Eric Davis <edavis@littlestreamsoftware.com> | 2010-08-25 14:59:16 +0000 |
|---|---|---|
| committer | Eric Davis <edavis@littlestreamsoftware.com> | 2010-08-25 14:59:16 +0000 |
| commit | daa4272c0e7f657a0070424224cabccbbeca6197 (patch) | |
| tree | fbae7898a9f1dd7e49a81aa5fa271b48b44b46af /app/controllers | |
| parent | bc951a9d4f849099f907c5a01b066b75f9d7bf4c (diff) | |
| download | redmine-daa4272c0e7f657a0070424224cabccbbeca6197.tar.gz redmine-daa4272c0e7f657a0070424224cabccbbeca6197.zip | |
Refactor: extract method in bulk_update.
git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4040 e93f8b46-1217-0410-a6f0-8f06a7374b81
Diffstat (limited to 'app/controllers')
| -rw-r--r-- | app/controllers/issues_controller.rb | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/app/controllers/issues_controller.rb b/app/controllers/issues_controller.rb index 29b131360..f10ba03bf 100644 --- a/app/controllers/issues_controller.rb +++ b/app/controllers/issues_controller.rb @@ -198,11 +198,8 @@ class IssuesController < ApplicationController def bulk_update @issues.sort! + attributes = parse_params_for_bulk_issue_attributes(params) - attributes = (params[:issue] || {}).reject {|k,v| v.blank?} - attributes.keys.each {|k| attributes[k] = '' if attributes[k] == 'none'} - attributes[:custom_field_values].reject! {|k,v| v.blank?} if attributes[:custom_field_values] - unsaved_issue_ids = [] @issues.each do |issue| issue.reload @@ -318,4 +315,11 @@ private return false end end + + def parse_params_for_bulk_issue_attributes(params) + attributes = (params[:issue] || {}).reject {|k,v| v.blank?} + attributes.keys.each {|k| attributes[k] = '' if attributes[k] == 'none'} + attributes[:custom_field_values].reject! {|k,v| v.blank?} if attributes[:custom_field_values] + attributes + end end |