Tabs::getLastTab() - Code Metrics - Inspection of "changelog" - Laravel-Backpack/CRUD - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 54fb66...bd2bad )

by Cristian

created 2017-11-06 11:51 UTC

Tabs::getLastTab() A

↳ Parent: Tabs

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2

Importance

Changes

Metric	Value
cc	2
eloc	5
nc	2
nop	0
dl	0
loc	10
ccs	5
cts	5
cp	1
crap	2
rs	9.4285
c	0
b	0
f	0

<?php

namespace Backpack\CRUD\PanelTraits;

trait Tabs
{
    public $tabsEnabled = false;
    public $tabsType = 'horizontal';

    public function enableTabs()
    {
        $this->tabsEnabled = true;
        $this->setTabsType(config('backpack.crud.tabs_type', 'horizontal'));

        return $this->tabsEnabled;
    }

    public function disableTabs()
    {
        $this->tabsEnabled = false;

        return $this->tabsEnabled;
    }

    public function tabsEnabled()

    {
        return $this->tabsEnabled;
    }

    public function tabsDisabled()
    {
        return ! $this->tabsEnabled;
    }

    public function setTabsType($type)

    {
        $this->tabsType = $type;

        return $this->tabsType;
    }

    public function getTabsType()

    {
        return $this->tabsType;
    }

    public function enableVerticalTabs()

    {
        return $this->setTabsType('vertical');
    }

    public function disableVerticalTabs()

    {
        return $this->setTabsType('horizontal');
    }

    public function enableHorizontalTabs()

    {
        return $this->setTabsType('horizontal');
    }

    public function disableHorizontalTabs()

    {
        return $this->setTabsType('vertical');
    }

    public function tabExists($label)
    {
        $tabs = $this->getTabs();

        return in_array($label, $tabs);
    }

    public function getLastTab()
    {
        $tabs = $this->getTabs();

        if (count($tabs)) {
            return last($tabs);
        }

        return false;
    }

    public function isLastTab($label)
    {
        return $this->getLastTab() == $label;
    }

    public function getFieldsWithoutATab()
    {
        $all_fields = $this->getCurrentFields();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        $fields_without_a_tab = collect($all_fields)->filter(function ($value, $key) {

            return ! isset($value['tab']);
        });

        return $fields_without_a_tab;
    }

    public function getTabFields($label)
    {
        if ($this->tabExists($label)) {
            $all_fields = $this->getCurrentFields();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

            $fields_for_current_tab = collect($all_fields)->filter(function ($value, $key) use ($label) {

                return isset($value['tab']) && $value['tab'] == $label;
            });

            return $fields_for_current_tab;
        }

        return [];
    }

    public function getTabs()
    {
        $tabs = [];
        $fields = $this->getCurrentFields();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        $fields_with_tabs = collect($fields)
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
            ->filter(function ($value, $key) {

                return isset($value['tab']);
            })
            ->each(function ($value, $key) use (&$tabs) {

                if (! in_array($value['tab'], $tabs)) {
                    $tabs[] = $value['tab'];
                }
            });

        return $tabs;
    }
}


1		<?php
2
3		namespace Backpack\CRUD\PanelTraits;
4
5		trait Tabs
6		{
7		public $tabsEnabled = false;
8		public $tabsType = 'horizontal';
9
10	9	public function enableTabs()
11		{
12	9	$this->tabsEnabled = true;
13	9	$this->setTabsType(config('backpack.crud.tabs_type', 'horizontal'));
14
15	9	return $this->tabsEnabled;
16		}
17
18	2	public function disableTabs()
19		{
20	2	$this->tabsEnabled = false;
21
22	2	return $this->tabsEnabled;
23		}
24
25	8	public function tabsEnabled()
		0 ignored issues – show Documentation introduced 2017-08-03 17:04 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
26		{
27	8	return $this->tabsEnabled;
28		}
29
30	1	public function tabsDisabled()
31		{
32	1	return ! $this->tabsEnabled;
33		}
34
35	14	public function setTabsType($type)
		0 ignored issues – show Documentation introduced 2017-08-03 17:04 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
36		{
37	14	$this->tabsType = $type;
38
39	14	return $this->tabsType;
40		}
41
42	5	public function getTabsType()
		0 ignored issues – show Documentation introduced 2017-08-03 17:04 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
43		{
44	5	return $this->tabsType;
45		}
46
47	1	public function enableVerticalTabs()
		0 ignored issues – show Documentation introduced 2017-08-03 17:04 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
48		{
49	1	return $this->setTabsType('vertical');
50		}
51
52	1	public function disableVerticalTabs()
		0 ignored issues – show Documentation introduced 2017-04-18 11:15 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
53		{
54	1	return $this->setTabsType('horizontal');
55		}
56
57	1	public function enableHorizontalTabs()
		0 ignored issues – show Documentation introduced 2017-04-18 11:15 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
58		{
59	1	return $this->setTabsType('horizontal');
60		}
61
62	1	public function disableHorizontalTabs()
		0 ignored issues – show Documentation introduced 2017-02-13 15:30 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
63		{
64	1	return $this->setTabsType('vertical');
65		}
66
67	3	public function tabExists($label)
68		{
69	3	$tabs = $this->getTabs();
70
71	3	return in_array($label, $tabs);
72		}
73
74	4	public function getLastTab()
75		{
76	4	$tabs = $this->getTabs();
77
78	4	if (count($tabs)) {
79	3	return last($tabs);
80		}
81
82	1	return false;
83		}
84
85	2	public function isLastTab($label)
86		{
87	2	return $this->getLastTab() == $label;
88		}
89
90	1	public function getFieldsWithoutATab()
91		{
92	1	$all_fields = $this->getCurrentFields();
		0 ignored issues – show Bug introduced 2017-02-20 20:06 UTC by Report Bug Copy Issue Report It seems like `getCurrentFields()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
93
94		$fields_without_a_tab = collect($all_fields)->filter(function ($value, $key) {
		0 ignored issues – show Unused Code introduced 2017-02-20 20:06 UTC by Report Bug Copy Issue Report The parameter `$key` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
95		return ! isset($value['tab']);
96		});
97
98		return $fields_without_a_tab;
99		}
100
101		public function getTabFields($label)
102		{
103		if ($this->tabExists($label)) {
104		$all_fields = $this->getCurrentFields();
		0 ignored issues – show Bug introduced 2017-02-13 15:30 UTC by Report Bug Copy Issue Report It seems like `getCurrentFields()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
105
106		$fields_for_current_tab = collect($all_fields)->filter(function ($value, $key) use ($label) {
		0 ignored issues – show Unused Code introduced 2017-02-20 20:06 UTC by Report Bug Copy Issue Report The parameter `$key` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
107		return isset($value['tab']) && $value['tab'] == $label;
108		});
109
110	1	return $fields_for_current_tab;
111		}
112
113	9	return [];
114		}
115	9
116	9	public function getTabs()
117		{
118	9	$tabs = [];
119		$fields = $this->getCurrentFields();
		0 ignored issues – show Bug introduced 2017-02-13 15:30 UTC by Report Bug Copy Issue Report It seems like `getCurrentFields()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
120	7
121	9	$fields_with_tabs = collect($fields)
		0 ignored issues – show Unused Code introduced 2017-04-18 10:54 UTC by Report Bug Copy Issue Report `$fields_with_tabs` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
122	9	->filter(function ($value, $key) {
		0 ignored issues – show Unused Code introduced 2017-04-18 10:54 UTC by Report Bug Copy Issue Report The parameter `$key` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
123	7	return isset($value['tab']);
124	7	})
125		->each(function ($value, $key) use (&$tabs) {
		0 ignored issues – show Unused Code introduced 2017-02-13 15:30 UTC by Report Bug Copy Issue Report The parameter `$key` is not used and could be removed. This check looks from parameters that have been defined for a function or method, but which are not used in the method body. Loading history...
126	9	if (! in_array($value['tab'], $tabs)) {
127		$tabs[] = $value['tab'];
128	9	}
129		});
130
131		return $tabs;
132		}
133		}
134

Laravel-Backpack / CRUD

Scrutinizer GitHub App not installed

Push — master ( 54fb66...bd2bad )

Tabs::getLastTab() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like