Fix issues caught by static analysis tests.

mailgun · Oct 13, 2021 · 054994c · 054994c
1 parent 276ad24
commit 054994c
Show file tree

Hide file tree

Showing 8 changed files with 13 additions and 10 deletions.
diff --git a/src/Api/Attachment.php b/src/Api/Attachment.php
@@ -12,11 +12,9 @@
 namespace Mailgun\Api;
 
 use Mailgun\Assert;
-use Mailgun\Exception\{
-    HttpClientException,
-    HttpServerException,
-    UnknownErrorException
-};
+use Mailgun\Exception\HttpClientException;
+use Mailgun\Exception\HttpServerException;
+use Mailgun\Exception\UnknownErrorException;
 use Psr\Http\Message\ResponseInterface;
 
 /**

diff --git a/src/Api/HttpApi.php b/src/Api/HttpApi.php
@@ -82,6 +82,7 @@ protected function hydrateResponse(ResponseInterface $response, string $class)
      * Throw the correct exception for this error.
      *
      * @throws HttpClientException|HttpServerException|UnknownErrorException
+     *
      * @return never
      */
     protected function handleErrors(ResponseInterface $response): void

diff --git a/src/Assert.php b/src/Assert.php
@@ -21,7 +21,10 @@
 final class Assert extends \Webmozart\Assert\Assert
 {
     /**
+     * @psalm-pure
+     *
      * @throws InvalidArgumentException
+     *
      * @return never
      */
     protected static function reportInvalidArgument($message): void

diff --git a/src/HttpClient/RequestBuilder.php b/src/HttpClient/RequestBuilder.php
@@ -130,8 +130,7 @@ private function createRequest(
         string $uri,
         array $headers,
         StreamInterface $stream
-    ): RequestInterface
-    {
+    ): RequestInterface {
         $request = $this->getRequestFactory()->createRequest($method, $uri);
         $request = $request->withBody($stream);
         foreach ($headers as $name => $value) {

diff --git a/src/Hydrator/ModelHydrator.php b/src/Hydrator/ModelHydrator.php
@@ -26,6 +26,7 @@ final class ModelHydrator implements Hydrator
      * @param class-string $class
      *
      * @throws HydrationException
+     *
      * @return ResponseInterface
      */
     public function hydrate(ResponseInterface $response, string $class)

diff --git a/src/Hydrator/NoopHydrator.php b/src/Hydrator/NoopHydrator.php
@@ -24,6 +24,7 @@ final class NoopHydrator implements Hydrator
      * @param class-string $class
      *
      * @throws \LogicException
+     *
      * @return never
      */
     public function hydrate(ResponseInterface $response, string $class)

diff --git a/src/Message/Exceptions/LimitExceeded.php b/src/Message/Exceptions/LimitExceeded.php
@@ -15,7 +15,7 @@
 
 class LimitExceeded extends \Exception implements Exception
 {
-    public static function create(string $field, int $limit): self
+    public static function create(string $field, int $limit)
     {
         return new self(sprintf('You\'ve exceeded the maximum (%d) %s for a single message.', $limit, $field));
     }

diff --git a/src/Message/Exceptions/TooManyRecipients.php b/src/Message/Exceptions/TooManyRecipients.php
@@ -16,12 +16,12 @@
 
 class TooManyRecipients extends LimitExceeded implements Exception
 {
-    public static function create(string $field, int $limit = MessageBuilder::RECIPIENT_COUNT_LIMIT): self
+    public static function create(string $field, int $limit = MessageBuilder::RECIPIENT_COUNT_LIMIT)
     {
         return new self(sprintf('You\'ve exceeded the maximum recipient count (%s) for filed "%s".', $limit, $field));
     }
 
-    public static function whenAutoSendDisabled(int $limit = MessageBuilder::RECIPIENT_COUNT_LIMIT): self
+    public static function whenAutoSendDisabled(int $limit = MessageBuilder::RECIPIENT_COUNT_LIMIT)
     {
         return new self(sprintf('You\'ve exceeded the maximum recipient count (%s) with autosend disabled.', $limit));
     }