SWP\MultiTenancyBundle\Command\CreateTenantCommand

Complexity

Total Complexity

14

Size/Duplication

Total Lines	126
Duplicated Lines	23.81 %

Coupling/Cohesion

Components	1
Dependencies	9

Importance

Changes	1
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	configure()	0	18	1
B	execute()	0	24	3
C	interact()	30	35	7
A	createTenant()	0	9	1
A	getEntityManager()	0	4	1
A	getTenantRepository()	0	4	1

<?php

/**
 * This file is part of the Superdesk Web Publisher MultiTenancyBundle.
 *
 * Copyright 2016 Sourcefabric z.u. and contributors.
 *
 * For the full copyright and license information, please see the
 * AUTHORS and LICENSE files distributed with this source code.
 *
 * @copyright 2016 Sourcefabric z.ú.
 * @license http://www.superdesk.org/license
 */
namespace SWP\MultiTenancyBundle\Command;

use Doctrine\ORM\EntityManagerInterface;
use SWP\Component\MultiTenancy\Model\Tenant;
use SWP\Component\MultiTenancy\Model\TenantInterface;
use SWP\Component\MultiTenancy\Repository\TenantRepositoryInterface;
use Symfony\Bundle\FrameworkBundle\Command\ContainerAwareCommand;
use Symfony\Component\Console\Input\InputArgument;
use Symfony\Component\Console\Input\InputInterface;
use Symfony\Component\Console\Input\InputOption;
use Symfony\Component\Console\Output\OutputInterface;

/**
 * Class CreateTenantCommand.
 */
class CreateTenantCommand extends ContainerAwareCommand
{
    /**
     * {@inheritdoc}
     */
    protected function configure()
    {
        $this
            ->setName('swp:tenant:create')
            ->setDescription('Creates a new tenant.')
            ->setDefinition([
                new InputArgument('subdomain', InputArgument::OPTIONAL, 'Subdomain name'),
                new InputArgument('name', InputArgument::OPTIONAL, 'Tenant name'),
                new InputOption('disabled', null, InputOption::VALUE_NONE, 'Set the tenant as a disabled'),
                new InputOption('default', null, InputOption::VALUE_NONE, 'Creates the default tenant'),
            ])
            ->setHelp(
                <<<EOT
                The <info>%command.name%</info> command creates a new tenant.
EOT
            )
        ;
    }

    /**
     * {@inheritdoc}
     */
    protected function execute(InputInterface $input, OutputInterface $output)
    {
        $subdomain = $input->getArgument('subdomain');
        $name = $input->getArgument('name');
        $default = $input->getOption('default');
        $disabled = $input->getOption('disabled');

        if ($default) {
            $subdomain = 'default';
            $name = 'Default tenant';
        }

        $tenant = $this->getTenantRepository()->findOneBySubdomain($subdomain);

The method findOneBySubdomain() does not exist on SWP\Component\MultiTenancy\Repository\TenantRepositoryInterface. Did you maybe mean findBySubdomain()?

        if (null !== $tenant) {
            throw new \InvalidArgumentException(sprintf('Tenant with subdomain "%s" already exists!', $subdomain));
        }

        $tenant = $this->createTenant($subdomain, $name, $disabled);

        $this->getEntityManager()->persist($tenant);
        $this->getEntityManager()->flush();

        $output->writeln(sprintf('Tenant <info>%s</info> has been created!', $name));
    }

    /**
     * {@inheritdoc}
     */
    protected function interact(InputInterface $input, OutputInterface $output)
    {
        $default = $input->getOption('default');
        if (!$input->getArgument('subdomain') && !$default) {

This code seems to be duplicated across your project.

            $subdomain = $this->getHelper('dialog')->askAndValidate(
                $output,
                '<question>Please enter subdomain:</question>',
                function ($subdomain) {
                    if (empty($subdomain)) {
                        throw new \RuntimeException('Subdomain can not be empty');
                    }

                    return $subdomain;
                }
            );

            $input->setArgument('subdomain', $subdomain);
        }

        if (!$input->getArgument('name') && !$default) {

This code seems to be duplicated across your project.

            $name = $this->getHelper('dialog')->askAndValidate(
                $output,
                '<question>Please enter name:</question>',
                function ($name) {
                    if (empty($name)) {
                        throw new \RuntimeException('Name can not be empty');
                    }

                    return $name;
                }
            );

            $input->setArgument('name', $name);
        }
    }

    /**
     * Creates a new tenant.
     *
     * @param $subdomain
     * @param $name
     * @param $disabled
     *
     * @return TenantInterface
     */
    protected function createTenant($subdomain, $name, $disabled)
    {
        $tenant = new Tenant();
        $tenant->setSubdomain($subdomain);
        $tenant->setName($name);
        $tenant->setEnabled(!$disabled);

        return $tenant;
    }

    /**
     * @return EntityManagerInterface
     */
    protected function getEntityManager()
    {
        return $this->getContainer()->get('doctrine.orm.entity_manager');
    }

    /**
     * @return TenantRepositoryInterface
     */
    protected function getTenantRepository()
    {
        return $this->getContainer()->get('swp_multi_tenancy.tenant_repository');
    }
}