Skip to content

Commit 51b5d20

Browse files
committed
UserMailer to return successful send boolean
1 parent cd04b5d commit 51b5d20

1 file changed

Lines changed: 18 additions & 17 deletions

File tree

src/Helper/User/UserMailer.php

Lines changed: 18 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -43,52 +43,52 @@ public function __construct(MailerInterface $mailer, ContainerInterface $contain
4343
$this->context = $context;
4444
}
4545

46-
public function sendPasswordResetEmail(AbstractUser $user): void
46+
public function sendPasswordResetEmail(AbstractUser $user): bool
4747
{
4848
$email = $this->container->get(PasswordResetEmailFactory::class)->create($user, $this->context);
49-
$this->send($email);
49+
return $this->send($email);
5050
}
5151

52-
public function sendChangeEmailConfirmationEmail(AbstractUser $user): void
52+
public function sendChangeEmailConfirmationEmail(AbstractUser $user): bool
5353
{
5454
$email = $this->container->get(ChangeEmailConfirmationEmailFactory::class)->create($user, $this->context);
55-
$this->send($email);
55+
return $this->send($email);
5656
}
5757

58-
public function sendEmailVerifyEmail(AbstractUser $user): void
58+
public function sendEmailVerifyEmail(AbstractUser $user): bool
5959
{
6060
$email = $this->container->get(VerifyEmailFactory::class)->create($user, $this->context);
61-
$this->send($email);
61+
return $this->send($email);
6262
}
6363

64-
public function sendWelcomeEmail(AbstractUser $user): void
64+
public function sendWelcomeEmail(AbstractUser $user): bool
6565
{
6666
$email = $this->container->get(WelcomeEmailFactory::class)->create($user, $this->context);
67-
$this->send($email);
67+
return $this->send($email);
6868
}
6969

70-
public function sendUserEnabledEmail(AbstractUser $user): void
70+
public function sendUserEnabledEmail(AbstractUser $user): bool
7171
{
7272
$email = $this->container->get(UserEnabledEmailFactory::class)->create($user, $this->context);
73-
$this->send($email);
73+
return $this->send($email);
7474
}
7575

76-
public function sendUsernameChangedEmail(AbstractUser $user): void
76+
public function sendUsernameChangedEmail(AbstractUser $user): bool
7777
{
7878
$email = $this->container->get(UsernameChangedEmailFactory::class)->create($user, $this->context);
79-
$this->send($email);
79+
return $this->send($email);
8080
}
8181

82-
public function sendPasswordChangedEmail(AbstractUser $user): void
82+
public function sendPasswordChangedEmail(AbstractUser $user): bool
8383
{
8484
$email = $this->container->get(PasswordChangedEmailFactory::class)->create($user, $this->context);
85-
$this->send($email);
85+
return $this->send($email);
8686
}
8787

88-
private function send(?RawMessage $message): void
88+
private function send(?RawMessage $message): bool
8989
{
9090
if (null === $message) {
91-
return;
91+
return false;
9292
}
9393

9494
try {
@@ -100,9 +100,10 @@ private function send(?RawMessage $message): void
100100
$logger->error($exception->getMessage(), [
101101
'exception' => $exception,
102102
]);
103-
return;
103+
return false;
104104
}
105105
throw $exception;
106106
}
107+
return true;
107108
}
108109
}

0 commit comments

Comments
 (0)