Colligator\Providers\CoverCacheServiceProvider

Complexity

Total Complexity

2

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	register()	0	12	1
A	provides()	0	4	1

<?php

namespace Colligator\Providers;

use Colligator\CoverCache;
use Illuminate\Support\ServiceProvider;
use Intervention\Image\ImageManager;
use League\Flysystem\Config as FlysystemConfig;

class CoverCacheServiceProvider extends ServiceProvider
{
	/**
     * Indicates if loading of the provider is deferred.
     *
     * @var bool
     */
    protected $defer = true;

    /**
     * Register the service provider.
     *
     * @return void
     */
    public function register()
    {
        $this->app->singleton(CoverCache::class, function ($app) {

The parameter $app is not used and could be removed.

            $fs = \Storage::disk('s3')->getAdapter();

The method getAdapter() does not seem to exist on object<Illuminate\Contracts\Filesystem\Filesystem>.

            $im = $this->app->make(ImageManager::class);
            $conf = new FlysystemConfig([
                // Default: 30 days

43% of this comment could be valid code. Did you maybe forget this after debugging?

                'CacheControl' => 'max-age=' . env('IMAGE_CACHE_TIME', 3153600) . ', public',
            ]);
            return new CoverCache($fs, $im, $conf);
        });
    }

    /**
     * Get the services provided by the provider.
     *
     * @return array
     */
    public function provides()
    {
        return [CoverCache::class];
    }
}