diff options
author | Stan Hu <stanhu@gmail.com> | 2018-01-10 17:20:50 -0800 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2018-01-10 17:22:56 -0800 |
commit | 95865da5499a794c539d43b1adfb33d98da97ba2 (patch) | |
tree | 4f0ad1af7be5337d292add3e2bc9da69ee221471 /app/services/issues | |
parent | 3576d59ae95a61dd20e997a619dbc6c8e8a70276 (diff) | |
download | gitlab-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.rb | 16 |
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 |