? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
28 Oct 2019

Summary of Changes

Removes unnecessary queries when sending mass mail, also fixing query error on PostgreSQL and MS SQL described in #26789.

Testing Instructions

Try sending mass mail. To all groups, to specific group, to group containing only you as user, include/exclude child groups.

Expected result

Error described in #26789 is gone and everything else works like before.

avatar SharkyKZ SharkyKZ - open - 28 Oct 2019
avatar SharkyKZ SharkyKZ - change - 28 Oct 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 28 Oct 2019
Category Administration com_users
avatar SharkyKZ SharkyKZ - change - 28 Oct 2019
Labels Added: ?
avatar SharkyKZ SharkyKZ - change - 28 Oct 2019
The description was changed
avatar SharkyKZ SharkyKZ - edited - 28 Oct 2019
avatar Quy Quy - test_item - 29 Oct 2019 - Tested successfully
avatar Quy
Quy - comment - 29 Oct 2019

I have tested this item successfully on 7b1825f


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

avatar viocassel viocassel - test_item - 29 Oct 2019 - Tested successfully
avatar viocassel
viocassel - comment - 29 Oct 2019

I have tested this item successfully on 7b1825f


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

avatar Quy Quy - change - 30 Oct 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 30 Oct 2019

RTC


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

avatar SharkyKZ SharkyKZ - change - 30 Oct 2019
Labels Added: ?
avatar HLeithner
HLeithner - comment - 30 Oct 2019

Merged last commit by code review. Thanks

avatar HLeithner HLeithner - change - 30 Oct 2019
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2019-10-30 10:08:21
Closed_By HLeithner
avatar HLeithner HLeithner - close - 30 Oct 2019
avatar HLeithner HLeithner - merge - 30 Oct 2019

Add a Comment

Login with GitHub to post a comment