Nip\Filesystem\FilesystemManager\HasCloudDriverTrait

Complexity

Total Complexity

5

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	createS3Driver()	0	9	1
A	formatS3Config()	0	14	4

<?php

namespace Nip\Filesystem\FilesystemManager;

use Aws\S3\S3Client;
use League\Flysystem\AwsS3v3\AwsS3Adapter as S3Adapter;

The type League\Flysystem\AwsS3v3\AwsS3Adapter was not found. Maybe you did not declare it correctly or list all dependencies?

use League\Flysystem\FilesystemInterface;

The type League\Flysystem\FilesystemInterface was not found. Maybe you did not declare it correctly or list all dependencies?

use Nip\Filesystem\FileDisk;

/**
 * Trait HasCloudDriverTrait
 * @package Nip\Filesystem\FilesystemManager
 */
trait HasCloudDriverTrait
{
    /**
     * Create an instance of the Amazon S3 driver.
     *
     * @param array $config
     * @return FilesystemInterface|FileDisk
     */
    public function createS3Driver(array $config)
    {
        $s3Config = $this->formatS3Config($config);
        $root = $s3Config['root'] ?? null;
        $options = $config['options'] ?? [];

        return $this->adapt($this->createDisk(

It seems like adapt() must be provided by classes using this trait. How about adding it as abstract method to this trait?

It seems like createDisk() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            new S3Adapter(new S3Client($s3Config), $s3Config['bucket'], $root, $options),
            $config
        ));
    }

    /**
     * Format the given S3 configuration with the default options.
     *
     * @param array $config
     * @return array
     */
    protected function formatS3Config(array $config)
    {
        $config += ['version' => 'latest'];
        if ($config['key'] && $config['secret']) {
            $config['credentials'] = [
                'key' => $config['key'],
                'secret' => $config['secret'],
            ];
        }
        if (empty($config['endpoint'])) {
            unset($config['endpoint']);
        }

        return $config;
    }
}