# [LOW] Wrong email recipient when replying to guest user comment

Closes gh-271
akeeba · Mar 30, 2023 · b72428f · b72428f
1 parent 3e648ff
commit b72428f
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 3 deletions.
diff --git a/.idea/php.xml b/.idea/php.xml
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,6 +1,7 @@
 <?php die(); ?>
 Akeeba Engage 3.2.6
 ================================================================================
+# [LOW] Wrong email recipient when replying to guest user comment (gh-271)
 # [HIGH] The DataCompliance plugin crashed when viewing the data options for a user
 
 Akeeba Engage 3.2.5

diff --git a/component/backend/src/Helper/TemplateEmails.php b/component/backend/src/Helper/TemplateEmails.php
@@ -286,7 +286,7 @@ public static function sendMail(string $key, array $data, User $user = null, str
 		}
 
 		//if ($user->guest || $user->block || !$user->sendEmail)
-		if ($user->guest || $user->block)
+		if ($user->block)
 		{
 			return false;
 		}

diff --git a/plugins/engage/email/src/Extension/Email.php b/plugins/engage/email/src/Extension/Email.php
@@ -390,7 +390,7 @@ protected function sendEmailMessages(string $type, CommentTable $comment, array
 			// Get the recipient Joomla user
 			try
 			{
-				$recipient = UserFetcher::getUser($userIDs[$email]);
+				$recipient = isset($userIDs[$email]) ? UserFetcher::getUser($userIDs[$email]) : null;
 			}
 			catch (Exception $e)
 			{