[Notifier] Change return type

OskarStark · OskarStark · commit d508013e4c38 · 2020-12-18T14:53:31.000+01:00
diff --git a/UPGRADE-5.3.md b/UPGRADE-5.3.md
@@ -23,6 +23,11 @@ HttpKernel
 
  * Marked the class `Symfony\Component\HttpKernel\EventListener\DebugHandlersListener` as internal
 
+Notifier
+-------
+
+* Changed the return type of `Symfony\Component\Notifier\Transport\AbstractTransportFactory::getEndpoint()` from `?string` to `string`
+
 PhpunitBridge
 -------------
 
diff --git a/src/Symfony/Component/Notifier/Bridge/Infobip/InfobipTransport.php b/src/Symfony/Component/Notifier/Bridge/Infobip/InfobipTransport.php
@@ -87,7 +87,7 @@ protected function doSend(MessageInterface $message): SentMessage
         return new SentMessage($message, (string) $this);
     }
 
-    protected function getEndpoint(): ?string
+    protected function getEndpoint(): string
     {
         return $this->host.($this->port ? ':'.$this->port : '');
     }
diff --git a/src/Symfony/Component/Notifier/Bridge/Mattermost/MattermostTransport.php b/src/Symfony/Component/Notifier/Bridge/Mattermost/MattermostTransport.php
@@ -87,7 +87,7 @@ protected function doSend(MessageInterface $message): SentMessage
         return $sentMessage;
     }
 
-    protected function getEndpoint(): ?string
+    protected function getEndpoint(): string
     {
         return rtrim($this->host.($this->port ? ':'.$this->port : '').($this->path ?? ''), '/');
     }
diff --git a/src/Symfony/Component/Notifier/CHANGELOG.md b/src/Symfony/Component/Notifier/CHANGELOG.md
@@ -1,6 +1,11 @@
 CHANGELOG
 =========
 
+5.3.0
+-----
+
+* [BC BREAK] Changed the return type of `AbstractTransportFactory::getEndpoint()` from `?string` to `string`
+
 5.2.0
 -----
 
diff --git a/src/Symfony/Component/Notifier/Transport/AbstractTransport.php b/src/Symfony/Component/Notifier/Transport/AbstractTransport.php
@@ -81,7 +81,7 @@ public function send(MessageInterface $message): SentMessage
 
     abstract protected function doSend(MessageInterface $message): SentMessage;
 
-    protected function getEndpoint(): ?string
+    protected function getEndpoint(): string
     {
         return ($this->host ?: $this->getDefaultHost()).($this->port ? ':'.$this->port : '');
     }

Original file line number	Diff line number	Diff line change
`@@ -87,7 +87,7 @@ protected function doSend(MessageInterface $message): SentMessage`
`87`	`87`	`return new SentMessage($message, (string) $this);`
`88`	`88`	`}`
`89`	`89`
`90`		`- protected function getEndpoint(): ?string`
	`90`	`+ protected function getEndpoint(): string`
`91`	`91`	`{`
`92`	`92`	`return $this->host.($this->port ? ':'.$this->port : '');`
`93`	`93`	`}`
Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,7 @@ public function send(MessageInterface $message): SentMessage`
`81`	`81`
`82`	`82`	`abstract protected function doSend(MessageInterface $message): SentMessage;`
`83`	`83`
`84`		`- protected function getEndpoint(): ?string`
	`84`	`+ protected function getEndpoint(): string`
`85`	`85`	`{`
`86`	`86`	`return ($this->host ?: $this->getDefaultHost()).($this->port ? ':'.$this->port : '');`
`87`	`87`	`}`