App\Containers\User\Tasks\UpdateUserTask

Complexity

Total Complexity

4

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	2

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
B	run()	0	36	3

<?php

namespace App\Containers\User\Tasks;

use App\Containers\Authentication\Exceptions\UpdateResourceFailedException;
use App\Containers\User\Contracts\UserRepositoryInterface;
use App\Ship\Parents\Tasks\Task;
use Illuminate\Support\Facades\Hash;

/**
 * Class UpdateUserTask.
 *
 * @author Mahmoud Zalt <mahmoud@zalt.me>
 */
class UpdateUserTask extends Task
{

    /**
     * @var  \App\Containers\User\Contracts\UserRepositoryInterface
     */
    private $userRepository;

    /**
     * UpdateUserTask constructor.
     *
     * @param \App\Containers\User\Contracts\UserRepositoryInterface $userRepository
     */
    public function __construct(UserRepositoryInterface $userRepository)
    {
        $this->userRepository = $userRepository;
    }

    /**
     * @param      $userId
     * @param null $password
     * @param null $name
     * @param null $email
     * @param null $gender
     * @param null $birth
     * @param null $token
     * @param null $expiresIn
     * @param null $refreshToken
     * @param null $tokenSecret
     *
     * @return  mixed
     * @throws \App\Containers\Authentication\Exceptions\UpdateResourceFailedException
     */
    public function run(
        $userId,
        $password = null,
        $name = null,
        $email = null,
        $gender = null,
        $birth = null,
        $token = null,
        $expiresIn = null,
        $refreshToken = null,
        $tokenSecret = null
    ) {

        // set all data in the array, then remove all null values and their keys
        $attributes = array_filter([
            'password'             => $password ? Hash::make($password) : null,
            'name'                 => $name,
            'email'                => $email,
            'gender'               => $gender,
            'birth'                => $birth,
            'social_token'         => $token,
            'social_expires_in'    => $expiresIn,
            'social_refresh_token' => $refreshToken,
            'social_token_secret'  => $tokenSecret,
        ]);

        // optionally, check if data is empty and return error
        if (!$attributes) {

The expression $attributes of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using empty($expr) instead to make it clear that you intend to check for an array without elements.

            throw new UpdateResourceFailedException('Inputs are empty.');
        }

        // updating the attributes
        $user = $this->userRepository->update($attributes, $userId);

        return $user;
    }

}