MultiTenantLaravel\App\Commands\CreateTenant

Complexity

Total Complexity

8

Size/Duplication

Total Lines	129
Duplicated Lines	50.39 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	handle()	0	15	3
A	createFakeTenant()	32	32	2
A	createNewTenant()	33	33	2

<?php

namespace MultiTenantLaravel\App\Commands;

use Illuminate\Console\Command;
use Faker\Generator as Faker;

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

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

    protected $faker;

    /**
     * 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(Faker $faker)
    {
        $this->faker = $faker;

        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();
            } else {
                $this->createFakeTenant();
            }

            $count--;
        }

    }

    /**
     * Setup a fake user using faker
     *
     * @return void
     */
    private function createFakeTenant()

This method seems to be duplicated in your project.

    {
        $name = $this->faker->name;

        $create_new = $this->anticipate('Would you like to create a new user, or use an existing?', ['New', 'Existing'], 'New');

        if ($create_new === "New") {
            $name = (string) $this->ask('Name');
            $email = (string) $this->ask('E-Mail');
            $user = config('multi-tenant.user_class')::create([
                'name' => $name,
                'email' => $email,
                'password' => bcrypt('tester'),
            ]);
        } else {
            $headers = ['Name', 'ID'];
            $tenants = config('multi-tenant.user_class')::all('name', 'id');
            $this->table($headers, $tenants->toArray());

            $user_id = (int) $this->ask('Please enter the id of the desired user.');

            $user = config('multi-tenant.user_class')::findOrFail($user_id);
        }

        $tenant = config('multi-tenant.tenant_class')::create([
            'name' => $name,
            'owner_id' => $user->id,
            'slug' => str_slug($name)
        ]);

        $this->comment('The user ' . $user->email . ' is now the owner of ' . $tenant->name . ' with the password `tester`');
    }

    /**
     * Setup a new tenant with user supplied settings for the tenant
     *
     * @return void
     */
    private function createNewTenant()

This method seems to be duplicated in your project.

    {

        $tenant_name = (string) $this->ask('Please enter a name for your new tenant.');

        $create_new = $this->anticipate('Would you like to create a new user, or use an existing?', ['New', 'Existing'], 'New');

        if ($create_new === "New") {
            $user_name = (string) $this->ask('Please enter a name for the new user');
            $user_email = (string) $this->ask('Please enter an e-mail for the new user');
            $user = config('multi-tenant.user_class')::create([
                'name' => $user_name,
                'email' => $user_email,
                'password' => bcrypt('tester'),
            ]);
        } else {
            $headers = ['Name', 'ID'];
            $tenants = config('multi-tenant.user_class')::all('name', 'id');
            $this->table($headers, $tenants->toArray());

            $user_id = (int) $this->ask('Please enter the id of the desired user.');

            $user = config('multi-tenant.user_class')::findOrFail($user_id);
        }

        $tenant = config('multi-tenant.tenant_class')::create([
            'name' => $tenant_name,
            'owner_id' => $user->id,
            'slug' => str_slug($tenant_name)
        ]);

        $this->comment('The user ' . $user->email . ' is now the owner of ' . $tenant->name . ' with the password `tester`');
    }

}