Sunnysideup\UpgradeToSilverstripe4\Tasks\IndividualTasks\FindFilesWithMoreThanOneClass::runActualTask()

Complexity

Conditions	14
Paths	10

Size

Total Lines	59
Code Lines	42

Duplication

Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	1	Features	0

<?php
    public function runActualTask($params = []): ?string
    {
        $errors = [];
        foreach ($this->mu()->getExistingModuleDirLocations() as $moduleDir) {

It seems like getExistingModuleDirLocations() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $searchPath = $this->mu()->findMyCodeDir($moduleDir);

It seems like findMyCodeDir() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            if (file_exists($searchPath)) {
                $this->mu()->colourPrint(

It seems like colourPrint() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                    'Searching in ' . $searchPath . ' for files with more than one class.',
                    'blue'
                );
                $fileFinder = new FindFiles();
                $flatArray = $fileFinder
                    ->setSearchPath($searchPath)
                    ->setExtensions(['php'])
                    ->getFlatFileArray();
                if (is_array($flatArray) && count($flatArray)) {
                    foreach ($flatArray as $path) {
                        // $className = basename($path, '.php');
                        $classNames = [];
                        $content = file_get_contents($path);
                        $tokens = token_get_all($content);
                        for ($index = 0; isset($tokens[$index]); $index++) {
                            if (! isset($tokens[$index][0])) {
                                continue;
                            }
                            if ($tokens[$index][0] === T_CLASS &&
                                $tokens[$index + 1][0] === T_WHITESPACE &&
                                $tokens[$index + 2][0] === T_STRING
                            ) {
                                $index += 2; // Skip class keyword and whitespace
                                $classNames[] = $tokens[$index][1];
                            }
                        }
                        if (count($classNames) > 1) {
                            $errors[] = $path . ': ' . implode(', ', $classNames);
                        }
                    }
                } else {
                    $this->mu()->colourPrint(
                        'Could not find any files in ' . $searchPath,
                        'red'
                    );
                }
            } elseif ($searchPath) {
                $this->mu()->colourPrint(
                    'Could not find the following path: "' . $searchPath,
                    'blue'
                );
            } else {
                $this->mu()->colourPrint(
                    'empty search path',
                    'blue'
                );
            }
        }
        if (count($errors)) {
            return 'Found files with multiple classes: ' . implode("\n\n ---\n\n", $errors);
        }
        return null;
    }