ONGR\ElasticsearchDSL\Aggregation\Metric\PercentilesAggregation

Complexity

Total Complexity

11

Size/Duplication

Total Lines	116
Duplicated Lines	23.28 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	9	9	1
A	getPercents()	0	4	1
A	setPercents()	0	6	1
A	getCompression()	0	4	1
A	setCompression()	0	6	1
A	getType()	0	4	1
A	getArray()	18	18	2
A	isRequiredParametersSet()	0	6	3

<?php

/*
 * This file is part of the ONGR package.
 *
 * (c) NFQ Technologies UAB <info@nfq.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace ONGR\ElasticsearchDSL\Aggregation\Metric;

use ONGR\ElasticsearchDSL\Aggregation\AbstractAggregation;
use ONGR\ElasticsearchDSL\Aggregation\Type\MetricTrait;
use ONGR\ElasticsearchDSL\ScriptAwareTrait;

/**
 * Class representing PercentilesAggregation.
 *
 * @link https://www.elastic.co/guide/en/elasticsearch/reference/current/search-aggregations-metrics-percentile-aggregation.html
 */
class PercentilesAggregation extends AbstractAggregation
{
    use MetricTrait;
    use ScriptAwareTrait;

    /**
     * @var array
     */
    private $percents;

    /**
     * @var int
     */
    private $compression;

    /**
     * Inner aggregations container init.
     *
     * @param string $name
     * @param string $field
     * @param array  $percents
     * @param string $script
     * @param int    $compression
     */
    public function __construct($name, $field = null, $percents = null, $script = null, $compression = null)

This method seems to be duplicated in your project.

    {
        parent::__construct($name);

        $this->setField($field);
        $this->setPercents($percents);

It seems like $percents defined by parameter $percents on line 47 can also be of type null; however, ONGR\ElasticsearchDSL\Aggregation\Metric\PercentilesAggregation::setPercents() does only seem to accept array, maybe add an additional type check?

        $this->setScript($script);
        $this->setCompression($compression);
    }

    /**
     * @return array
     */
    public function getPercents()
    {
        return $this->percents;
    }

    /**
     * @param array $percents
     *
     * @return $this
     */
    public function setPercents($percents)
    {
        $this->percents = $percents;

        return $this;
    }

    /**
     * @return int
     */
    public function getCompression()
    {
        return $this->compression;
    }

    /**
     * @param int $compression
     *
     * @return $this
     */
    public function setCompression($compression)
    {
        $this->compression = $compression;

        return $this;
    }

    /**
     * {@inheritdoc}
     */
    public function getType()
    {
        return 'percentiles';
    }

    /**
     * {@inheritdoc}
     */
    public function getArray()

This method seems to be duplicated in your project.

    {
        $out = array_filter(
            [
                'compression' => $this->getCompression(),
                'percents' => $this->getPercents(),
                'field' => $this->getField(),
                'script' => $this->getScript(),
            ],
            function ($val) {
                return ($val || is_numeric($val));
            }
        );

        $this->isRequiredParametersSet($out);

        return $out;
    }

    /**
     * @param array $a
     *
     * @throws \LogicException
     */
    private function isRequiredParametersSet($a)
    {
        if (!array_key_exists('field', $a) && !array_key_exists('script', $a)) {
            throw new \LogicException('Percentiles aggregation must have field or script set.');
        }
    }
}