RouteListItem - Code Metrics - Inspection of "Merge pull request #1678 from hydephp/refactor-to-..." - hydephp/develop - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c38b6c...e54c12 )

by Caen

created 2024-04-25 17:28 UTC

RouteListItem A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	42
Duplicated Lines	0 %

Importance

Changes

Metric	Value
eloc	12
dl	0
loc	42
rs	10
c	0
b	0
f	0
wmc	10

7 Methods

Rating	Name	Size	Complexity
A	styleOutputPath()	3	1
A	toArray()	7	1
A	styleRouteKey()	3	1
A	stylePageType()	3	2
A	styleSourcePath()	3	2
A	__construct()	3	1
A	isPageDiscoverable()	3	2

<?php

declare(strict_types=1);

namespace Hyde\Support\Internal;

use Hyde\Hyde;
use Hyde\Pages\InMemoryPage;
use Hyde\Support\Models\Route;
use Illuminate\Contracts\Support\Arrayable;

use function class_basename;
use function str_starts_with;

/**
 * @internal This class is internal and should not be depended on outside the HydePHP framework code.
 */
class RouteListItem implements Arrayable
{
    protected Route $route;

    public function __construct(Route $route)
    {
        $this->route = $route;
    }

    public function toArray(): array
    {
        return [
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
            'page_type' => $this->stylePageType($this->route->getPageClass()),
            'source_file' => $this->styleSourcePath($this->route->getSourcePath()),
            'output_file' => $this->styleOutputPath($this->route->getOutputPath()),
            'route_key' => $this->styleRouteKey($this->route->getRouteKey()),
        ];
    }

    protected function stylePageType(string $class): string
    {
        return str_starts_with($class, 'Hyde') ? class_basename($class) : $class;
    }

    protected function styleSourcePath(string $path): string
    {
        return $this->isPageDiscoverable() ? $path : 'none';
    }

    protected function styleOutputPath(string $path): string
    {
        return Hyde::getOutputDirectory()."/$path";
    }

    protected function styleRouteKey(string $key): string
    {
        return $key;
    }

    protected function isPageDiscoverable(): bool
    {
        return $this->route->getSourcePath() && ! $this->route->getPage() instanceof InMemoryPage;
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Hyde\Support\Internal;
6
7			use Hyde\Hyde;
8			use Hyde\Pages\InMemoryPage;
9			use Hyde\Support\Models\Route;
10			use Illuminate\Contracts\Support\Arrayable;
11
12			use function class_basename;
13			use function str_starts_with;
14
15			/**
16			* @internal This class is internal and should not be depended on outside the HydePHP framework code.
17			*/
18			class RouteListItem implements Arrayable
19			{
20			protected Route $route;
21
22			public function __construct(Route $route)
23			{
24			$this->route = $route;
25			}
26
27			public function toArray(): array
28			{
29			return [
			0 ignored issues – show Bug Best Practice introduced 2023-02-24 20:36 UTC by Report Bug Copy Issue Report The expression `return array('page_type'...>route->getRouteKey()))` returns the type `array<string,string>` which is incompatible with the return type mandated by `Illuminate\Contracts\Support\Arrayable::toArray()` of `Illuminate\Contracts\Support\TValue[]`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
30			'page_type' => $this->stylePageType($this->route->getPageClass()),
31			'source_file' => $this->styleSourcePath($this->route->getSourcePath()),
32			'output_file' => $this->styleOutputPath($this->route->getOutputPath()),
33			'route_key' => $this->styleRouteKey($this->route->getRouteKey()),
34			];
35			}
36
37			protected function stylePageType(string $class): string
38			{
39			return str_starts_with($class, 'Hyde') ? class_basename($class) : $class;
40			}
41
42			protected function styleSourcePath(string $path): string
43			{
44			return $this->isPageDiscoverable() ? $path : 'none';
45			}
46
47			protected function styleOutputPath(string $path): string
48			{
49			return Hyde::getOutputDirectory()."/$path";
50			}
51
52			protected function styleRouteKey(string $key): string
53			{
54			return $key;
55			}
56
57			protected function isPageDiscoverable(): bool
58			{
59			return $this->route->getSourcePath() && ! $this->route->getPage() instanceof InMemoryPage;
60			}
61			}
62

hydephp / develop

Push — master ( c38b6c...e54c12 )

RouteListItem A

Complexity

Size/Duplication

Importance

7 Methods

Duplication Side-by-Side

Filter issues like