WebDevEtc\BlogEtc\Requests\Traits\HasCategoriesTrait

Complexity

Total Complexity

3

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Importance

Changes	4
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	categories()	0	14	3

<?php

namespace WebDevEtc\BlogEtc\Requests\Traits;

use WebDevEtc\BlogEtc\Models\Category;

/**
 * Class HasCategoriesTrait.
 */
trait HasCategoriesTrait
{
    /**
     * If $_GET['category'] slugs were submitted, then it should return an array of the IDs.
     *
     * @return array
     */
    public function categories()
    {
        if (!$this->get('category') || !is_array($this->get('category'))) {

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

            return [];
        }

        //$this->get("category") is an array of category SLUGs and we need IDs

        // check they are valid, return the IDs
        // limit to 1000 ... just in case someone submits with too many for the web server. No error is given if they submit more than 1k.
        $vals = Category::whereIn('id', array_keys($this->get('category')))->select('id')->limit(1000)->get();
        $vals = array_values($vals->pluck('id')->toArray());

        return $vals;
    }
}