Backpack\CRUD\app\Console\Commands\Themes\InstallsTheme::installTheme()

Complexity

Conditions	5
Paths	6

Size

Total Lines	55
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function installTheme()
    {
        // Check if it is installed
        if ($this->isInstalled()) {
            $this->newLine();

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

            $this->line(sprintf('  %s was already installed', self::$addon['name']), 'fg=red');

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

            $this->newLine();

            return;
        }

        $this->newLine();
        $this->progressBlock($this->description);

        // Require package
        try {
            $this->composerRequire(self::$addon['repo']);
            $this->closeProgressBlock();
        } catch (\Throwable $e) {
            $this->errorProgressBlock();
            $this->line('  '.$e->getMessage(), 'fg=red');
            $this->newLine();

            return;
        }

        // Display general error in case it failed
        if (! $this->isInstalled()) {
            $this->errorProgressBlock();
            $this->note('For further information please check the log file.');
            $this->note('You can also follow the manual installation process documented on Github.');
            $this->newLine();

            return;
        }

        // Publish the theme config file
        $this->progressBlock('Publish theme config file');

        // manually include the provider in the run-time
        if (! class_exists(self::$addon['provider'])) {
            include self::$addon['provider_path'] ?? self::$addon['path'].'/src/AddonServiceProvider.php';
            app()->register(self::$addon['provider']);

The method register() does not exist on Illuminate\Container\Container. Are you sure you never get this type here, but always one of the subclasses?

        }

        $this->executeArtisanProcess('vendor:publish', [
            '--tag' => self::$addon['publish_tag'],
        ]);
        $this->closeProgressBlock();

        // add this theme's view namespace to the ui config file
        $this->progressBlock('Use theme as view namespace in <fg=blue>config/backpack/ui.php</>');
        $this->useViewNamespaceInConfigFile();
        $this->closeProgressBlock();
        $this->newLine();
    }