diff options
author | AlexDenisov <1101.debian@gmail.com> | 2013-05-16 10:32:16 +0000 |
---|---|---|
committer | AlexDenisov <1101.debian@gmail.com> | 2013-05-16 10:32:27 +0000 |
commit | 6e8cb844499dfcc77bd92f3daf0f7890400ec072 (patch) | |
tree | 5fb1909e5da126030d23c0f0c37482aa657027c3 /app/contexts | |
parent | d08e1db93091418694130870406995f21502d478 (diff) | |
download | gitlab-ce-6e8cb844499dfcc77bd92f3daf0f7890400ec072.tar.gz |
Added few tests on Issues::BulkUpdateContext
Diffstat (limited to 'app/contexts')
-rw-r--r-- | app/contexts/issues/bulk_update_context.rb | 18 |
1 files changed, 9 insertions, 9 deletions
diff --git a/app/contexts/issues/bulk_update_context.rb b/app/contexts/issues/bulk_update_context.rb index 8fe84d0e47c..73a3c353523 100644 --- a/app/contexts/issues/bulk_update_context.rb +++ b/app/contexts/issues/bulk_update_context.rb @@ -8,11 +8,14 @@ module Issues assignee_id = update_data[:assignee_id] status = update_data[:status] - unless status.present? - return { - count: 0, - success: false - } + new_state = nil + + if status.present? + if status == 'closed' + new_state = :close + else + new_state = :reopen + end end opts = {} @@ -22,12 +25,9 @@ module Issues issues = Issue.where(id: issues_ids).all issues = issues.select { |issue| can?(current_user, :modify_issue, issue) } - new_state = :reopen - new_state = :close if status == 'closed' - issues.each do |issue| issue.update_attributes(opts) - issue.send new_state + issue.send new_state if new_state end { |