CalendarController::userEventsAction() - Code Metrics - Inspection of "[WIP] Alpha1" - geekhub-php/serve-seattle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#30)

by nonanerz

created 2017-03-27 06:01 UTC

CalendarController::userEventsAction() A

↳ Parent: CalendarController

Complexity

Conditions	4
Paths	6

Size

Total Lines	19
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	19
rs	9.2
c	0
b	0
f	0
cc	4
eloc	13
nc	6
nop	0

<?php

namespace AppBundle\Controller\Api;

use AppBundle\Entity\DTO\DtoEvent;
use AppBundle\Entity\Event;
use AppBundle\Exception\JsonHttpException;
use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
use Symfony\Bundle\FrameworkBundle\Controller\Controller;
use Symfony\Component\HttpFoundation\JsonResponse;
use Symfony\Component\Routing\Annotation\Route;

class CalendarController extends Controller
{
    /**
     * @Route("/events")
     * @Method({"GET"})
     */
    public function userEventsAction()
    {
        $user = $this->getUser();
        $events = $user->getEvents();
        $calendar = $this->get('app.google_calendar');
        $googleEvents = [];
        foreach ($events as $event) {
            $googleEvents[] = $calendar
                ->getEventById($event->getGoogleId());
        }
        $events = [];
        foreach ($googleEvents as $event) {
            if ($event) {
                $events[] = new DtoEvent($event);
            }
        }

        return new JsonResponse(['events' => $events]);
    }

    /**
     * @Route("/events/{id}")
     * @Method("GET")
     */
    public function singleEventAction($id)
    {
        /** @var Event $event */
        $event = $this->getDoctrine()->getRepository('AppBundle:Event')
            ->findByGoogleId($id);
        $user = $event->getUsers()->first();

        if (!$user) {
            throw new JsonHttpException(404, 'User not found.');
        }
        $googleEvent = $this->get('app.google_calendar')
            ->getEventById($id);
        $event = new DtoEvent($googleEvent);

        return new JsonResponse(['event' => $event]);
    }
}


1			<?php
2
3			namespace AppBundle\Controller\Api;
4
5			use AppBundle\Entity\DTO\DtoEvent;
6			use AppBundle\Entity\Event;
7			use AppBundle\Exception\JsonHttpException;
8			use Sensio\Bundle\FrameworkExtraBundle\Configuration\Method;
9			use Symfony\Bundle\FrameworkBundle\Controller\Controller;
10			use Symfony\Component\HttpFoundation\JsonResponse;
11			use Symfony\Component\Routing\Annotation\Route;
12
13			class CalendarController extends Controller
14			{
15			/**
16			* @Route("/events")
17			* @Method({"GET"})
18			*/
19			public function userEventsAction()
20			{
21			$user = $this->getUser();
22			$events = $user->getEvents();
23			$calendar = $this->get('app.google_calendar');
24			$googleEvents = [];
25			foreach ($events as $event) {
26			$googleEvents[] = $calendar
27			->getEventById($event->getGoogleId());
28			}
29			$events = [];
30			foreach ($googleEvents as $event) {
31			if ($event) {
32			$events[] = new DtoEvent($event);
33			}
34			}
35
36			return new JsonResponse(['events' => $events]);
37			}
38
39			/**
40			* @Route("/events/{id}")
41			* @Method("GET")
42			*/
43			public function singleEventAction($id)
44			{
45			/** @var Event $event */
46			$event = $this->getDoctrine()->getRepository('AppBundle:Event')
47			->findByGoogleId($id);
48			$user = $event->getUsers()->first();
			0 ignored issues – show Bug introduced 2017-03-24 21:34 UTC by Report Bug Copy Issue Report The method `first` cannot be called on `$event->getUsers()` (of type `array`). Methods can only be called on objects. This check looks for methods being called on variables that have been inferred to never be objects. Loading history...
49			if (!$user) {
50			throw new JsonHttpException(404, 'User not found.');
51			}
52			$googleEvent = $this->get('app.google_calendar')
53			->getEventById($id);
54			$event = new DtoEvent($googleEvent);
55
56			return new JsonResponse(['event' => $event]);
57			}
58			}
59

geekhub-php / serve-seattle

Pull Request — master (#30)

CalendarController::userEventsAction() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like