NestedModuleController::getBrowserItems() - Code Metrics - Inspection of "Fix bad merge" - area17/twill - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — feature/permissions-management ( 3c8a9f...1cc208 )

by Harings

created 2022-01-19 09:48 UTC

NestedModuleController::getBrowserItems() A

↳ Parent: NestedModuleController

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	2
eloc	8
c	1
b	0
f	0
nc	2
nop	1
dl	0
loc	12
rs	10

<?php

namespace A17\Twill\Http\Controllers\Admin;

abstract class NestedModuleController extends ModuleController
{
    /**
     * Indicates if only parent items should be displayed when browsing for this module
     * within a browser field.
     *
     * @var bool
     */
    protected $showOnlyParentItemsInBrowsers = true;

    /**
     * The maximum depth allowed for nested items. A value of `1` means parent & child.
     *
     * @var int
     */
    protected $nestedItemsDepth = 1;

    protected function indexData($request)
    {
        return [
            'nested' => true,
            'nestedDepth' => $this->nestedItemsDepth,
        ];
    }

    protected function transformIndexItems($items)
    {
        return $items->toTree();
    }

    protected function indexItemData($item)
    {
        return ($item->children ? [
            'children' => $this->getIndexTableData($item->children),
        ] : []);
    }

    protected function getBrowserItems($scopes = [])
    {
        if ($this->showOnlyParentItemsInBrowsers) {
            return $this->getIndexItems($scopes, true);
        }

        return $this->repository->get(
            $this->indexWith,
            $scopes,
            $this->orderScope(),
            request('offset') ?? $this->perPage ?? 50,
            true
        );
    }
}


1			<?php
2
3			namespace A17\Twill\Http\Controllers\Admin;
4
5			abstract class NestedModuleController extends ModuleController
6			{
7			/**
8			* Indicates if only parent items should be displayed when browsing for this module
9			* within a browser field.
10			*
11			* @var bool
12			*/
13			protected $showOnlyParentItemsInBrowsers = true;
14
15			/**
16			* The maximum depth allowed for nested items. A value of `1` means parent & child.
17			*
18			* @var int
19			*/
20			protected $nestedItemsDepth = 1;
21
22			protected function indexData($request)
23			{
24			return [
25			'nested' => true,
26			'nestedDepth' => $this->nestedItemsDepth,
27			];
28			}
29
30			protected function transformIndexItems($items)
31			{
32			return $items->toTree();
33			}
34
35			protected function indexItemData($item)
36			{
37			return ($item->children ? [
38			'children' => $this->getIndexTableData($item->children),
39			] : []);
40			}
41
42			protected function getBrowserItems($scopes = [])
43			{
44			if ($this->showOnlyParentItemsInBrowsers) {
45			return $this->getIndexItems($scopes, true);
46			}
47
48			return $this->repository->get(
49			$this->indexWith,
50			$scopes,
51			$this->orderScope(),
52			request('offset') ?? $this->perPage ?? 50,
53			true
54			);
55			}
56			}
57

area17 / twill

Push — feature/permissions-management ( 3c8a9f...1cc208 )

NestedModuleController::getBrowserItems() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like