Handler::render() - Code Metrics - Inspection of "Implement new API and Route Updates" - Pterodactyl/Panel - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — develop (#371)

by Dane

created 2017-04-09 23:15 UTC

Handler::render() C

↳ Parent: Handler

Complexity

Conditions	10
Paths	8

Size

Total Lines	22
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	22
rs	6.1368
c	0
b	0
f	0
cc	10
eloc	13
nc	8
nop	2

How to fix Complexity

<?php

namespace Pterodactyl\Exceptions;

use Log;
use Exception;
use DisplayException;
.
|-- OtherDir
|   |-- Bar.php
|   `-- Foo.php
`-- SomeDir
    `-- Foo.php
use Illuminate\Auth\AuthenticationException;
use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;

class Handler extends ExceptionHandler
{
    /**
     * A list of the exception types that should not be reported.
     *
     * @var array
     */
    protected $dontReport = [
        \Illuminate\Auth\AuthenticationException::class,
        \Illuminate\Auth\Access\AuthorizationException::class,
        \Symfony\Component\HttpKernel\Exception\HttpException::class,
        \Illuminate\Database\Eloquent\ModelNotFoundException::class,
        \Illuminate\Session\TokenMismatchException::class,
        \Illuminate\Validation\ValidationException::class,
    ];

    /**
     * Report or log an exception.
     *
     * This is a great spot to send exceptions to Sentry, Bugsnag, etc.
     *
     * @param  \Exception  $exception
     * @return void
     */
    public function report(Exception $exception)
    {
        return parent::report($exception);
    }

    /**
     * Render an exception into an HTTP response.
     *
     * @param  \Illuminate\Http\Request  $request
     * @param  \Exception                $exception
     * @return \Illuminate\Http\Response
     */
    public function render($request, Exception $exception)
    {
        if ($request->expectsJson() || $request->isJson() || $request->is(...config('pterodactyl.json_routes'))) {
            $exception = $this->prepareException($exception);

            if (config('app.debug')) {
                $report = [
                    'code' => (! $this->isHttpException($exception)) ?: $exception->getStatusCode(),
                    'message' => class_basename($exception) . ' in ' . $exception->getFile() . ' on line ' . $exception->getLine(),
                ];
            }

            $response = response()->json([
                'error' => (config('app.debug')) ? $exception->getMessage() : 'An unhandled exception was encountered with this request.',
                'exception' => ! isset($report) ?: $report,
            ], ($this->isHttpException($exception)) ? $exception->getStatusCode() : 500, [], JSON_UNESCAPED_SLASHES);

            parent::report($exception);
class Daddy
{
    protected function getFirstName()
    {
        return "Eidur";
    }

    protected function getSurName()
    {
        return "Gudjohnsen";
    }
}

class Son
{
    public function getFirstName()
    {
        return parent::getSurname();
    }
}
        }

        return (isset($response)) ? $response : parent::render($request, $exception);

    }

    /**
     * Convert an authentication exception into an unauthenticated response.
     *
     * @param  \Illuminate\Http\Request                  $request
     * @param  \Illuminate\Auth\AuthenticationException  $exception
     * @return \Illuminate\Http\Response
     */
    protected function unauthenticated($request, AuthenticationException $exception)

    {
        if ($request->expectsJson()) {
            return response()->json(['error' => 'Unauthenticated.'], 401);
        }

        return redirect()->guest(route('auth.login'));
    }
}


GitHub Access Token became invalid

Pull Request — develop (#371)

Handler::render() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

1			<?php
2
3			namespace Pterodactyl\Exceptions;
4
5			use Log;
6			use Exception;
7			use DisplayException;
			0 ignored issues – show Bug introduced 2016-12-07 22:58 UTC by Report Bug Copy Issue Report This use statement conflicts with another class in this namespace, `Pterodactyl\Exceptions\DisplayException`. 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...
8			use Illuminate\Auth\AuthenticationException;
9			use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
10
11			class Handler extends ExceptionHandler
12			{
13			/**
14			* A list of the exception types that should not be reported.
15			*
16			* @var array
17			*/
18			protected $dontReport = [
19			\Illuminate\Auth\AuthenticationException::class,
20			\Illuminate\Auth\Access\AuthorizationException::class,
21			\Symfony\Component\HttpKernel\Exception\HttpException::class,
22			\Illuminate\Database\Eloquent\ModelNotFoundException::class,
23			\Illuminate\Session\TokenMismatchException::class,
24			\Illuminate\Validation\ValidationException::class,
25			];
26
27			/**
28			* Report or log an exception.
29			*
30			* This is a great spot to send exceptions to Sentry, Bugsnag, etc.
31			*
32			* @param \Exception $exception
33			* @return void
34			*/
35			public function report(Exception $exception)
36			{
37			return parent::report($exception);
38			}
39
40			/**
41			* Render an exception into an HTTP response.
42			*
43			* @param \Illuminate\Http\Request $request
44			* @param \Exception $exception
45			* @return \Illuminate\Http\Response
46			*/
47			public function render($request, Exception $exception)
48			{
49			if ($request->expectsJson() \|\| $request->isJson() \|\| $request->is(...config('pterodactyl.json_routes'))) {
50			$exception = $this->prepareException($exception);
51
52			if (config('app.debug')) {
53			$report = [
54			'code' => (! $this->isHttpException($exception)) ?: $exception->getStatusCode(),
55			'message' => class_basename($exception) . ' in ' . $exception->getFile() . ' on line ' . $exception->getLine(),
56			];
57			}
58
59			$response = response()->json([
60			'error' => (config('app.debug')) ? $exception->getMessage() : 'An unhandled exception was encountered with this request.',
61			'exception' => ! isset($report) ?: $report,
62			], ($this->isHttpException($exception)) ? $exception->getStatusCode() : 500, [], JSON_UNESCAPED_SLASHES);
63
64			parent::report($exception);
			0 ignored issues – show Comprehensibility Bug introduced 2017-01-12 20:50 UTC by Report Bug Copy Issue Report It seems like you call parent on a different method (`report()` instead of `render()`). Are you sure this is correct? If so, you might want to change this to `$this->report()`. This check looks for a call to a parent method whose name is different than the method from which it is called. Consider the following code: class Daddy { protected function getFirstName() { return "Eidur"; } protected function getSurName() { return "Gudjohnsen"; } } class Son { public function getFirstName() { return parent::getSurname(); } } The `getFirstName()` method in the `Son` calls the wrong method in the parent class. Loading history...
65			}
66
67			return (isset($response)) ? $response : parent::render($request, $exception);
			0 ignored issues – show Bug Compatibility introduced 2016-12-07 22:58 UTC by Report Bug Copy Issue Report The expression `isset($response) ? $resp...($request, $exception);` of type `Illuminate\Http\JsonResp...HttpFoundation\Response` adds the type `Illuminate\Http\JsonResponse` to the return on line 67 which is incompatible with the return type documented by `Pterodactyl\Exceptions\Handler::render` of type `Illuminate\Http\Response`. Loading history...
68			}
69
70			/**
71			* Convert an authentication exception into an unauthenticated response.
72			*
73			* @param \Illuminate\Http\Request $request
74			* @param \Illuminate\Auth\AuthenticationException $exception
75			* @return \Illuminate\Http\Response
76			*/
77			protected function unauthenticated($request, AuthenticationException $exception)
			0 ignored issues – show Unused Code introduced 2016-12-07 22:58 UTC by Report Bug Copy Issue Report The parameter `$exception` 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...
78			{
79			if ($request->expectsJson()) {
80			return response()->json(['error' => 'Unauthenticated.'], 401);
81			}
82
83			return redirect()->guest(route('auth.login'));
84			}
85			}
86

Pterodactyl / Panel

GitHub Access Token became invalid

Pull Request — develop (#371)

Handler::render() C

Complexity

Size

Duplication

Importance

How to fix Complexity

Long Method

Duplication Side-by-Side

Filter issues like