A17\Twill\Http\Controllers\Admin\IconsController

Complexity

Total Complexity

6

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	index()	0	28	3
A	show()	0	11	2

<?php

namespace A17\Twill\Http\Controllers\Admin;

use A17\Twill\Services\Blocks\BlockMaker;
use Illuminate\Filesystem\Filesystem;
use Illuminate\Support\Collection;
use Str;
use Symfony\Component\Finder\SplFileInfo;

class IconsController extends Controller
{
    public function __construct(Filesystem $files, BlockMaker $blockMaker)
    {
        $this->files = $files;

The property files does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->blockMaker = $blockMaker;

The property blockMaker does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    public function index()
    {
        $icons = collect(
            config('twill.block_editor.directories.source.icons')
        )->reduce(function (Collection $keep, $path) {
            if (!$this->files->exists($path)) {
                return $keep;
            }

            $files = collect($this->files->files($path))->map(function (
                SplFileInfo $file
            ) {
                if (in_array($file->getFilename(), config('twill.internal_icons'))) {
                    return null;
                }

                return [
                    'name' => Str::before($file->getFilename(), '.svg'),
                    'url' => route('admin.icons.show', [
                        'file' => $file->getFilename(),
                    ]),
                ];
            })->filter();

            return $keep->merge($files);
        }, collect());

        return view('twill::blocks.icons', compact('icons'));
    }

    public function show($file)
    {
        $file = $this->blockMaker->getIconFile($file, false);

        if (!$this->files->exists($file)) {
            abort(404);
        }

        return response()->stream(function () use ($file) {
            echo $this->files->get($file);
        }, 200, ["Content-Type" => "image/svg+xml"]);

    }
}