summaryrefslogtreecommitdiff
path: root/app/services/issues
diff options
context:
space:
mode:
authorStan Hu <stanhu@gmail.com>2018-01-10 17:20:50 -0800
committerStan Hu <stanhu@gmail.com>2018-01-10 17:22:56 -0800
commit95865da5499a794c539d43b1adfb33d98da97ba2 (patch)
tree4f0ad1af7be5337d292add3e2bc9da69ee221471 /app/services/issues
parent3576d59ae95a61dd20e997a619dbc6c8e8a70276 (diff)
downloadgitlab-ce-95865da5499a794c539d43b1adfb33d98da97ba2.tar.gz
Fix bug where award emojis would be lost when moving issues between projectssh-fix-award-emoji-move-issues
Closes #33423
Diffstat (limited to 'app/services/issues')
-rw-r--r--app/services/issues/move_service.rb16
1 files changed, 11 insertions, 5 deletions
diff --git a/app/services/issues/move_service.rb b/app/services/issues/move_service.rb
index 29def25719d..2f511ab44b7 100644
--- a/app/services/issues/move_service.rb
+++ b/app/services/issues/move_service.rb
@@ -24,7 +24,7 @@ module Issues
@new_issue = create_new_issue
rewrite_notes
- rewrite_award_emoji
+ rewrite_issue_award_emoji
add_note_moved_from
# Old issue tasks
@@ -76,7 +76,7 @@ module Issues
end
def rewrite_notes
- @old_issue.notes.find_each do |note|
+ @old_issue.notes_with_associations.find_each do |note|
new_note = note.dup
new_params = { project: @new_project, noteable: @new_issue,
note: rewrite_content(new_note.note),
@@ -84,13 +84,19 @@ module Issues
updated_at: note.updated_at }
new_note.update(new_params)
+
+ rewrite_award_emoji(note, new_note)
end
end
- def rewrite_award_emoji
- @old_issue.award_emoji.each do |award|
+ def rewrite_issue_award_emoji
+ rewrite_award_emoji(@old_issue, @new_issue)
+ end
+
+ def rewrite_award_emoji(old_awardable, new_awardable)
+ old_awardable.award_emoji.each do |award|
new_award = award.dup
- new_award.awardable = @new_issue
+ new_award.awardable = new_awardable
new_award.save
end
end