SprintExpired::handle() - Code Metrics - renatomarinho/laravel-gitscrum - Measure and Improve Code Quality continuously with Scrutinizer

SprintExpired::handle() A
last analyzed 2017-01-12 17:42 UTC

↳ Parent: SprintExpired

Complexity

Conditions	3
Paths	3

Size

Total Lines	16
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	16
rs	9.4285
c	0
b	0
f	0
cc	3
eloc	9
nc	3
nop	2

<?php
/**
 * GitScrum v0.1.
 *
 * @author  Renato Marinho <[email protected]>
 * @license http://opensource.org/licenses/GPL-3.0 GPLv3
 */

namespace GitScrum\Http\Middleware;

use Closure;
use GitScrum\Models\Sprint;
use Carbon\Carbon;

class SprintExpired
{
    /**
     * Handle an incoming request.
     *
     * @param \Illuminate\Http\Request $request
     * @param \Closure                 $next
     *
     * @return mixed
     */
    public function handle($request, Closure $next)
    {
        if ($slug = $request->slug) {
            $sprint = Sprint::slug($slug)
                ->whereDate('date_finish', '<', Carbon::now()->format('Y-m-d'))
                //->whereNull('closed_at')
                ->first();

            if ($sprint) {
                $msg = trans('This Sprint has been expired at').' '.$sprint->date_finish;
                view()->share('notification', ['alert' => 'danger', 'message' => $msg]);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
            }
        }

        return $next($request);
    }
}


1			<?php
2			/**
3			* GitScrum v0.1.
4			*
5			* @author Renato Marinho <[email protected]>
6			* @license http://opensource.org/licenses/GPL-3.0 GPLv3
7			*/
8
9			namespace GitScrum\Http\Middleware;
10
11			use Closure;
12			use GitScrum\Models\Sprint;
13			use Carbon\Carbon;
14
15			class SprintExpired
16			{
17			/**
18			* Handle an incoming request.
19			*
20			* @param \Illuminate\Http\Request $request
21			* @param \Closure $next
22			*
23			* @return mixed
24			*/
25			public function handle($request, Closure $next)
26			{
27			if ($slug = $request->slug) {
28			$sprint = Sprint::slug($slug)
29			->whereDate('date_finish', '<', Carbon::now()->format('Y-m-d'))
30			//->whereNull('closed_at')
31			->first();
32
33			if ($sprint) {
34			$msg = trans('This Sprint has been expired at').' '.$sprint->date_finish;
35			view()->share('notification', ['alert' => 'danger', 'message' => $msg]);
			0 ignored issues – show Bug introduced 2016-11-26 09:47 UTC by Report Bug Copy Issue Report The method `share` does only exist in `Illuminate\Contracts\View\Factory`, but not in `Illuminate\View\View`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
36			}
37			}
38
39			return $next($request);
40			}
41			}
42

renatomarinho / laravel-gitscrum

SprintExpired::handle() A last analyzed 2017-01-12 17:42 UTC

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like

SprintExpired::handle() A
last analyzed 2017-01-12 17:42 UTC