Salah3id\Domains\Traits\DomainCommandTrait

Complexity

Total Complexity

4

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getDomainName()	0	7	2
A	getDomainNameForRepo()	0	18	2

<?php

namespace Salah3id\Domains\Traits;

use Symfony\Component\Console\Question\ChoiceQuestion;

trait DomainCommandTrait
{
    /**
     * Get the domain name.
     *
     * @return string
     */
    public function getDomainName()
    {
        $domain = $this->argument('domain') ?: app('domains')->getUsedNow();

It seems like argument() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $domain = app('domains')->findOrFail($domain);

        return $domain->getStudlyName();
    }

    /**
     * Get the domain name for Repository Commands.
     *
     * @return string
     */
    public function getDomainNameForRepo()
    {
        try {
            $domain = app('domains')->getUsedNow();

            $domain = app('domains')->findOrFail($domain)->getStudlyName();

        } catch (\Throwable $th) {
            $helper = $this->getHelper('question');

It seems like getHelper() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            $question = new ChoiceQuestion(
                'Select Domain :',
                array_values($this->laravel['domains']->all()),
                0
            );
            $question->setErrorMessage('Domain %s is invalid.');
            $domain = $helper->ask($this->input, $this->output, $question);;
        }
        return $domain;
    }
}