[#26072] - [4.0] Reset controller name in administrator's com_login
- Fixed in Code Base
- 25 Oct 2019
- Medium
- Build: 4.0-dev
- # 26072
- Diff
- dryabov:patch-1
- Pending Hound Hound is busy sniffing around... Details
User tests: Successful: Unsuccessful:
Summary of Changes
Joomla\CMS\Application\AdministratorApplication::findOption method changes value of option to com_login for expired sessions, but it keeps other query params. That's why controller should be reset there to avoid "Invalid controller class: ..." error message displayed in the case of such a findOption-redirecting from actual extension page with specified controller in the URL.
Testing Instructions
In the /administrator area, open any 3rdparty extension page with specified controller parameter in the URL and clear cookies (or wait for session cookie expiration), and then refresh the page.
Expected result
Login form is displayed.
Actual result
"Invalid controller class: ..." message is displayed.
Comments
Core components are not affected as they are not use multiple display controllers.
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Administration com_login |
franz-wohlkoenig
- | Title |
|
||||||
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26072.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26072.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2019-10-25 22:19:44 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
?
|
||
Thanks for your first PR to Joomla. Thanks!
I have tested this item✅ successfully on 0d32f69
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/26072.