CreateVhostCommand   A
last analyzed

Complexity

Total Complexity 5

Size/Duplication

Total Lines 61
Duplicated Lines 0 %

Coupling/Cohesion

Components 1
Dependencies 4

Test Coverage

Coverage 0%

Importance

Changes 0
Metric Value
wmc 5
lcom 1
cbo 4
dl 0
loc 61
ccs 0
cts 39
cp 0
rs 10
c 0
b 0
f 0

2 Methods

Rating   Name   Duplication   Size   Complexity  
A configure() 0 23 1
A execute() 0 24 4
1
<?php
2
3
/**
4
 * admin
5
 *
6
 * @category    Tollwerk
7
 * @package     Tollwerk\Admin
8
 * @subpackage  Tollwerk\Admin\Infrastructure
9
 * @author      Joschi Kuphal <[email protected]> / @jkphl
10
 * @copyright   Copyright © 2018 Joschi Kuphal <[email protected]> / @jkphl
11
 * @license     http://opensource.org/licenses/MIT The MIT License (MIT)
12
 */
13
14
/***********************************************************************************
15
 *  The MIT License (MIT)
16
 *
17
 *  Copyright © 2018 Joschi Kuphal <[email protected]> / @jkphl
18
 *
19
 *  Permission is hereby granted, free of charge, to any person obtaining a copy of
20
 *  this software and associated documentation files (the "Software"), to deal in
21
 *  the Software without restriction, including without limitation the rights to
22
 *  use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of
23
 *  the Software, and to permit persons to whom the Software is furnished to do so,
24
 *  subject to the following conditions:
25
 *
26
 *  The above copyright notice and this permission notice shall be included in all
27
 *  copies or substantial portions of the Software.
28
 *
29
 *  THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
30
 *  IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS
31
 *  FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR
32
 *  COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER
33
 *  IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN
34
 *  CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
35
 ***********************************************************************************/
36
37
namespace Tollwerk\Admin\Infrastructure\Commands\Vhost;
38
39
use Symfony\Component\Console\Input\InputArgument;
40
use Symfony\Component\Console\Input\InputInterface;
41
use Symfony\Component\Console\Output\OutputInterface;
42
use Tollwerk\Admin\Domain\Vhost\Vhost as DomainVhost;
43
use Tollwerk\Admin\Infrastructure\Commands\AbstractCommand;
44
use Tollwerk\Admin\Ports\Facade\Vhost;
45
46
/**
47
 * vhost:create command
48
 *
49
 * @package Tollwerk\Admin
50
 * @subpackage Tollwerk\Admin\Infrastructure
51
 */
52
class CreateVhostCommand extends AbstractCommand
53
{
54
    /**
55
     * Configure the command
56
     */
57
    protected function configure()
58
    {
59
        $this
60
            // the name of the command (the part after "bin/console")
61
            ->setName('vhost:create')
62
            // the short description shown while running "php bin/console list"
63
            ->setDescription('Create a new virtual host')
64
            // the full command description shown when running the command with
65
            // the "--help" option
66
            ->setHelp("This command allows you to create a virtual host and add it to an account")
67
            // configure the virtual host account name
68
            ->addArgument(
69
                'account',
70
                InputArgument::REQUIRED,
71
                'The name of the account the virtual host should be added to'
72
            )
73
            // configure the virtual host domain name
74
            ->addArgument('domain', InputArgument::REQUIRED, 'The name of the virtual hosts\'s primary domain')
75
            // configure the virtual host document root
76
            ->addArgument('docroot', InputArgument::OPTIONAL, 'The virtual hosts\'s document root', '')
77
            // configure the virttual host type
78
            ->addArgument('type', InputArgument::OPTIONAL, 'The virtual host type', DomainVhost::TYPE_APACHE);
79
    }
80
81
    /**
82
     * Execute the command
83
     *
84
     * @param InputInterface $input
85
     * @param OutputInterface $output
86
     * @return int Status code
87
     */
88
    protected function execute(InputInterface $input, OutputInterface $output)
89
    {
90
        $account = $input->getArgument('account');
91
        $domain = $input->getArgument('domain');
92
        $docroot = $input->getArgument('docroot');
93
        $type = $input->getArgument('type');
94
        try {
95
            Vhost::create($account, $domain, $docroot, $type);
0 ignored issues
show
Bug introduced by
It seems like $account defined by $input->getArgument('account') on line 90 can also be of type array<integer,string> or null; however, Tollwerk\Admin\Ports\Facade\Vhost::create() does only seem to accept string, maybe add an additional type check?

If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:

/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.

Loading history...
Bug introduced by
It seems like $domain defined by $input->getArgument('domain') on line 91 can also be of type array<integer,string> or null; however, Tollwerk\Admin\Ports\Facade\Vhost::create() does only seem to accept string, maybe add an additional type check?

If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:

/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.

Loading history...
Bug introduced by
It seems like $docroot defined by $input->getArgument('docroot') on line 92 can also be of type array<integer,string> or null; however, Tollwerk\Admin\Ports\Facade\Vhost::create() does only seem to accept string, maybe add an additional type check?

If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:

/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.

Loading history...
Bug introduced by
It seems like $type defined by $input->getArgument('type') on line 93 can also be of type array<integer,string> or null; however, Tollwerk\Admin\Ports\Facade\Vhost::create() does only seem to accept string, maybe add an additional type check?

If a method or function can return multiple different values and unless you are sure that you only can receive a single value in this context, we recommend to add an additional type check:

/**
 * @return array|string
 */
function returnsDifferentValues($x) {
    if ($x) {
        return 'foo';
    }

    return array();
}

$x = returnsDifferentValues($y);
if (is_array($x)) {
    // $x is an array.
}

If this a common case that PHP Analyzer should handle natively, please let us know by opening an issue.

Loading history...
96
            $output->writeln(sprintf('<info>Virtual host "%s" created successfully</info>', $docroot ?: '/'));
97
            $this->printMessages($output);
98
            return 0;
99
        } catch (\Exception $e) {
100
            $output->writeln(
101
                sprintf(
102
                    '<error>Error creating virtual host "%s": %s (%s)</error>',
103
                    $docroot ?: '/',
104
                    $e->getMessage(),
105
                    $e->getCode()
106
                )
107
            );
108
            $this->printMessages($output);
109
            return $e->getCode();
110
        }
111
    }
112
}
113