Rinvex\Support\Traits\ConsoleTools

Complexity

Total Complexity

15

Size/Duplication

Total Lines	97
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	publishMigrations()	0	23	4
A	publishConfig()	0	11	3
A	publishViews()	0	11	3
A	publishLang()	0	11	3
A	registerCommands()	0	9	2

<?php

declare(strict_types=1);

namespace Rinvex\Support\Traits;

trait ConsoleTools
{
    /**
     * Publish package migrations.
     *
     * @return void
     */
    protected function publishMigrations(string $package): void
    {
        $namespace = str_replace('laravel-', '', $package);
        $namespace = str_replace(['/', '\\', '.', '_'], '-', $namespace);
        $basePath = starts_with($package, 'cortex') ? $this->app->path($package)

The property app does not exist. Did you maybe forget to declare it?

The function starts_with() has been deprecated with message: Str::startsWith() should be used directly instead. Will be removed in Laravel 5.9.

            : $this->app->basePath('vendor/'.$package);

        if (file_exists($path = $basePath.'/database/migrations')) {
            $stubs = $this->app['files']->glob($path.'/*.php.stub');
            $existing = $this->app['files']->glob($this->app->databasePath('migrations/'.$package.'/*.php'));

            $migrations = collect($stubs)->flatMap(function ($migration) use ($existing, $package) {
                $sequence = substr(basename($migration), 0, 2);
                $match = collect($existing)->first(function ($item, $key) use ($migration, $sequence) {

The parameter $key is not used and could be removed.

                    return strpos($item, str_replace(['.stub', $sequence], '', basename($migration))) !== false;
                });

                return [$migration => $this->app->databasePath('migrations/'.$package.'/'.($match ? basename($match) : date('Y_m_d_His', time() + $sequence).str_replace(['.stub', $sequence], '', basename($migration))))];

This line exceeds maximum limit of 120 characters; contains 220 characters

            })->toArray();

            $this->publishes($migrations, $namespace.'-migrations');

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

        }
    }

    /**
     * Publish package config.
     *
     * @return void
     */
    protected function publishConfig(string $package): void
    {
        $namespace = str_replace('laravel-', '', $package);
        $namespace = str_replace(['/', '\\', '.', '_'], '-', $namespace);
        $basePath = starts_with($package, 'cortex') ? $this->app->path($package)

The function starts_with() has been deprecated with message: Str::startsWith() should be used directly instead. Will be removed in Laravel 5.9.

            : $this->app->basePath('vendor/'.$package);

        if (file_exists($path = $basePath.'/config/config.php')) {
            $this->publishes([$path => $this->app->configPath(str_replace('-', '.', $namespace).'.php')], $namespace.'-config');

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

This line exceeds maximum limit of 120 characters; contains 128 characters

        }
    }

    /**
     * Publish package views.
     *
     * @return void
     */
    protected function publishViews(string $package): void
    {
        $namespace = str_replace('laravel-', '', $package);
        $namespace = str_replace(['/', '\\', '.', '_'], '-', $namespace);
        $basePath = starts_with($package, 'cortex') ? $this->app->path($package)

The function starts_with() has been deprecated with message: Str::startsWith() should be used directly instead. Will be removed in Laravel 5.9.

            : $this->app->basePath('vendor/'.$package);

        if (file_exists($path = $basePath.'/resources/views')) {
            $this->publishes([$path => $this->app->resourcePath('views/vendor/'.$package)], $namespace.'-views');

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

        }
    }

    /**
     * Publish package lang.
     *
     * @return void
     */
    protected function publishLang(string $package): void
    {
        $namespace = str_replace('laravel-', '', $package);
        $namespace = str_replace(['/', '\\', '.', '_'], '-', $namespace);
        $basePath = starts_with($package, 'cortex') ? $this->app->path($package)

The function starts_with() has been deprecated with message: Str::startsWith() should be used directly instead. Will be removed in Laravel 5.9.

            : $this->app->basePath('vendor/'.$package);

        if (file_exists($path = $basePath.'/resources/lang')) {
            $this->publishes([$path => $this->app->resourcePath('lang/vendor/'.$package)], $namespace.'-lang');

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

        }
    }

    /**
     * Register console commands.
     *
     * @return void
     */
    protected function registerCommands(): void
    {
        // Register artisan commands
        foreach ($this->commands as $key => $value) {

The property commands does not exist. Did you maybe forget to declare it?

            $this->app->singleton($value, $key);
        }

        $this->commands(array_values($this->commands));

The method commands() does not exist on Rinvex\Support\Traits\ConsoleTools. Did you maybe mean registerCommands()?

    }
}