remove move logic from controller

DiscussionMover class not actually doing the move logic.

Refactor to move this logic into the DiscussionMover class and tidy up the controller.
(caught by Jon's ever watching eyes) :eyes: 
@jdoud, @jonlemmon