ConsoleMessages::warning() - 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::warning() A

↳ Parent: ConsoleMessages

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

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

<?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::warning() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like