Swis\Filesystem\Encrypted\LocalEncryptedFilesystemServiceProvider

Complexity

Total Complexity

5

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Test Coverage

Coverage

93.75%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	registerResponseMacro()	0	24	2
A	registerStorageDriver()	0	22	2
A	boot()	0	4	1

<?php

namespace Swis\Filesystem\Encrypted;

use Illuminate\Contracts\Foundation\Application;
use Illuminate\Support\Arr;
use Illuminate\Support\Facades\Response;
use Illuminate\Support\Facades\Storage;
use Illuminate\Support\ServiceProvider;
use Illuminate\Support\Str;
use League\Flysystem\Adapter\Local as LocalAdapter;
use League\Flysystem\Filesystem;
use Swis\Flysystem\Encrypted\EncryptedAdapter;

class LocalEncryptedFilesystemServiceProvider extends ServiceProvider
{
    /**
     * @throws \LogicException
     */
    public function boot()
    {
        $this->registerStorageDriver();
        $this->registerResponseMacro();
    }

    protected function registerStorageDriver()
    {
        Storage::extend(
            'local-encrypted',
            function (Application $app, array $config) {
                $permissions = $config['permissions'] ?? [];

                $links = ($config['links'] ?? null) === 'skip'
                    ? LocalAdapter::SKIP_LINKS
                    : LocalAdapter::DISALLOW_LINKS;

                return new Filesystem(
                    new EncryptedAdapter(
                        new LocalAdapter(
                            $config['root'],
                            $config['lock'] ?? LOCK_EX,
                            $links,
                            $permissions
                        ),
                        $app->make('encrypter')
                    ),
                    Arr::only($config, ['visibility', 'disable_asserts', 'url'])
                );
            }
        );
    }

    protected function registerResponseMacro()
    {
        Response::macro(
            'downloadEncrypted',
            /**
             * @param \SplFileInfo|string $file
             * @param null $name

Are you sure the doc-type for parameter $name is correct as it would always require null to be passed?

             * @param array $headers
             * @param string $disposition
             *
             * @return \Swis\Filesystem\Encrypted\BinaryFileResponse
             */
            function ($file, $name = null, array $headers = [], $disposition = 'attachment') {
                $response = new BinaryFileResponse($file, 200, $headers, false, $disposition);

                if (null !== $name) {

The condition null !== $name is always false.

                    return $response->setContentDisposition(
                        $disposition,
                        $name,
                        str_replace('%', '', Str::ascii($name))
                    );
                }

                return $response;
            }
        );
    }
}