SidebarService::call() - Code Metrics - Inspection of "Merge pull request #5 from webfactor/2.1" - webfactor/laravel-generators - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 672a66...8833c5 )

by Thomas

created 2018-03-30 17:07 UTC

SidebarService::call() A

↳ Parent: SidebarService

Complexity

Conditions	2
Paths	2

Size

Total Lines	9
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	9
ccs	0
cts	8
cp	0
rs	9.6666
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	0
crap	6

<?php

namespace Webfactor\Laravel\Generators\Services;

use Illuminate\Filesystem\Filesystem;
use Webfactor\Laravel\Generators\Commands\MakeEntity;
use Webfactor\Laravel\Generators\Contracts\ServiceAbstract;
use Webfactor\Laravel\Generators\Contracts\ServiceInterface;

class SidebarService extends ServiceAbstract implements ServiceInterface
{
    protected $relativeToBasePath = 'resources/views/vendor/backpack/base/inc';

    protected $fileName = 'sidebar_content.blade.php';

    private $sidebarFile;

    public function call()
    {
        $this->sidebarFile = $this->getFilePath();

        if ($this->filesystem->exists($this->sidebarFile)) {
            $this->writeFile();
            $this->addLatestFileToIdeStack();
        }
    }

    private function getRouteName(): string
    {
        return strtolower($this->command->entity);
    }

    private function getLanguageName(): string
    {
        return snake_case($this->command->entity);
    }

    /**
     * Build the class with the given name.
     *
     * @param string $name
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
     *
     * @return string

     */
    private function writeFile()
    {
        $this->filesystem->append($this->sidebarFile, $this->getRouteString());
    }

    private function getFilePath()
    {
        return base_path($this->relativeToBasePath) . '/' . $this->fileName;
    }

    private function getRouteString()

    {
        return <<<FILE

<li>
    <a href="{{ backpack_url('{$this->getRouteName()}') }}">
        <i class="fa fa-question"></i><span>{{ trans('models.{$this->getLanguageName()}.plural') }}</span>
    </a>
</li>
FILE;
    }
}


1			<?php
2
3			namespace Webfactor\Laravel\Generators\Services;
4
5			use Illuminate\Filesystem\Filesystem;
6			use Webfactor\Laravel\Generators\Commands\MakeEntity;
7			use Webfactor\Laravel\Generators\Contracts\ServiceAbstract;
8			use Webfactor\Laravel\Generators\Contracts\ServiceInterface;
9
10			class SidebarService extends ServiceAbstract implements ServiceInterface
11			{
12			protected $relativeToBasePath = 'resources/views/vendor/backpack/base/inc';
13
14			protected $fileName = 'sidebar_content.blade.php';
15
16			private $sidebarFile;
17
18			public function call()
19			{
20			$this->sidebarFile = $this->getFilePath();
21
22			if ($this->filesystem->exists($this->sidebarFile)) {
23			$this->writeFile();
24			$this->addLatestFileToIdeStack();
25			}
26			}
27
28			private function getRouteName(): string
29			{
30			return strtolower($this->command->entity);
31			}
32
33			private function getLanguageName(): string
34			{
35			return snake_case($this->command->entity);
36			}
37
38			/**
39			* Build the class with the given name.
40			*
41			* @param string $name
			0 ignored issues – show Bug introduced 2018-03-30 17:09 UTC by Report Bug Copy Issue Report There is no parameter named `$name`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
42			*
43			* @return string
			0 ignored issues – show Documentation introduced 2018-03-30 17:09 UTC by Report Bug Copy Issue Report Should the return type not be `string\|null`? This check compares the return type specified in the `@return` annotation of a function or method doc comment with the types returned by the function and raises an issue if they mismatch. Loading history...
44			*/
45			private function writeFile()
46			{
47			$this->filesystem->append($this->sidebarFile, $this->getRouteString());
48			}
49
50			private function getFilePath()
51			{
52			return base_path($this->relativeToBasePath) . '/' . $this->fileName;
53			}
54
55			private function getRouteString()
			0 ignored issues – show Documentation introduced 2018-03-30 17:09 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history...
56			{
57			return <<<FILE
58
59			<li>
60			<a href="{{ backpack_url('{$this->getRouteName()}') }}">
61			<i class="fa fa-question"></i><span>{{ trans('models.{$this->getLanguageName()}.plural') }}</span>
62			</a>
63			</li>
64			FILE;
65			}
66			}
67

webfactor / laravel-generators

Push — master ( 672a66...8833c5 )

SidebarService::call() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like