Apply suggestions from code review

OskarStark · web-flow · commit bcfeb1637cbd · 2020-12-17T10:50:01.000+01: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 host (use "default" by default).', $dsn));
         }
 
-        $user = isset($parsedDsn['user']) && '' !== ($parsedDsn['user'] ?? '') ? urldecode($parsedDsn['user']) : null;
-        $password = isset($parsedDsn['pass']) && '' !== ($parsedDsn['pass'] ?? '') ? urldecode($parsedDsn['pass']) : null;
+        $user = '' !== ($parsedDsn['user'] ?? '') ? urldecode($parsedDsn['user']) : null;
+        $password = '' !== ($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 host (use "default" by default).', $dsn));`
`50`	`50`	`}`
`51`	`51`
`52`		`- $user = isset($parsedDsn['user']) && '' !== ($parsedDsn['user'] ?? '') ? urldecode($parsedDsn['user']) : null;`
`53`		`- $password = isset($parsedDsn['pass']) && '' !== ($parsedDsn['pass'] ?? '') ? urldecode($parsedDsn['pass']) : null;`
	`52`	`+ $user = '' !== ($parsedDsn['user'] ?? '') ? urldecode($parsedDsn['user']) : null;`
	`53`	`+ $password = '' !== ($parsedDsn['pass'] ?? '') ? urldecode($parsedDsn['pass']) : null;`
`54`	`54`	`$port = $parsedDsn['port'] ?? null;`
`55`	`55`	`parse_str($parsedDsn['query'] ?? '', $query);`
`56`	`56`