diff options
| author | Florian Pritz <bluewind@xinu.at> | 2013-09-22 17:09:57 +0200 |
|---|---|---|
| committer | Florian Pritz <bluewind@xinu.at> | 2013-09-22 17:18:56 +0200 |
| commit | 41c5abd4e8849e6a84ed2af405e9c25a74e018f3 (patch) | |
| tree | 9c8a62934a5b725330d9def35653e04adca20a08 /application/controllers/file.php | |
| parent | 9538bea0908c8a3758b41967d977455af731c344 (diff) | |
Move user_logged_in check to controller constructor
TL;DR: Allows us to show a proper error page if encryption_key is
missing from the config.
muser->logged_in() can load the session class which will die if
encryption_key is not set in the config causing an error to be
displayed.
Because the header is also loaded when we display an error
loading the class will be tried again. CI maintains an array with
information which classes have been tried to be loaded and will simply
return true without loading again.
muser->logged_in() will then try to access $this->session which doesn't
exist. Since all of this happens when we are already in the header the
error message appears in the navigation being hard to read.
Signed-off-by: Florian Pritz <bluewind@xinu.at>
Diffstat (limited to 'application/controllers/file.php')
| -rw-r--r-- | application/controllers/file.php | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/application/controllers/file.php b/application/controllers/file.php index 98ccae884..b02cdd8ce 100644 --- a/application/controllers/file.php +++ b/application/controllers/file.php @@ -20,7 +20,6 @@ class File extends MY_Controller { parent::__construct(); $this->load->model('mfile'); - $this->load->model('muser'); if (is_cli_client()) { $this->var->view_dir = "file_plaintext"; |