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

Complexity

Conditions	10
Paths	4

Size

Total Lines	42
Code Lines	32

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?

            $fileFinder = new FindFiles();
            $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)) {
                $flatArray = $fileFinder
                    ->setSearchPath($searchPath)
                    ->setExtensions(['php'])
                    ->getFlatFileArray();
                if (is_array($flatArray) && count($flatArray)) {
                    foreach ($flatArray as $path) {
                        $tableName = basename($path, '.php');
                        $newLine = 'private static $table_name = \'' . $tableName . '\';';
                        $filecontent = file_get_contents($path);
                        $hasNewLine = strpos($filecontent, 'private static $table_name');
                        if (! $hasNewLine) {
                            $positionOfPrivateStatic = 0;
                            foreach ($this->listToSearchFor as $item) {
                                $positionOfPrivateStatic = strpos($filecontent, $item);
                                if ($positionOfPrivateStatic) {
                                    break;
                                }
                            }
                            if ($positionOfPrivateStatic) {
                                $this->mu()->colourPrint('Adding  ' . $newLine . ' to ' . $path, 'green');

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

                                $filecontent =
                                    substr($filecontent, 0, $positionOfPrivateStatic) .
                                    "\r\n" . "\r\n" . '    ' . $newLine . "\r\n" . "\r\n" .
                                    substr($filecontent, $positionOfPrivateStatic);
                                file_put_contents($path, $filecontent);
                            }
                        }
                    }
                } else {
                    $this->mu()->colourPrint('Could not find any files in ' . $searchPath, 'red');
                }
            } else {
                $this->mu()->colourPrint('Could not find ' . $searchPath, 'blue');
            }
        }
        return null;
    }