RecentlyViewed\Models\Traits\CanBeViewed

Complexity

Total Complexity

5

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getRecentlyViewsLimit()	0	3	2
A	whereRecentlyViewedIn()	0	9	3

<?php

namespace RecentlyViewed\Models\Traits;

use Illuminate\Database\Eloquent\Builder;

/**
 * Trait CanBeViewed
 * @package RecentlyViewed\Models\Traits
 *
 */
trait CanBeViewed
{
    /**
     * @param  array  $values
     *
     * @return null|Builder
     */
    public function whereRecentlyViewedIn(array $values): ?Builder
    {
        $values = array_filter($values, fn ($v) => (is_int($v) || is_string($v)));

        if (count($values)) {
            return static::whereIn($this->getKeyName(), $values);

It seems like getKeyName() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        return null;
    }

    /**
     * Get recently viewed items count.
     *
     * @return int
     */
    public function getRecentlyViewsLimit(): int
    {
        return (int) (property_exists($this, 'recentlyViewsLimit') ? $this->recentlyViewsLimit : 10);
    }
}