SilverStripe\FullTextSearch\Solr\Writers\SolrSearchQueryWriterBasic

Complexity

Total Complexity

4

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	generateQueryString()	0	8	1
A	getComparisonConjunction()	0	3	1
A	getComparisonPolarity()	0	7	2

<?php

namespace SilverStripe\FullTextSearch\Solr\Writers;

use SilverStripe\FullTextSearch\Search\Criteria\SearchCriterion;
use SilverStripe\FullTextSearch\Search\Queries\AbstractSearchQueryWriter;

/**
 * Class SolrSearchQueryWriter_Basic
 * @package SilverStripe\FullTextSearch\Solr\Writers
 */
class SolrSearchQueryWriterBasic extends AbstractSearchQueryWriter
{
    /**
     * @var SearchCriterion $searchCriterion
     * @return string
     */
    public function generateQueryString(SearchCriterion $searchCriterion)
    {
        return sprintf(
            '%s(%s%s%s)',
            $this->getComparisonPolarity($searchCriterion->getComparison()),
            addslashes($searchCriterion->getTarget()),
            $this->getComparisonConjunction(),
            $searchCriterion->getQuoteValue($searchCriterion->getValue())

It seems like $searchCriterion->getValue() can also be of type array; however, parameter $value of SilverStripe\FullTextSearch\Search\Criteria\SearchCriterion::getQuoteValue() does only seem to accept string, maybe add an additional type check?

        );
    }

    /**
     * Is this a positive (+) or negative (-) Solr comparison.
     *
     * @param string $comparison
     * @return string
     */
    protected function getComparisonPolarity($comparison)
    {
        switch ($comparison) {
            case SearchCriterion::NOT_EQUAL:
                return '-';
            default:
                return '+';
        }
    }

    /**
     * Decide how we are comparing our left and right values.
     *
     * @return string
     */
    protected function getComparisonConjunction()
    {
        return ':';
    }
}