[#45835] - [5.4] Convert site mod_login to service provider
- Fixed in Code Base
- 6 Aug 2025
- Medium
- Build: 5.4-dev
- # 45835
- Diff
- joomlaweby:mod_login
User tests: Successful: Unsuccessful:
Summary of Changes
Converts the site login module to service provider.
Testing Instructions
- Make sure you have site module mod_login published
- You should see module output
- Try also to be logged-in/out
Actual result BEFORE applying this Pull Request
Module works as expected
Expected result AFTER applying this Pull Request
Module works same way as before applying pull request
Link to documentations
Please select:
-
Documentation link for docs.joomla.org:
-
No documentation changes for docs.joomla.org needed
-
Pull Request link for manual.joomla.org:
-
No documentation changes for manual.joomla.org needed
| Status | New | ⇒ | Pending |
joomla-cms-bot
- | Category | ⇒ | Modules Front End |
I have tested this item ✅ successfully on a9d3b12
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45835.
| Status | Pending | ⇒ | Ready to Commit |
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45835.
I have tested this item ✅ successfully on a9d3b12I have successfully tested this - nice work @joomlaweby - I don't know how many more of these need converting to service provider, but keep 'em coming ;) I'll gladly test them ALL :) Cheers.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45835.
I think there are 2 more modules...I've got them prepared :-)
| Labels |
Added:
RTC
|
||
I think there are 2 more modules...I've got them prepared :-)
@joomlaweby Would be good to get them ready and tested before the end of next week when we will build our packages for Beta 1.
| Labels |
Added:
PR-5.4-dev
|
||
RTC is still valid, commits after that were just clean branch updates and code style (removal of empty line).
✅ Final test before merge, with JBT graft Joomla_5.4.0-alpha4-dev+pr.45835-Development-Full_Package.zip
- Tested with Firefox, Safari, Chrome and Edge
- Created user with only Editor rights, logged in with admin and editor
- Tested forgot your password and forgot your user name
- Tested with and without remember me after browser restart
- Tested disabling title, pre- and post-text
- Installed German language paket and switch to German as default language, checked login form in German
- Checked logs/error.php for logged login failures
- Enabled user registration, checked link is working
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2025-08-06 04:46:58 |
| Closed_By | ⇒ | muhme |
Thank you @joomlaweby for your contribution. Thank you @exlemor and @RickR2H for testing.
I have tested this item ✅ successfully on a9d3b12
I have successfully tested this - nice work @joomlaweby - I don't know how many more of these need converting to service provider, but keep 'em coming ;) I'll gladly test them ALL :) Cheers.
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45835.