404_override doesn't work as expected.. |
[eluser]Glazz[/eluser]
Hello, I'm using 404_override in my routes to show my custom error page, it works but only for non existing controllers, if you have a non existing method in a controller it shows the default error page. Example: Code: http://domain.com/non-controller Code: http://domain.com/existing-controller/non-method Is this a bug? Or do i need to something else so it can show error for non existing methods inside existing controllers? Regards - Glazz
[eluser]idealws[/eluser]
I am experiencing this same issue using CI 2.0.0 now. Anyone have a fix for this? Regards, Ray
[eluser]Eric Barnes[/eluser]
I am testing this now and it is doing something weird. It is late here and I don't have time to fully test this to find a fix. Can you post this as a bug in the reactor? https://bitbucket.org/ellislab/codeignit...status=new Please be specific and thorough in the report.
[eluser]Glazz[/eluser]
Eric, https://bitbucket.org/ellislab/codeignit...de-problem Don't know if i done it right though. - G l a z z
[eluser]Eric Barnes[/eluser]
I just commited a fix for this: https://bitbucket.org/ellislab/codeignit...a0e75a2101 Can you give that change a try and report back?
[eluser]Glazz[/eluser]
Hello, With those changes now whenever the method doesn't exist, it shows the index of the controller, it's better, but yet not as expected :c
[eluser]Eric Barnes[/eluser]
Rats! I tested it but not thorough enough I think this should do it if you want to test again. Code: // is_callable() returns TRUE on some versions of PHP 5 for private and protected
[eluser]Glazz[/eluser]
Yeaps, i believe it is working now, at least i've tested with some pages and it shows my custom error page x) Thanks - G l a z z Edit: Now i have another problem, if i try to access a non existing controller it shows the page fine, but for some reason if i try to access a non existing method it shows this: Code: A PHP Error was encountered This is my custom error controller page: Code: <?php $this->load->view('topo'); ?> If i comment out the load->view('topo'); line the error doesn't show up, but this only happens for non existing method's..
[eluser]Eric Barnes[/eluser]
Giazz, I just made another commit and in all my tests it is fully working. Can you give this a shot? https://bitbucket.org/ellislab/codeignit...gniter.php |
Welcome Guest, Not a member yet? Register Sign In |