ElkArte Community

It's not that useful for now, but my heart says: let's use an interface for the admin controllers.. All controllers need at least the action_index() method, IMO this should be declared in an interface.

interface admin_action_controller
{
     function action_index();
     function settings(); // deprecated, I know..
}

and in the related controller:

class ManageBBC_Controller implements admin_action_controller
{
	/**
	 * BBC settings form
	 * @var Settings_Form
	 */
...

It's just a quick'n'dirty sample..

Opinions?

Surely. :) Or an abstract base class.

Also, I believe pre_dispatch() (https://github.com/elkarte/Elkarte/blob/master/sources/Dispatcher.class.php#L301) is part of it.
Thinking at contextual stuff they do, such as load templates and languages.

There's no particular certain reason for a base class vs an interface. But it can always be filled with a method they need.