Tinyissue\Repository\User\UpdaterRepository

Complexity

Total Complexity

7

Size/Duplication

Total Lines	82
Duplicated Lines	51.22 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	create()	15	15	1
A	delete()	11	11	1
A	update()	5	10	3
A	updateMessagesSettings()	11	11	1

<?php

/*
 * This file is part of the Tinyissue package.
 *
 * (c) Mohamed Alsharaf <mohamed.alsharaf@gmail.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Tinyissue\Repository\User;

use Hash;
use Tinyissue\Contracts\Model\UserInterface;
use Tinyissue\Contracts\Repository\User\UpdaterRepository as UserUpdater;
use Tinyissue\Model\Project\User as ProjectUser;
use Tinyissue\Model\User;
use Tinyissue\Repository\RepositoryUpdater;

/**
 * Class UpdaterRepository.
 *
 * @package Tinyissue\Repository\User
 */
class UpdaterRepository extends RepositoryUpdater implements UserUpdater
{
    public function __construct(UserInterface $model)
    {
        $this->model = $model;

It seems like $model of type object<Tinyissue\Contracts\Model\UserInterface> is incompatible with the declared type object<Illuminate\Database\Eloquent\Model> of property $model.

    }

    /**
     * Add a new user.
     *
     * @param array $info
     *
     * @return bool
     */
    public function create(array $info)

This method seems to be duplicated in your project.

    {
        $insert = [
            'email'     => $info['email'],
            'firstname' => $info['firstname'],
            'lastname'  => $info['lastname'],
            'role_id'   => $info['role_id'],
            'private'   => (boolean) $info['private'],
            'password'  => Hash::make($info['password']),
            'status'    => $info['status'],
            'language'  => app('tinyissue.settings')->getLanguage(),
        ];

        return $this->model->fill($insert)->save();
    }

    /**
     * Soft deletes a user and empties the email.
     *
     * @return bool
     */
    public function delete()

This method seems to be duplicated in your project.

    {
        $this->model->update([
            'email'   => $this->email . '_deleted',

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

            'deleted' => User::DELETED_USERS,
        ]);
        $this->model->projects()->delete();
//        Project\User::where('user_id', '=', $this->id)->delete();

        return true;
    }

    /**
     * Updates the users settings, validates the fields.
     *
     * @param array $info

There is no parameter named $info. Was it maybe removed?

     *
     * @return Eloquent\Model
     */
    public function update(array $attributes = [], array $options = [])
    {
        if ($attributes['password']) {

This code seems to be duplicated across your project.

            $attributes['password'] = Hash::make($attributes['password']);
        } elseif (empty($attributes['password'])) {
            unset($attributes['password']);
        }

        return $this->model->update($attributes, $options);
    }

    /**
     * Update user messages setting.
     *
     * @param array $input
     */
    public function updateMessagesSettings(array $input)

This method seems to be duplicated in your project.

    {
        return (new ProjectUser())

The method where does not exist on object<Tinyissue\Model\Project\User>? Since you implemented __call, maybe consider adding a @method annotation.

            ->where('user_id', '=', $this->model->id)
            ->whereIn('project_id', array_keys($input))
            ->get()
            ->each(function (ProjectUser $project) use ($input) {
                $project->message_id = $input[$project->project_id];
                $project->save();
            });
    }
}