Solr\Paginator\Adapter\SolrAdapter

Complexity

Total Complexity

6

Size/Duplication

Total Lines	102
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	3
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	7	1
A	getItems()	0	7	1
A	count()	0	5	1
A	getResponse()	0	16	3

<?php
/**
 * YAWIK
 *
 * @copyright (c) 2013 - 2016 Cross Solution (http://cross-solution.de)
 * @license   MIT
 */

namespace Solr\Paginator\Adapter;

use Solr\Exception\ServerException;
use Solr\Filter\AbstractPaginationQuery;
use Zend\Paginator\Adapter\AdapterInterface;
use Zend\Stdlib\Parameters;
use Solr\Bridge\ResultConverter;

class SolrAdapter implements AdapterInterface
{
    /**
     * @var \SolrClient
     */
    protected $client;

    /**
     * @var Parameters
     */
    protected $params;

    /**
     * @var int
     */
    protected $count;

    /**
     * @var \SolrQuery
     */
    protected $query;

    /**
     * Store current query response from solr server
     *
     * @var \SolrQueryResponse
     */
    protected $response;

    /**
     * @var AbstractPaginationQuery
     */
    protected $filter;

    /**
     * @var ResultConverter
     */
    protected $resultConverter;

    /**
     * SolrAdapter constructor.
     *
     * @param   \SolrClient                 $client
     * @param   AbstractPaginationQuery     $filter
     * @param   ResultConverter             $resultConverter
     * @param   array                       $params
     */
    public function __construct($client,$filter,$resultConverter,$params=array())
    {
        $this->client           = $client;
        $this->filter           = $filter;
        $this->resultConverter  = $resultConverter;
        $this->params           = $params;

It seems like $params of type array is incompatible with the declared type object<Zend\Stdlib\Parameters> of property $params.

    }
    
    /**
     * @inheritdoc
     */
    public function getItems($offset, $itemCountPerPage)
    {
        return $this->resultConverter->convert(
            $this->filter,
            $this->getResponse($offset,$itemCountPerPage)
        );
    }

    /**
     * @inheritdoc
     * @return  mixed
     * @throws \Exception
     */
    public function count()
    {
        $response = $this->getResponse()->getArrayResponse();
        return $response['response']['numFound'];
    }

    /**
     * Process query into server
     *
     * @param   int     $offset
     * @param   int     $itemCountPerPage
     * @return  \SolrQueryResponse
     * @throws  \Exception
     */
    protected function getResponse($offset=0,$itemCountPerPage=5)
    {
        if(!is_object($this->response)){
            $query = new \SolrQuery();
            $query = $this->filter->filter($this->params,$query);

The call to AbstractPaginationQuery::filter() has too many arguments starting with $query.

            $query->setStart($offset);
            $query->setRows($itemCountPerPage);
            try{
                $this->response = $this->client->query($query);
            }catch (\Exception $e){
                $message = 'Failed to process query';
                throw new ServerException($message,$e->getCode(),$e);
            }
        }
        return $this->response;
    }
}