Chriscreates\Projects\Traits\HasUsers

Complexity

Total Complexity

10

Size/Duplication

Total Lines	96
Duplicated Lines	33.33 %

Coupling/Cohesion

Components	2
Dependencies	2

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	users()	0	9	1
A	addUser()	0	4	1
A	addUsers()	17	17	2
A	removeUser()	0	4	2
A	removeUsers()	15	15	2
A	hasUser()	0	4	2

<?php

namespace Chriscreates\Projects\Traits;

use Illuminate\Database\Eloquent\Relations\BelongsToMany;

trait HasUsers
{
    /**
     * @return Illuminate\Database\Eloquent\Relations\BelongsToMany
     */
    public function users() : BelongsToMany
    {
        return $this->belongsToMany(

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

            get_class(user_model()),
            'project_users',
            'user_id',
            'project_id'
        );
    }

    /**
     * Add a user to the current Project.
     *
     * @param  string|int|\Illuminate\Database\Eloquent\Model $user
     * @param  array                   $pivot_columns
     * @return self
     */
    public function addUser($user, array $pivot_columns = []) : self
    {
        return $this->addUsers([$user], $pivot_columns);
    }

    /**
     * Add one or multiple users to the current Project.
     *
     * @param  array                   $users
     * @param  array                   $pivot_columns
     * @return self
     */
    public function addUsers(array $users = [], array $pivot_columns = []) : self

This method seems to be duplicated in your project.

    {
        $keyed = collect($users)
        ->mapWithKeys(function ($user) use ($pivot_columns) {
            return [
                is_a($user, user_model()) ? $user->id : $user => $pivot_columns,
            ];
        })->toArray();

        $this->users()->syncWithoutDetaching($keyed);

        $this->load('users');

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

        $this->refresh('users');

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

        return $this;
    }

    /**
     * Remove the given user from the current Project
     *
     * @param  string|int|\Illuminate\Database\Eloquent\Model $user
     * @return self
     */
    public function removeUser($user) : self
    {
        return $this->users()->detach(is_a($user, user_model()) ? $user->id : $user);
    }

    /**
     * Remove one or multiple users from the current Project
     *
     * @param  array                   $users
     * @return self
     */
    public function removeUsers(array $users = []) : self

This method seems to be duplicated in your project.

    {
        $keyed = collect($users)
        ->map(function ($user) {
            return is_a($user, user_model()) ? $user->id : $user;
        })->toArray();

        $this->users()->detach($keyed);

        $this->load('users');

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

        $this->refresh('users');

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

        return $this;
    }

    /**
     * Check if the given user is on the current Project
     *
     * @param  string|int|\Illuminate\Database\Eloquent\Model $user
     * @return bool
     */
    public function hasUser($user) : bool
    {
        return $this->users->contains('id', is_a($user, user_model()) ? $user->id : $user);

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

    }
}