Piou-piou / RibsAdminBundle

Service/UserLog.php

<?php

namespace PiouPiou\RibsAdminBundle\Service;

use Doctrine\ORM\EntityManagerInterface;
use PiouPiou\RibsAdminBundle\Entity\UserLogs;
use Symfony\Component\HttpKernel\Event\RequestEvent;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;

class UserLog
{
    /**
     * @var TokenStorageInterface
     */
    private $token_storage;

    /**
     * @var EntityManagerInterface a
     */
    private $em;

    public function __construct(TokenStorageInterface $token_storage, EntityManagerInterface $em)
    {
        $this->token_storage = $token_storage;
        $this->em = $em;
    }

    /**
     * @param RequestEvent $request_event
     */
    public function onKernelRequest(RequestEvent $request_event)
    {
        if ($request_event->isMasterRequest()) {
            $user = null;
            if ($this->token_storage->getToken() && is_object($this->token_storage->getToken()->getUser()) && $this->token_storage->getToken()->getUser()->getUser()) {
                $user = $this->token_storage->getToken()->getUser()->getUser();
            }
            $request = $request_event->getRequest();
            $route = $request->get("_route");

            if (in_array($route, ["_profiler", "_profiler_search_bar", "_wdt", "ribsadmin_userlogs", "ribsadmin_userlogs_show"])) {
                return;
            }

            if ($user) {
                $user_log = new UserLogs();
                $user_log->setMethod($request->getMethod());
                $user_log->setUser($user);
                $user_log->setRoute($request->get("_route"));

It seems like $request->get('_route') can also be of type null; however, parameter $route of PiouPiou\RibsAdminBundle\Entity\UserLogs::setRoute() does only seem to accept string, maybe add an additional type check?

                $user_log->setUrl($request->getRequestUri());
                $user_log->setFullUrl($request->getUri());
                $user_log->setRequestFormat($request->getRequestFormat());

It seems like $request->getRequestFormat() can also be of type null; however, parameter $request_format of PiouPiou\RibsAdminBundle\Entity\UserLogs::setRequestFormat() does only seem to accept string, maybe add an additional type check?

                $user_log->setRequestParameters($request->request->all());
                $this->em->persist($user_log);
                $this->em->flush();
            }
        }
    }
}