minor #35081 [Notifier] Add more specific types at documentation level when php engine can't (Philippe Segatori)

This PR was squashed before being merged into the 5.0 branch (closes #35081).

Discussion
----------

[Notifier] Add more specific types at documentation level when php engine can't

| Q             | A
| ------------- | ---
| Branch?       | 5.0 <!-- see below -->
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | Fix #35078  <!-- prefix each issue number with "Fix #", if any -->
| License       | MIT
| Doc PR        | . <!-- required for new features -->

This an example of the proposal made in #35078 to document a narrower type when php type declaration cannot.

Commits
-------

b9931f571c [Notifier] Add more specific types at documentation level when php engine can't
This commit is contained in:
Fabien Potencier 2020-01-08 10:13:33 +01:00
commit ad7d24109c
6 changed files with 16 additions and 1 deletions

View File

@ -23,6 +23,9 @@ use Symfony\Component\Notifier\Transport\TransportInterface;
*/
final class NexmoTransportFactory extends AbstractTransportFactory
{
/**
* @return NexmoTransport
*/
public function create(Dsn $dsn): TransportInterface
{
$scheme = $dsn->getScheme();

View File

@ -23,6 +23,9 @@ use Symfony\Component\Notifier\Transport\TransportInterface;
*/
final class SlackTransportFactory extends AbstractTransportFactory
{
/**
* @return SlackTransport
*/
public function create(Dsn $dsn): TransportInterface
{
$scheme = $dsn->getScheme();

View File

@ -24,6 +24,9 @@ use Symfony\Component\Notifier\Transport\TransportInterface;
*/
final class TelegramTransportFactory extends AbstractTransportFactory
{
/**
* @return TelegramTransport
*/
public function create(Dsn $dsn): TransportInterface
{
$scheme = $dsn->getScheme();

View File

@ -23,6 +23,9 @@ use Symfony\Component\Notifier\Transport\TransportInterface;
*/
final class TwilioTransportFactory extends AbstractTransportFactory
{
/**
* @return TwilioTransport
*/
public function create(Dsn $dsn): TransportInterface
{
$scheme = $dsn->getScheme();

View File

@ -58,7 +58,7 @@ final class SmsMessage implements MessageInterface
return $this->phone;
}
public function getRecipientId(): ?string
public function getRecipientId(): string
{
return $this->phone;
}

View File

@ -20,6 +20,9 @@ use Symfony\Component\Notifier\Exception\UnsupportedSchemeException;
*/
final class NullTransportFactory extends AbstractTransportFactory
{
/**
* @return NullTransport
*/
public function create(Dsn $dsn): TransportInterface
{
if ('null' === $dsn->getScheme()) {