User tests: Successful: Unsuccessful:
The $table->load
check for deleting group rows has been moved above the loop that deletes them.
Please refer to #33771 for the reason behind this.
This PR mimics what was done for Content Map's Delete All Bug (#33771) to User Groups where a similar bug was left un-noticed
- B
: - B1
- C
: - C1
Before the PR: only the first Parent-Child pair would be deleted and the 2nd Parent-Child was skipped. This happened because the B1 was deleted with B (it's parent) and the loop returned false when it iterated over B1 as it was no longer present in the table.
After the PR: all selected groups will be deleted.
None
Status | New | ⇒ | Pending |
Category | ⇒ | Administration com_users |
I have tested this item
Status | Pending | ⇒ | Ready to Commit |
RTC
Status | Ready to Commit | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2021-05-23 13:12:27 |
Closed_By | ⇒ | Quy | |
Labels |
Added:
?
?
|
Thanks!
Thank you
I have tested this item✅ successfully on 79ad657
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/34094.