Psonic\Commands\Search\SuggestCommand

Complexity

Total Complexity

2

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	10	2

<?php

namespace Psonic\Commands\Search;

use Psonic\Commands\Command;

final class SuggestCommand extends Command
{
    private $command    = 'SUGGEST';
    private $parameters = [];

    /**
     * SuggestCommand constructor.
     * @param string $collection
     * @param string $bucket
     * @param string $terms
     * @param null $limit

Are you sure the doc-type for parameter $limit is correct as it would always require null to be passed?

     */
    public function __construct(string $collection, string $bucket, string $terms, $limit = null)
    {
        $this->parameters = [
            'collection' => $collection,
            'bucket'     => $bucket,
            'terms'      => self::wrapInQuotes($terms),

The method Psonic\Commands\Command::wrapInQuotes() is not static, but was called statically.

            'limit'      => $limit ? "LIMIT($limit)" : null,

$limit is of type null, thus it always evaluated to false.

        ];

        parent::__construct($this->command, $this->parameters);
    }
}