App\Controller\UserController

Complexity

Total Complexity

4

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	1
Bugs	1	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	profile()	0	19	2
A	getId()	0	3	1

<?php

namespace App\Controller;

use App\Controller;
use App\Entity\User;
use Cycle\ORM\ORMInterface;
use Psr\Http\Message\ResponseInterface as Response;
use Psr\Http\Message\ServerRequestInterface as Request;

class UserController extends Controller
{
    protected function getId(): string
    {
        return 'user';
    }

    public function index(ORMInterface $orm): Response
    {
        $repository = $orm->getRepository(User::class);
        $response = $this->responseFactory->createResponse();

        $data = [
            'items' => $repository->findAll(),
        ];

        $output = $this->render('index', $data);

        $response->getBody()->write($output);
        return $response;
    }

    public function profile(Request $request, ORMInterface $orm): Response
    {
        $userRepo = $orm->getRepository(User::class);
        $login = $request->getAttribute('login', null);

        $item = $userRepo->findByLogin($login);

The method findByLogin() does not exist on Cycle\ORM\RepositoryInterface. It seems like you code against a sub-type of Cycle\ORM\RepositoryInterface such as App\Repository\UserRepository.

        if ($item === null) {
            return $this->responseFactory->createResponse(404);
        }

        $data = [
            'item' => $item,
        ];
        $response = $this->responseFactory->createResponse();

        $output = $this->render('profile', $data);
        $response->getBody()->write($output);

        return $response;
    }
}