Guard::isSafeMode() - Code Metrics - Inspection of "Fix #1295" - overtrue/wechat - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c56f2f...7cf3e7 )

by Carlos

created 2018-07-11 07:19 UTC

Guard::isSafeMode() A

↳ Parent: Guard

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

/*
 * This file is part of the overtrue/wechat.
 *
 * (c) overtrue <[email protected]>
 *
 * This source file is subject to the MIT license that is bundled
 * with this source code in the file LICENSE.
 */

namespace EasyWeChat\Work\Server;

use EasyWeChat\Kernel\ServerGuard;

/**
 * Class Guard.
 *
 * @author overtrue <[email protected]>
 */
class Guard extends ServerGuard
{
    /**
     * @return bool
     */
    public function validate()
    {
        return $this;

    }
    
    /**
     * Check the request message safe mode.
     *
     * @return bool
     */
    protected function isSafeMode(): bool
    {
        return true;
    }

    /**
     * @return bool
     */
    protected function shouldReturnRawResponse(): bool
    {
        return !is_null($this->app['request']->get('echostr'));
    }
}


1			<?php
2
3			/*
4			* This file is part of the overtrue/wechat.
5			*
6			* (c) overtrue <[email protected]>
7			*
8			* This source file is subject to the MIT license that is bundled
9			* with this source code in the file LICENSE.
10			*/
11
12			namespace EasyWeChat\Work\Server;
13
14			use EasyWeChat\Kernel\ServerGuard;
15
16			/**
17			* Class Guard.
18			*
19			* @author overtrue <[email protected]>
20			*/
21			class Guard extends ServerGuard
22			{
23			/**
24			* @return bool
25			*/
26			public function validate()
27			{
28			return $this;
			0 ignored issues – show Bug Best Practice introduced 2018-07-11 07:21 UTC by Report Bug Copy Issue Report The expression `return $this` returns the type `EasyWeChat\Work\Server\Guard` which is incompatible with the documented return type `boolean`. Loading history...
29			}
30
31			/**
32			* Check the request message safe mode.
33			*
34			* @return bool
35			*/
36			protected function isSafeMode(): bool
37			{
38			return true;
39			}
40
41			/**
42			* @return bool
43			*/
44			protected function shouldReturnRawResponse(): bool
45			{
46			return !is_null($this->app['request']->get('echostr'));
47			}
48			}
49

overtrue / wechat

Push — master ( c56f2f...7cf3e7 )

Guard::isSafeMode() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like