ag84ark\AwsSesBounceComplaintHandler\AwsSesBounceComplaintHandler

Complexity

Total Complexity

6

Size/Duplication

Total Lines	34
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	canSendToEmail()	0	15	3
A	clearEmail()	0	4	1
A	ignoreEmail()	0	8	2

<?php

namespace ag84ark\AwsSesBounceComplaintHandler;

use ag84ark\AwsSesBounceComplaintHandler\Models\WrongEmail;
use Illuminate\Support\Collection;

class AwsSesBounceComplaintHandler
{
    public function canSendToEmail(string $email): bool
    {
        /** @var WrongEmail[]|Collection $emails */
        $emails = WrongEmail::active()
            ->bounced()
            ->where('email', '=', $email)
            ->get();

        foreach ($emails as $wrongEmail) {
            if (! $wrongEmail->canBouncedSend()) {
                return false;
            }
        }

        return true;
    }

    public function ignoreEmail(string $email): void
    {
        /** @var WrongEmail[]|Collection $emails */
        $emails = WrongEmail::where('email', '=', $email)->get();

        foreach ($emails as $wrongEmail) {
            $wrongEmail->ignore = true;

The property $ignore was declared of type integer, but true is of type true. Maybe add a type cast?

            $wrongEmail->save();
        }
    }

    public function clearEmail(string $email): void
    {
        /* @var WrongEmail[]|Collection $emails */
        WrongEmail::where('email', '=', $email)->delete();
    }
}