Hubph\Internal\EventLogger

Complexity

Total Complexity

8

Size/Duplication

Total Lines	65
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	start()	0	4	1
A	stop()	0	3	1
A	log()	0	6	1
A	writeHeader()	0	6	2
A	getHeader()	0	8	1
A	getEntry()	0	13	1

<?php

namespace Hubph\Internal;

use Hubph\EventLoggerInterface;

class EventLogger implements EventLoggerInterface
{
    protected $filename;

    /**
     * EventLogger constructor
     */
    public function __construct($filename)
    {
        $this->filename = $filename;
    }

    public function start()
    {
        @unlink($this->filename);

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

    }

    public function stop()
    {
    }

    /**
     * Write an event into the event log
     * @param string $event_name
     * @param array $args
     * @param array $params
     * @param array $response
     */
    public function log($event_name, $args, $params, $response)
    {
        $this->writeHeader();
        $entry = $this->getEntry($event_name, $args, $params, $response);
        file_put_contents($this->filename, $entry, FILE_APPEND);
    }

    protected function writeHeader()
    {
        if (!file_exists($this->filename)) {
            file_put_contents($this->filename, $this->getHeader());
        }
    }

    protected function getHeader()
    {
        return <<<EOT
# Event log
# ---------

EOT;
    }

    protected function getEntry($event_name, $args, $params, $response)
    {
        $args_string = json_encode($args);
        $params_string = json_encode($params);
        $response_string = json_encode($response);

        return <<<EOT
name    : $event_name
args    : $args_string
params  : $params_string
response: $response_string
EOT;
    }
}