Sunnysideup\UpgradeToSilverstripe4\Tasks\IndividualTasks\FindYmlExamplesFilesAndChangeExtension

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getTitle()	0	3	1
A	getDescription()	0	3	1
A	runActualTask()	0	8	1
A	hasCommitAndPush()	0	3	1

<?php

namespace Sunnysideup\UpgradeToSilverstripe4\Tasks\IndividualTasks;

use Sunnysideup\UpgradeToSilverstripe4\Api\FindFiles;
use Sunnysideup\UpgradeToSilverstripe4\Tasks\Task;

/**
 * Run a dev/build as a smoke test to see if all is well
 */
class FindYmlExamplesFilesAndChangeExtension extends Task
{
    protected $taskStep = 's30';

    protected $extensions = [
        'example',
        'Example',
        'EXAMPLE',
    ];

    public function getTitle()
    {
        return 'Find YML extension files and change extension temporarily.';
    }

    public function getDescription()
    {
        return 'In our modules we include a lot of .yml.example files - we want to update those as well.';
    }

    public function runActualTask($params = []): ?string
    {
        $files = FindFiles::inst()
            ->setSearchPath($this->mu()->getGitRootDir())

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

            ->setExtensions($this->extensions)
            ->getFlatFileArray();
        print_r($files);
        return null;
    }

    protected function hasCommitAndPush()
    {
        return true;
    }
}