QueryShortcut::getPlugins() - Code Metrics - Inspection of "change(internal): 微信 shortcut 从 plugin 文件夹独立出来 (#9..." - yansongda/pay - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( a85346...8e58b4 )

by Songda

created 2023-12-22 09:34 UTC

QueryShortcut::getPlugins() A

↳ Parent: QueryShortcut

Complexity

Conditions	3
Paths	3

Size

Total Lines	13
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

declare(strict_types=1);

namespace Yansongda\Pay\Shortcut\Wechat;

use Yansongda\Pay\Contract\ShortcutInterface;
use Yansongda\Pay\Exception\Exception;
use Yansongda\Pay\Exception\InvalidParamsException;
use Yansongda\Pay\Plugin\ParserPlugin;
use Yansongda\Pay\Plugin\Wechat\LaunchPlugin;
use Yansongda\Pay\Plugin\Wechat\Pay\Common\QueryPlugin;
use Yansongda\Pay\Plugin\Wechat\Pay\Common\QueryRefundPlugin;
use Yansongda\Pay\Plugin\Wechat\PreparePlugin;
use Yansongda\Pay\Plugin\Wechat\RadarSignPlugin;
use Yansongda\Supports\Str;

class QueryShortcut implements ShortcutInterface
{
    /**
     * @throws InvalidParamsException
     */
    public function getPlugins(array $params): array
    {
        if (isset($params['combine_out_trade_no'])) {
            return $this->combinePlugins();
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.
    }
}
        }

        $typeMethod = Str::camel($params['_action'] ?? 'default').'Plugins';

        if (method_exists($this, $typeMethod)) {
            return $this->{$typeMethod}();
        }

        throw new InvalidParamsException(Exception::PARAMS_SHORTCUT_ACTION_INVALID, "Query action [{$typeMethod}] not supported");
    }

    protected function defaultPlugins(): array
    {
        return [
            PreparePlugin::class,
            QueryPlugin::class,
            RadarSignPlugin::class,
            LaunchPlugin::class,
            ParserPlugin::class,
        ];
    }

    protected function refundPlugins(): array
    {
        return [
            PreparePlugin::class,
            QueryRefundPlugin::class,
            RadarSignPlugin::class,
            LaunchPlugin::class,
            ParserPlugin::class,
        ];
    }

    protected function combinePlugins(): array
    {
        return [
            PreparePlugin::class,
            \Yansongda\Pay\Plugin\Wechat\Pay\Combine\QueryPlugin::class,
            RadarSignPlugin::class,
            LaunchPlugin::class,
            ParserPlugin::class,
        ];
    }
}


1			<?php
2
3			declare(strict_types=1);
4
5			namespace Yansongda\Pay\Shortcut\Wechat;
6
7			use Yansongda\Pay\Contract\ShortcutInterface;
8			use Yansongda\Pay\Exception\Exception;
9			use Yansongda\Pay\Exception\InvalidParamsException;
10			use Yansongda\Pay\Plugin\ParserPlugin;
11			use Yansongda\Pay\Plugin\Wechat\LaunchPlugin;
12			use Yansongda\Pay\Plugin\Wechat\Pay\Common\QueryPlugin;
13			use Yansongda\Pay\Plugin\Wechat\Pay\Common\QueryRefundPlugin;
14			use Yansongda\Pay\Plugin\Wechat\PreparePlugin;
15			use Yansongda\Pay\Plugin\Wechat\RadarSignPlugin;
16			use Yansongda\Supports\Str;
17
18			class QueryShortcut implements ShortcutInterface
19			{
20			/**
21			* @throws InvalidParamsException
22			*/
23			public function getPlugins(array $params): array
24			{
25			if (isset($params['combine_out_trade_no'])) {
26			return $this->combinePlugins();
			0 ignored issues – show Bug Best Practice introduced 2021-06-21 08:24 UTC by Report Bug Copy Issue Report The expression `return $this->combinePlugins()` 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...
27			}
28
29			$typeMethod = Str::camel($params['_action'] ?? 'default').'Plugins';
30
31			if (method_exists($this, $typeMethod)) {
32			return $this->{$typeMethod}();
33			}
34
35			throw new InvalidParamsException(Exception::PARAMS_SHORTCUT_ACTION_INVALID, "Query action [{$typeMethod}] not supported");
36			}
37
38			protected function defaultPlugins(): array
39			{
40			return [
41			PreparePlugin::class,
42			QueryPlugin::class,
43			RadarSignPlugin::class,
44			LaunchPlugin::class,
45			ParserPlugin::class,
46			];
47			}
48
49			protected function refundPlugins(): array
50			{
51			return [
52			PreparePlugin::class,
53			QueryRefundPlugin::class,
54			RadarSignPlugin::class,
55			LaunchPlugin::class,
56			ParserPlugin::class,
57			];
58			}
59
60			protected function combinePlugins(): array
61			{
62			return [
63			PreparePlugin::class,
64			\Yansongda\Pay\Plugin\Wechat\Pay\Combine\QueryPlugin::class,
65			RadarSignPlugin::class,
66			LaunchPlugin::class,
67			ParserPlugin::class,
68			];
69			}
70			}
71

yansongda / pay

Push — master ( a85346...8e58b4 )

QueryShortcut::getPlugins() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like