Distilleries\Expendable\Http\Controllers\Frontend\AssetController::getIndex()

Complexity

Conditions	4
Paths	4

Size

Total Lines	28
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	15
CRAP Score	4

Importance

Changes

0

<?php namespace Distilleries\Expendable\Http\Controllers\Frontend;
    public function getIndex($path = '')
    {

        $directory   = explode('/', $path);
        $directory   = reset($directory);
        $white_liste = config('expendable.folder_whitelist');
        if (in_array($directory, $white_liste))
        {
            $path = storage_path($path);
            $filesystem = app('files');
            if ($filesystem->isFile($path))
            {
                $mimetype = mime_content_type($path);
                if (@is_array(getimagesize($path)))
                {
                    return response()->make($filesystem->get($path), 200, array('content-type' => $mimetype));
                } else
                {
                    $name = explode('/', $path);
                    $name = end($name);

                    return response()->download($path, $name, array('content-type' => $mimetype));
                }
            }
        }


        return abort(404);

Are you sure the usage of abort(404) is correct as it seems to always return null.

    }
}