style: add void return type

Author: carlalexander

src/Autoloader.php

@@ -21,7 +21,7 @@ class Autoloader
     /**
      * Handles autoloading of Ymir plugin classes.
      */
-    public static function autoload(string $class)
+    public static function autoload(string $class): void
     {
         if (!str_starts_with($class, __NAMESPACE__)) {
             return;
@@ -38,7 +38,7 @@ public static function autoload(string $class)
     /**
      * Registers the plugin autoloader as an SPL autoloader.
      */
-    public static function register(bool $prepend = false)
+    public static function register(bool $prepend = false): void
     {
         spl_autoload_register([static::class, 'autoload'], true, $prepend);
     }

src/CloudProvider/Aws/CloudFrontClient.php

@@ -50,15 +50,15 @@ public function __construct(ClientInterface $client, string $distributionId, str
     /**
      * {@inheritdoc}
      */
-    public function clearAll()
+    public function clearAll(): void
     {
         $this->addPath('/*');
     }
 
     /**
      * {@inheritdoc}
      */
-    public function clearUrl(string $url)
+    public function clearUrl(string $url): void
     {
         $path = parse_url($url, PHP_URL_PATH);
 
@@ -72,7 +72,7 @@ public function clearUrl(string $url)
     /**
      * {@inheritdoc}
      */
-    public function clearUrls($urls)
+    public function clearUrls($urls): void
     {
         if (is_array($urls) || is_string($urls)) {
             $urls = new Collection($urls);
@@ -90,7 +90,7 @@ public function clearUrls($urls)
     /**
      * Invalidate the given paths.
      */
-    public function invalidatePaths(array $paths)
+    public function invalidatePaths(array $paths): void
     {
         $concretePaths = array_filter($paths, function (string $path) {
             return !str_ends_with($path, '*');
@@ -110,7 +110,7 @@ public function invalidatePaths(array $paths)
     /**
      * {@inheritdoc}
      */
-    public function sendClearRequest(?callable $guard = null)
+    public function sendClearRequest(?callable $guard = null): void
     {
         if (empty($this->invalidationPaths)) {
             return;
@@ -146,7 +146,7 @@ protected function getService(): string
     /**
      * Add the given path to the list.
      */
-    private function addPath(string $path)
+    private function addPath(string $path): void
     {
         if (in_array($path, ['*', '/*'])) {
             $this->invalidationPaths = ['/*'];
@@ -242,7 +242,7 @@ private function generateInvalidationPayload(array $paths): string
     /**
      * Send an invalidation request.
      */
-    private function sendInvalidation(array $paths)
+    private function sendInvalidation(array $paths): void
     {
         $attempts = 0;
         $maxAttempts = 3;

src/CloudProvider/Aws/DynamoDbClient.php

@@ -41,7 +41,7 @@ public function batchGetItem(array $arguments): array
     /**
      * Deletes a single item in a table by primary key.
      */
-    public function deleteItem(array $arguments)
+    public function deleteItem(array $arguments): void
     {
         $response = $this->perform('DeleteItem', $arguments);
 
@@ -73,7 +73,7 @@ public function getItem(array $arguments)
     /**
      * Creates a new item, or replaces an old item with a new item.
      */
-    public function putItem(array $arguments)
+    public function putItem(array $arguments): void
     {
         $response = $this->perform('PutItem', $arguments);
 

src/CloudProvider/Aws/LambdaClient.php

@@ -50,7 +50,7 @@ public function __construct(ClientInterface $client, string $functionName, strin
     /**
      * {@inheritdoc}
      */
-    public function createAttachmentMetadata($attachment, bool $async = false)
+    public function createAttachmentMetadata($attachment, bool $async = false): void
     {
         $this->runWpCliCommand(sprintf('%s %s', Console\CreateAttachmentMetadataCommand::getName(), $this->getAttachmentId($attachment)), $async);
     }
@@ -90,7 +90,7 @@ public function createCroppedAttachmentImage($attachment, int $width, int $heigh
     /**
      * {@inheritdoc}
      */
-    public function editAttachmentImage($attachment, string $changes, string $apply = 'all')
+    public function editAttachmentImage($attachment, string $changes, string $apply = 'all'): void
     {
         $this->runWpCliCommand(sprintf('%s %s \'%s\' %s', Console\EditAttachmentImageCommand::getName(), $this->getAttachmentId($attachment), $changes, $this->buildOptions([
             'apply' => $apply,
@@ -100,7 +100,7 @@ public function editAttachmentImage($attachment, string $changes, string $apply
     /**
      * {@inheritdoc}
      */
-    public function resizeAttachmentImage($attachment, int $width, int $height)
+    public function resizeAttachmentImage($attachment, int $width, int $height): void
     {
         $this->runWpCliCommand(sprintf('%s %s %s', Console\ResizeAttachmentImageCommand::getName(), $this->getAttachmentId($attachment), $this->buildOptions([
             'width' => $width,

src/CloudProvider/Aws/S3Client.php

@@ -41,7 +41,7 @@ public function __construct(ClientInterface $client, string $bucket, string $key
     /**
      * {@inheritdoc}
      */
-    public function copyObject(string $sourceKey, string $targetKey, string $acl = 'public-read')
+    public function copyObject(string $sourceKey, string $targetKey, string $acl = 'public-read'): void
     {
         $response = $this->request('put', $targetKey, null, [
             'x-amz-acl' => $acl,
@@ -66,7 +66,7 @@ public function createPutObjectRequest(string $key, string $acl = 'public-read')
     /**
      * {@inheritdoc}
      */
-    public function deleteObject(string $key)
+    public function deleteObject(string $key): void
     {
         $response = $this->request('delete', $key);
 
@@ -176,7 +176,7 @@ public function objectExists(string $key): bool
     /**
      * {@inheritdoc}
      */
-    public function putObject(string $key, string $object, string $acl = 'public-read', string $contentType = '')
+    public function putObject(string $key, string $object, string $acl = 'public-read', string $contentType = ''): void
     {
         $headers = [
             'x-amz-acl' => $acl,

src/CloudProvider/Aws/SesClient.php

@@ -46,7 +46,7 @@ public function canSendEmails(): bool
     /**
      * {@inheritdoc}
      */
-    public function sendEmail(Email $email)
+    public function sendEmail(Email $email): void
     {
         $response = $this->request('post', '/', http_build_query([
             'Action' => 'SendRawEmail',

src/CloudStorage/AbstractCloudStorageStreamWrapper.php

@@ -87,7 +87,7 @@ public static function getProtocol(): string
     /**
      * Register the cloud storage stream wrapper.
      */
-    public static function register(CloudStorageClientInterface $client, ?\ArrayObject $cache = null)
+    public static function register(CloudStorageClientInterface $client, ?\ArrayObject $cache = null): void
     {
         if (in_array(static::getProtocol(), stream_get_wrappers())) {
             stream_wrapper_unregister(static::getProtocol());
@@ -279,7 +279,7 @@ public function stream_cast(): bool
      *
      * @see https://www.php.net/manual/en/streamwrapper.stream-close.php
      */
-    public function stream_close()
+    public function stream_close(): void
     {
         $this->cache = null;
         fclose($this->openedStreamObjectResource);
@@ -804,7 +804,7 @@ private function parsePath(string $path): string
     /**
      * Remove the cache value for the given key.
      */
-    private function removeCacheValue(string $key)
+    private function removeCacheValue(string $key): void
     {
         $cache = $this->getCache();
 
@@ -818,7 +818,7 @@ private function removeCacheValue(string $key)
     /**
      * Set the given cache value for the given key.
      */
-    private function setCacheValue(string $key, $value)
+    private function setCacheValue(string $key, $value): void
     {
         $this->getCache()->offsetSet($key, $value);
     }

src/Configuration/AssetsConfiguration.php

@@ -25,7 +25,7 @@ class AssetsConfiguration implements ContainerConfigurationInterface
     /**
      * {@inheritdoc}
      */
-    public function modify(Container $container)
+    public function modify(Container $container): void
     {
         $container['assets_path'] = $container->service(function () {
             return getenv('YMIR_ASSETS_PATH') ?: '';

src/Configuration/AttachmentConfiguration.php

@@ -25,7 +25,7 @@ class AttachmentConfiguration implements ContainerConfigurationInterface
     /**
      * {@inheritdoc}
      */
-    public function modify(Container $container)
+    public function modify(Container $container): void
     {
         $container['file_manager'] = $container->service(function (Container $container) {
             return new AttachmentFileManager($container['uploads_basedir']);

src/Configuration/CloudProviderConfiguration.php

@@ -24,7 +24,7 @@ class CloudProviderConfiguration implements ContainerConfigurationInterface
     /**
      * {@inheritdoc}
      */
-    public function modify(Container $container)
+    public function modify(Container $container): void
     {
         $container['cloud_provider_function_name'] = $container->service(function () {
             $functionName = getenv('AWS_LAMBDA_FUNCTION_NAME');