Merodiro\Friendships\Friendable

Complexity

Total Complexity

17

Size/Duplication

Total Lines	121
Duplicated Lines	16.53 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

98.55%

Importance

Changes	2
Bugs	1	Features	0

9 Methods

Rating	Name	Duplication	Size	Complexity
A	friendRequestsReceived()	10	10	1
A	friendRequestsSent()	10	10	1
A	isFriendsWith()	0	6	1
A	deleteFriend()	0	12	2
B	checkFriendship()	0	23	6
A	friends()	0	17	1
A	addFriend()	0	14	2
A	acceptFriend()	0	12	2
A	mutualFriends()	0	6	1

<?php

namespace Merodiro\Friendships;

trait Friendable
{
    public function checkFriendship($user)
    {
        if ($this->id == $user->id) {

The property id does not exist. Did you maybe forget to declare it?

            return 'same_user';
        }

        $friendship = Friendship::betweenUsers($this, $user)
            ->first();

        if (!$friendship) {
            return 'not_friends';
        }

        if ($friendship->status == 1) {
            return 'friends';
        }
        if ($friendship->requester == $this->id) {
            return 'waiting';
        }
        if ($friendship->user_requested == $this->id) {
            return 'pending';
        }
    }

    public function addFriend($recipient)
    {
        $friendshipStatus = $this->checkFriendship($recipient);

        if ($friendshipStatus == 'not_friends') {
            Friendship::create([
                    'requester'      => $this->id,
                    'user_requested' => $recipient->id,
                ]);
            event('friendrequest.sent', [$this, $recipient]);
        }

        return $friendshipStatus == 'not_friends';
    }

    public function acceptFriend($sender)
    {
        $friendshipStatus = $this->checkFriendship($sender);

        if ($friendshipStatus == 'pending') {
            Friendship::betweenUsers($this, $sender)
                ->update(['status' => 1]);
            event('friendrequest.accepted', [$this, $sender]);
        }

        return $friendshipStatus == 'pending';
    }

    public function deleteFriend($user)
    {
        $friendshipStatus = $this->checkFriendship($user);

        if ($friendshipStatus != 'not_friends') {
            Friendship::betweenUsers($this, $user)
                ->delete();
            event('friendship.deleted', [$this, $user]);
        }

        return $friendshipStatus != 'not_friends';
    }

    public function friends()
    {
        $recipients = Friendship::whereSender($this)
            ->accepted(1)
            ->get(['user_requested'])
            ->toArray();

        $senders = Friendship::whereRecipient($this)
            ->accepted(1)
            ->get(['requester'])
            ->toArray();

        $friendsIds = array_merge($recipients, $senders);

        return static::whereIn('id', $friendsIds)
            ->get();
    }

    public function friendRequestsReceived()

This method seems to be duplicated in your project.

    {
        $senders = Friendship::whereRecipient($this)
            ->accepted(0)
            ->get(['requester'])
            ->toArray();

        return static::whereIn('id', $senders)
            ->get();
    }

    public function friendRequestsSent()

This method seems to be duplicated in your project.

    {
        $recipients = Friendship::whereSender($this)
            ->accepted(0)
            ->get(['user_requested'])
            ->toArray();

        return static::whereIn('id', $recipients)
            ->get();
    }

    public function isFriendsWith($user)
    {
        $friendshipStatus = $this->checkFriendship($user);

        return $friendshipStatus === 'friends';
    }

    public function mutualFriends($user)
    {
        $userFriends = $user->friends();
        $friends = $this->friends();
        return $userFriends->intersect($friends);
    }
}