App\Services\ContactService

Complexity

Total Complexity

5

Size/Duplication

Total Lines	74
Duplicated Lines	0 %

Test Coverage

Coverage

91.67%

Importance

Changes	1
Bugs	0	Features	0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	contacts()	0	3	1
A	setContacts()	0	12	1
A	contact()	0	3	1
A	addContact()	0	8	1
A	removeContact()	0	6	1

<?php

namespace App\Services;

use App\Interfaces\ContactServiceInterface;
use App\Models\Contact;
use App\Models\User;
use Illuminate\Contracts\Pagination\LengthAwarePaginator;
use Illuminate\Support\Collection;

class ContactService implements ContactServiceInterface
{
    /**
     * All contacts.
     *
     * @param User $user
     * @return LengthAwarePaginator
     */
    public function contacts(User $user): LengthAwarePaginator
    {
        return Contact::forUser($user->id)->with('source')->paginate();
    }

    /**
     * Retrieve a contact.
     *
     * @param string $contact_id
     * @param User $user
     * @return Contact|null
     */
    public function contact(string $contact_id, User $user): ?Contact
    {
        return Contact::where('id', $contact_id)->forUser($user->id)->first();
    }

    /**
     * Creates contact.
     * @param User $user
     * @param User $contact
     * @return Contact
     */
    public function addContact(User $user, User $contact): Contact
    {
        return Contact::updateOrCreate([
            'user_id' => $user->id,
            'source_type' => get_class($contact),
            'source_id' => $contact->id,
        ], [
            'name' => $contact->name,

The property name does not exist on App\Models\User. Since you implemented __get, consider adding a @property annotation.

        ]);
    }

    /**
     * Creates contact by user collection and returns contact.
     *
     * @param Collection $users
     * @return Collection
     */
    public function setContacts(Collection $users): Collection
    {
        return $users->map(function ($user) use ($users) {
            return $users
                    ->filter(function ($item) use ($user) {
                        return ! $item->is($user);
                    })
                    ->map(function ($contact) use ($user) {
                        return $this->addContact($user, $contact);
                    });
        })
                ->flatten();
    }

    /**
     * Remove a contact.
     *
     * @param string $contact_id
     * @return Contact
     */
    public function removeContact(string $contact_id): Contact
    {
        $contact = Contact::find($contact_id);
        $contact->delete();

        return $contact;

The expression return $contact could return the type null which is incompatible with the type-hinted return App\Models\Contact. Consider adding an additional type-check to rule them out.

    }
}