MultiTenantLaravel\App\Commands\CreateTenant

Complexity

Total Complexity

6

Size/Duplication

Total Lines	60
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	handle()	0	15	3
A	createFakeTenant()	0	7	1
A	createNewTenant()	0	4	1

<?php

namespace MultiTenantLaravel\App\Commands;

use Illuminate\Console\Command;

class CreateTenant extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'tenant:create {--fake}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Create a new tenant';

    /**
     * Create a new command instance.
     *
     * @return void

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $count = (int) $this->ask('How many would you like to create?');

        while ($count > 0) {
            if (!$this->option('fake')) {
                $this->createNewTenant();

The call to the method MultiTenantLaravel\App\Commands\CreateTenant::createNewTenant() seems un-needed as the method has no side-effects.

            } else {
                $this->createFakeTenant();
            }

            $count--;
        }

    }

    private function createFakeTenant()
    {
        // Start faker and create a fake tenant
        $faker = new Faker\Generator();

        dd($faker);
    }

    private function createNewTenant()
    {
        // Ask for some user input and create a new tenant
    }
}