Apps\Controller\Admin\Content\Boot::boot()

Complexity

Conditions	3
Paths	1

Size

Total Lines	60
Code Lines	37

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    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);
            });
}