Modules\Account\Services\AccountService

Complexity

Total Complexity

7

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	find()	0	7	2
A	delete()	0	3	1
A	heartbeat()	0	5	1
A	update()	0	7	1
A	getByUserId()	0	3	1
A	create()	0	6	1

<?php
/**
 * Created by PhpStorm.
 * User: arthur
 * Date: 04.10.18
 * Time: 16:17.
 */

namespace Modules\Account\Services;

use Carbon\Carbon;
use Modules\Account\Contracts\AccountServiceContract;
use Modules\Account\Entities\Account;
use Modules\Account\Events\AccountCreatedEvent;
use Modules\Account\Events\AccountUpdatedEvent;

class AccountService implements AccountServiceContract
{
    public function getByUserId($userId)
    {
        return Account::where('user_id', $userId)->get();
    }

    public function find($id): ?Account
    {
        if ($id instanceof Account) {
            return $id;
        }

        return Account::find($id);
    }

    public function update($id, $data): Account
    {
        $Account = $this->find($id);
        $Account->update($data);
        event(new AccountUpdatedEvent($Account));

        return $Account;
    }

    public function create($data): Account
    {
        $Account = Account::create($data);
        event(new AccountCreatedEvent($Account));

        return $Account;
    }

    public function delete($id): bool
    {
        return Account::destroy($id);

The expression return Modules\Account\Entities\Account::destroy($id) returns the type integer which is incompatible with the type-hinted return boolean.

    }

    public function heartbeat($id, $data): void
    {
        $this->update($id, [
            'last_heartbeat' => Carbon::now(),
            'online'         => true,
        ]);
    }
}