Question

I have a set of controllers that should only be accessible if you are an admin (as opposed to a regular user).

Thus, in the constructor for each of the controllers, I would do:

public function __construct() {
    parent::__construct();
    if (! is_admin()) {
         show_404();
    }
}

Instead of adding this code to the constructor of every Admin Controller, is there a better way to do this?

I was thinking I could create a Base controller called Admin_Controller that would look like this:

public class Admin_Controller extends CI_Controller {

    public function __construct() {
         //the above code goes here
    }
}

And then all my other controllers can extend this class, instead of the CI_Controller class. The only problem with this is, I need to include this file at the top of my other controllers, or CodeIgniter cannot find Admin_Controller.

Is there a better way to do this?

Answer 1

place this in your application/core folder: MY_Controller.php (note the correct use of capitals)

class MY_Controller extends CI_Controller
{
public function __construct() 
{
     parent::__construct();
     if (! is_admin())
     {
          show_404();
     }
} 

Then in all your normal controllers that you want users to be logged in

class Whatever extends MY_Controller
{
   public function __construct() 
    {
         parent::__construct();
    } 
}

See userguide here on extending core classes