App\Http\Controllers\ContactController

Complexity

Total Complexity

3

Size/Duplication

Total Lines	40
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	show()	0	4	1
A	__construct()	0	3	1
A	index()	0	5	1

<?php

namespace App\Http\Controllers;

use App\Http\Resources\ContactResource;
use App\Interfaces\ContactServiceInterface;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;

class ContactController extends Controller
{
    /**
     * @var ContactServiceInterface
     */
    private ContactServiceInterface $service;

    /**
     * ContactController constructor.
     *
     * @param ContactServiceInterface $service
     */
    public function __construct(ContactServiceInterface $service)
    {
        $this->service = $service;
    }

    /**
     * Returns user by id.
     *
     * @param string $id
     * @return ContactResource
     */
    public function show(string $id)
    {
        return new ContactResource(
            $this->service->contact($id)
        );
    }

    /**
     * Returns contacts by user.
     *
     * @return \Illuminate\Http\Resources\Json\AnonymousResourceCollection
     */
    public function index()
    {
        $user = Auth::user();

        return ContactResource::collection($this->service->contacts($user));

It seems like $user can also be of type null; however, parameter $user of App\Interfaces\ContactServiceInterface::contacts() does only seem to accept App\Models\User, maybe add an additional type check?

    }
}