Fenos\Notifynder\Artisan\DeleteCategory

Complexity

Total Complexity

6

Size/Duplication

Total Lines	73
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	5
Bugs	4	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getArguments()	0	6	1
A	isIntegerValue()	0	4	1
A	fire()	0	17	3

<?php

namespace Fenos\Notifynder\Artisan;

use Fenos\Notifynder\Contracts\NotifynderCategory;
use Illuminate\Console\Command;
use Symfony\Component\Console\Input\InputArgument;

class DeleteCategory extends Command
{
    /**
     * The console command name.
     *
     * @var string
     */
    protected $name = 'notifynder:delete:category';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Delete a notifynder category by ID or Name given';

    /**
     * @var \\Fenos\Notifynder\Contracts\NotifynderCategory
     */
    private $notifynderCategory;

    /**
     * Create a new command instance.
     *
     * @param  NotifynderCategory                       $notifynderCategory
     * @return \Fenos\Notifynder\Artisan\DeleteCategory

Adding a @return annotation to constructors is generally not recommended as a constructor does not have a meaningful return value.

     */
    public function __construct(NotifynderCategory $notifynderCategory)
    {
        parent::__construct();
        $this->notifynderCategory = $notifynderCategory;

It seems like $notifynderCategory of type object<Fenos\Notifynder\Contracts\NotifynderCategory> is incompatible with the declared type object<\Fenos\Notifynder\Contracts\NotifynderCategory> of property $notifynderCategory.

    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function fire()
    {
        $identifier = $this->argument('identifier');

        if ($this->isIntegerValue($identifier)) {
            $delete = $this->notifynderCategory->delete($identifier);
        } else {
            $delete = $this->notifynderCategory->deleteByName($identifier);
        }

        if (! $delete) {
            $this->error('Category Not found');

            return false;
        }
        $this->info('Category has been deleted');
    }

    public function isIntegerValue($identifier)
    {
        return preg_match('/[0-9]/', $identifier);
    }

    /**
     * Get the console command arguments.
     *
     * @return array
     */
    protected function getArguments()
    {
        return [
            ['identifier', InputArgument::REQUIRED, '1 - nameCategory'],
        ];
    }
}