Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#18774] - [4.0] com_users routing issue

?
avatar joomdonation
joomdonation
21 Nov 2017

Steps to reproduce the issue

Install latest Joomla 4 branch with sample data. Check urls of the following pages:

  • Login /index.php/login?view=login
  • Forgot username: /index.php/username-reminder?view=remind
  • Forgot password; /index.php/password-reset?view=reset

Expected result

Url should not contain view in query string, so;

  • Login /index.php/login
  • Forgot username: /index.php/username-reminder
  • Forgot password; /index.php/password-reset

Actual result

Urls still contain view in query string (which is not needed)

avatar joomdonation joomdonation - open - 21 Nov 2017
avatar joomla-cms-bot joomla-cms-bot - change - 21 Nov 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 21 Nov 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Nov 2017
Category ⇒ Router / SEF
avatar franz-wohlkoenig franz-wohlkoenig - change - 21 Nov 2017
Status New ⇒ Confirmed
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 21 Nov 2017

Issue confirmed.

This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18774.

avatar csthomas
csthomas - comment - 21 Nov 2017

You should have the same on 3.8 with modern routing.
Please test PR from 3.8 #18429 on branch 4.0-dev or 3.8

curl https://patch-diff.githubusercontent.com/raw/joomla/joomla-cms/pull/18429.diff | git apply
avatar joomdonation
joomdonation - comment - 21 Nov 2017

@csthomas I did a quick code review at your PR and it looks OK to me. I will check it more carefully before posting my test result. Will try to get it merged to staging, so I will close this issue now

avatar joomdonation joomdonation - change - 21 Nov 2017
Status Confirmed ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2017-11-21 16:32:50
Closed_By ⇒ joomdonation
avatar joomdonation joomdonation - close - 21 Nov 2017

Add a Comment

Login with GitHub to post a comment