HasFactories::loadFactories() - Code Metrics - ARCANEDEV/Support - Measure and Improve Code Quality continuously with Scrutinizer

HasFactories::loadFactories() A
last analyzed 2021-02-26 17:52 UTC

↳ Parent: HasFactories

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

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

<?php

declare(strict_types=1);

namespace Arcanedev\Support\Providers\Concerns;

/**
 * Trait     HasFactories
 *
 * @author   ARCANEDEV <[email protected]>
 */
trait HasFactories
{
    /* -----------------------------------------------------------------
     |  Main Methods
     | -----------------------------------------------------------------
     */

    /**
     * Get the migrations path.
     *
     * @return string
     */
    protected function getFactoriesPath(): string
    {
        return $this->getBasePath().DIRECTORY_SEPARATOR.'database'.DIRECTORY_SEPARATOR.'factories';
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * Publish the factories.
     *
     * @param  string|null  $path
     */
    protected function publishFactories(?string $path = null): void
    {
        $this->publishes([
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->getFactoriesPath() => $path ?: database_path('factories'),
        ], $this->getPublishedTags('factories'));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * Load the factories.
     */
    protected function loadFactories(): void
    {
        $this->loadFactoriesFrom($this->getFactoriesPath());

    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Arcanedev\Support\Providers\Concerns;
6
7			/**
8			* Trait HasFactories
9			*
10			* @author ARCANEDEV <[email protected]>
11			*/
12			trait HasFactories
13			{
14			/* -----------------------------------------------------------------
15			\| Main Methods
16			\| -----------------------------------------------------------------
17			*/
18
19			/**
20			* Get the migrations path.
21			*
22			* @return string
23			*/
24			protected function getFactoriesPath(): string
25			{
26			return $this->getBasePath().DIRECTORY_SEPARATOR.'database'.DIRECTORY_SEPARATOR.'factories';
			0 ignored issues – show Bug introduced 2020-03-04 08:19 UTC by Report Bug Copy Issue Report It seems like `getBasePath()` 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...
27			}
28
29			/**
30			* Publish the factories.
31			*
32			* @param string\|null $path
33			*/
34			protected function publishFactories(?string $path = null): void
35			{
36			$this->publishes([
			0 ignored issues – show Bug introduced 2020-03-04 08:19 UTC by Report Bug Copy Issue Report It seems like `publishes()` 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...
37			$this->getFactoriesPath() => $path ?: database_path('factories'),
38			], $this->getPublishedTags('factories'));
			0 ignored issues – show Bug introduced 2020-03-08 13:21 UTC by Report Bug Copy Issue Report It seems like `getPublishedTags()` 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...
39			}
40
41			/**
42			* Load the factories.
43			*/
44			protected function loadFactories(): void
45			{
46			$this->loadFactoriesFrom($this->getFactoriesPath());
			0 ignored issues – show Bug introduced 2020-03-04 08:19 UTC by Report Bug Copy Issue Report The method `loadFactoriesFrom()` does not exist on `Arcanedev\Support\Providers\Concerns\HasFactories`. Did you maybe mean `loadFactories()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
47			}
48			}
49

ARCANEDEV / Support

HasFactories::loadFactories() A last analyzed 2021-02-26 17:52 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

HasFactories::loadFactories() A
last analyzed 2021-02-26 17:52 UTC