Response - Code Metrics - Inspection of "Merge pull request #13 from cybercog/feature/reque..." - cybercog/laravel-sense - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0d7181...2d779c )

by Anton

created 2018-10-07 00:53 UTC

Response A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Importance

Changes

Metric	Value
eloc	7
dl	0
loc	25
rs	10
c	0
b	0
f	0
wmc	5

4 Methods

Rating	Name	Size	Complexity
A	toJson()	3	1
A	toHtml()	4	1
A	toResponse()	3	2
A	__construct()	3	1

<?php

/*
 * This file is part of Laravel Sense.
 *
 * (c) Anton Komarev <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

declare(strict_types=1);

namespace Cog\Laravel\Sense\Http\Controllers\Urls\Get;

use Illuminate\Contracts\Support\Responsable;

class Response implements Responsable
{
    /** @var \Cog\Laravel\Sense\Url\Models\Url */
    private $url;

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

    public function toResponse($request)
    {
        return $request->wantsJson() ? $this->toJson() : $this->toHtml();
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.
    }
}
    }

    private function toHtml()
    {
        return view('sense::urls.view', [
            'url' => $this->url,
        ]);
    }

    private function toJson()
    {
        return [];
    }
}


1			<?php
2
3			/*
4			* This file is part of Laravel Sense.
5			*
6			* (c) Anton Komarev <[email protected]>
7			*
8			* For the full copyright and license information, please view the LICENSE
9			* file that was distributed with this source code.
10			*/
11
12			declare(strict_types=1);
13
14			namespace Cog\Laravel\Sense\Http\Controllers\Urls\Get;
15
16			use Illuminate\Contracts\Support\Responsable;
17
18			class Response implements Responsable
19			{
20			/** @var \Cog\Laravel\Sense\Url\Models\Url */
21			private $url;
22
23			public function __construct($url)
24			{
25			$this->url = $url;
26			}
27
28			public function toResponse($request)
29			{
30			return $request->wantsJson() ? $this->toJson() : $this->toHtml();
			0 ignored issues – show Bug Best Practice introduced 2018-10-06 23:30 UTC by Report Bug Copy Issue Report The expression `return $request->wantsJs...son() : $this->toHtml()` returns the type `Illuminate\View\View\|array` which is incompatible with the return type mandated by `Illuminate\Contracts\Sup...sponsable::toResponse()` of `Illuminate\Http\Response`. 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...
31			}
32
33			private function toHtml()
34			{
35			return view('sense::urls.view', [
36			'url' => $this->url,
37			]);
38			}
39
40			private function toJson()
41			{
42			return [];
43			}
44			}
45

cybercog / laravel-sense

Push — master ( 0d7181...2d779c )

Response A

Complexity

Size/Duplication

Importance

4 Methods

Duplication Side-by-Side

Filter issues like