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

Complexity

Conditions	11
Paths	96

Size

Total Lines	89
Code Lines	66

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

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

        $fixer = FileSystemFixes::inst($this->mu())

The assignment to $fixer is dead and can be removed.

            ->removeDirOrFile($this->mu()->getWebRootDirLocation(), $this->mu()->getAboveWebRootDirLocation())

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

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

The method getWebRootDirLocation() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

The method getAboveWebRootDirLocation() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

            ->mkDir($this->mu()->getWebRootDirLocation(), $this->mu()->getAboveWebRootDirLocation());
        if (! $this->versionToLoad) {
            $this->versionToLoad = $this->mu()->getFrameworkComposerRestraint();
        }

        //install project / silverstripe clean.
        $altBranch = $this->mu()->getParentProjectForModuleBranchOrTag();

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

The method getParentProjectForModuleBranchOrTag() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

        if (! $altBranch) {
            $altBranch = 'master';
        }
        if ($this->mu()->getIsModuleUpgrade()) {

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

The method getIsModuleUpgrade() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

            $alternativeGitLink = $this->mu()->getParentProjectForModule();

The method getParentProjectForModule() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

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

            if ($alternativeGitLink) {
                Git::inst($this->mu())
                    ->Clone(
                        $this->mu()->getWebRootDirLocation(),
                        $alternativeGitLink,
                        $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?

                        $altBranch
                    );
            } else {
                $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()->getAboveWebRootDirLocation(),

It seems like $this->mu()->getAboveWebRootDirLocation() can also be of type Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader and Sunnysideup\UpgradeToSilverstripe4\Traits\Creator and null; however, parameter $newDir of Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader::execMe() does only seem to accept string, maybe add an additional type check?

                    $this->mu()->getComposerEnvironmentVars() . ' composer create-project -n ' . $this->defaultSilverstripeProject . ' ' . $this->mu()->getWebRootDirLocation() . ' ' . $this->versionToLoad,

The method getComposerEnvironmentVars() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

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

                    'set up vanilla install of ' . $this->defaultSilverstripeProject . ' - version: ' . $this->versionToLoad,
                    false
                );
                foreach($this->allowedPlugins as $plugin) {
                    $this->mu()->execMe(
                        $this->mu()->getWebRootDirLocation(),
                        $this->mu()->getComposerEnvironmentVars() . 'composer config --no-interaction allow-plugins.' . $plugin . ' true',
                        'composer config --no-interaction allow-plugins.' . $plugin . ' true',
                        false
                    );
                }
            }
        }
        if ($this->installModuleAsVendorModule) {
            Composer::inst($this->mu())
                ->ClearCache()
                ->Require(
                    $this->mu()->getVendorName() . '/' . $this->mu()->getPackageName(),

The method getVendorName() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

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

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

The method getPackageName() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

                    'dev-' . $this->mu()->getNameOfTempBranch(),

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

The method getNameOfTempBranch() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

                    $this->composerOptions
                );
            $branch = $this->mu()->getParentProjectForModuleBranchOrTag() ?: 'master';
            if($this->mu()->getNameOfTempBranch() !== $branch) {
                $gitLink = $this->mu()->getGitLink();

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

The method getGitLink() does not exist on Sunnysideup\UpgradeToSilverstripe4\ModuleUpgrader. Since you implemented __call, consider adding a @method annotation.

                $command = '
                    git init;
                    git remote add origin ' . $gitLink . ';
                    git pull origin ' . $altBranch . ';
                    git status;';
                $this->mu()->execMe(
                    $this->mu()->getGitRootDir(),
                    $command,
                    'Make sure it is a git repo',
                    false
                );
            }
        } else {
            Git::inst($this->mu())
                ->Clone(
                    $this->mu()->getWebRootDirLocation(),
                    $this->mu()->getGitLink(),
                    $this->mu()->getGitRootDir(),
                    $this->mu()->getNameOfTempBranch()
                );
            if ($this->mu()->getIsModuleUpgrade()) {
                $this->workoutExtraRequirementsFromModule();
            }
        }
        foreach ($this->alsoRequire as $package => $version) {
            Composer::inst($this->mu())
                ->ClearCache()
                ->Require(
                    $package,
                    $version,
                    $this->composerOptions
                );
        }

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