App\Modules\Security\Api\Controller\SecurityQueryController

Complexity

Total Complexity

2

Size/Duplication

Total Lines	23
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getPostsByUserId()	0	6	1

<?php

namespace App\Modules\Security\Api\Controller;

use App\Modules\Security\Api\Query\FindPostsByUserIdQuery;
use App\Modules\Security\Api\SecurityApiInterface;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\Routing\Annotation\Route;
use Symfony\Component\Uid\Ulid;

#[Route('/api/security')]
class SecurityQueryController extends AbstractController
{

    /**
     * @param SecurityApiInterface $securityApi
     */
    public function __construct(
        private SecurityApiInterface $securityApi
    )
    {
    }

    /**
     * @param string $userId
     * @return Response
     */
    #[Route('/{userId}', methods: ['GET'])]
    function getPostsByUserId(string $userId, Request $request): Response

It is generally recommended to explicitly declare the visibility for methods.

    {
        $pageNo = $request->query->get("pageNo", "1");
        return $this->json(
            $this->securityApi->findPostsByUserId(new FindPostsByUserIdQuery(new Ulid($userId), intval($pageNo)))
        );
    }
}