This worries me.. It would be better if you directly assign the associations.. ie:

motion.outcome_author = user
motion.outcome = motion_params[:outcome]

return false unless motion.save