LeKoala\DevToolkit\Tasks\ClearCacheFolderTask

Complexity

Total Complexity

5

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	run()	0	25	5

<?php

namespace LeKoala\DevToolkit\Tasks;

use Exception;
use SilverStripe\Dev\BuildTask;
use SilverStripe\Control\Director;
use LeKoala\Base\Helpers\FileHelper;

The type LeKoala\Base\Helpers\FileHelper was not found. Maybe you did not declare it correctly or list all dependencies?

/**
 */
class ClearCacheFolderTask extends BuildTask
{
    protected $title = "Clear cache folder";
    protected $description = 'Clear silverstripe-cache folder.';
    private static $segment = 'ClearCacheFolderTask';

The private property $segment is not used, and could be removed.

    public function run($request)
    {
        $this->request = $request;

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

        $folder = Director::baseFolder() . '/silverstripe-cache';
        $create = $_GET['create'] ?? false;
        if (!is_dir($folder)) {
            if ($create) {
                mkdir($folder, 0755);
            } else {
                throw new Exception("silverstripe-cache folder does not exist in root");
            }
        }

        $result = FileHelper::rmDir($folder);
        if ($result) {
            $this->message("Removed $folder");

The method message() does not exist on LeKoala\DevToolkit\Tasks\ClearCacheFolderTask. Since you implemented __call, consider adding a @method annotation.

        } else {
            $this->message("Failed to remove $folder", "error");
        }
        $result = mkdir($folder, 0755);
        if ($result) {
            $this->message("A new folder has been created at $folder");
        } else {
            $this->message("Failed to create a new folder at $folder", "error");
        }
    }
}