Atrapalo\Monolog\Formatter\ElasticsearchFormatter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	49
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	format()	0	10	1
A	formatBatch()	0	17	2

<?php

namespace Atrapalo\Monolog\Formatter;

use DateTime;
use Monolog\Formatter\NormalizerFormatter;

class ElasticsearchFormatter extends NormalizerFormatter
{
    /**
     * @var string
     */
    private $type;

    /**
     * @var string
     */
    private $index;

    public function __construct($index, $type)
    {
        parent::__construct(DateTime::ISO8601);

        $this->index = $index;
        $this->type = $type;
    }

    public function format(array $record)
    {
        $record = parent::format($record);

        return [
            'type'      => $this->type,
            'index'     => $this->index,
            'body'      => $record
        ];
    }

    public function formatBatch(array $records)
    {
        $bulk = ['body' => []];

        foreach ($records as $record) {
            $bulk['body'][] = [
                'index' => [
                    '_index' => $this->index,
                    '_type'  => $this->type,
                ]
            ];

            $bulk['body'][] = parent::format($record);

It seems like you call parent on a different method (format() instead of formatBatch()). Are you sure this is correct? If so, you might want to change this to $this->format().

        }

        return $bulk;
    }
}