Feeling Lucky
?
?
Maintainers Checked
[#39251] - [4.2] Fix logout redirect error on multilingual site
- Fixed in Code Base
- 28 Nov 2022
- Medium
- Build: 4.2-dev
- # 39251
- Diff
- joomdonation:fix_logout_rediect
User tests: Successful: Unsuccessful:
Pull Request for Issue #38720.
Summary of Changes
This PR fix issue described in #38720.
Testing Instructions
- Prepare a multilingual website
- Create a menu item link to Login Form menu item type. In that menu item, set Logout Redirect parameter to any menu item
- Access to that menu item to login
- After logged in, access to that menu item again and click on Logout button
Actual result BEFORE applying this Pull Request
You get fatal error, something like:
Joomla\Component\Users\Site\Model\LoginModel::getMenuLanguage(): Argument #1 ($id) must be of type int, string given....
Expected result AFTER applying this Pull Request
No error. You are being redirected to the selected menu item on logout.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Front End com_users |
alikon
- comment
- 26 Nov 2022
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39251.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
Maintainers Checked
|
||
richard67
- comment
- 26 Nov 2022
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39251.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-11-28 09:03:03 |
| Closed_By | ⇒ | laoneo | |
| Labels |
Added:
?
|
||
laoneo
- comment
- 28 Nov 2022
Thank you!
funartist
- comment
- 13 Dec 2022
Thank you!
I have tested this item✅ successfully on d0c6b58
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39251.