Re: Topic and MoveTopic (and MergeTopic) controllers
Reply #1 –
I think the reason why it wasn't done is because it makes the controller a lot bigger. I would say if it does make it too big, call it something like "TopicExtra" or "TopicAdmin" or "TopicModeration" and combine all of the moderation stuff in there.
Re: Topic and MoveTopic (and MergeTopic) controllers
Reply #3 –
I agree with moving printpage to the display. Don't agree with using Topic to redirect to the other controllers. That would make it harder to find what you're looking for. Lessen code duplication by loading Topics.subs.php there and checking "if (empty($topic))" (not by much). Even better would be to move the call to the function that loads $topic to pre_dispatch().