Zicht\Bundle\UrlBundle\Logging\Logging

Complexity

Total Complexity

3

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	3
Bugs	1	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	createLog()	0	9	1
A	flush()	0	4	1

<?php
/**
 * @author Gerard van Helden <gerard@zicht.nl>
 * @copyright Zicht Online <http://zicht.nl>
 */
namespace Zicht\Bundle\UrlBundle\Logging;

use Doctrine\ORM\EntityManager;
use Symfony\Component\HttpFoundation\Request;
use Zicht\Bundle\UrlBundle\Entity\ErrorLog;

/**
 * Keeps a database log of URL issues, such as 404 and 500 errors
 */
class Logging
{
    /**
     * Constructor
     *
     * @param \Doctrine\ORM\EntityManager $manager
     */
    public function __construct(EntityManager $manager)
    {
        $this->manager = $manager;

The property manager does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }


    /**
     * Create a log entry for the passed request.
     *
     * @param \Symfony\Component\HttpFoundation\Request $request
     * @param string $message
     * @return \Zicht\Bundle\UrlBundle\Entity\ErrorLog
     */
    public function createLog(Request $request, $message)
    {
        return new ErrorLog(
            $message,
            new \DateTime(),
            $request->headers->get('referer', null),
            $request->headers->get('user-agent', null),
            $request->getClientIp(),
            $request->getRequestUri()
        );
    }


    /**
     * Persist the log and flush the manager.
     *
     * @param ErrorLog $entry
     * @return void
     */
    public function flush($entry)
    {
        $this->manager->persist($entry);
        $this->manager->flush($entry);
    }
}