BeyondCode\Mailbox\Console\CleanEmails::handle()

Complexity

Conditions	3
Paths	2

Size

Total Lines	34
Code Lines	17

Duplication

Lines	0
Ratio	0 %

Importance

Changes	5
Bugs	2	Features	0

<?php
    public function handle()
    {
        $this->comment('Cleaning old incoming email logs...');

        $maxAgeInDays = config('mailbox.store_incoming_emails_for_days');

        if ($maxAgeInDays === INF) {
            $this->error($this->signature.' is disabled because store_incoming_emails_for_days is set to INF.');

            return 1;
        }

        $cutOffDate = Carbon::now()->subDay($maxAgeInDays)->format('Y-m-d H:i:s');

The call to Carbon\Carbon::subDay() has too many arguments starting with $maxAgeInDays.

        /** @var InboundEmail $modelClass */
        $modelClass = config('mailbox.model');

        // chunk the deletion to avoid memory issues

        $this->amountDeleted = 0;

        $modelClass::where('created_at', '<', $cutOffDate)
            ->select('id')
            ->chunk(100, function ($models) use ($modelClass) {
                foreach ($models as $model) {
                    $modelInstance = $modelClass::find($model->id);
                    $modelInstance->delete();
                    $this->amountDeleted++;
                }
            });

        $this->info("Deleted {$this->amountDeleted} record(s) from the Mailbox logs.");

        $this->comment('All done!');
    }