MissingScopeHandler - Code Metrics - Inspection of "Changed way to map exceptions to be more extensibl..." - sfelix-martins/json-exception-handler - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Branch — 2.0 (f5ddb0)

by Samuel

created 2018-05-19 15:30 UTC

MissingScopeHandler A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	12
Duplicated Lines	0 %

Importance

Changes

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

1 Method

Rating	Name	Duplication	Size	Complexity
A	handle()	0	7	1

<?php

namespace SMartins\Exceptions\Handlers;

use SMartins\Exceptions\JsonApi\Error;
use SMartins\Exceptions\JsonApi\Source;

class MissingScopeHandler extends AbstractHandler
{
    /**
     * {@inheritDoc}
     */
    public function handle()
    {
        return (new Error)->setStatus(403)
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.
    }
}
            ->setCode($this->getCode('missing_scope'))
            ->setSource((new Source())->setPointer($this->getDefaultPointer()))
            ->setTitle($this->getDefaultTitle())
            ->setDetail($this->exception->getMessage());
    }
}


1			<?php
2
3			namespace SMartins\Exceptions\Handlers;
4
5			use SMartins\Exceptions\JsonApi\Error;
6			use SMartins\Exceptions\JsonApi\Source;
7
8			class MissingScopeHandler extends AbstractHandler
9			{
10			/**
11			* {@inheritDoc}
12			*/
13			public function handle()
14			{
15			return (new Error)->setStatus(403)
			0 ignored issues – show Bug Best Practice introduced 2018-05-19 15:34 UTC by Report Bug Copy Issue Report The expression `return new SMartins\Exce...xception->getMessage())` returns the type `SMartins\Exceptions\JsonApi\Error` which is incompatible with the return type mandated by `SMartins\Exceptions\Hand...stractHandler::handle()` of `array\|Illuminate\Support\Collection`. 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...
16			->setCode($this->getCode('missing_scope'))
17			->setSource((new Source())->setPointer($this->getDefaultPointer()))
18			->setTitle($this->getDefaultTitle())
19			->setDetail($this->exception->getMessage());
20			}
21			}
22

sfelix-martins / json-exception-handler

Branch — 2.0 (f5ddb0)

MissingScopeHandler A

Complexity

Size/Duplication

Importance

1 Method

Duplication Side-by-Side

Filter issues like