Merge pull request #4 from screenfluent/fix/mailer-debug-check

mychidarko · web-flow · commit 6e51cda6a81c · 2024-11-27T15:36:15.000Z
fix: safely access debug config with null coalescing
diff --git a/src/Mail/Mailer.php b/src/Mail/Mailer.php
@@ -124,10 +124,12 @@ protected static function validate(Mail $mail)
      */
     public static function send(Mail $mail)
     {
-        if (static::$config['debug'] === "SERVER") {
+        $debug = static::$config['debug'] ?? SMTP::DEBUG_OFF;
+
+        if ($debug === "SERVER") {
             static::$mailer->SMTPDebug = SMTP::DEBUG_SERVER;
         } else {
-            static::$mailer->SMTPDebug = static::$config['debug'] ?? SMTP::DEBUG_OFF;
+            static::$mailer->SMTPDebug = $debug;
         }
 
         if (static::validate($mail)) {

Original file line number	Diff line number	Diff line change
`@@ -124,10 +124,12 @@ protected static function validate(Mail $mail)`
`124`	`124`	`*/`
`125`	`125`	`public static function send(Mail $mail)`
`126`	`126`	`{`
`127`		`- if (static::$config['debug'] === "SERVER") {`
	`127`	`+ $debug = static::$config['debug'] ?? SMTP::DEBUG_OFF;`
	`128`	`+`
	`129`	`+ if ($debug === "SERVER") {`
`128`	`130`	`static::$mailer->SMTPDebug = SMTP::DEBUG_SERVER;`
`129`	`131`	`} else {`
`130`		`- static::$mailer->SMTPDebug = static::$config['debug'] ?? SMTP::DEBUG_OFF;`
	`132`	`+ static::$mailer->SMTPDebug = $debug;`
`131`	`133`	`}`
`132`	`134`
`133`	`135`	`if (static::validate($mail)) {`