[#4003] - [#32934] Updated and improved PR - Update view.html.php #2665
- Closed
- 16 Jun 2015
- Medium
- Build: staging
- # 4003
- Diff
- Kubik-Rubik:com_users-view-redirect-jroute
- Foreign ID 32934
- Success The Travis CI build passed Details
User tests: Successful: Unsuccessful:
Small improvement of the PR by @andykirk.
Thank you for the work!
Hi
I've created a pluging that makes use of $app->getRouter()->attachBuildRule(...), redirecting things like 'index.php?option=com_users&view=login' to just 'login', and in components\com_users\views\profile\view.html.php on line 66 the redirect is:
$app->redirect(JUri::base() . 'index.php?option=com_users&view=login', '', 302);
Note it doesn't use JRoute::() as I believe it should, which means in those circumstances my plugin doesn't work.
Changing it to:
$app->redirect(JRoute::('index.php?option=com_users&view=login', false), '', 302);
Fixes it to work as expected.
I guess it would be difficult for people to test this without installing a plugin that does what my plugin is doing, so I'm happy to provide a reduced plugin zip for people to install if necessary.
Please let me know.
Thanks,
Andy
nicksavov
- | Labels |
Removed:
?
|
||
| Title |
|
||||||
| Status | New | ⇒ | Pending |
| Category | ⇒ | Components |
| Status | Pending | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2015-06-16 12:44:33 |
| Closed_By | ⇒ | Bakual |
Merged based on review as it's a simple PR and unlikely to be tested.
Gave atribution to @andykirk.
Thanks @Kubik-Rubik for redoing it. 
Copied details from joomlacode before they disappear
This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/4003.