No Code Attached Yet J4 Issue bug
avatar chandanndeep
chandanndeep
20 Aug 2019

Steps to reproduce the issue

Logout Redirection Page "Default" view is not working.

According to tooltip = The default is to stay on the same page.
image

Expected result

Stay on the same page.

Actual result

Redirect to homepage from Default view.

avatar chandanndeep chandanndeep - open - 20 Aug 2019
avatar joomla-cms-bot joomla-cms-bot - change - 20 Aug 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 20 Aug 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Aug 2019
Labels Added: J3 Issue
avatar franz-wohlkoenig franz-wohlkoenig - labeled - 20 Aug 2019
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 20 Aug 2019

Which Joomla-Version? There is a Reason why you are asked for "System Information".

avatar infograf768
infograf768 - comment - 20 Aug 2019
  1. This screenshot has nothing to do with J 3.x or J4.
  2. In a normal j3 environment we first get a page containing only a Logout button. But therafter we are redirected. If nothing is entered in the Menu Item Logout Redirect or Logout Redirect fields, then the default page can't be the logout view and therefore displays the Home Page.
  3. I agree that the tip is wrong
avatar chandanndeep
chandanndeep - comment - 20 Aug 2019

Joomla V3.9.11

avatar HLeithner
HLeithner - comment - 20 Aug 2019

@chandanndeep that doesn't look like a core menu item. can you show us the details tab?

avatar simbus82
simbus82 - comment - 20 Aug 2019

@infograf768 @HLeithner
It's the menu voice "Logout" under "Users" section in "menu voice type" and it's a core element of Joomla.

image

@chandanndeep Is the "System - Logout" plugin enabled or disabled?

If it is active, it overrides the setting of the menu item and always redirects to the home.
I don't know if this is the intended functionality, a warning should be shown ...

avatar franz-wohlkoenig franz-wohlkoenig - change - 20 Aug 2019
Status New Discussion
avatar brianteeman
brianteeman - comment - 20 Aug 2019

@HLeithner its the standard logout page! The user just has several plugins installed which add extra tabs

avatar HLeithner
HLeithner - comment - 20 Aug 2019

Oh ok I found it, thx @simbus82
funy the german translation is completely different.

avatar HLeithner
HLeithner - comment - 20 Aug 2019

The description has been changed by @infograf768 in PR #12337 but I don't see a functional change in this pr maybe I didn't see a reference to a code change?

In my opinion the description is wrong and should be changed back to the original text if there wars no intention to change this behavior in j3.7

avatar simbus82
simbus82 - comment - 20 Aug 2019

Oh ok I found it, thx @simbus82
funy the german translation is completely different.

I was also in doubt, I changed from Italian to English and found it ;-)
About the PR #12337, am I wrong or is it related to the module and not to the menu item?

avatar infograf768
infograf768 - comment - 20 Aug 2019

@HLeithner
In fact I did not change the part concerned i.e. redirection to the same page.
That was and is still wrong. We should keep the rest of my changes and just modify that part IMHO.

avatar brianteeman
brianteeman - comment - 20 Aug 2019

@simbus82 its both
@HLeithner the change in the string was really just to account for the fact that the menu selection was no longer in a dropdown

avatar brianteeman
brianteeman - comment - 20 Aug 2019

Also dont forget that the string is also used for the module and not just the menu item where the behaviour may be different

avatar infograf768
infograf768 - comment - 20 Aug 2019

Also dont forget that the string is also used for the module and not just the menu item where the behaviour may be different

Not that I am aware of.
The login module uses:
MOD_LOGIN_FIELD_LOGIN_REDIRECTURL_DESC
MOD_LOGIN_FIELD_LOGOUT_REDIRECTURL_DESC

avatar HLeithner
HLeithner - comment - 20 Aug 2019

change has been done in this commit: 463bbb6#diff-7bead94ab2f4003deaee80abfddcf323L206

From "default" to "same" which is not the same ;-)

anyway so we should change this back to the correct value.

avatar infograf768
infograf768 - comment - 20 Aug 2019

lol, indeed. Different from the module indeed.

avatar infograf768
infograf768 - comment - 20 Aug 2019

Anyone makes the patch?

avatar brianteeman
brianteeman - comment - 20 Aug 2019

Also dont forget that the string is also used for the module and not just the menu item where the behaviour may be different

Sorry if I wasnt clear (typing on my phone). The module strings were chan ged in the same PR that Harald linked to and they will be different to the component behaviour

avatar HLeithner
HLeithner - comment - 20 Aug 2019

I will make the pr.

avatar HLeithner
HLeithner - comment - 20 Aug 2019

Closing for PR #25946

thx all for the help.

avatar HLeithner HLeithner - close - 20 Aug 2019
avatar HLeithner HLeithner - change - 20 Aug 2019
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2019-08-20 13:28:38
Closed_By HLeithner
avatar simbus82
simbus82 - comment - 20 Aug 2019

Sorry guys but... i have tested it before.

Is the "System - Logout" plugin enabled or disabled?
If it is active, it overrides the setting of the menu item and always redirects to the home.
I don't know if this is the intended functionality, a warning should be shown ..."

If this plugin is disabled, the menu voice with Logout Redirection Page set to "Default" works well, as the expected behaviour.

avatar HLeithner
HLeithner - comment - 20 Aug 2019

Based on the source of system - logout it would only redirect you to the main page if the current page is not accessible by the current user. Which is no user if you hit the logout button.

But if you set a redirect the redirect should be executed before system - logout get called to redirect you...

are you sure the page you are redirecting to has access level public? also the component takes the return parameter and should redirect you to the url (menu item) you set. if none is set it redirects you to the default page.

Or I misinterpret something in the source

avatar simbus82
simbus82 - comment - 21 Aug 2019

I have do this test, i have created a menu voice "logout" with same settings as @chandanndeep.
Example with Plugin ON
I'm logged-in and i'm browsing around. Now i'm in the public url www.mysite/page-public.
Then I see a menu voice called "logout", i click it: i'm redirected to Home Page.
This behaviour is wrong most of the times, because broke the User Experience.

Example with Plugin OFF
I'm logged-in and i'm browsing around. Now i go to the public url www.mysite/page-public.
Then I see a menu voice called "logout", i click it: i'm redirected to www.mysite/page-public, the same page that i'm visiting.
This behaviour is correct, it's what is expected by user.

Maybe we need only to inform the admin user that the plugin affects this behavior.

avatar franz-wohlkoenig franz-wohlkoenig - change - 13 Sep 2019
Status Closed New
Closed_Date 2019-08-20 13:28:38
Closed_By HLeithner
avatar franz-wohlkoenig franz-wohlkoenig - reopen - 13 Sep 2019
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 13 Sep 2019

Reopened as #25946 is closed.

avatar franz-wohlkoenig franz-wohlkoenig - change - 13 Sep 2019
Status New Discussion
avatar joomdonation joomdonation - change - 12 Nov 2022
Labels Added: No Code Attached Yet J4 Issue
Removed: J3 Issue ?
avatar joomdonation joomdonation - unlabeled - 12 Nov 2022
avatar joomdonation joomdonation - labeled - 12 Nov 2022
avatar joomdonation
joomdonation - comment - 12 Nov 2022

The issue still happens in J4. Unfortunately, with the way menu logout is implemented, it's quite hard to fix.

avatar Hackwar Hackwar - change - 19 Feb 2023
Labels Added: bug
avatar Hackwar Hackwar - labeled - 19 Feb 2023

Add a Comment

Login with GitHub to post a comment