[com_users][fix] User profil changes and password reset won't work if logout menu exists and no login menu is shown

[conseilgouz]

Pull Request for Issue # .

Summary of Changes

com_users/router.php

Testing Instructions

Joomla 3.6.2, no specific module/extension/plugin

1. create a logout menu
2. create a login menu that will disappear when you are logged in (shown if the user is "invited")
3. set "force password reset" to yes on one user
4. login with this user
5. user profile is displayed but you cannot access the passord change form.

Documentation Changes Required: no

If a logout menu item exists, users can not access to profil changes.

It means that the "force password reset" function won't work either.

com_users/Router.php creates a default link index.php/logout/profile?layout=edit that won't work, it blocks profile changes button et stays on profile display form.

The problem is that logout menu creates the following link :
index.php?option=com_users&view=login&layout=logout&task=user.menulogout

and the router just checks view=login, ignoring the view part of the link.

A "cleaner" solution could be to change the logout menu to become index.php?option=com_users&view=logout.

Pascal

[brianteeman]

I can NOT confirm your findings. As seen by the video everything works correctly - or do i misunderstand

[conseilgouz]

Hi Brian,

It does work because you have both login and logout menus.

In my configuration, after user login, the login menu disappears (only shown if you are "invited"), so there 's only Logout Menu. In this case, you cannot access the profile changes.

Pascal

[brianteeman]

I tried again and I still cannot replicate it - see https://www.dropbox.com/s/1eqn8wwgvg6sedg/reset.mp4?dl=0

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/12037.}

[conseilgouz]

OK, one more condition: SEF mode should be ON (which is the default configuration).

Pascal

[brianteeman]

And now I can replicate the issue -thanks. Will test soon

[brianteeman]

I have tested this item ✅ successfully on 763887b

Thank you for being patient with me
I have now confirmed the issue and have applied the PR and it resolves the issue

Thanks!!!

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/joomla-cms/12037.}

[ghost]

I have tested this item ✅ successfully on 763887b

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12037.}

[zero-24]

@Hackwar can you take a final look here?

As this are the merge conflicts we have with 3.7.0 router changes.

[Hackwar]

This PR does not seem to be valid anymore. The tests by @brianteeman are from before a complete rewrite of the routers. The code like this has no function right now. I did not test the whole thing, but I would be surprised if that is still an issue in the current staging. In any case, if this PR is merged in the current form, it will just create a bunch of notices and nothing more. There will be no change to the router by this.

[roland-d]

@pmleconte Can you have a look to see if the issue still exists in 3.7.0 and if needed update your PR? Thank you.

[ghost]

If this PR get no Response, it will be closed at 23th July 2017.

[joomla-cms-bot]

Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/12037

[ghost]

closed as stated above.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12037.}

[procajlok]

Hello! O Joomla 3.8.1 I have the same problem. When i have 'logout' menu item (visible only for login user), user can't reset password.

---

_{This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/12037.}

[ghost]

@procajlok Please update on latest Release and ask further help on the forums. This repository concerns in first Place Joomla-Core coding, thanks.