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

Complexity

Conditions	5
Paths	6

Size

Total Lines	54
Code Lines	39

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?

            $moduleDir = $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?

            //Start search machine from the module location. replace API
            $textSearchMachine = new SearchAndReplaceAPI($moduleDir);
            $textSearchMachine->setIsReplacingEnabled(true);
            $textSearchMachine->setFileReplacementMaxCount(1);
            $textSearchMachine->setIgnoreFileIfFound(['private static $table_name']);
            $textSearchMachine->addToIgnoreFolderArray($this->ignoreFolderArray);
            $this->mu()->colourPrint("Checking ${moduleDir}");

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

            $moduleDir = $this->mu()->checkIfPathExistsAndCleanItUp($moduleDir);

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

            if (! file_exists($moduleDir)) {
                $this->mu()->colourPrint("SKIPPING ${moduleDir} as it does not exist.");
            } else {
                $textSearchMachine->setSearchPath($moduleDir);
                $textSearchMachine->setExtensions($this->extensionArray); //setting extensions to search files within
                $this->mu()->colourPrint(
                    "++++++++++++++++++++++++++++++++++++\n" .
                    "CHECKING\n" .
                    "IN ${moduleDir}\n" .
                    'FOR ' . implode(',', $this->extensionArray) . " FILES\n" .
                    'BASE ' . $moduleDir . "\n" .
                    "++++++++++++++++++++++++++++++++++++\n"
                );
                foreach ($this->findArray as $finalFind) {
                    $caseSensitive = true;
                    $replacementType = 'COMPLEX';
                    $comment = 'Check that is class indeed extends DataObject and that it is not a data-extension!';
                    $finalReplace = '
    private static $table_name = \'[SEARCH_REPLACE_CLASS_NAME_GOES_HERE]\';

    ' . $finalFind;
                    $this->mu()->colourPrint(
                        '    --- FIND: ' . $finalFind . "\n" .
                        '    --- REPLACE: ' . $finalReplace . "\n"
                    );

                    $textSearchMachine->setSearchKey($finalFind, $caseSensitive, $replacementType);
                    $textSearchMachine->setReplacementKey($finalReplace);
                    $textSearchMachine->setComment($comment);
                    $textSearchMachine->startSearchAndReplace();
                }

                //SHOW TOTALS
                $replacements = $textSearchMachine->showFormattedSearchTotals();
                if (! $replacements) {
                    //flush output anyway!
                    $this->mu()->colourPrint('No replacements for  ' . implode(',', $this->extensionArray));
                }
                $this->mu()->colourPrint($textSearchMachine->getOutput());
            }
        }
        return null;
    }