This logic seems strange to me. Was this pair programmed? Did you guys have Rob take a look?

I think it should just be `if DiscussionMover.move!(current_user, discussion, destination)`. And delete line 82 and stick the group_id assignment into `DiscussionMover.move!`.