Controllers with aliases are no longer working correctly #128

New issue

Closed

opened 2018-06-29 03:20:10 +00:00 by jamie · 2 comments

jamie commented

2018-06-29 03:20:10 +00:00

(Migrated from git.hazaar.io)

With the new __runAction() method in the Hazaar\Contoller\Basic class we check that the requested controller is actually the currentl controller and if not, we have made a poor assumption that we have ended up in this controller because we've been re-routed to the default controller. I completely forgot that it's possible to be re-routed to another controller with an alias, which we don't need to worry about as much.

I will need to fix up the __runAction() method to support alias re-directions.

With the new `__runAction()` method in the `Hazaar\Contoller\Basic` class we check that the requested controller is actually the currentl controller and if not, we have made a poor assumption that we have ended up in this controller because we've been re-routed to the default controller. I completely forgot that it's possible to be re-routed to another controller with an alias, which we don't need to worry about as much. I will need to fix up the `__runAction()` method to support alias re-directions.

jamie commented

2018-06-29 05:13:45 +00:00

(Migrated from git.hazaar.io)

Turns out this wasn't the issue and it was just an incomplete change from 93111e9ebc

Turns out this wasn't the issue and it was just an incomplete change from 93111e9ebc9233dbc36ea65c28ab43ba2c5c3249

jamie commented

2018-06-29 05:13:45 +00:00

(Migrated from git.hazaar.io)

closed