crocodicstudio\crudbooster\commands\Mailqueues

Complexity

Total Complexity

3

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	21	3

<?php namespace crocodicstudio\crudbooster\commands;

use Cache;
use CRUDBooster;

The type CRUDBooster was not found. Maybe you did not declare it correctly or list all dependencies?

use DB;
use Illuminate\Console\Command;
use Request;

class Mailqueues extends Command
{
    /**
     * The console command name.
     *
     * @var string
     */
    protected $name = 'mailqueues';

    /**
     * The console command description.
     *
     * @var string
     */
    protected $description = 'Run Mail Queues';

    /**
     * Execute the console command.
     *
     * @return mixed
     */
    public function handle()
    {


        $now = date('Y-m-d H:i:s');

        $this->comment('Mail Queues Started '.$now);

        $queues = DB::table('cms_email_queues')->where('send_at', '<=', $now)->take($limit_an_hour)->get();

The variable $limit_an_hour seems to be never defined.

        $this->comment('Total Queues : '.count($queues));

        Cache::increment('total_email_sent', count($queues));
        Cache::put('last_email_sent', date('Y-m-d H:i:s'));

The call to Illuminate\Support\Facades\Cache::put() has too few arguments starting with ttl.

        foreach ($queues as $q) {
            if (filter_var($q->email_recipient, FILTER_VALIDATE_EMAIL) !== false) {
                CRUDBooster::sendEmailQueue($queue);

The variable $queue does not exist. Did you maybe mean $queues?

                $this->comment('Email send -> '.$q->subject);
            }
            DB::table('mailqueues')->where('id', $q->id)->delete();
        }
    }
}