Nedmas\FortrabbitStorage\Providers\FortrabbitStorageServiceProvider

Complexity

Total Complexity

8

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	5

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
C	boot()	0	23	7
A	register()	0	4	1

<?php

namespace Nedmas\FortrabbitStorage\Providers;

use Aws\S3\S3Client;
use Illuminate\Support\Arr;
use Illuminate\Support\Facades\Storage;
use Illuminate\Support\ServiceProvider;
use League\Flysystem\Filesystem;
use Nedmas\FortrabbitStorage\Filesystem\FortrabbitAdapter;

class FortrabbitStorageServiceProvider extends ServiceProvider
{
    /**
     * Perform post-registration booting of services.
     *
     * @return void
     */
    public function boot()
    {
        Storage::extend('fortrabbit', function ($app, $config) {
            $s3Config = $config + ['version' => 'latest'];

            if ($s3Config['key'] && $s3Config['secret']) {
                $s3Config['credentials'] = Arr::only($s3Config, ['key', 'secret']);
            }

            $root = isset($s3Config['root']) ? $s3Config['root'] : null;

            $options = isset($config['options']) ? $config['options'] : [];

            $storageUrl = isset($s3Config['storage_url']) ? $s3Config['storage_url'] : null;

            $config = Arr::only($config, ['visibility', 'disable_asserts']);

            return new Filesystem(
                new FortrabbitAdapter(new S3Client($s3Config), $s3Config['bucket'], $root, $options, $storageUrl),

It seems like $root defined by isset($s3Config['root']) ? $s3Config['root'] : null on line 28 can also be of type array or null; however, Nedmas\FortrabbitStorage\Filesystem\FortrabbitAdapter::__construct() does only seem to accept string, maybe add an additional type check?

It seems like $options defined by isset($config['options']) ? $config['options'] : array() on line 30 can also be of type string; however, Nedmas\FortrabbitStorage\Filesystem\FortrabbitAdapter::__construct() does only seem to accept array, maybe add an additional type check?

It seems like $storageUrl defined by isset($s3Config['storage_url']) ? $s3Config['storage_url'] : null on line 32 can also be of type array or null; however, Nedmas\FortrabbitStorage\Filesystem\FortrabbitAdapter::__construct() does only seem to accept string, maybe add an additional type check?

                count($config) > 0 ? $config : null
            );
        });
    }

    /**
     * Register bindings in the container.
     *
     * @return void
     */
    public function register()
    {
        //
    }
}