Skip to content

Commit

Permalink
Apply fixes from StyleCI
Browse files Browse the repository at this point in the history
  • Loading branch information
StyleCIBot committed Nov 22, 2024
1 parent efc917f commit fd34263
Show file tree
Hide file tree
Showing 2 changed files with 14 additions and 3 deletions.
5 changes: 3 additions & 2 deletions src/Driver/Config.php
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@ public function config(): array
$this->validator->assertValid($config);
} catch (IlluminateValidationException $e) {
$this->logger->error('[fof-s3-assets] Invalid S3 disk configuration', ['errors' => $e->errors()]);

return [];
}

Expand Down Expand Up @@ -80,7 +81,7 @@ protected function buildConfigArray(string $key, string $secret, string $region,
'ACL' => $acl,
];
}

return $config;
}

Expand Down Expand Up @@ -148,7 +149,7 @@ protected function createAwsUrlFromBucketAndRegion(?string $bucket, ?string $reg
protected function getSettingsPrefix(): string
{
$shareWithFoFUpload = (bool) $this->settings->get('fof-s3-assets.share_s3_config_with_fof_upload');

if ($shareWithFoFUpload) {
return 'fof-upload';
}
Expand Down
12 changes: 11 additions & 1 deletion src/Repository/S3Repository.php
Original file line number Diff line number Diff line change
@@ -1,5 +1,14 @@
<?php

/*
* This file is part of fof/s3-assets.
*
* Copyright (c) FriendsOfFlarum
*
* For the full copyright and license information, please view the LICENSE.md
* file that was distributed with this source code.
*/

namespace FoF\S3Assets\Repository;

use Psr\Log\LoggerInterface;
Expand All @@ -8,7 +17,8 @@ class S3Repository
{
public function __construct(
protected LoggerInterface $logger
) {}
) {
}

/**
* Get a list of AWS regions available for S3.
Expand Down

0 comments on commit fd34263

Please sign in to comment.