Apps\Controller\Admin\Content\Boot

Complexity

Total Complexity

3

Size/Duplication

Total Lines	66
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	boot()	0	60	3

<?php

namespace Apps\Controller\Admin\Content;


use Apps\ActiveRecord\Content;
use Apps\ActiveRecord\ContentCategory;
use Apps\Controller\Admin\Main;
use Apps\Model\Admin\Main\AbstractSearchItem;
use Apps\Model\Admin\Main\CollectionSearchResults;
use Ffcms\Core\App;
use Ffcms\Core\Helper\Text;
use Ffcms\Core\Helper\Type\Str;
use Illuminate\Support\Collection;

/**
 * Trait Boot
 * @package Apps\Controller\Admin\Content
 */
trait Boot
{
    /**
     * Implement search boot features
     * @return void
     */
    public static function boot(): void
    {
        App::$Event->on(Main::SEARCH_EVENT_NAME, function ($model) {
            /** @var CollectionSearchResults $model */
            $limit = $model->getLimit();
            $query = $model->getQuery();

            // relevant search by string query
            $records = Content::search($query)
                ->take($limit)
                ->get();

            /** @var Content[]|Collection $records */
            $records->each(function($item) use ($model) {
                /** @var Content $item */
                $title = $item->getLocaled('title');
                $text = App::$Security->strip_tags($item->getLocaled('text'));
                $snippet = Text::snippet($text);

It seems like $text can also be of type array and array and null; however, parameter $text of Ffcms\Core\Helper\Text::snippet() does only seem to accept string, maybe add an additional type check?

                // prevent empty items
                if (Str::likeEmpty($title)) {

It seems like $title can also be of type array; however, parameter $string of Ffcms\Core\Helper\Type\Str::likeEmpty() does only seem to accept null|string, maybe add an additional type check?

                    return;
                }

                // initialize abstract response pattern
                $res = new AbstractSearchItem();
                $res->setTitle($title);

It seems like $title can also be of type array; however, parameter $value of Apps\Model\Front\Search\AbstractSearchResult::setTitle() does only seem to accept string, maybe add an additional type check?

                $res->setSnippet($snippet);
                $res->setDate($item->created_at);
                $res->setRelevance((int)$item->relevance);

The property relevance does not seem to exist on Apps\ActiveRecord\Content. Are you sure there is no database migration missing?

                $res->setUrl('content/update', [$item->id]);
                $res->setMarker('Content');

                $model->add($res);
            });
            // search in categories
            $records = ContentCategory::search($query)
                ->take($limit)
                ->get();

            /** @var ContentCategory[]|Collection $records */
            $records->each(function($item) use ($model) {
                /** @var ContentCategory $item */
                $title = $item->getLocaled('title');
                $text = App::$Security->strip_tags($item->getLocaled('description'));
                $snippet = Text::snippet($text);

It seems like $text can also be of type array and array and null; however, parameter $text of Ffcms\Core\Helper\Text::snippet() does only seem to accept string, maybe add an additional type check?

                // prevent empty items
                if (Str::likeEmpty($title)) {

It seems like $title can also be of type array; however, parameter $string of Ffcms\Core\Helper\Type\Str::likeEmpty() does only seem to accept null|string, maybe add an additional type check?

                    return;
                }

                // initialize abstract response pattern
                $res = new AbstractSearchItem();
                $res->setTitle($title);

It seems like $title can also be of type array; however, parameter $value of Apps\Model\Front\Search\AbstractSearchResult::setTitle() does only seem to accept string, maybe add an additional type check?

                $res->setSnippet($snippet);
                $res->setDate($item->created_at);
                $res->setRelevance((int)$item->relevance);

The property relevance does not exist on Apps\ActiveRecord\ContentCategory. Since you implemented __get, consider adding a @property annotation.

                $res->setUrl('content/categoryupdate', [$item->id]);
                $res->setMarker('Content');

                $model->add($res);
            });
        });
    }
}