Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3.x] Update registration.php #36993

Closed
wants to merge 2 commits into from
Closed

Conversation

brianteeman
Copy link
Contributor

Send mail to all administrators with access to com_users and set to receive system emails instead of sending the mail to all users with "receive system emails"

Please test very carefully with all 3 user registration options

@coolcat-creations
Copy link
Contributor

Tested unsuccessful

My steps:

Using my own website I created two users

  1. Testuser1, registered, system mails enabled
  2. Testuser2, registered, system mails disabled

I set the "Send Mail to Administrators" Option in the User Options to yes

I registered from the Frontend with a new user
Newuser1

  • The Administrator got an email
  • The registered user with system mails enabled got an email

I downloaded the prebuilt package Joomla_3.10.5_to_3.10.6-rc2-dev+pr.36993-Development-Patch_Package.zip and updated Joomla with it.

I registered from the Frontend with a new user
Newuser2

  • The Administrator got an email
  • The registered user with system mails enabled got an email

@brianteeman
Copy link
Contributor Author

Could you check the other options as well please

@brianteeman
Copy link
Contributor Author

image

I may have missed one

@coolcat-creations
Copy link
Contributor

Tested now with:

Self - Failed
None - Failed
Administrator - Success

@brianteeman
Copy link
Contributor Author

and can you confirm you had three fail before?

@coolcat-creations
Copy link
Contributor

I tested now the other two options without the patch and its the same.

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

@brianteeman
Copy link
Contributor Author

ok I will take a real look. I only tested it on j4 so probably made a mistake when just typing it into j3

@coolcat-creations
Copy link
Contributor

Thank you. Could there be something wrong with the prebuilt package, can I use it as an update like I did?

@brianteeman
Copy link
Contributor Author

no - its more likely to be me

@brianteeman
Copy link
Contributor Author

please try #36997

@RickR2H RickR2H reopened this Apr 21, 2022
@RickR2H
Copy link
Member

RickR2H commented Apr 21, 2022

Open en closed to try fix issues with the issue tracker

@RickR2H RickR2H closed this Apr 21, 2022
@brianteeman brianteeman deleted the patch-8 branch July 12, 2022 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants