mathewparet\LaravelInvites\Commands\GenerateInvitations

Complexity

Total Complexity

5

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	handle()	0	18	4
A	__construct()	0	3	1

<?php

namespace mathewparet\LaravelInvites\Commands;

use mathewparet\LaravelInvites\Facades\LaravelInvites;

use Illuminate\Console\Command;

class GenerateInvitations extends Command
{
    /**
     * The name and signature of the console command.
     *
     * @var string
     */
    protected $signature = 'invites:generate {email?} 
        {--a|allow=1 : Number of times the code can be used} 
        {--c|count=1 : The number of codes to be generated}
        {--d|days= : The number of days until expiry (preceeds hours option)}
        {--r|hours= : Number of hours until expiry}';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Generates invitation codes';

    /**
     * Create a new command instance.
     *
     * @return void
     */
    public function __construct()
    {
        parent::__construct();
    }

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {
        $email = $this->argument('email') ? : null;
        $allow = $this->option('allow');
        $count = $this->option('count');
        $hours = $this->option('hours');
        $days = $this->option('days');

        $invite = LaravelInvites::for($email)->allow($allow);

The method for() does not exist on mathewparet\LaravelInvites\Facades\LaravelInvites. Since you implemented __callStatic, consider adding a @method annotation.

        if($days)
            $invite->setExpiry(now()->addDays($days));

$days of type string is incompatible with the type integer expected by parameter $value of Carbon\Carbon::addDays().

        else if($hours)
            $invite->setExpiry(now()->addHours($hours));

$hours of type string is incompatible with the type integer expected by parameter $value of Carbon\Carbon::addHours().

        $invites = $invite->generate($count);

The assignment to $invites is dead and can be removed.

        $this->info($count." invitations generated.");
    }
}