Merodiro\Friendships\Friendable

Complexity

Total Complexity

17

Size/Duplication

Total Lines	103
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	0

Test Coverage

Coverage

98.15%

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	checkFriendship()	0	18	5
A	addFriend()	0	12	2
A	acceptFriend()	0	13	2
A	deleteFriend()	0	13	2
A	friends()	0	5	1
A	friendRequestsReceived()	0	5	1
A	friendRequestsSent()	0	5	1
A	isFriendsWith()	0	4	1
A	mutualFriendsCount()	0	7	1
A	mutualFriends()	0	9	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);

The method betweenUsers() does not exist on Merodiro\Friendships\Friendship. Did you maybe mean scopeBetweenUsers()?

        if ($friendship->count() == 0) {
            return 'not_friends';
        } elseif ($friendship->count() == 2) {
            return 'friends';
        } elseif ($friendship->first()->user_id == $this->id) {
            return 'waiting';
        } else {
            return 'pending';
        }
    }

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

        if ($friendshipStatus == 'not_friends') {
            $this->friends()->attach($recipient);

            event('friendrequest.sent', [$this, $recipient]);

            return 'waiting';
        }
    }

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

        if ($friendshipStatus == 'pending') {
            $this->friends()->attach($sender, ['status' => 1]);
            $sender->friends()->updateExistingPivot($this, ['status' => 1]);

            event('friendrequest.accepted', [$this, $sender]);

            return 'friends';
        }
    }

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

        if ($friendshipStatus != 'not_friends') {
            $this->friends()->detach($user);
            $user->friends()->detach($this);

            event('friendship.deleted', [$this, $user]);

            return 'not_friends';
        }
    }

    public function friends()
    {
        return $this->belongsToMany(config('friendships.user_model'), 'friendships', 'user_id', 'friend_id')

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

            ->where('status', 1);
    }

    public function friendRequestsReceived()
    {
        return $this->belongsToMany(config('friendships.user_model'), 'friendships', 'friend_id', 'user_id')

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

            ->where('status', 0);
    }

    public function friendRequestsSent()
    {
        return $this->belongsToMany(config('friendships.user_model'), 'friendships', 'user_id', 'friend_id')

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

            ->where('status', 0);
    }

    public function isFriendsWith($user)
    {
        return $this->friends->contains($user);

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

    }

    public function mutualFriendsCount($user)
    {
        $userFriends = $user->friends->pluck('id');
        $friends = $this->friends->pluck('id');

        return $userFriends->intersect($friends)->count();
    }

    public function mutualFriends($user)
    {
        $userFriends = $user->friends->pluck('id');
        $friends = $this->friends->pluck('id');

        $mutualIds = $userFriends->intersect($friends);

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