? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
10 Feb 2022

Only send email notificiations only to all users with user creating permissions and receiving system emails

145132b 10 Feb 2022 avatar brianteeman .
avatar brianteeman brianteeman - open - 10 Feb 2022
avatar brianteeman brianteeman - change - 10 Feb 2022
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 10 Feb 2022
Category Front End com_users
avatar coolcat-creations
coolcat-creations - comment - 10 Feb 2022

Tested with this PR - unfortunately the same thing:

None - Users with no Admin rights receives the mail
Self - Users with no Admin rights receives the mail
Administrator - Only Admin gets the mail

avatar brianteeman
brianteeman - comment - 10 Feb 2022

All 3 cases worked for me

Untitled.Project.mp4
avatar coolcat-creations
coolcat-creations - comment - 10 Feb 2022

Thank you for rechecking, I checked the files and the update package did not update the model. Weird. I Will copy paste your code and check again.

avatar coolcat-creations
coolcat-creations - comment - 10 Feb 2022

Ok I tested now:

None - Test successful
Self - Test successful
Admin - My Provider Blocked me hahaha - will report this in a few

avatar tecpromotion tecpromotion - test_item - 10 Feb 2022 - Tested successfully
avatar tecpromotion
tecpromotion - comment - 10 Feb 2022

I have tested this item successfully on 145132b

Thanks @brianteeman


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

avatar coolcat-creations
coolcat-creations - comment - 10 Feb 2022

Oh I hope my partial test and Brians recording and yours is enough no idea when hosting.de will unblock me. And thank you @brianteeman for this fast reaction and fix!

avatar brianteeman
brianteeman - comment - 10 Feb 2022

@coolcat-creations reconfigure global config to test with mailtrap.io

avatar brianteeman
brianteeman - comment - 10 Feb 2022

@zero-24 technically this is a JSST issue so please check it carefully, merge quickly and it will also need to be merged into 4.1

avatar coolcat-creations
coolcat-creations - comment - 10 Feb 2022

I could test now the 3rd case and it was successful! Thank you!

avatar coolcat-creations coolcat-creations - test_item - 10 Feb 2022 - Tested successfully
avatar coolcat-creations
coolcat-creations - comment - 10 Feb 2022

I have tested this item successfully on 145132b

Successful tested as stated above


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

avatar Quy Quy - change - 10 Feb 2022
Status Pending Ready to Commit
Labels Added: ?
avatar Quy
Quy - comment - 10 Feb 2022

RTC


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

avatar coolcat-creations
coolcat-creations - comment - 15 Feb 2022

Seems it did not make it into todays release altough its security related :-(

avatar brianteeman
brianteeman - comment - 15 Feb 2022

Only so much I can do :(

avatar brianteeman brianteeman - change - 15 Feb 2022
Labels Added: ?
avatar zero-24
zero-24 - comment - 15 Feb 2022

Yes will be shipped with 3.10.7 sorry took the wrong milestone thanks for correction @Quy

avatar zero-24 zero-24 - change - 20 Feb 2022
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2022-02-20 09:23:40
Closed_By zero-24
avatar zero-24 zero-24 - close - 20 Feb 2022
avatar zero-24 zero-24 - merge - 20 Feb 2022
avatar zero-24
zero-24 - comment - 20 Feb 2022

Merging thanks :-)

avatar brianteeman
brianteeman - comment - 20 Feb 2022

thanks

avatar coolcat-creations
coolcat-creations - comment - 21 Feb 2022

Thank you everyone

Add a Comment

Login with GitHub to post a comment