MailMarketing\Models\InMailMarketing

Complexity

Total Complexity

2

Size/Duplication

Total Lines	19
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	updateMemberInMailMarketingList()	0	7	1
A	mailMarketingMemberData()	0	7	1

<?php

namespace MailMarketing\Models;

use Illuminate\Foundation\Bus\PendingDispatch;
use MailMarketing\Jobs\UpdateMemberInMailMarketingListJob;

trait InMailMarketing
{
    public function mailMarketingMemberData(): array
    {
        return $this->only([

It seems like only() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            'email',
            'first_name',
            'last_name',
            'phone',
        ]);
    }

    public function updateMemberInMailMarketingList($listId, bool $isAdd = true, array $tags = []): PendingDispatch
    {
        return UpdateMemberInMailMarketingListJob::dispatch(
            $listId,
            $this->mailMarketingMemberData(),
            $isAdd,
            $tags
        );
    }
}