Acacha\ForgePublish\Commands\PublishDomain

Complexity

Total Complexity

8

Size/Duplication

Total Lines	85
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	envVar()	0	4	1
A	argKey()	0	4	1
A	questionText()	0	4	1
A	default()	0	4	2
A	defaultDomain()	0	7	2

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 12 and the first side effect is on line 96.

namespace Acacha\ForgePublish\Commands;

use Acacha\ForgePublish\Parser\ForgePublishRCParser;

/**
 * Class PublishDomain.
 *
 * @package Acacha\ForgePublish\Commands
 */
class PublishDomain extends SaveEnvVariable
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'publish:domain {domain?}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Save acacha forge domain';

    /**
     * ForgePublishRCParser
     *
     * @var ForgePublishRCParser
     */
    protected $parser;

    /**
     * SaveEnvVariable constructor.
     *
     */
    public function __construct(ForgePublishRCParser $parser)
    {
        parent::__construct();
        $this->parser = $parser;
    }

    /**
     * Env var to set.
     *
     * @return mixed
     */
    protected function envVar()
    {
        return 'ACACHA_FORGE_DOMAIN';
    }

    /**
     * Argument key.
     *
     * @return mixed
     */
    protected function argKey()
    {
        return 'domain';
    }

    /**
     * Question text.
     *
     * @return mixed
     */
    protected function questionText()
    {
        return 'Acacha forge domain?';
    }

    /**
     * Default proposed value when asking.
     *
     */
    protected function default()

Possible parse error: non-abstract method defined as abstract

    {
        return $default = fp_env('ACACHA_FORGE_DOMAIN') ? fp_env('ACACHA_FORGE_DOMAIN') : $this->defaultDomain();

$default is not used, you could remove the assignment.

It is generally advisable to only define one property per statement.

The visibility should be declared for property $default.

    }

    /**
     * Default domain.
     *
     * @return string
     */
    protected function defaultDomain()
    {
        if ($suffix = $this->parser->getDomainSuffix()) {
            return strtolower(camel_case(basename(getcwd()))) . '.' . $suffix;
        }
        return '';
    }
}