[Notifier] [Mattermost] [BC BREAK] Change constructor signature

OskarStark · OskarStark · commit 9a4d2bd59d4d · 2020-12-28T08:08:05.000+01:00
| Q | A | ------------- | --- | Branch? | 5.x, but BC BREAK for experimental bridge | Bug fix? | no | New feature? | no | Deprecations? | no | Tickets | --- | License | MIT | Doc PR | --- Follows #39545
diff --git a/src/Symfony/Component/Notifier/Bridge/Mattermost/CHANGELOG.md b/src/Symfony/Component/Notifier/Bridge/Mattermost/CHANGELOG.md
@@ -5,6 +5,10 @@ CHANGELOG
 -----
 
  * The bridge is not marked as `@experimental` anymore
+* [BC BREAK] Changed signature of `MattermostTransport::__construct()` method from:
+  `public function __construct(string $token, string $channel, HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null, string $path = null)`
+  to:
+  `public function __construct(string $token, string $channel, ?string $path = null, HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null)`
 
 5.1.0
 -----
diff --git a/src/Symfony/Component/Notifier/Bridge/Mattermost/MattermostTransport.php b/src/Symfony/Component/Notifier/Bridge/Mattermost/MattermostTransport.php
@@ -29,7 +29,7 @@ final class MattermostTransport extends AbstractTransport
     private $channel;
     private $path;
 
-    public function __construct(string $token, string $channel, HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null, string $path = null)
+    public function __construct(string $token, string $channel, ?string $path = null, HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null)
     {
         $this->token = $token;
         $this->channel = $channel;
diff --git a/src/Symfony/Component/Notifier/Bridge/Mattermost/MattermostTransportFactory.php b/src/Symfony/Component/Notifier/Bridge/Mattermost/MattermostTransportFactory.php
@@ -41,7 +41,7 @@ public function create(Dsn $dsn): TransportInterface
         $host = $dsn->getHost();
         $port = $dsn->getPort();
 
-        return (new MattermostTransport($token, $channel, $this->client, $this->dispatcher, $path))->setHost($host)->setPort($port);
+        return (new MattermostTransport($token, $channel, $path, $this->client, $this->dispatcher))->setHost($host)->setPort($port);
     }
 
     protected function getSupportedSchemes(): array
diff --git a/src/Symfony/Component/Notifier/Bridge/Mattermost/Tests/MattermostTransportTest.php b/src/Symfony/Component/Notifier/Bridge/Mattermost/Tests/MattermostTransportTest.php
@@ -29,7 +29,7 @@ final class MattermostTransportTest extends TransportTestCase
      */
     public function createTransport(?HttpClientInterface $client = null): TransportInterface
     {
-        return (new MattermostTransport('testAccessToken', 'testChannel', $client ?: $this->createMock(HttpClientInterface::class)))->setHost('host.test');
+        return (new MattermostTransport('testAccessToken', 'testChannel', null, $client ?: $this->createMock(HttpClientInterface::class)))->setHost('host.test');
     }
 
     public function toStringProvider(): iterable

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,7 @@ final class MattermostTransport extends AbstractTransport`
`29`	`29`	`private $channel;`
`30`	`30`	`private $path;`
`31`	`31`
`32`		`- public function __construct(string $token, string $channel, HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null, string $path = null)`
	`32`	`+ public function __construct(string $token, string $channel, ?string $path = null, HttpClientInterface $client = null, EventDispatcherInterface $dispatcher = null)`
`33`	`33`	`{`
`34`	`34`	`$this->token = $token;`
`35`	`35`	`$this->channel = $channel;`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public function create(Dsn $dsn): TransportInterface`
`41`	`41`	`$host = $dsn->getHost();`
`42`	`42`	`$port = $dsn->getPort();`
`43`	`43`
`44`		`- return (new MattermostTransport($token, $channel, $this->client, $this->dispatcher, $path))->setHost($host)->setPort($port);`
	`44`	`+ return (new MattermostTransport($token, $channel, $path, $this->client, $this->dispatcher))->setHost($host)->setPort($port);`
`45`	`45`	`}`
`46`	`46`
`47`	`47`	`protected function getSupportedSchemes(): array`