crocodicstudio\crudbooster\commands\Mailqueues::handle()

Complexity

Conditions	3
Paths	3

Size

Total Lines	21
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php namespace crocodicstudio\crudbooster\commands;
    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();
        }