Modules\Machine\Services\Notification

Complexity

Total Complexity

4

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	find()	0	3	1
A	delete()	0	3	1
A	update()	0	6	1
A	create()	0	6	1

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

namespace Modules\Machine\Services;

use Modules\Machine\Contracts\NotificationServiceContract;

The type Modules\Machine\Contracts\NotificationServiceContract was not found. Maybe you did not declare it correctly or list all dependencies?

use Modules\Machine\Entities\Machine;
use Modules\Machine\Events\MachineRegisteredEvent;

class Notification implements NotificationServiceContract
{
    public function find($id): ?Machine
    {
        return Machine::find($id);
    }

    public function update($id, $data): Machine
    {
        $user = $this->find($id);
        $user->update($data);

        return $user;

The expression return $user could return the type null which is incompatible with the type-hinted return Modules\Machine\Entities\Machine. Consider adding an additional type-check to rule them out.

    }

    public function create($data): Machine
    {
        $machine = Machine::create($data);
        event(new MachineRegisteredEvent($machine));

        return $machine;
    }

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

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

    }
}