Blok\LaravelPackageGenerator\Commands\Traits\InteractsWithComposer

Complexity

Total Complexity

5

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	composerRunCommand()	0	12	2
A	composerDumpAutoload()	0	3	1
A	composerUpdatePackage()	0	3	1
A	composerRemovePackage()	0	3	1

<?php

namespace Blok\LaravelPackageGenerator\Commands\Traits;

use Blok\LaravelPackageGenerator\Exceptions\RuntimeException;

trait InteractsWithComposer
{
    /**
     * Run "composer dump-autoload".
     */
    protected function composerDumpAutoload()
    {
        $this->composerRunCommand('composer dump-autoload');
    }

    /**
     * Run "composer update $vendor/$package".
     *
     * @param string $vendor
     * @param string $package
     */
    protected function composerUpdatePackage($vendor, $package)
    {
        $this->composerRunCommand("composer update --ignore-platform-reqs $vendor/$package");
    }

    /**
     * Run "composer remove $vendor/$package".
     *
     * @param string $vendor
     * @param string $package
     */
    protected function composerRemovePackage($vendor, $package)
    {
        $this->composerRunCommand("composer remove --ignore-platform-reqs $vendor/$package");
    }

    /**
     * Run arbitrary composer command.
     *
     * @param $command
     */
    protected function composerRunCommand($command)
    {
        $this->info("Run \"$command\".");

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

        $output = [];
        exec($command, $output, $returnStatusCode);

        if ($returnStatusCode !== 0) {
            throw RuntimeException::commandExecutionFailed($command, $returnStatusCode);
        }

        $this->info("\"$command\" was successfully ran.");
    }
}