loadTasks - Code Metrics - consolidation/robo - Measure and Improve Code Quality continuously with Scrutinizer

loadTasks A
last analyzed 2020-08-27 17:44 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

Metric	Value
wmc	2
lcom	0
cbo	0
dl	0
loc	22
rs	10
c	0
b	0
f	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	taskBowerInstall()	0	4	1
A	taskBowerUpdate()	0	4	1

<?php

namespace Robo\Task\Bower;

trait loadTasks
{
    /**
     * @param null|string $pathToBower
     *
     * @return \Robo\Task\Bower\Install|\Robo\Collection\CollectionBuilder
     */
    protected function taskBowerInstall($pathToBower = null)
    {
        return $this->task(Install::class, $pathToBower);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * @param null|string $pathToBower
     *
     * @return \Robo\Task\Bower\Update|\Robo\Collection\CollectionBuilder
     */
    protected function taskBowerUpdate($pathToBower = null)
    {
        return $this->task(Update::class, $pathToBower);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1			<?php
2
3			namespace Robo\Task\Bower;
4
5			trait loadTasks
6			{
7			/**
8			* @param null\|string $pathToBower
9			*
10			* @return \Robo\Task\Bower\Install\|\Robo\Collection\CollectionBuilder
11			*/
12			protected function taskBowerInstall($pathToBower = null)
13			{
14			return $this->task(Install::class, $pathToBower);
			0 ignored issues – show Bug introduced 2016-09-13 23:40 UTC by Report Bug Copy Issue Report It seems like `task()` 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...
15			}
16
17			/**
18			* @param null\|string $pathToBower
19			*
20			* @return \Robo\Task\Bower\Update\|\Robo\Collection\CollectionBuilder
21			*/
22			protected function taskBowerUpdate($pathToBower = null)
23			{
24			return $this->task(Update::class, $pathToBower);
			0 ignored issues – show Bug introduced 2016-09-13 23:40 UTC by Report Bug Copy Issue Report It seems like `task()` 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...
25			}
26			}
27

consolidation / robo

loadTasks A last analyzed 2020-08-27 17:44 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

2 Methods

Duplication Side-by-Side

Filter issues like

loadTasks A
last analyzed 2020-08-27 17:44 UTC