SoliDry\Extension\JWTTrait::createJwtUser()

Complexity

Conditions	2
Paths	2

Size

Total Lines	20
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function createJwtUser()
    {
        if(empty($this->model->password)) {
            Json::outputErrors(
                [
                    [
                        JSONApiInterface::ERROR_TITLE  => 'Password should be provided',
                        JSONApiInterface::ERROR_DETAIL => 'To get refreshed token in future usage of application - user password should be provided',
                    ],
                ]
            );
        }

        /** @var \Illuminate\Database\Eloquent\Model $model */
        $model           = $this->getEntity($this->model->id);

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

The property id does not seem to exist on SoliDry\Extension\ApiController.

        $model->jwt      = Jwt::create($this->model->id);
        $model->password = password_hash($this->model->password, PASSWORD_DEFAULT);
        $model->save();
        $this->model = $model;

It seems like $model of type Illuminate\Database\Eloquent\Model is incompatible with the declared type SoliDry\Extension\ApiController of property $model.

        unset($this->model->password);
    }
}