Firesphere\SolrPermissions\Extensions\BaseIndexExtension

Complexity

Total Complexity

2

Size/Duplication

Total Lines	27
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	onBeforeSearch()	0	19	2

<?php


namespace Firesphere\SolrPermissions\Extensions;

use Firesphere\SolrSearch\Indexes\BaseIndex;
use Firesphere\SolrSearch\Queries\BaseQuery;
use Minimalcode\Search\Criteria;
use SilverStripe\Core\Extension;
use SilverStripe\Security\Security;
use Solarium\QueryType\Select\Query\Query;

/**
 * Class \Firesphere\SolrPermissions\Extensions\BaseIndexExtension
 *
 * Add Member specific view capabilities to the Index
 *
 * @package Firesphere\SolrPermissions\Extensions
 * @property BaseIndex|BaseIndexExtension $owner
 */
class BaseIndexExtension extends Extension
{
    /**
     * Before search, add the member view filtering
     *
     * @param BaseQuery $query
     * @param Query $clientQuery
     */
    public function onBeforeSearch($query, $clientQuery)
    {
        $member = Security::getCurrentUser();

        $filter = ['null'];

        if ($member) {

$member is of type SilverStripe\Security\Member, thus it always evaluated to true.

            $filter[] = sprintf('1-%s', $member->ID);
        }

        $query->addFilter('MemberView', $filter);

$filter of type array<integer,string> is incompatible with the type string expected by parameter $value of Firesphere\SolrSearch\Queries\BaseQuery::addFilter().

        $criteria = Criteria::where('MemberView')->in($filter);

        $clientQuery->createFilterQuery('MemberView')
            ->setQuery($criteria->getQuery());
        $filterQueries = $clientQuery->getFilterQueries();
        unset($filterQueries['ViewStatus']);
        $clientQuery->setFilterQueries($filterQueries);
    }
}