Slides\Saml2\Commands\DeleteTenant

Complexity

Total Complexity

6

Size/Duplication

Total Lines	77
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	36	5
A	__construct()	0	5	1

<?php

namespace Slides\Saml2\Commands;

use Slides\Saml2\Repositories\TenantRepository;

/**
 * Class DeleteTenant
 *
 * @package Slides\Saml2\Commands
 */
class DeleteTenant extends \Illuminate\Console\Command
{
    use RendersTenants;

The trait Slides\Saml2\Commands\RendersTenants requires some properties which are not provided by Slides\Saml2\Commands\DeleteTenant: $uuid, $relay_state_url, $idp_x509_cert, $key, $idp_entity_id, $id, $name_id_format, $metadata, $created_at, $idp_logout_url, $idp_login_url, $updated_at, $deleted_at

    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'saml2:delete-tenant {tenant}
                            { --safe : Safe deletion }';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Delete a tenant by ID, key or UUID';

    /**
     * @var TenantRepository
     */
    protected $tenants;

    /**
     * DeleteTenant constructor.
     *
     * @param TenantRepository $tenants
     */
    public function __construct(TenantRepository $tenants)
    {
        $this->tenants = $tenants;

        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return void
     */
    public function handle()
    {
        $tenants = $this->tenants->findByAnyIdentifier($this->argument('tenant'), false);

It seems like $this->argument('tenant') can also be of type array; however, parameter $key of Slides\Saml2\Repositories\TenantRepository::findByAnyIdentifier() does only seem to accept integer|string, maybe add an additional type check?

        if($tenants->isEmpty()) {
            $this->error('Cannot find a matching tenant by "' . $this->argument('tenant') . '" identifier');

Are you sure $this->argument('tenant') of type array|null|string can be used in concatenation?

            return;
        }

        $this->renderTenants($tenants, 'Found tenant(s)');

        if($tenants->count() > 1) {
            $deletingId = $this->ask('We have found several tenants, which one would you like to delete? (enter its ID)');
        }
        else {
            $deletingId = $tenants->first()->id;
        }

        $tenant = $tenants->firstWhere('id', $deletingId);

        if($this->option('safe')) {
            $tenant->delete();

            $this->info('The tenant #' . $deletingId . ' safely deleted. To restore it, run:');
            $this->output->block('php artisan saml2:restore-tenant ' . $deletingId);

            return;
        }

        if(!$this->confirm('Would you like to forcely delete the tenant #' . $deletingId . '? It cannot be reverted.')) {
            return;
        }

        $tenant->forceDelete();

        $this->info('The tenant #' . $deletingId . ' safely deleted.');
    }
}