Issues in Kernel.php (master) - Issues in master - Codexshaper/wpb-foundation - Measure and Improve Code Quality continuously with Scrutinizer

Issues (57)

Security Analysis not enabled

Cross-Site Scripting

Cross-Site Scripting enables an attacker to inject code into the response of a web-request that is viewed by other users. It can for example be used to bypass access controls, or even to take over other users' accounts.

File Exposure

File Exposure allows an attacker to gain access to local files that he should not be able to access. These files can for example include database credentials, or other configuration files.

File Manipulation

File Manipulation enables an attacker to write custom data to files. This potentially leads to injection of arbitrary code on the server.

Object Injection

Object Injection enables an attacker to inject an object into PHP code, and can lead to arbitrary code execution, file exposure, or file manipulation attacks.

Code Injection

Code Injection enables an attacker to execute arbitrary code on the server.

Response Splitting

Response Splitting can be used to send arbitrary responses.

File Inclusion

File Inclusion enables an attacker to inject custom files into PHP's file loading mechanism, either explicitly passed to include, or for example via PHP's auto-loading mechanism.

Command Injection

Command Injection enables an attacker to inject a shell command that is execute with the privileges of the web-server. This can be used to expose sensitive data, or gain access of your server.

SQL Injection

SQL Injection enables an attacker to execute arbitrary SQL code on your database server gaining access to user data, or manipulating user data.

XPath Injection

XPath Injection enables an attacker to modify the parts of XML document that are read. If that XML document is for example used for authentication, this can lead to further vulnerabilities similar to SQL Injection.

LDAP Injection

LDAP Injection enables an attacker to inject LDAP statements potentially granting permission to run unauthorized queries, or modify content inside the LDAP tree.

Header Injection

Other Vulnerability

This category comprises other attack vectors such as manipulating the PHP runtime, loading custom extensions, freezing the runtime, or similar.

Regex Injection

Regex Injection enables an attacker to execute arbitrary code in your PHP process.

XML Injection

XML Injection enables an attacker to read files on your local filesystem including configuration files, or can be abused to freeze your web-server process.

Variable Injection

Variable Injection enables an attacker to overwrite program variables with custom data, and can lead to further vulnerabilities.

Unfortunately, the security analysis is currently not available for your project. If you are a non-commercial open-source project, please contact support to gain access.

src/Http/Kernel.php (10 issues)

Labels

Severity

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

<?php

namespace CodexShaper\WP\Http;

use CodexShaper\WP\Http\Events\RequestHandled;
use Illuminate\Contracts\Container\Container;
use Illuminate\Contracts\Debug\ExceptionHandler;
use Illuminate\Contracts\Http\Kernel as KernelContract;
use Illuminate\Routing\Pipeline;
use Illuminate\Routing\Router;
use Illuminate\Support\Facades\Facade;
use InvalidArgumentException;
use Throwable;

class Kernel implements KernelContract
{
    /**
     * The application implementation.
     *
     * @var \Illuminate\Contracts\Foundation\Application
     */
    protected $app;

    /**
     * The router instance.
     *
     * @var \Illuminate\Routing\Router
     */
    protected $router;

    /**
     * The bootstrap classes for the application.
     *
     * @var array
     */
    protected $bootstrappers = [];

    /**
     * The application's middleware stack.
     *
     * @var array
     */
    protected $middleware = [];

    /**
     * The application's route middleware groups.
     *
     * @var array
     */
    protected $middlewareGroups = [];

    /**
     * The application's route middleware.
     *
     * @var array
     */
    protected $routeMiddleware = [];

    /**
     * The priority-sorted list of middleware.
     *
     * Forces non-global middleware to always be in the given order.
     *
     * @var array
     */
    protected $middlewarePriority = [
        \Illuminate\Session\Middleware\StartSession::class,
        \Illuminate\View\Middleware\ShareErrorsFromSession::class,
        \Illuminate\Contracts\Auth\Middleware\AuthenticatesRequests::class,
        \Illuminate\Routing\Middleware\ThrottleRequests::class,
        \Illuminate\Session\Middleware\AuthenticateSession::class,
        \Illuminate\Routing\Middleware\SubstituteBindings::class,
        \Illuminate\Auth\Middleware\Authorize::class,
    ];

    /**
     * Create a new HTTP kernel instance.
     *
     * @param \Illuminate\Contracts\Foundation\Application $app
     * @param \Illuminate\Routing\Router                   $router
     *
     * @return void

     */
    public function __construct(Container $app, Router $router)
    {
        $this->app = $app;
class Alien {}

class Dalek extends Alien {}

class Plot
{
    /** @var  Dalek */
    public $villain;
}

$alien = new Alien();
$plot = new Plot();
if ($alien instanceof Dalek) {
    $plot->villain = $alien;
}
        $this->router = $router;

        $this->syncMiddlewareToRouter();
    }

    /**
     * Handle an incoming HTTP request.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return \Illuminate\Http\Response
     */
    public function handle($request)
    {
        try {
            $request->enableHttpMethodParameterOverride();
            $response = $this->sendRequestThroughRouter($request);
        } catch (Throwable $e) {
            throw new \Exception($e, 1);
            $this->reportException($e);
function fx() {
    try {
        doSomething();
        return true;
    }
    catch (\Exception $e) {
        return false;
    }

    return false;
}

            $response = $this->renderException($request, $e);
        }

        $this->app['events']->dispatch(
            new RequestHandled($request, $response)
        );

        return $response;
    }

    /**
     * Send the given request through the middleware / router.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return \Illuminate\Http\Response
     */
    protected function sendRequestThroughRouter($request)
    {
        $this->app->instance('request', $request);

        Facade::clearResolvedInstance('request');

        $this->bootstrap();

        return (new Pipeline($this->app))
                    ->send($request)
                    // ->through($this->app->shouldSkipMiddleware() ? [] : $this->middleware)
                    ->through($this->middleware)
                    ->then($this->dispatchToRouter());
    }

    /**
     * Bootstrap the application for HTTP requests.
     *
     * @return void
     */
    public function bootstrap()
    {
        // if (! $this->app->hasBeenBootstrapped()) {
        //     $this->app->bootstrapWith($this->bootstrappers());
        // }
    }

    /**
     * Get the route dispatcher callback.
     *
     * @return \Closure
     */
    protected function dispatchToRouter()
    {
        return function ($request) {
            $this->app->instance('request', $request);

            return $this->router->dispatch($request);
        };
    }

    /**
     * Call the terminate method on any terminable middleware.
     *
     * @param \Illuminate\Http\Request  $request
     * @param \Illuminate\Http\Response $response
     *
     * @return void
     */
    public function terminate($request, $response)
    {
        $this->terminateMiddleware($request, $response);

        $this->app->terminate();
    }

    /**
     * Call the terminate method on any terminable middleware.
     *
     * @param \Illuminate\Http\Request  $request
     * @param \Illuminate\Http\Response $response
     *
     * @return void
     */
    protected function terminateMiddleware($request, $response)
    {
        $middlewares = $this->app->shouldSkipMiddleware() ? [] : array_merge(
            $this->gatherRouteMiddleware($request),
            $this->middleware
        );

        foreach ($middlewares as $middleware) {
            if (!is_string($middleware)) {
                continue;
            }

            [$name] = $this->parseMiddleware($middleware);


            $instance = $this->app->make($name);

            if (method_exists($instance, 'terminate')) {
                $instance->terminate($request, $response);
            }
        }
    }

    /**
     * Gather the route middleware for the given request.
     *
     * @param \Illuminate\Http\Request $request
     *
     * @return array
     */
    protected function gatherRouteMiddleware($request)
    {
        if ($route = $request->route()) {
            return $this->router->gatherRouteMiddleware($route);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        }

        return [];
    }

    /**
     * Parse a middleware string to get the name and parameters.
     *
     * @param string $middleware
     *
     * @return array
     */
    protected function parseMiddleware($middleware)
    {
        [$name, $parameters] = array_pad(explode(':', $middleware, 2), 2, []);
function someFunction() {
    $x = 5;
    echo $x;
}

        if (is_string($parameters)) {
function someFunction() {
    $x = 5;
    echo $x;
}
            $parameters = explode(',', $parameters);
function someFunction() {
    $x = 5;
    echo $x;
}
        }

        return [$name, $parameters];
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
    }

    /**
     * Determine if the kernel has a given middleware.
     *
     * @param string $middleware
     *
     * @return bool
     */
    public function hasMiddleware($middleware)
    {
        return in_array($middleware, $this->middleware);
    }

    /**
     * Add a new middleware to beginning of the stack if it does not already exist.
     *
     * @param string $middleware
     *
     * @return $this
     */
    public function prependMiddleware($middleware)
    {
        if (array_search($middleware, $this->middleware) === false) {
            array_unshift($this->middleware, $middleware);
        }

        return $this;
    }

    /**
     * Add a new middleware to end of the stack if it does not already exist.
     *
     * @param string $middleware
     *
     * @return $this
     */
    public function pushMiddleware($middleware)
    {
        if (array_search($middleware, $this->middleware) === false) {
            $this->middleware[] = $middleware;
        }

        return $this;
    }

    /**
     * Prepend the given middleware to the given middleware group.
     *
     * @param string $group
     * @param string $middleware
     *
     * @throws \InvalidArgumentException
     *
     * @return $this
     */
    public function prependMiddlewareToGroup($group, $middleware)
    {
        if (!isset($this->middlewareGroups[$group])) {
            throw new InvalidArgumentException("The [{$group}] middleware group has not been defined.");
        }

        if (array_search($middleware, $this->middlewareGroups[$group]) === false) {
            array_unshift($this->middlewareGroups[$group], $middleware);
        }

        $this->syncMiddlewareToRouter();

        return $this;
    }

    /**
     * Append the given middleware to the given middleware group.
     *
     * @param string $group
     * @param string $middleware
     *
     * @throws \InvalidArgumentException
     *
     * @return $this
     */
    public function appendMiddlewareToGroup($group, $middleware)
    {
        if (!isset($this->middlewareGroups[$group])) {
            throw new InvalidArgumentException("The [{$group}] middleware group has not been defined.");
        }

        if (array_search($middleware, $this->middlewareGroups[$group]) === false) {
            $this->middlewareGroups[$group][] = $middleware;
        }

        $this->syncMiddlewareToRouter();

        return $this;
    }

    /**
     * Prepend the given middleware to the middleware priority list.
     *
     * @param string $middleware
     *
     * @return $this
     */
    public function prependToMiddlewarePriority($middleware)
    {
        if (!in_array($middleware, $this->middlewarePriority)) {
            array_unshift($this->middlewarePriority, $middleware);
        }

        $this->syncMiddlewareToRouter();

        return $this;
    }

    /**
     * Append the given middleware to the middleware priority list.
     *
     * @param string $middleware
     *
     * @return $this
     */
    public function appendToMiddlewarePriority($middleware)
    {
        if (!in_array($middleware, $this->middlewarePriority)) {
            $this->middlewarePriority[] = $middleware;
        }

        $this->syncMiddlewareToRouter();

        return $this;
    }

    /**
     * Sync the current state of the middleware to the router.
     *
     * @return void
     */
    protected function syncMiddlewareToRouter()
    {
        $this->router->middlewarePriority = $this->middlewarePriority;

        foreach ($this->middlewareGroups as $key => $middleware) {
            $this->router->middlewareGroup($key, $middleware);
        }

        foreach ($this->routeMiddleware as $key => $middleware) {
            $this->router->aliasMiddleware($key, $middleware);
        }
    }

    /**
     * Get the bootstrap classes for the application.
     *
     * @return array
     */
    protected function bootstrappers()
    {
        return $this->bootstrappers;
    }

    /**
     * Report the exception to the exception handler.
     *
     * @param \Throwable $e
     *
     * @return void
     */
    protected function reportException(Throwable $e)
    {
        $this->app[ExceptionHandler::class]->report($e);
    }

    /**
     * Render the exception to a response.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Throwable               $e
     *
     * @return \Symfony\Component\HttpFoundation\Response
     */
    protected function renderException($request, Throwable $e)
    {
        return $this->app[ExceptionHandler::class]->render($request, $e);
    }

    /**
     * Get the application's route middleware groups.
     *
     * @return array
     */
    public function getMiddlewareGroups()
    {
        return $this->middlewareGroups;
    }

    /**
     * Get the application's route middleware.
     *
     * @return array
     */
    public function getRouteMiddleware()
    {
        return $this->routeMiddleware;
    }

    /**
     * Get the Laravel application instance.
     *
     * @return \Illuminate\Contracts\Foundation\Application
     */
    public function getApplication()
    {
        return $this->app;
    }
}


Issues (57)

Security Analysis not enabled

src/Http/Kernel.php (10 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Available Fixes

1			<?php
2
3			namespace CodexShaper\WP\Http;
4
5			use CodexShaper\WP\Http\Events\RequestHandled;
6			use Illuminate\Contracts\Container\Container;
7			use Illuminate\Contracts\Debug\ExceptionHandler;
8			use Illuminate\Contracts\Http\Kernel as KernelContract;
9			use Illuminate\Routing\Pipeline;
10			use Illuminate\Routing\Router;
11			use Illuminate\Support\Facades\Facade;
12			use InvalidArgumentException;
13			use Throwable;
14
15			class Kernel implements KernelContract
16			{
17			/**
18			* The application implementation.
19			*
20			* @var \Illuminate\Contracts\Foundation\Application
21			*/
22			protected $app;
23
24			/**
25			* The router instance.
26			*
27			* @var \Illuminate\Routing\Router
28			*/
29			protected $router;
30
31			/**
32			* The bootstrap classes for the application.
33			*
34			* @var array
35			*/
36			protected $bootstrappers = [];
37
38			/**
39			* The application's middleware stack.
40			*
41			* @var array
42			*/
43			protected $middleware = [];
44
45			/**
46			* The application's route middleware groups.
47			*
48			* @var array
49			*/
50			protected $middlewareGroups = [];
51
52			/**
53			* The application's route middleware.
54			*
55			* @var array
56			*/
57			protected $routeMiddleware = [];
58
59			/**
60			* The priority-sorted list of middleware.
61			*
62			* Forces non-global middleware to always be in the given order.
63			*
64			* @var array
65			*/
66			protected $middlewarePriority = [
67			\Illuminate\Session\Middleware\StartSession::class,
68			\Illuminate\View\Middleware\ShareErrorsFromSession::class,
69			\Illuminate\Contracts\Auth\Middleware\AuthenticatesRequests::class,
70			\Illuminate\Routing\Middleware\ThrottleRequests::class,
71			\Illuminate\Session\Middleware\AuthenticateSession::class,
72			\Illuminate\Routing\Middleware\SubstituteBindings::class,
73			\Illuminate\Auth\Middleware\Authorize::class,
74			];
75
76			/**
77			* Create a new HTTP kernel instance.
78			*
79			* @param \Illuminate\Contracts\Foundation\Application $app
80			* @param \Illuminate\Routing\Router $router
81			*
82			* @return void
			0 ignored issues – show Comprehensibility Best Practice introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this Adding a `@return` annotation to constructors is generally not recommended as a constructor does not have a meaningful return value. Adding a `@return` annotation to a constructor is not recommended, since a constructor does not have a meaningful return value. Please refer to the PHP core documentation on constructors. Loading history...
83			*/
84			public function __construct(Container $app, Router $router)
85			{
86			$this->app = $app;
			0 ignored issues – show Documentation Bug introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$app` is of type `object<Illuminate\Contracts\Container\Container>`, but the property `$app` was declared to be of type `object<Illuminate\Contra...Foundation\Application>`. Are you sure that you always receive this specific sub-class here, or does it make sense to add an `instanceof` check? Our type inference engine has found a suspicous assignment of a value to a property. This check raises an issue when a value that can be of a given class or a super-class is assigned to a property that is type hinted more strictly. Either this assignment is in error or an instanceof check should be added for that assignment. class Alien {} class Dalek extends Alien {} class Plot { /** @var Dalek */ public $villain; } $alien = new Alien(); $plot = new Plot(); if ($alien instanceof Dalek) { $plot->villain = $alien; } Loading history...
87			$this->router = $router;
88
89			$this->syncMiddlewareToRouter();
90			}
91
92			/**
93			* Handle an incoming HTTP request.
94			*
95			* @param \Illuminate\Http\Request $request
96			*
97			* @return \Illuminate\Http\Response
98			*/
99			public function handle($request)
100			{
101			try {
102			$request->enableHttpMethodParameterOverride();
103			$response = $this->sendRequestThroughRouter($request);
104			} catch (Throwable $e) {
105			throw new \Exception($e, 1);
106			$this->reportException($e);
			0 ignored issues – show Unused Code introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$this->reportException($e);` does not seem to be reachable. This check looks for unreachable code. It uses sophisticated control flow analysis techniques to find statements which will never be executed. Unreachable code is most often the result of `return`, `die` or `exit` statements that have been added for debug purposes. function fx() { try { doSomething(); return true; } catch (\Exception $e) { return false; } return false; } In the above example, the last `return false` will never be executed, because a return statement has already been met in every possible execution path. Loading history...
107
108			$response = $this->renderException($request, $e);
109			}
110
111			$this->app['events']->dispatch(
112			new RequestHandled($request, $response)
113			);
114
115			return $response;
116			}
117
118			/**
119			* Send the given request through the middleware / router.
120			*
121			* @param \Illuminate\Http\Request $request
122			*
123			* @return \Illuminate\Http\Response
124			*/
125			protected function sendRequestThroughRouter($request)
126			{
127			$this->app->instance('request', $request);
128
129			Facade::clearResolvedInstance('request');
130
131			$this->bootstrap();
132
133			return (new Pipeline($this->app))
134			->send($request)
135			// ->through($this->app->shouldSkipMiddleware() ? [] : $this->middleware)
136			->through($this->middleware)
137			->then($this->dispatchToRouter());
138			}
139
140			/**
141			* Bootstrap the application for HTTP requests.
142			*
143			* @return void
144			*/
145			public function bootstrap()
146			{
147			// if (! $this->app->hasBeenBootstrapped()) {
148			// $this->app->bootstrapWith($this->bootstrappers());
149			// }
150			}
151
152			/**
153			* Get the route dispatcher callback.
154			*
155			* @return \Closure
156			*/
157			protected function dispatchToRouter()
158			{
159			return function ($request) {
160			$this->app->instance('request', $request);
161
162			return $this->router->dispatch($request);
163			};
164			}
165
166			/**
167			* Call the terminate method on any terminable middleware.
168			*
169			* @param \Illuminate\Http\Request $request
170			* @param \Illuminate\Http\Response $response
171			*
172			* @return void
173			*/
174			public function terminate($request, $response)
175			{
176			$this->terminateMiddleware($request, $response);
177
178			$this->app->terminate();
179			}
180
181			/**
182			* Call the terminate method on any terminable middleware.
183			*
184			* @param \Illuminate\Http\Request $request
185			* @param \Illuminate\Http\Response $response
186			*
187			* @return void
188			*/
189			protected function terminateMiddleware($request, $response)
190			{
191			$middlewares = $this->app->shouldSkipMiddleware() ? [] : array_merge(
192			$this->gatherRouteMiddleware($request),
193			$this->middleware
194			);
195
196			foreach ($middlewares as $middleware) {
197			if (!is_string($middleware)) {
198			continue;
199			}
200
201			[$name] = $this->parseMiddleware($middleware);
			0 ignored issues – show Bug introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$name` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history...
202
203			$instance = $this->app->make($name);
204
205			if (method_exists($instance, 'terminate')) {
206			$instance->terminate($request, $response);
207			}
208			}
209			}
210
211			/**
212			* Gather the route middleware for the given request.
213			*
214			* @param \Illuminate\Http\Request $request
215			*
216			* @return array
217			*/
218			protected function gatherRouteMiddleware($request)
219			{
220			if ($route = $request->route()) {
221			return $this->router->gatherRouteMiddleware($route);
			0 ignored issues – show Documentation introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this `$route` is of type `object\|string`, but the function expects a `object<Illuminate\Routing\Route>`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
222			}
223
224			return [];
225			}
226
227			/**
228			* Parse a middleware string to get the name and parameters.
229			*
230			* @param string $middleware
231			*
232			* @return array
233			*/
234			protected function parseMiddleware($middleware)
235			{
236			[$name, $parameters] = array_pad(explode(':', $middleware, 2), 2, []);
			0 ignored issues – show Bug introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$name` does not exist. Did you forget to declare it? This check marks access to variables or properties that have not been declared yet. While PHP has no explicit notion of declaring a variable, accessing it before a value is assigned to it is most likely a bug. Loading history... Bug introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$parameters` seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition? This error can happen if you refactor code and forget to move the variable initialization. Let’s take a look at a simple example: function someFunction() { $x = 5; echo $x; } The above code is perfectly fine. Now imagine that we re-order the statements: function someFunction() { echo $x; $x = 5; } In that case, `$x` would be read before it is initialized. This was a very basic example, however the principle is the same for the found issue. Loading history...
237
238			if (is_string($parameters)) {
			0 ignored issues – show Bug introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$parameters` seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition? This error can happen if you refactor code and forget to move the variable initialization. Let’s take a look at a simple example: function someFunction() { $x = 5; echo $x; } The above code is perfectly fine. Now imagine that we re-order the statements: function someFunction() { echo $x; $x = 5; } In that case, `$x` would be read before it is initialized. This was a very basic example, however the principle is the same for the found issue. Loading history...
239			$parameters = explode(',', $parameters);
			0 ignored issues – show Bug introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$parameters` seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition? This error can happen if you refactor code and forget to move the variable initialization. Let’s take a look at a simple example: function someFunction() { $x = 5; echo $x; } The above code is perfectly fine. Now imagine that we re-order the statements: function someFunction() { echo $x; $x = 5; } In that case, `$x` would be read before it is initialized. This was a very basic example, however the principle is the same for the found issue. Loading history...
240			}
241
242			return [$name, $parameters];
			0 ignored issues – show Bug introduced 2020-06-18 07:41 UTC by Report Bug Copy Issue Report Show Similar Issues like this The variable `$parameters` does not seem to be defined for all execution paths leading up to this point. If you define a variable conditionally, it can happen that it is not defined for all execution paths. Let’s take a look at an example: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } // $x is potentially undefined here. echo $x; } In the above example, the variable `$x` is defined if you pass “foo” or “bar” as argument for `$a`. However, since the `switch` statement has no default case statement, if you pass any other value, the variable `$x` would be undefined. Available Fixes Check for existence of the variable explicitly: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } if (isset($x)) { // Make sure it's always set. echo $x; } } Define a default value for the variable: function myFunction($a) { $x = ''; // Set a default which gets overridden for certain paths. switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; } echo $x; } Add a value for the missing path: function myFunction($a) { switch ($a) { case 'foo': $x = 1; break; case 'bar': $x = 2; break; // We add support for the missing case. default: $x = ''; break; } echo $x; } Loading history...
243			}
244
245			/**
246			* Determine if the kernel has a given middleware.
247			*
248			* @param string $middleware
249			*
250			* @return bool
251			*/
252			public function hasMiddleware($middleware)
253			{
254			return in_array($middleware, $this->middleware);
255			}
256
257			/**
258			* Add a new middleware to beginning of the stack if it does not already exist.
259			*
260			* @param string $middleware
261			*
262			* @return $this
263			*/
264			public function prependMiddleware($middleware)
265			{
266			if (array_search($middleware, $this->middleware) === false) {
267			array_unshift($this->middleware, $middleware);
268			}
269
270			return $this;
271			}
272
273			/**
274			* Add a new middleware to end of the stack if it does not already exist.
275			*
276			* @param string $middleware
277			*
278			* @return $this
279			*/
280			public function pushMiddleware($middleware)
281			{
282			if (array_search($middleware, $this->middleware) === false) {
283			$this->middleware[] = $middleware;
284			}
285
286			return $this;
287			}
288
289			/**
290			* Prepend the given middleware to the given middleware group.
291			*
292			* @param string $group
293			* @param string $middleware
294			*
295			* @throws \InvalidArgumentException
296			*
297			* @return $this
298			*/
299			public function prependMiddlewareToGroup($group, $middleware)
300			{
301			if (!isset($this->middlewareGroups[$group])) {
302			throw new InvalidArgumentException("The [{$group}] middleware group has not been defined.");
303			}
304
305			if (array_search($middleware, $this->middlewareGroups[$group]) === false) {
306			array_unshift($this->middlewareGroups[$group], $middleware);
307			}
308
309			$this->syncMiddlewareToRouter();
310
311			return $this;
312			}
313
314			/**
315			* Append the given middleware to the given middleware group.
316			*
317			* @param string $group
318			* @param string $middleware
319			*
320			* @throws \InvalidArgumentException
321			*
322			* @return $this
323			*/
324			public function appendMiddlewareToGroup($group, $middleware)
325			{
326			if (!isset($this->middlewareGroups[$group])) {
327			throw new InvalidArgumentException("The [{$group}] middleware group has not been defined.");
328			}
329
330			if (array_search($middleware, $this->middlewareGroups[$group]) === false) {
331			$this->middlewareGroups[$group][] = $middleware;
332			}
333
334			$this->syncMiddlewareToRouter();
335
336			return $this;
337			}
338
339			/**
340			* Prepend the given middleware to the middleware priority list.
341			*
342			* @param string $middleware
343			*
344			* @return $this
345			*/
346			public function prependToMiddlewarePriority($middleware)
347			{
348			if (!in_array($middleware, $this->middlewarePriority)) {
349			array_unshift($this->middlewarePriority, $middleware);
350			}
351
352			$this->syncMiddlewareToRouter();
353
354			return $this;
355			}
356
357			/**
358			* Append the given middleware to the middleware priority list.
359			*
360			* @param string $middleware
361			*
362			* @return $this
363			*/
364			public function appendToMiddlewarePriority($middleware)
365			{
366			if (!in_array($middleware, $this->middlewarePriority)) {
367			$this->middlewarePriority[] = $middleware;
368			}
369
370			$this->syncMiddlewareToRouter();
371
372			return $this;
373			}
374
375			/**
376			* Sync the current state of the middleware to the router.
377			*
378			* @return void
379			*/
380			protected function syncMiddlewareToRouter()
381			{
382			$this->router->middlewarePriority = $this->middlewarePriority;
383
384			foreach ($this->middlewareGroups as $key => $middleware) {
385			$this->router->middlewareGroup($key, $middleware);
386			}
387
388			foreach ($this->routeMiddleware as $key => $middleware) {
389			$this->router->aliasMiddleware($key, $middleware);
390			}
391			}
392
393			/**
394			* Get the bootstrap classes for the application.
395			*
396			* @return array
397			*/
398			protected function bootstrappers()
399			{
400			return $this->bootstrappers;
401			}
402
403			/**
404			* Report the exception to the exception handler.
405			*
406			* @param \Throwable $e
407			*
408			* @return void
409			*/
410			protected function reportException(Throwable $e)
411			{
412			$this->app[ExceptionHandler::class]->report($e);
413			}
414
415			/**
416			* Render the exception to a response.
417			*
418			* @param \Illuminate\Http\Request $request
419			* @param \Throwable $e
420			*
421			* @return \Symfony\Component\HttpFoundation\Response
422			*/
423			protected function renderException($request, Throwable $e)
424			{
425			return $this->app[ExceptionHandler::class]->render($request, $e);
426			}
427
428			/**
429			* Get the application's route middleware groups.
430			*
431			* @return array
432			*/
433			public function getMiddlewareGroups()
434			{
435			return $this->middlewareGroups;
436			}
437
438			/**
439			* Get the application's route middleware.
440			*
441			* @return array
442			*/
443			public function getRouteMiddleware()
444			{
445			return $this->routeMiddleware;
446			}
447
448			/**
449			* Get the Laravel application instance.
450			*
451			* @return \Illuminate\Contracts\Foundation\Application
452			*/
453			public function getApplication()
454			{
455			return $this->app;
456			}
457			}
458

Codexshaper / wpb-foundation

Issues (57)

Security Analysis not enabled

src/Http/Kernel.php (10 issues)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Available Fixes

Duplication Side-by-Side

Filter issues like