[Mailer] fixed logic

fabpot · fabpot · commit 288772478cb4 · 2019-07-18T10:29:30.000+02:00
diff --git a/src/Symfony/Component/Mailer/Transport/Dsn.php b/src/Symfony/Component/Mailer/Transport/Dsn.php
@@ -49,8 +49,8 @@ public static function fromString(string $dsn): self
             throw new InvalidArgumentException(sprintf('The "%s" mailer DSN must contain a mailer name.', $dsn));
         }
 
-        $user = urldecode($parsedDsn['user'] ?? null);
-        $password = urldecode($parsedDsn['pass'] ?? null);
+        $user = isset($parsedDsn['user']) ? urldecode($parsedDsn['user']) : null;
+        $password = isset($parsedDsn['pass']) ? urldecode($parsedDsn['pass']) : null;
         $port = $parsedDsn['port'] ?? null;
         parse_str($parsedDsn['query'] ?? '', $query);
 

Original file line number	Diff line number	Diff line change
`@@ -49,8 +49,8 @@ public static function fromString(string $dsn): self`
`49`	`49`	`throw new InvalidArgumentException(sprintf('The "%s" mailer DSN must contain a mailer name.', $dsn));`
`50`	`50`	`}`
`51`	`51`
`52`		`- $user = urldecode($parsedDsn['user'] ?? null);`
`53`		`- $password = urldecode($parsedDsn['pass'] ?? null);`
	`52`	`+ $user = isset($parsedDsn['user']) ? urldecode($parsedDsn['user']) : null;`
	`53`	`+ $password = isset($parsedDsn['pass']) ? urldecode($parsedDsn['pass']) : null;`
`54`	`54`	`$port = $parsedDsn['port'] ?? null;`
`55`	`55`	`parse_str($parsedDsn['query'] ?? '', $query);`
`56`	`56`