Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#38720] - logout getMenuLanguage

No Code Attached Yet
avatar thalueng
thalueng
8 Sep 2022

Steps to reproduce the issue

PHP throws a TypeError on logout from the site. This is because getMenuLanguage is called with the wrong argument in com_users/src/Controller/UserController.php within method logout on line 146 (Joomla 4.2.2), which was introduced with Joomla 4.2.0.

Expected result

getMenuLanguage needs to be called with $return == menu id (xxx) as int, not with $return == 'index.php?Itemid=xxx'

Actual result

System information (as much as possible)

Joomla 4.2.2, Apache 2.4, PHP 7.4

Additional comments

avatar thalueng thalueng - open - 8 Sep 2022
avatar thalueng thalueng - change - 8 Sep 2022
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 8 Sep 2022
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 8 Sep 2022
avatar brianteeman
brianteeman - comment - 8 Sep 2022

This is the pull request where the change was made #38299

avatar chmst chmst - change - 11 Sep 2022
Status New ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2022-09-11 11:38:15
Closed_By ⇒ chmst
avatar chmst chmst - close - 11 Sep 2022
avatar chmst
chmst - comment - 11 Sep 2022

Closing as there is a pull request

avatar brianteeman
brianteeman - comment - 11 Sep 2022

@chmst I think you may have misunderstood. The PR I referenced is the one causing the reported broken behaviour

avatar Fedik Fedik - change - 11 Sep 2022
Status Closed ⇒ New
Closed_Date 2022-09-11 11:38:15 ⇒
Closed_By chmst ⇒
avatar Fedik Fedik - reopen - 11 Sep 2022
avatar joomdonation joomdonation - change - 19 Nov 2022
Status New ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2022-11-19 11:41:14
Closed_By ⇒ joomdonation
avatar joomdonation joomdonation - close - 19 Nov 2022
avatar joomdonation
joomdonation - comment - 19 Nov 2022

PR #39251 should fix this issue. Please help testing. Thanks !

Add a Comment

Login with GitHub to post a comment