ConsoleMessages - Code Metrics - Inspection of "Extended Commands" - nWidart/laravel-modules - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1088)

unknown

created 2020-10-07 20:01 UTC

ConsoleMessages A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

50%

Importance

Changes

Metric	Value
dl	0
loc	22
ccs	4
cts	8
cp	0.5
rs	10
c	0
b	0
f	0
wmc	4
lcom	0
cbo	0

4 Methods

Rating	Name	Size	Complexity
A	info()	4	1
A	success()	4	1
A	warning()	4	1
A	critical()	4	1

<?php

namespace Nwidart\Modules\Console\Traits;

trait ConsoleMessages
{
    public function info($string, $verbosity = NULL)

    {
        return $this->line("🠲 <options=bold;>{$string} ");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    public function success($string, $verbosity = NULL)

    {
        return $this->line("✅ <options=bold;fg=blue>{$string} ");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    public function warning($string, $verbosity = NULL)

    {
        return $this->line("⚠️  <fg=yellow;options=bold>{$string} ");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    public function critical($string, $verbosity = NULL)

    {
        return $this->line("❌ <fg=white;bg=red;options=bold>{$string} ");
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }
}


1		<?php
2
3		namespace Nwidart\Modules\Console\Traits;
4
5		trait ConsoleMessages
6		{
7	4	public function info($string, $verbosity = NULL)
		0 ignored issues – show Unused Code introduced 2020-10-07 19:24 UTC by Report Bug Copy Issue Report The parameter `$verbosity` 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...
8		{
9	4	return $this->line("🠲 <options=bold;>{$string} ");
		0 ignored issues – show Bug introduced 2020-10-07 19:24 UTC by Report Bug Copy Issue Report It seems like `line()` 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...
10		}
11
12	4	public function success($string, $verbosity = NULL)
		0 ignored issues – show Unused Code introduced 2020-10-07 19:24 UTC by Report Bug Copy Issue Report The parameter `$verbosity` 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...
13		{
14	4	return $this->line("✅ <options=bold;fg=blue>{$string} ");
		0 ignored issues – show Bug introduced 2020-10-07 19:24 UTC by Report Bug Copy Issue Report It seems like `line()` 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		public function warning($string, $verbosity = NULL)
		0 ignored issues – show Unused Code introduced 2020-10-07 19:24 UTC by Report Bug Copy Issue Report The parameter `$verbosity` 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...
18		{
19		return $this->line("⚠️ <fg=yellow;options=bold>{$string} ");
		0 ignored issues – show Bug introduced 2020-10-07 19:24 UTC by Report Bug Copy Issue Report It seems like `line()` 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...
20		}
21
22		public function critical($string, $verbosity = NULL)
		0 ignored issues – show Unused Code introduced 2020-10-07 19:24 UTC by Report Bug Copy Issue Report The parameter `$verbosity` 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...
23		{
24		return $this->line("❌ <fg=white;bg=red;options=bold>{$string} ");
		0 ignored issues – show Bug introduced 2020-10-07 19:24 UTC by Report Bug Copy Issue Report It seems like `line()` 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

nWidart / laravel-modules

Pull Request — master (#1088)

ConsoleMessages A

Complexity

Size/Duplication

Coupling/Cohesion

Test Coverage

Importance

4 Methods

Duplication Side-by-Side

Filter issues like