A17\Twill\Repositories\Behaviors\HandleNesting

Complexity

Total Complexity

4

Size/Duplication

Total Lines	33
Duplicated Lines	0 %

Importance

Changes	3
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	setNewOrder()	0	4	1
A	forNestedSlug()	0	11	3

<?php

namespace A17\Twill\Repositories\Behaviors;

use A17\Twill\Jobs\ReorderNestedModuleItems;

trait HandleNesting
{
    /**
     * The Laravel queue name to be used for the reordering operation.
     *
     * @var string
     */
    protected $reorderNestedModuleItemsJobQueue = 'default';

    /**
     * @param string $nestedSlug
     * @param array $with
     * @param array $withCount
     * @param array $scopes
     * @return \A17\Twill\Models\Model|null
     */
    public function forNestedSlug($nestedSlug, $with = [], $withCount = [], $scopes = [])
    {
        $targetSlug = collect(explode('/', $nestedSlug))->last();

        $targetItem = $this->forSlug($targetSlug, $with, $withCount, $scopes);

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

        if (!$targetItem || $nestedSlug !== $targetItem->nestedSlug) {
            return null;
        }

        return $targetItem;
    }

    public function setNewOrder($ids)
    {
        ReorderNestedModuleItems::dispatch($this->model, $ids)
            ->onQueue($this->reorderNestedModuleItemsJobQueue);
    }
}