Scrutinizer Auto-Fixes (#343)

This commit consists of patches automatically generated for this project on https://scrutinizer-ci.com

src/Viserio/Cache/CacheManager.php

@@ -23,7 +23,6 @@
 use Memcached;
 use MongoDB\Driver\Manager as MongoDBManager;
 use Predis\Client as PredisClient;
-use Psr\Cache\CacheItemPoolInterface;
 use Redis;
 use Viserio\{
     Filesystem\FilesystemManager,

src/Viserio/Connect/ConnectManager.php

@@ -106,7 +106,7 @@ protected function createMariadbConnection(array $config): PDO
      *
      * @param array $config
      *
-     * @return \Mongo|\MongoClient|\MongoDB\Client
+     * @return PDO
      */
     protected function createMongoConnection(array $config): PDO
     {

src/Viserio/Container/Container.php

@@ -3,7 +3,6 @@
 namespace Viserio\Container;
 
 use Interop\Container\ContainerInterface as ContainerInteropInterface;
-use Nucleus\Invoker\Invoker;
 use Viserio\Container\Exception\BindingResolutionException;
 use Viserio\Container\Exception\ContainerException;
 use Viserio\Container\Exception\NotFoundException;

src/Viserio/Contracts/Exception/Exception/FlattenException.php

@@ -2,9 +2,9 @@
 declare(strict_types=1);
 namespace Viserio\Contracts\Exception\Exception;
 
-use __PHP_Incomplete_Class;
 use ArrayObject;
 use Throwable;
+use __PHP_Incomplete_Class;
 
 /**
  * @author Fabien Potencier <fabien@symfony.com>

src/Viserio/Contracts/Exception/Handler.php

@@ -74,11 +74,13 @@ public function addShouldntReport(Throwable $exception): Handler;
 
     /**
      * Register the exception / Error handlers for the application.
+     * @return void
      */
     public function register();
 
     /**
      * Unregister the PHP error handler.
+     * @return void
      */
     public function unregister();
 
@@ -96,6 +98,7 @@ public function unregister();
      * @param null   $context
      *
      * @throws \ErrorException
+     * @return void
      */
     public function handleError(
         int $level,
@@ -113,11 +116,13 @@ public function handleError(
      * be handled differently since they are not normal exceptions.
      *
      * @param \Throwable $exception
+     * @return null|string
      */
     public function handleException(Throwable $exception);
 
     /**
      * Handle the PHP shutdown event.
+     * @return void
      */
     public function handleShutdown();
 }

src/Viserio/Contracts/Mail/GPGMailer.php

@@ -81,7 +81,7 @@ public function sign($text): string;
      *
      * @throws \Exception
      *
-     * @return string
+     * @return boolean
      */
     public function verify($text, string $fingerprint): bool;
 }

src/Viserio/Contracts/StaticalProxy/AliasLoader.php

@@ -25,6 +25,7 @@ public function load(string $alias): bool;
 
     /**
      * Removes an alias.
+     * @return void
      */
     public function removeAlias();
 
@@ -42,6 +43,7 @@ public function resolveAlias(string $alias);
      *
      * @param string|string[] $patterns
      * @param string|null     $translation
+     * @return void
      */
     public function aliasPattern($patterns, string $translation = null);
 
@@ -50,6 +52,7 @@ public function aliasPattern($patterns, string $translation = null);
      *
      * @param string      $pattern
      * @param string|null $translation
+     * @return void
      */
     public function removeAliasPattern(string $pattern, string $translation = null);
 
@@ -58,6 +61,7 @@ public function removeAliasPattern(string $pattern, string $translation = null);
      *
      * @param string $class
      * @param string $alias
+     * @return void
      */
     public function aliasNamespace(string $class, string $alias);
 
@@ -72,11 +76,13 @@ public function resolveNamespaceAlias(string $alias);
 
     /**
      * Removes a namespace alias.
+     * @return void
      */
     public function removeNamespaceAlias();
 
     /**
      * Register the loader on the auto-loader stack.
+     * @return void
      */
     public function register();
 
@@ -89,13 +95,15 @@ public function isRegistered(): bool;
 
     /**
      * Unregisters the autoloader function.
+     * @return void
      */
     public function unregister();
 
     /**
      * Set the registered aliases.
      *
      * @param array $aliases
+     * @return void
      */
     public function setAliases(array $aliases);
 

src/Viserio/Contracts/View/Virtuoso.php

@@ -83,6 +83,7 @@ public function yieldContent(string $section, string $default = ''): string;
      *
      * @param string $section
      * @param string $content
+     * @return void
      */
     public function startSection(string $section, string $content = '');
 
@@ -91,6 +92,7 @@ public function startSection(string $section, string $content = '');
      *
      * @param string $section
      * @param string $content
+     * @return void
      */
     public function inject(string $section, string $content);
 
@@ -116,21 +118,25 @@ public function appendSection(): string;
 
     /**
      * Clear all of the section contents.
+     * @return void
      */
     public function clearSections();
 
     /**
      * Clear all of the section contents if done rendering.
+     * @return void
      */
     public function clearSectionsIfDoneRendering();
 
     /**
      * Increment the rendering counter.
+     * @return void
      */
     public function incrementRender();
 
     /**
      * Decrement the rendering counter.
+     * @return void
      */
     public function decrementRender();
 

src/Viserio/Mail/Mailer.php

@@ -3,12 +3,11 @@
 namespace Viserio\Mail;
 
 use Closure;
-use Exception;
 use InvalidArgumentException;
 use Narrowspark\Arr\StaticArr as Arr;
 use Swift_Mailer;
-use Swift_Mime_Message;
 use Swift_Message;
+use Swift_Mime_Message;
 use Viserio\Contracts\{
     Events\Traits\EventsAwareTrait,
     Mail\Mailer as MailerContract,

src/Viserio/Mail/Transport/Mandrill.php

@@ -69,7 +69,7 @@ public function getKey(): string
      *
      * @param string $key
      *
-     * @return string
+     * @return Mandrill
      */
     public function setKey(string $key): Mandrill
     {

src/Viserio/Mail/Transport/Postmark.php

@@ -77,7 +77,7 @@ public function getServerToken(): string
      *
      * @param string $serverToken
      *
-     * @return string
+     * @return Postmark
      */
     public function setServerToken(string $serverToken): Postmark
     {
@@ -90,7 +90,7 @@ public function setServerToken(string $serverToken): Postmark
      * Convert email dictionary with emails and names
      * to array of emails with names.
      *
-     * @param array $emails
+     * @param string[] $emails
      *
      * @return array
      */
@@ -115,7 +115,7 @@ protected function convertEmailsArray(array $emails): array
      * @param Swift_Mime_Message $message
      * @param string             $mimeType
      *
-     * @return \Swift_Mime_MimePart|null
+     * @return \Swift_Mime_MimeEntity|null
      */
     protected function getMIMEPart(Swift_Mime_Message $message, $mimeType)
     {

src/Viserio/Mail/Transport/Ses.php

@@ -30,7 +30,7 @@ public function __construct(SesClient $ses)
      * @param \Swift_Mime_Message $message
      * @param string[]|null       $failedRecipients
      *
-     * @return Log|null
+     * @return integer
      */
     public function send(Swift_Mime_Message $message, &$failedRecipients = null)
     {

src/Viserio/Mail/Transport/SparkPost.php

@@ -89,7 +89,7 @@ public function getKey(): string
      *
      * @param string $key
      *
-     * @return string
+     * @return SparkPost
      */
     public function setKey(string $key): SparkPost
     {

src/Viserio/Mail/TransportManager.php

@@ -3,12 +3,11 @@
 namespace Viserio\Mail;
 
 use Aws\Ses\SesClient;
-use Interop\Container\ContainerInterface;
-use Swift_SmtpTransport as SmtpTransport;
-use Swift_MailTransport as MailTransport;
-use Narrowspark\Arr\StaticArr as Arr;
 use GuzzleHttp\Client as HttpClient;
+use Narrowspark\Arr\StaticArr as Arr;
 use Psr\Log\LoggerInterface;
+use Swift_MailTransport as MailTransport;
+use Swift_SmtpTransport as SmtpTransport;
 use Viserio\Support\AbstractManager;
 use Viserio\Mail\Transport\{
     Log as LogTransport,

src/Viserio/Queue/Connectors/SyncQueue.php

@@ -3,8 +3,8 @@
 namespace Viserio\Queue\Connectors;
 
 use Throwable;
-use Viserio\Contracts\Queue\Job as JobContract;
 use Viserio\Contracts\Exception\Exception\FatalThrowableError;
+use Viserio\Contracts\Queue\Job as JobContract;
 use Viserio\Queue\Jobs\SyncJob;
 
 class SyncQueue extends AbstractQueue

src/Viserio/StaticalProxy/StaticalProxy.php

@@ -154,7 +154,7 @@ protected static function resolveStaticalProxyInstance($name)
      *
      * @param string $name
      *
-     * @return object
+     * @return MockInterface
      */
     protected static function createFreshMockInstance(string $name)
     {
@@ -168,7 +168,7 @@ protected static function createFreshMockInstance(string $name)
     /**
      * Create a fresh mock instance for the given class.
      *
-     * @return object
+     * @return MockInterface
      */
     protected static function createMock()
     {