Skip to content

Commit 6fc330a

Browse files
author
Yansell Rivas Diaz
committed
feat(EmailBundle) change required value
* added: At least one email is required and mailTo is not required * Note: update the database
1 parent 90f5b75 commit 6fc330a

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

Entity/Email.php

100644100755
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,7 @@ class Email
5454
/**
5555
* @var string
5656
*
57-
* @ORM\Column(name="message_to", type="string", length=255)
57+
* @ORM\Column(name="message_to", type="string", length=255, nullable=true)
5858
*/
5959
private $messageTo;
6060

Service/Mailer.php

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -304,6 +304,11 @@ private function processEmail($from, $to, $cc = array(), $bcc = array(), $subjec
304304
$subject = "TEST - ".$subject;
305305
}
306306

307+
if(!$to && !$cc && !$bcc) {
308+
$this->container->get('logger')->log(\Monolog\Logger::CRITICAL, "At least one email is required");
309+
return false;
310+
}
311+
307312
/** @var Swift_Message $message */
308313
$message = new \Swift_Message($subject);
309314
$message->setFrom($from);
@@ -322,9 +327,6 @@ private function processEmail($from, $to, $cc = array(), $bcc = array(), $subjec
322327
}
323328
} elseif($to != "" && filter_var($to, FILTER_VALIDATE_EMAIL)) {
324329
$message->setTo($to);
325-
} else {
326-
$this->container->get('logger')->log(\Monolog\Logger::CRITICAL, "Invalid recipient: ".json_encode($to));
327-
return false;
328330
}
329331

330332
if(is_array($bcc)) {

0 commit comments

Comments
 (0)