Sunnysideup\UpgradeToSilverstripe4\Tasks\IndividualTasks\ComposerInstallSimple

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

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

<?php

namespace Sunnysideup\UpgradeToSilverstripe4\Tasks\IndividualTasks;

use Sunnysideup\UpgradeToSilverstripe4\Tasks\Helpers\Composer;
use Sunnysideup\UpgradeToSilverstripe4\Tasks\Task;

/**
 * Install a basic / standard install of Silverstripe ('.$this->versionToLoad.')
 * using composer' ;
 */
class ComposerInstallSimple extends Task
{
    protected $taskStep = 's20';

    protected $versionToLoad = '';

    /**
     * @var string
     */
    protected $composerOptions = '--prefer-dist';

    public function getTitle()
    {
        return 'use Composer to install requirements.';
    }

    public function getDescription()
    {
        return '';
    }

    public function runActualTask($params = []): ?string
    {
        $this->mu()->setBreakOnAllErrors(true);

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

        $command = 'composer install ' . $this->composerOptions;
        $this->mu()->execMe(

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

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

            $command,
            'Basic composer install',
            false
        );

        $this->mu()->setBreakOnAllErrors(false);
        return null;
    }

    protected function hasCommitAndPush()
    {
        return false;
    }
}