WatchedMovieController::postWatchedMovies() - Code Metrics - Inspection of "Merge pull request #73 from svbackend/Filters" - svbackend/my-art-lib - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0b4911...4851fa )

by Valentyn

created 2019-08-15 14:24 UTC

WatchedMovieController::postWatchedMovies() A

↳ Parent: WatchedMovieController

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	6
CRAP Score	2.0116

Importance

Changes

Metric	Value
dl	0
loc	13
ccs	6
cts	7
cp	0.8571
rs	9.8333
c	0
b	0
f	0
cc	2
nc	2
nop	3
crap	2.0116

<?php

namespace App\Users\Controller;

use App\Controller\BaseController;
use App\Guests\Entity\GuestSession;
use App\Movies\Repository\MovieRepository;
use App\Movies\Request\AddWatchedMovieRequest;
use App\Movies\Request\UpdateWatchedMovieRequest;
use App\Movies\Service\WatchedMovieService;
use App\Movies\Transformer\MovieTransformer;
use App\Pagination\CustomPaginatedCollection;
use App\Users\Entity\User;
use App\Users\Entity\UserWatchedMovie;
use App\Users\Repository\WatchedMovieRepository;
use App\Users\Request\MergeWatchedMoviesRequest;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\HttpFoundation\Request;
use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException;
use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
use Symfony\Component\Routing\Annotation\Route;

class WatchedMovieController extends BaseController
{
    /**
     * @Route("/api/users/watchedMovies", methods={"POST"});
     *
     * @param AddWatchedMovieRequest $addWatchedMovieRequest
     * @param Request                $request
     * @param WatchedMovieService    $watchedMovieService
     *
     * @throws \Exception|NotFoundHttpException
     *
     * @return JsonResponse
     */
    public function postWatchedMovies(AddWatchedMovieRequest $addWatchedMovieRequest, Request $request, WatchedMovieService $watchedMovieService)
    {
        $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');

        $watchedMovieDTO = $addWatchedMovieRequest->getWatchedMovieDTO();
        $isMovieAdded = $watchedMovieService->addUserWatchedMovie($this->getUser(), $watchedMovieDTO, $request->getLocale());
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        if ($isMovieAdded === false) {
            throw new NotFoundHttpException('Movie not found by provided ID / TMDB ID');
        }

        return new JsonResponse(null, 202);
    }

    /**
     * @Route("/api/users/{user}/watchedMovies/{watchedMovie}", methods={"PATCH"});
     *
     * @param UserWatchedMovie          $watchedMovie
     * @param UpdateWatchedMovieRequest $request
     * @param WatchedMovieService       $watchedMovieService
     *
     * @throws \Exception
     *
     * @return JsonResponse
     */
    public function patchWatchedMoviesById(UserWatchedMovie $watchedMovie, UpdateWatchedMovieRequest $request, WatchedMovieService $watchedMovieService)
    {
        $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');

        if ($watchedMovie->getUser()->getId() !== $this->getUser()->getId()) {
            throw new AccessDeniedHttpException();
        }

        $watchedMovieDTO = $request->getWatchedMovieDTO();
        $watchedMovieService->updateUserWatchedMovie($watchedMovie, $watchedMovieDTO);

        $this->getDoctrine()->getManager()->flush();

        return new JsonResponse(null, 200);
    }

    /**
     * @Route("/api/users/{user}/watchedMovies/movie/{movieId}", methods={"PATCH"});
     *
     * @param int                       $movieId
     * @param UpdateWatchedMovieRequest $request
     * @param WatchedMovieService       $watchedMovieService
     *
     * @throws \Exception
     *
     * @return JsonResponse
     */
    public function patchWatchedMoviesByMovieId(int $movieId, UpdateWatchedMovieRequest $request, WatchedMovieService $watchedMovieService, WatchedMovieRepository $repository)
    {
        $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');

        /** @var $user User */
        $user = $this->getUser();

        if (null === $watchedMovie = $repository->findOneByMovieId($movieId, $user->getId())) {
            throw new NotFoundHttpException();
        }

        $watchedMovieDTO = $request->getWatchedMovieDTO();
        $watchedMovieService->updateUserWatchedMovie($watchedMovie, $watchedMovieDTO);

        $this->getDoctrine()->getManager()->flush();

        return new JsonResponse(null, 200);
    }

    /**
     * @Route("/api/users/{id<\d+>}/watchedMovies", methods={"GET"});
     *
     * @param Request         $request
     * @param User            $profileOwner
     * @param MovieRepository $repository
     *
     * @throws
     *
     * @return JsonResponse
     */
    public function getAll(Request $request, User $profileOwner, MovieRepository $repository)
    {
        $offset = (int) $request->get('offset', 0);
        $limit = $request->get('limit', null);

        $currentUser = $this->getUser();

        [$items, $ids, $count] = $repository->getAllWatchedMoviesByUserId($profileOwner, $currentUser);

        $collection = new CustomPaginatedCollection($items, $ids, $count, $offset, $limit);

        return $this->items($collection, MovieTransformer::list());
    }

    /**
     * @Route("/api/users/{user}/watchedMovies/{watchedMovieId}", methods={"DELETE"});
     *
     * @param int                    $watchedMovieId
     * @param WatchedMovieRepository $repository
     *
     * @return JsonResponse
     */
    public function deleteWatchedMovies(int $watchedMovieId, WatchedMovieRepository $repository)
    {
        $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');

        /** @var $currentUser User */
        $currentUser = $this->getUser();

        if (null === $watchedMovie = $repository->findOneById($watchedMovieId, $currentUser->getId())) {
            $watchedMovie = $repository->findOneByMovieId($watchedMovieId, $currentUser->getId());
        }

        if (null === $watchedMovie) {
            throw new NotFoundHttpException();
        }

        $this->getDoctrine()->getManager()->remove($watchedMovie);
        $this->getDoctrine()->getManager()->flush();

        return new JsonResponse(null, 202);
    }

    /**
     * @Route("/api/users/mergeWatchedMovies", methods={"POST"});
     *
     * @param MergeWatchedMoviesRequest $mergeWatchedMoviesRequest
     * @param WatchedMovieService       $watchedMovieService
     *
     * @throws \Exception
     *
     * @return JsonResponse
     */
    public function postMergeWatchedMovies(MergeWatchedMoviesRequest $mergeWatchedMoviesRequest, WatchedMovieService $watchedMovieService)
    {
        $this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
        $guestSessionRepository = $this->getDoctrine()->getRepository(GuestSession::class);
        $guestSession = $guestSessionRepository->findOneBy([
            'token' => $mergeWatchedMoviesRequest->get('token'),
        ]);

        /** @var $guestSession GuestSession|null */
        if ($guestSession === null) {
            throw new NotFoundHttpException('Guest session not found by provided token');
        }

        $watchedMovieService->mergeWatchedMovies($guestSession, $this->getUser());
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        return new JsonResponse(null, 202);
    }
}


1		<?php
2
3		namespace App\Users\Controller;
4
5		use App\Controller\BaseController;
6		use App\Guests\Entity\GuestSession;
7		use App\Movies\Repository\MovieRepository;
8		use App\Movies\Request\AddWatchedMovieRequest;
9		use App\Movies\Request\UpdateWatchedMovieRequest;
10		use App\Movies\Service\WatchedMovieService;
11		use App\Movies\Transformer\MovieTransformer;
12		use App\Pagination\CustomPaginatedCollection;
13		use App\Users\Entity\User;
14		use App\Users\Entity\UserWatchedMovie;
15		use App\Users\Repository\WatchedMovieRepository;
16		use App\Users\Request\MergeWatchedMoviesRequest;
17		use Symfony\Component\HttpFoundation\JsonResponse;
18		use Symfony\Component\HttpFoundation\Request;
19		use Symfony\Component\HttpKernel\Exception\AccessDeniedHttpException;
20		use Symfony\Component\HttpKernel\Exception\NotFoundHttpException;
21		use Symfony\Component\Routing\Annotation\Route;
22
23		class WatchedMovieController extends BaseController
24		{
25		/**
26		* @Route("/api/users/watchedMovies", methods={"POST"});
27		*
28		* @param AddWatchedMovieRequest $addWatchedMovieRequest
29		* @param Request $request
30		* @param WatchedMovieService $watchedMovieService
31		*
32		* @throws \Exception\|NotFoundHttpException
33		*
34		* @return JsonResponse
35		*/
36	6	public function postWatchedMovies(AddWatchedMovieRequest $addWatchedMovieRequest, Request $request, WatchedMovieService $watchedMovieService)
37		{
38	6	$this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
39
40	6	$watchedMovieDTO = $addWatchedMovieRequest->getWatchedMovieDTO();
41	6	$isMovieAdded = $watchedMovieService->addUserWatchedMovie($this->getUser(), $watchedMovieDTO, $request->getLocale());
		0 ignored issues – show Documentation introduced 2018-05-06 11:38 UTC by Report Bug Copy Issue Report `$this->getUser()` is of type `null\|object`, but the function expects a `object<App\Users\Entity\User>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
42
43	6	if ($isMovieAdded === false) {
44		throw new NotFoundHttpException('Movie not found by provided ID / TMDB ID');
45		}
46
47	6	return new JsonResponse(null, 202);
48		}
49
50		/**
51		* @Route("/api/users/{user}/watchedMovies/{watchedMovie}", methods={"PATCH"});
52		*
53		* @param UserWatchedMovie $watchedMovie
54		* @param UpdateWatchedMovieRequest $request
55		* @param WatchedMovieService $watchedMovieService
56		*
57		* @throws \Exception
58		*
59		* @return JsonResponse
60		*/
61	1	public function patchWatchedMoviesById(UserWatchedMovie $watchedMovie, UpdateWatchedMovieRequest $request, WatchedMovieService $watchedMovieService)
62		{
63	1	$this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
64
65	1	if ($watchedMovie->getUser()->getId() !== $this->getUser()->getId()) {
66		throw new AccessDeniedHttpException();
67		}
68
69	1	$watchedMovieDTO = $request->getWatchedMovieDTO();
70	1	$watchedMovieService->updateUserWatchedMovie($watchedMovie, $watchedMovieDTO);
71
72	1	$this->getDoctrine()->getManager()->flush();
73
74	1	return new JsonResponse(null, 200);
75		}
76
77		/**
78		* @Route("/api/users/{user}/watchedMovies/movie/{movieId}", methods={"PATCH"});
79		*
80		* @param int $movieId
81		* @param UpdateWatchedMovieRequest $request
82		* @param WatchedMovieService $watchedMovieService
83		*
84		* @throws \Exception
85		*
86		* @return JsonResponse
87		*/
88	1	public function patchWatchedMoviesByMovieId(int $movieId, UpdateWatchedMovieRequest $request, WatchedMovieService $watchedMovieService, WatchedMovieRepository $repository)
89		{
90	1	$this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
91
92		/** @var $user User */
93	1	$user = $this->getUser();
94
95	1	if (null === $watchedMovie = $repository->findOneByMovieId($movieId, $user->getId())) {
96		throw new NotFoundHttpException();
97		}
98
99	1	$watchedMovieDTO = $request->getWatchedMovieDTO();
100	1	$watchedMovieService->updateUserWatchedMovie($watchedMovie, $watchedMovieDTO);
101
102	1	$this->getDoctrine()->getManager()->flush();
103
104	1	return new JsonResponse(null, 200);
105		}
106
107		/**
108		* @Route("/api/users/{id<\d+>}/watchedMovies", methods={"GET"});
109		*
110		* @param Request $request
111		* @param User $profileOwner
112		* @param MovieRepository $repository
113		*
114		* @throws
115		*
116		* @return JsonResponse
117		*/
118	2	public function getAll(Request $request, User $profileOwner, MovieRepository $repository)
119		{
120	2	$offset = (int) $request->get('offset', 0);
121	2	$limit = $request->get('limit', null);
122
123	2	$currentUser = $this->getUser();
124
125	2	[$items, $ids, $count] = $repository->getAllWatchedMoviesByUserId($profileOwner, $currentUser);
		0 ignored issues – show Bug introduced 2018-10-19 18:58 UTC by Report Bug Copy Issue Report The variable `$items` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history... Bug introduced 2018-10-19 18:58 UTC by Report Bug Copy Issue Report The variable `$ids` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history... Bug introduced 2018-10-19 18:58 UTC by Report Bug Copy Issue Report The variable `$count` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
126	2	$collection = new CustomPaginatedCollection($items, $ids, $count, $offset, $limit);
127
128	2	return $this->items($collection, MovieTransformer::list());
129		}
130
131		/**
132		* @Route("/api/users/{user}/watchedMovies/{watchedMovieId}", methods={"DELETE"});
133		*
134		* @param int $watchedMovieId
135		* @param WatchedMovieRepository $repository
136		*
137		* @return JsonResponse
138		*/
139		public function deleteWatchedMovies(int $watchedMovieId, WatchedMovieRepository $repository)
140		{
141		$this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
142
143		/** @var $currentUser User */
144		$currentUser = $this->getUser();
145
146		if (null === $watchedMovie = $repository->findOneById($watchedMovieId, $currentUser->getId())) {
147		$watchedMovie = $repository->findOneByMovieId($watchedMovieId, $currentUser->getId());
148		}
149
150		if (null === $watchedMovie) {
151		throw new NotFoundHttpException();
152		}
153
154		$this->getDoctrine()->getManager()->remove($watchedMovie);
155		$this->getDoctrine()->getManager()->flush();
156
157		return new JsonResponse(null, 202);
158		}
159
160		/**
161		* @Route("/api/users/mergeWatchedMovies", methods={"POST"});
162		*
163		* @param MergeWatchedMoviesRequest $mergeWatchedMoviesRequest
164		* @param WatchedMovieService $watchedMovieService
165		*
166		* @throws \Exception
167		*
168		* @return JsonResponse
169		*/
170		public function postMergeWatchedMovies(MergeWatchedMoviesRequest $mergeWatchedMoviesRequest, WatchedMovieService $watchedMovieService)
171		{
172		$this->denyAccessUnlessGranted('IS_AUTHENTICATED_FULLY');
173		$guestSessionRepository = $this->getDoctrine()->getRepository(GuestSession::class);
174		$guestSession = $guestSessionRepository->findOneBy([
175		'token' => $mergeWatchedMoviesRequest->get('token'),
176		]);
177
178		/** @var $guestSession GuestSession\|null */
179		if ($guestSession === null) {
180		throw new NotFoundHttpException('Guest session not found by provided token');
181		}
182
183		$watchedMovieService->mergeWatchedMovies($guestSession, $this->getUser());
		0 ignored issues – show Documentation introduced 2018-05-06 11:38 UTC by Report Bug Copy Issue Report `$this->getUser()` is of type `null\|object`, but the function expects a `object<App\Users\Entity\User>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
184
185		return new JsonResponse(null, 202);
186		}
187		}
188

svbackend / my-art-lib

Push — master ( 0b4911...4851fa )

WatchedMovieController::postWatchedMovies() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like