ShopSearch::get_searchable_classes() - Code Metrics - Inspection of "Merge pull request #11 from helpfulrobot/convert-t..." - markguinn/silverstripe-shop-search - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 08af57...01b1ac )

by Mark

created 2015-12-21 11:02 UTC

ShopSearch::get_searchable_classes() C

↳ Parent: ShopSearch

Complexity

Conditions	8
Paths	9

Size

Total Lines	22
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	22
rs	6.6038
cc	8
eloc	12
nc	9
nop	0

<?php
/**
 * Fulltext search index for shop buyables
 *
 * @author Mark Guinn <[email protected]>
 * @date 08.29.2013
 * @package shop_search
 */
class ShopSearch extends Object
namespace YourVendor;

class YourClass { }
{
    const FACET_TYPE_LINK       = 'link';
    const FACET_TYPE_CHECKBOX   = 'checkbox';
    const FACET_TYPE_RANGE      = 'range';

    /** @var string - class name of adapter class to use */
    private static $adapter_class = 'ShopSearchSimple';


    /** @var array - these classes will be added to the index - e.g. Category, Page, etc. */
    private static $searchable = array();


    /** @var bool - if true, all buyable models will be added to the index automatically  */
    private static $buyables_are_searchable = true;


    /** @var int - size of paging in the search */
    private static $page_size = 10;


    /** @var bool */
    private static $suggest_enabled = true;


    /** @var int - how many suggestions to provide */
    private static $suggest_limit = 5;


    /** @var bool */
    private static $search_as_you_type_enabled = true;


    /** @var int - how may sayt (search-as-you-type) entries to provide */
    private static $sayt_limit = 5;


    /** @var bool - automatically create facets for static attributes */
    private static $auto_facet_attributes = false;


    /** @var string - optionally, a different template to run ajax results through (sans-Page.ss) */
    private static $ajax_results_template = '';


    /** @var string - these allow you to use different querystring params in you need to */
    private static $qs_query         = 'q';

    private static $qs_filters       = 'f';

    private static $qs_parent_search = '__ps';

    private static $qs_title         = '__t';

    private static $qs_source        = '__src'; // used to log searches from search-as-you-type

    private static $qs_sort          = 'sort';


    /** @var array - I'm leaving this particularly bare b/c with config merging it's a pain to remove items */
    private static $sort_options = array(

        'score desc'            => 'Relevance',
//		'SiteTree_Title asc'    => 'Alphabetical (A-Z)',

//		'SiteTree_Title dsc'    => 'Alphabetical (Z-A)',
    );

    /**
     * @var array - default search facets (price, category, etc)
     *   Key    field name - e.g. Price - can be a VirtualFieldIndex field
     *   Value  facet label - e.g. Search By Category - if the value is a relation or returns an array or
     *          list all values will be faceted individually
     *          NOTE: this can also be another array with keys: Label, Type, and Values (for checkbox only)
     */
    private static $facets = array();


    /** @var array - field definition for Solr only */
    private static $solr_fulltext_fields = array();


    /** @var array - field definition for Solr only */
    private static $solr_filter_fields = array();


    /** @var string - if present, will create a copy of SiteTree_Title that's suited for alpha sorting */
    private static $solr_title_sort_field = '';


    /**
     * @var string - If present, everything matching the following regex will be removed from
     *               keyword search queries before passing to the search adapter.
     */
    private static $keyword_filter_regex = '/[^a-zA-Z0-9\s\-]/';



    /**
     * @return array
     */
    public static function get_searchable_classes()
    {
        // First get any explicitly declared searchable classes
        $searchable = Config::inst()->get('ShopSearch', 'searchable');
        if (is_string($searchable) && strlen($searchable) > 0) {
            $searchable = array($searchable);
        } elseif (!is_array($searchable)) {
            $searchable = array();
        }

        // Add in buyables automatically if asked
        if (Config::inst()->get('ShopSearch', 'buyables_are_searchable')) {
            $buyables = SS_ClassLoader::instance()->getManifest()->getImplementorsOf('Buyable');
            if (is_array($buyables) && count($buyables) > 0) {
                foreach ($buyables as $c) {
                    $searchable[] = $c;
                }
            }
        }

        return array_unique($searchable);
    }

    /**
     * Returns an array of categories suitable for a dropdown menu
     * TODO: cache this
     *
     * @param int $parentID [optional]
     * @param string $prefix [optional]
     * @param int $maxDepth [optional]
     * @return array
     * @static
     */
    public static function get_category_hierarchy($parentID = 0, $prefix = '', $maxDepth = 999)
    {
        $out = array();
        $cats = ProductCategory::get()
            ->filter(array(
                'ParentID'      => $parentID,
                'ShowInMenus'   => 1,
            ))
            ->sort('Sort');

        // If there is a single parent category (usually "Products" or something), we
        // probably don't want that in the hierarchy.
        if ($parentID == 0 && $cats->count() == 1) {
            return self::get_category_hierarchy($cats->first()->ID, $prefix, $maxDepth);
        }

        foreach ($cats as $cat) {
            $out[$cat->ID] = $prefix . $cat->Title;
            if ($maxDepth > 1) {
                $out += self::get_category_hierarchy($cat->ID, $prefix . $cat->Title . ' > ', $maxDepth - 1);
            }
        }

        return $out;
    }

    /**
     * @return ShopSearchAdapter
     */
    public static function adapter()
    {
        $adapterClass = Config::inst()->get('ShopSearch', 'adapter_class');
        return Injector::inst()->get($adapterClass);
    }

    /**
     * @return ShopSearch
     */
    public static function inst()
    {
        return Injector::inst()->get('ShopSearch');
    }

    /**
     * The result will contain at least the following:
     *      Matches - SS_List of results
     *      TotalMatches - total # of results, unlimited
     *      Query - query string
     * Also saves a log record.
     *
     * @param array $vars
     * @param bool $logSearch [optional]
     * @param bool $useFacets [optional]
     * @param int $start [optional]
     * @param int $limit [optional]
     * @return ArrayData
     */
    public function search(array $vars, $logSearch=true, $useFacets=true, $start=-1, $limit=-1)
    {
        $qs_q   = $this->config()->get('qs_query');
        $qs_f   = $this->config()->get('qs_filters');
        $qs_ps  = $this->config()->get('qs_parent_search');
        $qs_t   = $this->config()->get('qs_title');
        $qs_sort= $this->config()->get('qs_sort');
        if ($limit < 0) {
            $limit  = $this->config()->get('page_size');
        }
        if ($start < 0) {
            $start  = !empty($vars['start']) ? (int)$vars['start'] : 0;
        } // as far as i can see, fulltextsearch hard codes 'start'
        $facets = $useFacets ? $this->config()->get('facets') : array();
        if (!is_array($facets)) {
            $facets = array();
        }
        if (empty($limit)) {
            $limit = -1;
        }

        // figure out and scrub the sort
        $sortOptions = $this->config()->get('sort_options');
        $sort        = !empty($vars[$qs_sort]) ? $vars[$qs_sort] : '';
        if (!isset($sortOptions[$sort])) {
            $sort    = current(array_keys($sortOptions));
        }

        // figure out and scrub the filters
        $filters  = !empty($vars[$qs_f]) ? FacetHelper::inst()->scrubFilters($vars[$qs_f]) : array();

        // do the search
        $keywords = !empty($vars[$qs_q]) ? $vars[$qs_q] : '';
        if ($keywordRegex = $this->config()->get('keyword_filter_regex')) {
            $keywords = preg_replace($keywordRegex, '', $keywords);
        }
        $results  = self::adapter()->searchFromVars($keywords, $filters, $facets, $start, $limit, $sort);

        // massage the results a bit
        if (!empty($keywords) && !$results->hasValue('Query')) {
            $results->Query = $keywords;
        }
        if (!empty($filters) && !$results->hasValue('Filters')) {
            $results->Filters = new ArrayData($filters);
        }
        if (!$results->hasValue('Sort')) {
            $results->Sort = $sort;
        }
        if (!$results->hasValue('TotalMatches')) {
            $results->TotalMatches = $results->Matches->hasMethod('getTotalItems')
                ? $results->Matches->getTotalItems()
                : $results->Matches->count();
        }

        // for some types of facets, update the state
        if ($results->hasValue('Facets')) {
            FacetHelper::inst()->transformHierarchies($results->Facets);
            FacetHelper::inst()->updateFacetState($results->Facets, $filters);
        }

        // make a hash of the search so we can know if we've already logged it this session
        $loggedFilters = !empty($filters) ? json_encode($filters) : null;
        $loggedQuery   = strtolower($results->Query);
//		$searchHash    = md5($loggedFilters . $loggedQuery);

//		$sessSearches  = Session::get('loggedSearches');
//		if (!is_array($sessSearches)) $sessSearches = array();
//		Debug::dump($searchHash, $sessSearches);

        // save the log record
        if ($start == 0 && $logSearch && (!empty($keywords) || !empty($filters))) { // && !in_array($searchHash, $sessSearches)) {

            $log = SearchLog::create(array(
                'Query'         => $loggedQuery,
                'Title'         => !empty($vars[$qs_t]) ? $vars[$qs_t] : '',
                'Link'          => Controller::curr()->getRequest()->getURL(true), // I'm not 100% happy with this, but can't think of a better way
                'NumResults'    => $results->TotalMatches,
                'MemberID'      => Member::currentUserID(),
                'Filters'       => $loggedFilters,
                'ParentSearchID'=> !empty($vars[$qs_ps]) ? $vars[$qs_ps] : 0,
            ));
            $log->write();
            $results->SearchLogID = $log->ID;
            $results->SearchBreadcrumbs = $log->getBreadcrumbs();

//			$sessSearches[] = $searchHash;

//			Session::set('loggedSearches', $sessSearches);
        }

        return $results;
    }

    /**
     * @param string $str
     * @return SS_Query
     */
    public function getSuggestQuery($str='')
    {
        $hasResults = 'CASE WHEN max("SearchLog"."NumResults") > 0 THEN 1 ELSE 0 END';
        $searchCount = 'count(distinct "SearchLog"."ID")';
        $q = new SQLQuery();

        $q = $q->setSelect('"SearchLog"."Query"')
            // TODO: what to do with filter?
            ->selectField($searchCount, 'SearchCount')
            ->selectField('max("SearchLog"."Created")', 'LastSearch')
            ->selectField('max("SearchLog"."NumResults")', 'NumResults')
            ->selectField($hasResults, 'HasResults')
            ->setFrom('"SearchLog"')
            ->setGroupBy('"SearchLog"."Query"')
            ->setOrderBy(array(
                "$hasResults DESC",
                "$searchCount DESC"
            ))
            ->setLimit(Config::inst()->get('ShopSearch', 'suggest_limit'))
        ;

        if (strlen($str) > 0) {
            $q = $q->addWhere(sprintf('"SearchLog"."Query" LIKE \'%%%s%%\'', Convert::raw2sql($str)));
        }

        return $q;
    }


    /**
     * @param string $str
     * @return array
     */
    public function suggest($str='')
    {
        $adapter = self::adapter();
        if ($adapter->hasMethod('suggest')) {
            return $adapter->suggest($str);

        } else {
            return $this->getSuggestQuery($str)->execute()->column('Query');
        }
    }


    /**
     * Returns an array that can be made into json and passed to the controller
     * containing both term suggestions and a few product matches.
     *
     * @param array $searchVars
     * @return array
     */
    public function suggestWithResults(array $searchVars)
    {
        $qs_q       = $this->config()->get('qs_query');
        $qs_f       = $this->config()->get('qs_filters');
        $keywords   = !empty($searchVars[$qs_q]) ? $searchVars[$qs_q] : '';
        $filters    = !empty($searchVars[$qs_f]) ? $searchVars[$qs_f] : array();

        $adapter = self::adapter();

        // get suggestions and product list from the adapter
        if ($adapter->hasMethod('suggestWithResults')) {
            $results = $adapter->suggestWithResults($keywords, $filters);
        } else {
            $limit      = (int)ShopSearch::config()->sayt_limit;
            $search     = self::adapter()->searchFromVars($keywords, $filters, array(), 0, $limit, 'Popularity DESC');
            //$search     = ShopSearch::inst()->search($searchVars, false, false, 0, $limit);


            $results = array(
                'products'      => $search->Matches,
                'suggestions'   => $this->suggest($keywords),
            );
        }

        // the adapter just gave us a list of products, which we need to process a little further
        if (!empty($results['products'])) {
            // this gets encoded into the product links
            $searchVars['total'] = $results['products']->hasMethod('getTotalItems')
                ? $results['products']->getTotalItems()
                : $results['products']->count();

            $products   = array();
            foreach ($results['products'] as $prod) {
                if (!$prod || !$prod->exists()) {
                    continue;
                }
                $img = $prod->hasMethod('ProductImage') ? $prod->ProductImage() : $prod->Image();
                $thumb = ($img && $img->exists()) ? $img->getThumbnail() : null;

                $json = array(
                    'link'  => $prod->Link() . '?' . ShopSearch::config()->qs_source . '=' . urlencode(base64_encode(json_encode($searchVars))),
                    'title' => $prod->Title,
                    'desc'  => $prod->obj('Content')->Summary(),
                    'thumb' => $thumb ? $thumb->Link() : '',
                    'price' => $prod->obj('Price')->Nice(),
                );

                if ($prod->hasExtension('HasPromotionalPricing') && $prod->hasValidPromotion()) {
                    $json['original_price'] = $prod->getOriginalPrice()->Nice();
                }

                $products[] = $json;
            }

            // replace the list of product objects with json
            $results['products'] = $products;
        }

        $this->extend('updateSuggestWithResults', $results, $keywords, $filters);

        return $results;
    }
}


1			<?php
2			/**
3			* Fulltext search index for shop buyables
4			*
5			* @author Mark Guinn <[email protected]>
6			* @date 08.29.2013
7			* @package shop_search
8			*/
9			class ShopSearch extends Object
			0 ignored issues – show Coding Style Compatibility introduced 2015-12-10 04:10 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
10			{
11			const FACET_TYPE_LINK = 'link';
12			const FACET_TYPE_CHECKBOX = 'checkbox';
13			const FACET_TYPE_RANGE = 'range';
14
15			/** @var string - class name of adapter class to use */
16			private static $adapter_class = 'ShopSearchSimple';
			0 ignored issues – show Unused Code introduced 2015-12-10 04:10 UTC by Report Bug Copy Issue Report The property `$adapter_class` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
17
18			/** @var array - these classes will be added to the index - e.g. Category, Page, etc. */
19			private static $searchable = array();
			0 ignored issues – show Unused Code introduced 2015-12-10 04:10 UTC by Report Bug Copy Issue Report The property `$searchable` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
20
21			/** @var bool - if true, all buyable models will be added to the index automatically */
22			private static $buyables_are_searchable = true;
			0 ignored issues – show Unused Code introduced 2015-12-10 04:10 UTC by Report Bug Copy Issue Report The property `$buyables_are_searchable` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
23
24			/** @var int - size of paging in the search */
25			private static $page_size = 10;
			0 ignored issues – show Unused Code introduced 2015-12-10 04:10 UTC by Report Bug Copy Issue Report The property `$page_size` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
26
27			/** @var bool */
28			private static $suggest_enabled = true;
			0 ignored issues – show Unused Code introduced 2015-12-10 04:10 UTC by Report Bug Copy Issue Report The property `$suggest_enabled` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
29
30			/** @var int - how many suggestions to provide */
31			private static $suggest_limit = 5;
			0 ignored issues – show Unused Code introduced 2015-12-10 04:10 UTC by Report Bug Copy Issue Report The property `$suggest_limit` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
32
33			/** @var bool */

markguinn / silverstripe-shop-search

Push — master ( 08af57...01b1ac )

ShopSearch::get_searchable_classes() C

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like