dev-think-one / laravel-recently-viewed

src/Models/Traits/CanBeViewed.php

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