ConsoleMessages::info() - 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 19:29 UTC

ConsoleMessages::info() A

↳ Parent: ConsoleMessages

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	2
dl	0
loc	4
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			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			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			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			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::info() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like