Skip to content

Fix issue #32210#32215

Merged
chmst merged 1 commit intojoomla:stagingfrom
joomdonation:patch-2
Feb 1, 2021
Merged

Fix issue #32210#32215
chmst merged 1 commit intojoomla:stagingfrom
joomdonation:patch-2

Conversation

@joomdonation
Copy link
Contributor

Pull Request for Issue #32210.

Summary of Changes

Added code to check to make sure menu item is active before getting redirect menu item from menu parameters to avoid fatal error on menulogout method.

I could not re-procedure the issue as mentioned in #32210. However, we can see that fatal error by access to direct link http://localhost/joomla/index.php/component/users?task=user.menulogout

Testing Instructions

  1. Login
  2. Access to this link http://localhost/joomla/index.php/component/users?task=user.menulogout (direct link to trigger menulogout method in users controller
  3. Before patch: You get fatal error.
  4. Apply patch: You are being logged out.

@alebak
Copy link

alebak commented Feb 1, 2021

I have tested this item ✅ successfully on 554dfa7


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

1 similar comment
@ghost
Copy link

ghost commented Feb 1, 2021

I have tested this item ✅ successfully on 554dfa7


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

@alikon
Copy link
Contributor

alikon commented Feb 1, 2021

RTC


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

@joomla-cms-bot joomla-cms-bot added the RTC This Pull Request is Ready To Commit label Feb 1, 2021
@chmst chmst added this to the Joomla! 3.9.25 milestone Feb 1, 2021
@chmst chmst merged commit cd4f219 into joomla:staging Feb 1, 2021
@joomla-cms-bot joomla-cms-bot added PR-staging and removed RTC This Pull Request is Ready To Commit labels Feb 1, 2021
@chmst
Copy link
Contributor

chmst commented Feb 1, 2021

Thanks

@joomdonation joomdonation deleted the patch-2 branch February 1, 2021 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants