or

etc…

It shows default CI error page too.

Hello,

I’ve been testing, and this error happens with most of the libraries.

I tried to fix it but i didn’t have success with it :c

I’m suffering the same issue here.

application/routes.php:

application/controllers/error.php:

As you can see I’m using a breadcrumb class to handle breadcrumbs. But this happens when throwing the 404 error:

A PHP Error was encountered
Severity: Notice
Message: Undefined property: Error::$breadcrumb
Filename: controllers/error.php
Line Number: 30

Fatal error: Call to a member function append() on a non-object in application/controllers/error.php on line 30

So it looks like libraries that are auto-loaded, aren’t loaded anymore or something. Manually loading the libraries in the controller doesn’t affect the error.

In short, it isn’t possible to use other libraries / helpers / plugins within the 404_override route.

for some reason, the autoloaded libraries are not loaded :/
i have the same problem too…

I’ve fixed it, in a nasty way. I had to change three lines of code in the system/ folder.

Create application/core/MY_Loader.php:

Replace “include_once()” with “include()” in system/core/Loader.php:

Add two lines in system/core/CodeIgniter.php ($CI->load->reset(); and $CI->load->_ci_autoloader();):

And it’s working.

Yes, the solution is working… But i still think this is a bug

Yeah, the problem is that the loader class (obviously) ignores loading classes twice, and the include_once() in system/core/Loader.php prevents the autoload array from being loaded a second time.

If somebody has a more decent solution, shoot! Or some CI / Reactor dev: commit this stuff to the repo (and adopt the loader->reset() method in the core).

Hmm. I’ve been doing a little work on the CI_Loader class myself and someone has already asked me to create a Reactor fork (done, just have to incorporate my work yet). I’ve strictly been focused on the library loader method, but I’m marking this thread. I’ll try playing with your solution some. You are calling a Loader method directly that’s supposed to be a private function to the class, but that’s a known holdover from CI < 2.0
I don’t see anything wrong right now with the change from include_once to include but there might be a better way.

using thePiet’s idea of adding a reset() function to the loader I’ve come up with this solution that fixes all the 404 override problems and allows to override the error page for any status code. I’ve put in a pull request so hopefully this will be fixed in the next release.

https://bitbucket.org/bubbafoley/codeigniter-error-overrides/changeset/36320ae89bbf

Your 404-error pull request link is… a 404 Error! You might want to fix that.

is fixed. somehow an extra ‘s’ got added to the end of the link

Don’t feel bad. I had a link a couple of days ago to my OWN site that I forgot the ‘.com’

@bubbafoley, Nice effort, using the status override variable in routes is a nice idea too. But you should not load a controller from the exceptions class. There should be only one point in the core where controllers are loaded.

wiredesignz - 26 February 2011 12:52 AM

@bubbafoley, Nice effort, using the status override variable in routes is a nice idea too. But you should not load a controller from the exceptions class. There should be only one point in the core where controllers are loaded.

yeah I didn’t like doing it that way but it was the only way I could find that a) didn’t have duplicate code all over the place, and b) worked in all cases.

For instance, the current broken implementation checks the 404 override in _validate_request() in system/core/Router.php before calling show_404() then does the same check in system/core/CodeIgniter.php if a controller function doesn’t exist. It doesn’t work for missing controllers in subdirectories or when uncallable functions are passed through the URL.

The simple fix then is that the same check needs to be applied everywhere show_404() is called in the boot process. That’s alot of redundant code. But it’s still broken because if you call show_404() from a controller, the override won’t take because show_error() doesn’t check for it.

Is there a better way to call the controller?

Does anyone got a fix for this? My classses aren’t loaded on the 404 page…