Issues in LaravelKernel.php - Fixed Issues - Inspection of "documentの修正" - polidog/LaravelBundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fc5d0f...a9d38a )

by Ryota

created 2016-07-17 16:54 UTC

LaravelKernel.php (1 issue)

Labels

Documentation 1

Severity

Minor 1

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
/**
 * Created by PhpStorm.
 * User: polidog
 * Date: 2016/07/17.
 */

namespace Polidog\LaravelBundle;

use Illuminate\Contracts\Http\Kernel;
use Illuminate\Foundation\Application;

class LaravelKernel implements Kernel
{
    /**
     * @var Kernel
     */
    private $kernel;

    /**
     * @var Application
     */
    private $app;

    /**
     * LaravelKernel constructor.
     *
     * @param string $bootstrapFile
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
     */
    public function __construct(Bootstrap $bootstrap)
    {
        $this->app = $bootstrap->load();
        $this->kernel = $this->app->make(Kernel::class);
    }

    /**
     * {@inheritdoc}
     */
    public function handle($request)
    {
        return $this->kernel->handle($request);
    }

    /**
     * {@inheritdoc}
     */
    public function terminate($request, $response)
    {
        $this->kernel->terminate($request, $response);
    }

    /**
     * {@inheritdoc}
     */
    public function getApplication()
    {
        return $this->app;
    }

    /**
     * {@inheritdoc}
     */
    public function bootstrap()
    {
        // no support.
    }

}


1			<?php
2			/**
3			* Created by PhpStorm.
4			* User: polidog
5			* Date: 2016/07/17.
6			*/
7
8			namespace Polidog\LaravelBundle;
9
10			use Illuminate\Contracts\Http\Kernel;
11			use Illuminate\Foundation\Application;
12
13			class LaravelKernel implements Kernel
14			{
15			/**
16			* @var Kernel
17			*/
18			private $kernel;
19
20			/**
21			* @var Application
22			*/
23			private $app;
24
25			/**
26			* LaravelKernel constructor.
27			*
28			* @param string $bootstrapFile
			0 ignored issues – show Documentation introduced 2016-07-17 16:08 UTC by Report Bug Copy Issue Report Show Similar Issues like this There is no parameter named `$bootstrapFile`. Did you maybe mean `$bootstrap`? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. It has, however, found a similar but not annotated parameter which might be a good fit. Consider the following example. The parameter `$ireland` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $ireland */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was changed, but the annotation was not. Loading history...
29			*/
30			public function __construct(Bootstrap $bootstrap)
31			{
32			$this->app = $bootstrap->load();
33			$this->kernel = $this->app->make(Kernel::class);
34			}
35
36			/**
37			* {@inheritdoc}
38			*/
39			public function handle($request)
40			{
41			return $this->kernel->handle($request);
42			}
43
44			/**
45			* {@inheritdoc}
46			*/
47			public function terminate($request, $response)
48			{
49			$this->kernel->terminate($request, $response);
50			}
51
52			/**
53			* {@inheritdoc}
54			*/
55			public function getApplication()
56			{
57			return $this->app;
58			}
59
60			/**
61			* {@inheritdoc}
62			*/
63			public function bootstrap()
64			{
65			// no support.
66			}
67
68			}
69

polidog / LaravelBundle

Push — master ( fc5d0f...a9d38a )

LaravelKernel.php (1 issue)

Labels

Severity

Introduced By

Upgrade to new PHP Analysis Engine

Duplication Side-by-Side

Filter issues like