[Notifier] cs

nicolas-grekas · nicolas-grekas · commit ef84e66b47cc · 2021-07-21T13:31:37.000+02:00
diff --git a/src/Symfony/Component/Notifier/Bridge/FakeChat/FakeChatTransportFactory.php b/src/Symfony/Component/Notifier/Bridge/FakeChat/FakeChatTransportFactory.php
@@ -38,17 +38,15 @@ public function create(Dsn $dsn): TransportInterface
     {
         $scheme = $dsn->getScheme();
 
-        if (!\in_array($scheme, $this->getSupportedSchemes())) {
+        if ('fakechat+email' !== $scheme) {
             throw new UnsupportedSchemeException($dsn, 'fakechat', $this->getSupportedSchemes());
         }
 
-        if ('fakechat+email' === $scheme) {
-            $mailerTransport = $dsn->getHost();
-            $to = $dsn->getRequiredOption('to');
-            $from = $dsn->getRequiredOption('from');
+        $mailerTransport = $dsn->getHost();
+        $to = $dsn->getRequiredOption('to');
+        $from = $dsn->getRequiredOption('from');
 
-            return (new FakeChatEmailTransport($this->mailer, $to, $from))->setHost($mailerTransport);
-        }
+        return (new FakeChatEmailTransport($this->mailer, $to, $from))->setHost($mailerTransport);
     }
 
     protected function getSupportedSchemes(): array
diff --git a/src/Symfony/Component/Notifier/Bridge/FakeSms/FakeSmsTransportFactory.php b/src/Symfony/Component/Notifier/Bridge/FakeSms/FakeSmsTransportFactory.php
@@ -39,17 +39,15 @@ public function create(Dsn $dsn): TransportInterface
     {
         $scheme = $dsn->getScheme();
 
-        if (!\in_array($scheme, $this->getSupportedSchemes())) {
+        if ('fakesms+email' !== $scheme) {
             throw new UnsupportedSchemeException($dsn, 'fakesms', $this->getSupportedSchemes());
         }
 
-        if ('fakesms+email' === $scheme) {
-            $mailerTransport = $dsn->getHost();
-            $to = $dsn->getRequiredOption('to');
-            $from = $dsn->getRequiredOption('from');
+        $mailerTransport = $dsn->getHost();
+        $to = $dsn->getRequiredOption('to');
+        $from = $dsn->getRequiredOption('from');
 
-            return (new FakeSmsEmailTransport($this->mailer, $to, $from))->setHost($mailerTransport);
-        }
+        return (new FakeSmsEmailTransport($this->mailer, $to, $from))->setHost($mailerTransport);
     }
 
     protected function getSupportedSchemes(): array

Original file line number	Diff line number	Diff line change
`@@ -38,17 +38,15 @@ public function create(Dsn $dsn): TransportInterface`
`38`	`38`	`{`
`39`	`39`	`$scheme = $dsn->getScheme();`
`40`	`40`
`41`		`- if (!\in_array($scheme, $this->getSupportedSchemes())) {`
	`41`	`+ if ('fakechat+email' !== $scheme) {`
`42`	`42`	`throw new UnsupportedSchemeException($dsn, 'fakechat', $this->getSupportedSchemes());`
`43`	`43`	`}`
`44`	`44`
`45`		`- if ('fakechat+email' === $scheme) {`
`46`		`- $mailerTransport = $dsn->getHost();`
`47`		`- $to = $dsn->getRequiredOption('to');`
`48`		`- $from = $dsn->getRequiredOption('from');`
	`45`	`+ $mailerTransport = $dsn->getHost();`
	`46`	`+ $to = $dsn->getRequiredOption('to');`
	`47`	`+ $from = $dsn->getRequiredOption('from');`
`49`	`48`
`50`		`- return (new FakeChatEmailTransport($this->mailer, $to, $from))->setHost($mailerTransport);`
`51`		`- }`
	`49`	`+ return (new FakeChatEmailTransport($this->mailer, $to, $from))->setHost($mailerTransport);`
`52`	`50`	`}`
`53`	`51`
`54`	`52`	`protected function getSupportedSchemes(): array`
Original file line number	Diff line number	Diff line change
`@@ -39,17 +39,15 @@ public function create(Dsn $dsn): TransportInterface`
`39`	`39`	`{`
`40`	`40`	`$scheme = $dsn->getScheme();`
`41`	`41`
`42`		`- if (!\in_array($scheme, $this->getSupportedSchemes())) {`
	`42`	`+ if ('fakesms+email' !== $scheme) {`
`43`	`43`	`throw new UnsupportedSchemeException($dsn, 'fakesms', $this->getSupportedSchemes());`
`44`	`44`	`}`
`45`	`45`
`46`		`- if ('fakesms+email' === $scheme) {`
`47`		`- $mailerTransport = $dsn->getHost();`
`48`		`- $to = $dsn->getRequiredOption('to');`
`49`		`- $from = $dsn->getRequiredOption('from');`
	`46`	`+ $mailerTransport = $dsn->getHost();`
	`47`	`+ $to = $dsn->getRequiredOption('to');`
	`48`	`+ $from = $dsn->getRequiredOption('from');`
`50`	`49`
`51`		`- return (new FakeSmsEmailTransport($this->mailer, $to, $from))->setHost($mailerTransport);`
`52`		`- }`
	`50`	`+ return (new FakeSmsEmailTransport($this->mailer, $to, $from))->setHost($mailerTransport);`
`53`	`51`	`}`
`54`	`52`
`55`	`53`	`protected function getSupportedSchemes(): array`