PapayShortcut::getInvoke() - Code Metrics - Inspection of "feat: 增加委托代扣 shortcut" - yansongda/pay - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#773)

by Songda

created 2023-04-09 15:34 UTC

PapayShortcut::getInvoke() A

↳ Parent: PapayShortcut

Complexity

Conditions	3
Paths	3

Size

Total Lines	9
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

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

<?php

declare(strict_types=1);

namespace Yansongda\Pay\Plugin\Wechat\Shortcut;

use Yansongda\Pay\Contract\ShortcutInterface;
use Yansongda\Pay\Plugin\ParserPlugin;
use Yansongda\Pay\Plugin\Wechat\Papay\ContractOrderPlugin;
use Yansongda\Pay\Plugin\Wechat\Pay\Common\InvokePrepayV2Plugin;
use Yansongda\Pay\Plugin\Wechat\PreparePlugin;
use Yansongda\Pay\Plugin\Wechat\RadarSignPlugin;

/**
 * 支付中签约.
 *
 * @see https://pay.weixin.qq.com/wiki/doc/api/wxpay_v2/papay/chapter3_5.shtml
 */
class PapayShortcut implements ShortcutInterface
{
    public function getPlugins(array $params): array
    {
        return [
interface HasName {
    /** @return string */
    public function getName();
}

class Name {
    public $name;
}

class User implements HasName {
    /** @return string|Name */
    public function getName() {
        return new Name('foo'); // This is a violation of the ``HasName`` interface
                                // which only allows a string value to be returned.
    }
}
            PreparePlugin::class,
            ContractOrderPlugin::class,
            RadarSignPlugin::class,
            $this->getInvoke($params),
            ParserPlugin::class,
        ];
    }

    protected function getInvoke(array $params): string
    {
        switch ($params['_type']) {
            case 'app':
                return \Yansongda\Pay\Plugin\Wechat\Pay\App\InvokePrepayV2Plugin::class;
            case 'mini':
                return \Yansongda\Pay\Plugin\Wechat\Pay\Mini\InvokePrepayV2Plugin::class;
            default:
                return InvokePrepayV2Plugin::class;
        }
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Yansongda\Pay\Plugin\Wechat\Shortcut;
6
7			use Yansongda\Pay\Contract\ShortcutInterface;
8			use Yansongda\Pay\Plugin\ParserPlugin;
9			use Yansongda\Pay\Plugin\Wechat\Papay\ContractOrderPlugin;
10			use Yansongda\Pay\Plugin\Wechat\Pay\Common\InvokePrepayV2Plugin;
11			use Yansongda\Pay\Plugin\Wechat\PreparePlugin;
12			use Yansongda\Pay\Plugin\Wechat\RadarSignPlugin;
13
14			/**
15			* 支付中签约.
16			*
17			* @see https://pay.weixin.qq.com/wiki/doc/api/wxpay_v2/papay/chapter3_5.shtml
18			*/
19			class PapayShortcut implements ShortcutInterface
20			{
21			public function getPlugins(array $params): array
22			{
23			return [
			0 ignored issues – show Bug Best Practice introduced 2023-04-09 15:36 UTC by Report Bug Copy Issue Report The expression `return array(Yansongda\P...in\ParserPlugin::class)` returns the type `array<integer,string>` which is incompatible with the return type mandated by `Yansongda\Pay\Contract\S...Interface::getPlugins()` of `Yansongda\Pay\Contract\PluginInterface[]`. In the issue above, the returned value is violating the contract defined by the mentioned interface. Let's take a look at an example: interface HasName { /** @return string / public function getName(); } class Name { public $name; } class User implements HasName { /* @return string\|Name */ public function getName() { return new Name('foo'); // This is a violation of the ``HasName`` interface // which only allows a string value to be returned. } } Loading history...
24			PreparePlugin::class,
25			ContractOrderPlugin::class,
26			RadarSignPlugin::class,
27			$this->getInvoke($params),
28			ParserPlugin::class,
29			];
30			}
31
32			protected function getInvoke(array $params): string
33			{
34			switch ($params['_type']) {
35			case 'app':
36			return \Yansongda\Pay\Plugin\Wechat\Pay\App\InvokePrepayV2Plugin::class;
37			case 'mini':
38			return \Yansongda\Pay\Plugin\Wechat\Pay\Mini\InvokePrepayV2Plugin::class;
39			default:
40			return InvokePrepayV2Plugin::class;
41			}
42			}
43			}
44

yansongda / pay

Pull Request — master (#773)

PapayShortcut::getInvoke() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like