Sausin\LaravelOvh\OVHServiceProvider

Complexity

Total Complexity

10

Size/Duplication

Total Lines	111
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	boot()	0	15	1
A	checkConfig()	0	12	2
A	makeClient()	0	20	1
A	getVars()	0	10	3
A	getLargeObjectConfig()	0	18	3

<?php

namespace Sausin\LaravelOvh;

use BadMethodCallException;
use Illuminate\Support\Facades\Storage;
use Illuminate\Support\ServiceProvider;
use League\Flysystem\Filesystem;
use OpenStack\OpenStack;

class OVHServiceProvider extends ServiceProvider
{
    /**
     * Bootstrap the application services.
     *
     * @return void
     */
    public function boot()
    {
        Storage::extend('ovh', function ($app, $config) {
            // check if the config is complete
            $this->checkConfig($config);

            // create the client
            $client = $this->makeClient($config);

            // get the container
            $container = $client->objectStoreV1()->getContainer($config['container']);

            return new Filesystem(new OVHSwiftAdapter($container, $this->getVars($config)));
        });
    }

    /**
     * Check that the config is properly setup.
     *
     * @param  array $config
     * @return void|BadMethodCallException
     */
    protected function checkConfig($config)
    {
        // needed keys
        $needKeys = ['server', 'region', 'user', 'pass', 'userDomain', 'projectId', 'container'];

        if (count(array_intersect($needKeys, array_keys($config))) === count($needKeys)) {
            return;
        }

        // if the configuration wasn't complete, throw an exception
        throw new BadMethodCallException('Need following keys '.implode(', ', $needKeys));
    }

    /**
     * Make the client needed for interaction with OVH OpenStack.
     *
     * @param  array $config
     * @return \OpenStack\OpenStack
     */
    protected function makeClient($config)
    {
        // setup the client for OpenStack
        return new OpenStack([
            'authUrl' => $config['server'],
            'region' => $config['region'],
            'user' => [
                'name' => $config['user'],
                'password' => $config['pass'],
                'domain' => [
                    'name' => $config['userDomain'],
                ],
            ],
            'scope' => [
                'project' => [
                    'id' => $config['projectId'],
                ],
            ],
        ]);
    }

    /**
     * Return the config variables required by the adapter.
     *
     * @param  array &$config
     * @return array
     */
    protected function getVars(&$config)
    {
        return [
            'region' => $config['region'],
            'projectId' => $config['projectId'],
            'container' => $config['container'],
            'urlKey' => isset($config['urlKey']) ? $config['urlKey'] : null,
            'endpoint' => isset($config['endpoint']) ? $config['endpoint'] : null,
        ] + $this->getLargeObjectConfig();

The call to getLargeObjectConfig() misses a required argument $config.

    }
    
    /**
     * Return the config variables required for large object.
     *
     * @param  array &$config
     * @return array
     */
    protected function getLargeObjectConfig(&$config)
    {
        $largeObjConfig = [];

        $largeObjVars = [
            'swiftLargeObjectThreshold',
            'swiftSegmentSize',
            'swiftSegmentContainer',
        ];

        foreach ($largeObjVars as $key) {
            if (isset($config[$key])) {
                $largeObjConfig[$key] = $config[$key];
            }
        }
        
        return $largeObjConfig;
    }
}