When a user account doesn't have an email, new user notification email attempt shouldn't be made

Created on 29 September 2022, about 2 years ago
Updated 23 March 2023, almost 2 years ago

Problem/Motivation

A user can be created anonymous or without an email address. However, postSave() hook tries to send an email after an user is created and it results in an error if the email handlers aren't checking for empty email addresses before hand.

Proposed resolution

User entity postSave() hook should check for the email address prior to making an email sending attempt.

๐Ÿ› Bug report
Status

Needs work

Version

9.5

Component
User systemย  โ†’

Last updated 3 days ago

Created by

fathima.asmat London, UK

Live updates comments and jobs are added and updated live.
  • Needs tests

    The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024