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

Complexity

Conditions	3
Paths	3

Size

Total Lines	19
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function runActualTask($params = []): ?string
    {
        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?

            $newClientFolder = $moduleDir . '/' . $this->clientFolderName . '/ ';
            $fixer = FileSystemFixes::inst($this->mu())
                ->mkDir($newClientFolder);
            $foldersToMoveName = [
                'javascript',
                'js',
                'images',
                'img',
                'css',
            ];
            foreach ($foldersToMoveName as $folderToMoveName) {
                $folderToMove = $moduleDir . '/' . $folderToMoveName . '/ ';
                $fixer->moveFolderOrFile($folderToMove, $newClientFolder);
            }
        }
        return null;
    }