Completed
Push — master ( 3c2674...5698f6 )
by Ricardo
06:00
created

AccessDeniedException   A

Complexity

Total Complexity 1

Size/Duplication

Total Lines 14
Duplicated Lines 0 %

Coupling/Cohesion

Components 0
Dependencies 0

Importance

Changes 0
Metric Value
dl 0
loc 14
rs 10
c 0
b 0
f 0
wmc 1
lcom 0
cbo 0

1 Method

Rating   Name   Duplication   Size   Complexity  
A render() 0 4 1
1
<?php
2
3
namespace Integrations\Exceptions;
4
5
use Exception;
0 ignored issues
show
Bug introduced by
This use statement conflicts with another class in this namespace, Integrations\Exceptions\Exception.

Let’s assume that you have a directory layout like this:

.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php

and let’s assume the following content of Bar.php:

// Bar.php
namespace OtherDir;

use SomeDir\Foo; // This now conflicts the class OtherDir\Foo

If both files OtherDir/Foo.php and SomeDir/Foo.php are loaded in the same runtime, you will see a PHP error such as the following:

PHP Fatal error:  Cannot use SomeDir\Foo as Foo because the name is already in use in OtherDir/Foo.php

However, as OtherDir/Foo.php does not necessarily have to be loaded and the error is only triggered if it is loaded before OtherDir/Bar.php, this problem might go unnoticed for a while. In order to prevent this error from surfacing, you must import the namespace with a different alias:

// Bar.php
namespace OtherDir;

use SomeDir\Foo as SomeDirFoo; // There is no conflict anymore.
Loading history...
6
7
class AccessDeniedException extends Exception
8
{
9
    /**
10
     * Render the exception into an HTTP response.
11
     *
12
     * @param  \Illuminate\Http\Request
13
     *
14
     * @return \Illuminate\Http\Response
15
     */
16
    public function render($request)
0 ignored issues
show
Unused Code introduced by
The parameter $request is not used and could be removed.

This check looks from parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
17
    {
18
        return response(view('errors.403', ['exception' => $this]), 403);
0 ignored issues
show
Bug introduced by
It seems like view('errors.403', array('exception' => $this)) targeting view() can also be of type object<Illuminate\Contracts\View\Factory>; however, response() does only seem to accept string|object<Illuminate\View\View>|array|null, maybe add an additional type check?

This check looks at variables that are passed out again to other methods.

If the outgoing method call has stricter type requirements than the method itself, an issue is raised.

An additional type check may prevent trouble.

Loading history...
19
    }
20
}
21