H5PExceptionHandler::renderException() - Code Metrics - djoudi/Laravel-H5P - Measure and Improve Code Quality continuously with Scrutinizer

H5PExceptionHandler::renderException() A
last analyzed 2020-03-26 06:08 UTC

↳ Parent: H5PExceptionHandler

Complexity

Conditions	3
Paths	3

Size

Total Lines	14
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
cc	3
eloc	10
c	1
b	1
f	0
nc	3
nop	1
dl	0
loc	14
rs	9.9332

<?php

/*
 *
 * @Project        Expression project.displayName is undefined on line 5, column 35 in Templates/Licenses/license-default.txt.
 * @Copyright      Djoudi
 * @Created        2017-02-21
 * @Filename       H5PExceptionHandler.php
 * @Description
 *
 */

namespace Djoudi\LaravelH5p\Exceptions;

use Exception;
use Illuminate\Database\Eloquent\ModelNotFoundException;
use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
use Symfony\Component\HttpKernel\Exception\HttpException;

class H5PExceptionHandler extends ExceptionHandler
{
    /**
     * A list of the exception types that should not be reported.
     *
     * @var array
     */
    protected $dontReport = [
        HttpException::class,
    ];

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

    }

    /**
     * Render an exception into an HTTP response.
     *
     * @param \Illuminate\Http\Request $request
     * @param  $e
     *
     * @return \Illuminate\Http\Response
     */
    public function render($request, Exception $e)
    {
        switch ($e) {

        case $e instanceof ModelNotFoundException:

            return $this->renderException($e);
            break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

        case $e instanceof H5PException:

            return $this->renderException($e);
            break;

        default:

            return parent::render($request, $e);
        }
    }

    protected function renderException($e)
    {
        switch ($e) {

        case $e instanceof ModelNotFoundException:
            return response()->view('errors.404', [], 404);
            break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}

        case $e instanceof H5PException:
            return response()->view('errors.friendly');
            break;
        default:
            return (new SymfonyDisplayer(config('app.debug')))
filter:
    dependency_paths: ["lib/*"]
                ->createResponse($e);
        }
    }
}


1			<?php
2
3			/*
4			*
5			* @Project Expression project.displayName is undefined on line 5, column 35 in Templates/Licenses/license-default.txt.
6			* @Copyright Djoudi
7			* @Created 2017-02-21
8			* @Filename H5PExceptionHandler.php
9			* @Description
10			*
11			*/
12
13			namespace Djoudi\LaravelH5p\Exceptions;
14
15			use Exception;
16			use Illuminate\Database\Eloquent\ModelNotFoundException;
17			use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
18			use Symfony\Component\HttpKernel\Exception\HttpException;
19
20			class H5PExceptionHandler extends ExceptionHandler
21			{
22			/**
23			* A list of the exception types that should not be reported.
24			*
25			* @var array
26			*/
27			protected $dontReport = [
28			HttpException::class,
29			];
30
31			/**
32			* Report or log an exception.
33			*
34			* This is a great spot to send exceptions to Sentry, Bugsnag, etc.
35			*
36			* @param \Exception $e
37			*
38			* @return void
39			*/
40			public function report(Exception $e)
41			{
42			return parent::report($e);
			0 ignored issues – show Bug Best Practice introduced 2019-03-12 05:27 UTC by Report Bug Copy Issue Report The expression `return parent::report($e)` also could return the type `callable` which is incompatible with the documented return type `void`. Loading history...
43			}
44
45			/**
46			* Render an exception into an HTTP response.
47			*
48			* @param \Illuminate\Http\Request $request
49			* @param $e
50			*
51			* @return \Illuminate\Http\Response
52			*/
53			public function render($request, Exception $e)
54			{
55			switch ($e) {
56
57			case $e instanceof ModelNotFoundException:
58
59			return $this->renderException($e);
60			break;
			0 ignored issues – show Unused Code introduced 2019-03-05 19:39 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
61
62			case $e instanceof H5PException:
63
64			return $this->renderException($e);
65			break;
66
67			default:
68
69			return parent::render($request, $e);
70			}
71			}
72
73			protected function renderException($e)
74			{
75			switch ($e) {
76
77			case $e instanceof ModelNotFoundException:
78			return response()->view('errors.404', [], 404);
79			break;
			0 ignored issues – show Unused Code introduced 2019-03-05 19:39 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
80
81			case $e instanceof H5PException:
82			return response()->view('errors.friendly');
83			break;
84			default:
85			return (new SymfonyDisplayer(config('app.debug')))
			0 ignored issues – show Bug introduced 2019-03-05 19:39 UTC by Report Bug Copy Issue Report The type `Djoudi\LaravelH5p\Exceptions\SymfonyDisplayer` was not found. Maybe you did not declare it correctly or list all dependencies? The issue could also be caused by a filter entry in the build configuration. If the path has been excluded in your configuration, e.g. `excluded_paths: ["lib/"]`, you can move it to the dependency path list as follows: filter: dependency_paths: ["lib/"] For further information see https://scrutinizer-ci.com/docs/tools/php/php-scrutinizer/#list-dependency-paths Loading history...
86			->createResponse($e);
87			}
88			}
89			}
90

djoudi / Laravel-H5P

H5PExceptionHandler::renderException() A last analyzed 2020-03-26 06:08 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

H5PExceptionHandler::renderException() A
last analyzed 2020-03-26 06:08 UTC