Running php-cs-fixer

Running php-cs-fixer to fix CS drift

Author: JimTools

SnsClient.php

@@ -140,6 +140,6 @@ private function resolveClient(): void
             }
         }
 
-        throw new \LogicException(sprintf('The input client must be an instance of "%s" or "%s" or a callable that returns one of those. Got "%s"', AwsSnsClient::class, MultiRegionClient::class, is_object($client) ? get_class($client) : gettype($client)));
+        throw new \LogicException(sprintf('The input client must be an instance of "%s" or "%s" or a callable that returns one of those. Got "%s"', AwsSnsClient::class, MultiRegionClient::class, is_object($client) ? $client::class : gettype($client)));
     }
 }

SnsDestination.php

@@ -34,7 +34,7 @@ public function getTopicName(): string
     /**
      * The policy that defines who can access your topic. By default, only the topic owner can publish or subscribe to the topic.
      */
-    public function setPolicy(string $policy = null): void
+    public function setPolicy(?string $policy = null): void
     {
         $this->setAttribute('Policy', $policy);
     }
@@ -47,7 +47,7 @@ public function getPolicy(): ?string
     /**
      * The display name to use for a topic with SMS subscriptions.
      */
-    public function setDisplayName(string $displayName = null): void
+    public function setDisplayName(?string $displayName = null): void
     {
         $this->setAttribute('DisplayName', $displayName);
     }
@@ -60,7 +60,7 @@ public function getDisplayName(): ?string
     /**
      * The display name to use for a topic with SMS subscriptions.
      */
-    public function setDeliveryPolicy(int $deliveryPolicy = null): void
+    public function setDeliveryPolicy(?int $deliveryPolicy = null): void
     {
         $this->setAttribute('DeliveryPolicy', $deliveryPolicy);
     }

SnsMessage.php

@@ -63,11 +63,11 @@ public function __construct(
         string $body = '',
         array $properties = [],
         array $headers = [],
-        array $messageAttributes = null,
-        string $messageStructure = null,
-        string $phoneNumber = null,
-        string $subject = null,
-        string $targetArn = null
+        ?array $messageAttributes = null,
+        ?string $messageStructure = null,
+        ?string $phoneNumber = null,
+        ?string $subject = null,
+        ?string $targetArn = null,
     ) {
         $this->body = $body;
         $this->properties = $properties;
@@ -192,7 +192,7 @@ public function setTargetArn(?string $targetArn): void
      * of each user is processed in a FIFO fashion.
      * For more information, see: https://docs.aws.amazon.com/sns/latest/dg/fifo-message-grouping.html
      */
-    public function setMessageGroupId(string $id = null): void
+    public function setMessageGroupId(?string $id = null): void
     {
         $this->messageGroupId = $id;
     }
@@ -210,7 +210,7 @@ public function getMessageGroupId(): ?string
      * aren't delivered during the 5-minute deduplication interval.
      * For more information, see https://docs.aws.amazon.com/sns/latest/dg/fifo-message-dedup.html
      */
-    public function setMessageDeduplicationId(string $id = null): void
+    public function setMessageDeduplicationId(?string $id = null): void
     {
         $this->messageDeduplicationId = $id;
     }

SnsProducer.php

@@ -99,7 +99,7 @@ public function send(Destination $destination, Message $message): void
      *
      * @return SnsProducer
      */
-    public function setDeliveryDelay(int $deliveryDelay = null): Producer
+    public function setDeliveryDelay(?int $deliveryDelay = null): Producer
     {
         if (null === $deliveryDelay) {
             return $this;
@@ -118,7 +118,7 @@ public function getDeliveryDelay(): ?int
      *
      * @return SnsProducer
      */
-    public function setPriority(int $priority = null): Producer
+    public function setPriority(?int $priority = null): Producer
     {
         if (null === $priority) {
             return $this;
@@ -137,7 +137,7 @@ public function getPriority(): ?int
      *
      * @return SnsProducer
      */
-    public function setTimeToLive(int $timeToLive = null): Producer
+    public function setTimeToLive(?int $timeToLive = null): Producer
     {
         if (null === $timeToLive) {
             return $this;

SnsSubscribe.php

@@ -6,7 +6,7 @@
 
 class SnsSubscribe
 {
-    const PROTOCOL_SQS = 'sqs';
+    public const PROTOCOL_SQS = 'sqs';
 
     /**
      * @var SnsDestination
@@ -23,22 +23,16 @@ class SnsSubscribe
      */
     private $protocol;
 
-    /**
-     * @var
-     */
     private $returnSubscriptionArn;
 
-    /**
-     * @var
-     */
     private $attributes;
 
     public function __construct(
         SnsDestination $topic,
         string $endpoint,
         string $protocol,
         bool $returnSubscriptionArn = false,
-        array $attributes = []
+        array $attributes = [],
     ) {
         $this->topic = $topic;
         $this->endpoint = $endpoint;

SnsUnsubscribe.php

@@ -24,7 +24,7 @@ class SnsUnsubscribe
     public function __construct(
         SnsDestination $topic,
         string $endpoint,
-        string $protocol
+        string $protocol,
     ) {
         $this->topic = $topic;
         $this->endpoint = $endpoint;

Tests/SnsConnectionFactoryConfigTest.php

@@ -41,9 +41,6 @@ public function testThrowIfDsnCouldNotBeParsed()
 
     /**
      * @dataProvider provideConfigs
-     *
-     * @param mixed $config
-     * @param mixed $expectedConfig
      */
     public function testShouldParseConfigurationAsExpected($config, $expectedConfig)
     {

examples/consume.php

@@ -12,7 +12,7 @@
 if ($autoload) {
     require_once $autoload;
 } else {
-    throw new \LogicException('Composer autoload was not found');
+    throw new LogicException('Composer autoload was not found');
 }
 
 use Enqueue\Sns\SnsConnectionFactory;
@@ -26,7 +26,7 @@
 while (true) {
     if ($m = $consumer->receive(20000)) {
         $consumer->acknowledge($m);
-        echo 'Received message: '.$m->getBody().PHP_EOL;
+        echo 'Received message: '.$m->getBody().\PHP_EOL;
     }
 }
 

examples/produce.php

@@ -12,7 +12,7 @@
 if ($autoload) {
     require_once $autoload;
 } else {
-    throw new \LogicException('Composer autoload was not found');
+    throw new LogicException('Composer autoload was not found');
 }
 
 use Enqueue\Sns\SnsConnectionFactory;