Feeling Lucky
Release Blocker
bug
PR-5.0-dev
[#41916] - [5.0] Fix user group deletion events compatibility
- Fixed in Code Base
- 25 Sep 2023
- Medium
- Build: 5.0-dev
- # 41916
- Diff
- Fedik:fix-user-group-del-event
User tests: Successful: Unsuccessful:
Pull Request for Issue #41906 .
Summary of Changes
Fix event compatibility for UserGroupDelete events
Testing Instructions
Try delete User group
Actual result BEFORE applying this Pull Request
An error
Expected result AFTER applying this Pull Request
Group is deleted without error
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: joomla/Manual#197
- No documentation changes for manual.joomla.org needed
joomla-cms-bot
- | Category | ⇒ | Administration com_users Libraries |
| Status | New | ⇒ | Pending |
| Labels |
Added:
Release Blocker
bug
PR-5.0-dev
|
||
Fedik
- comment
- 25 Sep 2023
Changed
richard67
- comment
- 25 Sep 2023
I have tested this item ✅ successfully on 1cf9784
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41916.
| Status | Pending | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2023-09-25 17:21:15 |
| Closed_By | ⇒ | HLeithner |
HLeithner
- comment
- 25 Sep 2023
thanks
Please introduce new extended classes for before and after, we don't know how long we really have this event and as you did it now you just pollute the constructor used for all other events