Feeling Lucky
?
?
[#39483] - [4.2] Fix move user to group
- Fixed in Code Base
- 28 Dec 2022
- Medium
- Build: 4.2-dev
- # 39483
- Diff
- heelc29:4.2/users-batch-group
User tests: Successful: Unsuccessful:
Summary of Changes
The feature to move an user to a group is broken after #38596.
Variable $users is not assigned in this if-case. Use the correct variable $userIds like it was before #38596
joomla-cms/administrator/components/com_users/src/Model/UserModel.php
Lines 725 to 727 in f520c98
Testing Instructions
Try moving a user to a group using the batch command.
Actual result BEFORE applying this Pull Request
Expected result AFTER applying this Pull Request
The user is moved to the new group.
Link to documentations
Please select:
- No documentation changes for docs.joomla.org needed
- No documentation changes for manual.joomla.org needed
Votes
# of Users Experiencing Issue
1/1
Average Importance Score
3.00
joomla-cms-bot
- | Category | ⇒ | Administration com_users |
| Status | New | ⇒ | Pending |
ceciogit
- comment
- 24 Dec 2022
I have tested this item
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39483.
| Status | Pending | ⇒ | Ready to Commit |
| Labels |
Added:
?
|
||
richard67
- comment
- 24 Dec 2022
RTC
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39483.
| Status | Ready to Commit | ⇒ | Fixed in Code Base |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2022-12-28 12:16:32 |
| Closed_By | ⇒ | wilsonge | |
| Labels |
Added:
?
|
||
wilsonge
- comment
- 28 Dec 2022
Thanks!
I have tested this item✅ successfully on e3326d8
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/39483.