PiouPiou\RibsAdminBundle\EventListener\ExceptionListener

Complexity

Total Complexity

7

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	4
A	onKernelException()	0	26	3

<?php

namespace PiouPiou\RibsAdminBundle\EventListener;

use PiouPiou\RibsAdminBundle\Entity\User;
use Symfony\Component\DependencyInjection\ParameterBag\ParameterBagInterface;
use Symfony\Component\HttpKernel\Event\ExceptionEvent;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;

class ExceptionListener
{
    /**
     * @var ParameterBagInterface
     */
    private $paramter;

    /**
     * @var User
     */
    private $user;

    public function __construct(ParameterBagInterface $parameterBag, TokenStorageInterface $tokenStorage)
    {
        $this->paramter = $parameterBag;
        if ($tokenStorage->getToken() && is_object($tokenStorage->getToken()->getUser()) && $tokenStorage->getToken()->getUser()->getUser()) {

The method getUser() does not exist on Stringable.

The method getUser() does not exist on Symfony\Component\Security\Core\User\UserInterface. Did you maybe mean getUsername()?

            $this->user = $tokenStorage->getToken()->getUser()->getUser();
        }
    }

    public function onKernelException(ExceptionEvent $event)
    {
        $slack_webhook = $this->paramter->get('ribs_admin.slack_webhook');
        if ($slack_webhook) {
            $data = array();
            $data['channel'] = '#errors';
            $data['username'] = $_SERVER['HTTP_HOST'];
            $data['text'] = "• *Erreur* : " . strip_tags($event->getThrowable()->getMessage());
            $data['text'] .= "\n• *Erreur File* : " . strip_tags($event->getThrowable()->getFile()) . " at line :" . strip_tags($event->getThrowable()->getLine());
            $data['text'] .= "\n• *URL* : " . $event->getRequest()->getUri();

            if ($this->user) {
                $data['text'] .= "\n• *Utilisateur* : " . $this->user . " with id : " . $this->user->getId();
            }

            $data['unfurl_links'] = false;
            $data_json = json_encode($data);

            $ch = curl_init();
            curl_setopt($ch, CURLOPT_URL, $slack_webhook);

It seems like $ch can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

            curl_setopt($ch, CURLOPT_POST, 1);
            curl_setopt($ch, CURLOPT_HTTPHEADER, array('Content-Type: application/json','Content-Length: ' . strlen($data_json)));
            curl_setopt($ch, CURLOPT_POSTFIELDS, $data_json);
            curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
            curl_exec($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

            curl_close($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

        }
    }
}