CaptainHook\App\Hook\File\Action\NotEmpty::execute()

Complexity

Conditions	6
Paths	14

Size

Total Lines	23
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function execute(Config $config, IO $io, Repository $repository, Config\Action $action): void
    {
        $pathToFolder = $action->getOptions()->get('pathSrc');
        $searchedFolder = $action->getOptions()->get('folderName');
        $path     = glob((getcwd() . $pathToFolder));
        $errors   = [];
        foreach ($path as $folder) {
            $folderName = $folder . $searchedFolder;
            if (!is_dir($folderName)) {
                $errors[] = '<error>Missing ' . $searchedFolder . ' folder in ' . basename($folder) . ' </error>';
            }
            if (is_dir($folderName)) {
                $filesInDir = array_diff(scandir($folderName), ['.', '..']);

It seems like scandir($folderName) can also be of type false; however, parameter $array1 of array_diff() does only seem to accept array, maybe add an additional type check?

                if (count($filesInDir) === 0) {
                    $errors[] = '<error>Empty   ' . $searchedFolder . ' folder in ' . basename($folder) . ' </error>';
                }
            }
        }
        if (empty($errors)) {
            $io->write('<info>No errors found while checking for ' . $searchedFolder . ' files</info>');
        } else {
            $io->writeError($errors);
            throw new Exception('Please check if ' . $searchedFolder . ' folder is not missing and is not empty');
        }