? ? Pending

User tests: Successful: Unsuccessful:

avatar alikon
alikon
28 Mar 2020

Summary of Changes

converted sql to prepared statements

Testing Instructions

  • create user group
  • delete user group
  • modify user group
  • view user group

Expected result

works as before

Actual result

N/A

avatar alikon alikon - open - 28 Mar 2020
avatar alikon alikon - change - 28 Mar 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Mar 2020
Category Libraries
avatar alikon alikon - change - 28 Mar 2020
Labels Added: ?
avatar Quy
Quy - comment - 28 Mar 2020

Create a new group with Group Parent Administrator. It is not listed under Administrator.

28491

avatar chmst
chmst - comment - 28 Mar 2020

Same effect as @Quy


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28491.

avatar alikon
alikon - comment - 29 Mar 2020

fixed

avatar chmst
chmst - comment - 29 Mar 2020

In a fresh installation, I deleted 2 usergroups at the same time and got this:
delete-usergroups

avatar Quy
Quy - comment - 30 Mar 2020

@chmst Please explain why your last report is an issue.

avatar chmst
chmst - comment - 30 Mar 2020

There is no menu any more, no toggle icon and no modules in the header.

avatar Quy
Quy - comment - 30 Mar 2020

It is not happening to me. I will install the latest version and test again.

avatar chmst
chmst - comment - 30 Mar 2020

Probably this is caused by other PRs in my system. But I had this effect several times. I'll re-test asap with a clean installation

avatar Quy
Quy - comment - 30 Mar 2020

I have tested this item successfully on 28c9b9e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28491.

avatar Quy Quy - test_item - 30 Mar 2020 - Tested successfully
avatar chmst
chmst - comment - 30 Mar 2020

Sorry, but I got this again on a fresh installation. I deleted groups with child groups: manager and editor. The result is as in my screen above. But it is nor related to this PR, It is also before theis PR.
Will make a new issue for it.

avatar chmst
chmst - comment - 30 Mar 2020

I have tested this item successfully on 28c9b9e


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28491.

avatar chmst chmst - test_item - 30 Mar 2020 - Tested successfully
avatar Quy Quy - change - 30 Mar 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 30 Mar 2020

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/28491.

avatar wilsonge wilsonge - change - 1 Apr 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-04-01 17:51:43
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 1 Apr 2020
avatar wilsonge wilsonge - merge - 1 Apr 2020

Add a Comment

Login with GitHub to post a comment