johnykvsky / spisywarka

src/Controller/Admin/AdminItemController.php

<?php

namespace App\Controller\Admin;

use App\Command\DeleteItemCommand;
use App\Repository\ItemRepository;
use Psr\Log\LoggerInterface;
use Ramsey\Uuid\Uuid;
use Symfony\Bundle\FrameworkBundle\Controller\AbstractController;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Response;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\Messenger\MessageBusInterface;
use Symfony\Component\Routing\Annotation\Route;
use Symfony\Component\Form\FormError;
use Knp\Component\Pager\PaginatorInterface; 
use Symfony\Component\HttpFoundation\RedirectResponse;
use App\Service\ItemService;
use Symfony\Component\Form\FormInterface;
use Swagger\Annotations as SWG;
use App\Traits\RequestQueryTrait;

class AdminItemController extends AbstractController
{
    use RequestQueryTrait;

The trait App\Traits\RequestQueryTrait requires the property $query which is not provided by App\Controller\Admin\AdminItemController.

    /**
     * @var MessageBusInterface
     */
    private $commandBus;
    /**
     * @var LoggerInterface
     */
    private $logger;
    /**
     * @var ItemRepository
     */
    private $repository;
    /**
     * @var ItemService
     */
    private $itemService;

    /**
     * @param MessageBusInterface $commandBus
     * @param LoggerInterface $logger
     * @param ItemRepository $repository
     * @param ItemService $itemService
     */
    public function __construct(
        MessageBusInterface $commandBus,
        LoggerInterface $logger,
        ItemRepository $repository,
        ItemService $itemService
    )
    {
        $this->commandBus = $commandBus;
        $this->logger = $logger;
        $this->repository = $repository;
        $this->itemService = $itemService;
    }

    /**
     * @Route("/admin/item/{id}", name="admin_item", defaults={"id"=null}, methods={"GET","POST"})
     * 
     * @param string|null $id
     * @param Request $rawRequest
     * @return Response|RedirectResponse
     */
    public function createOrEditItem(?string $id, Request $rawRequest): Response
    {
        $form = $this->getForm($id);
        $form->handleRequest($rawRequest);

        try {
            if ($form->isSubmitted() && $form->isValid()) {
                $itemDTO = $form->getData();
                $itemDTO->setId($id);
                $command = $this->itemService->getCommand($itemDTO);
                $this->commandBus->dispatch($command);
                $this->addFlash('success','Your changes were saved!');
                return $this->redirectToRoute('admin_item', ['id' => $command->getId()]);

The method getId() does not exist on App\Command\CommandInterface. It seems like you code against a sub-type of said class. However, the method does not exist in App\Command\ResetPasswordCommand or App\Command\ResetPasswordConfirmationCommand or App\Command\AddItemToCollectionCommand or App\Command\RemoveItemFromCollectionCommand. Are you sure you never get one of those?

            }
        } catch (\Exception $e) {
            $this->addFlash('danger','Error while saving changes');
            $error = new FormError("There is an error: ".$e->getMessage());
            $form->addError($error);
        }

        return $this->render('admin/item/form.html.twig', [
            'form' => $form->createView(),
            'id' => $id
        ]);
    }

    /** 
     * @Route("/admin/items/list", name="admin_items") 
     * @param PaginatorInterface $paginator
     * @param Request $request
     * @return Response
     */ 
    public function itemsList(PaginatorInterface $paginator, Request $request): Response 
    {         
        $searchQuery = $request->query->getAlnum('search');

        return $this->render('admin/item/list.html.twig', [ 
            'pagination' => $paginator->paginate(
             $this->repository->listAllItems($searchQuery), $request->query->getInt('page', 1),10),
             'searchQuery'  => $searchQuery
        ]); 
    }

    /**
     * 
     * @Route("/admin/delete/item/{id}", name="admin_delete_item", methods={"GET"})
     * @param string $id
     * @param Request $rawRequest
     * @return Response|RedirectResponse
     */
    public function deleteItem(string $id, Request $rawRequest): Response

The parameter $rawRequest is not used and could be removed.

    {
        try {
            $itemId = Uuid::fromString($id);
            $command = new DeleteItemCommand($itemId);
            $this->commandBus->dispatch($command);
            $this->addFlash('success','Item deleted!');
        } catch (\Exception $e) {
            $this->addFlash('danger','Error while deleting item: '.$e->getMessage());
        }

        return $this->redirectToRoute('admin_items');
    }

    /** 
     * @Route("/autocomplete/items", name="admin_items_autocomplete", methods={"GET"}) 
     *
     * @SWG\Tag(name="Autocomplete")
     * @SWG\Get(
     *     @SWG\Parameter(name="q", in="path", type="string", description="Search query")
     * )
     * @SWG\Response(
     *     response="200",
     *     description="Matched items"
     * )
     *
     * @param Request $request
     * @return JsonResponse
     */ 
    public function autocompleteAction(Request $request): JsonResponse
    {
        $result = array();
        $q = $request->query->get('q');
        $searchQuery = filter_var(urldecode($q), FILTER_SANITIZE_STRING);

The assignment to $searchQuery is dead and can be removed.

        $items = $this->repository->autocomplete($this->getFromRequest($request, 'q', true));

        foreach ($items as $item) {
            $result[] = ['id' => $item->getId()->toString(), 'text'=>$item->getName()];
        }

        return $this->json(['items' => $result], 200);
    }

    /**
     * @param string|null $id
     * @return FormInterface
     */
    private function getForm($id = null): FormInterface
    {
        if (empty($id)) {
            return $this->createForm(\App\Form\Type\ItemType::class);
        }

        $item = $this->repository->getItem(Uuid::fromString($id));
        $itemDTO = $this->itemService->fillItemDTO($item);
        return $this->createForm(\App\Form\Type\ItemType::class, $itemDTO);
    }
}