Completed
Push — master ( c25059...aa3601 )
by i
04:27
created

ScanPayment   A

Complexity

Total Complexity 1

Size/Duplication

Total Lines 25
Duplicated Lines 0 %

Test Coverage

Coverage 0%

Importance

Changes 0
Metric Value
dl 0
loc 25
ccs 0
cts 7
cp 0
rs 10
c 0
b 0
f 0
wmc 1

1 Method

Rating   Name   Duplication   Size   Complexity  
A toPay() 0 9 1
1
<?php
2
3
namespace Nilnice\Payment\Wechat;
4
5
use Illuminate\Support\Collection;
6
use Nilnice\Payment\Constant;
7
use Nilnice\Payment\Log;
8
9
class ScanPayment extends AbstractWechat
10
{
11
    /**
12
     * Use scan to pay for order.
13
     *
14
     * @param string $gateway
15
     * @param array  $payload
16
     *
17
     * @return \Illuminate\Support\Collection
18
     *
19
     * @throws \InvalidArgumentException
20
     * @throws \Nilnice\Payment\Exception\GatewayException
21
     * @throws \Nilnice\Payment\Exception\InvalidKeyException
22
     * @throws \Nilnice\Payment\Exception\InvalidSignException
23
     * @throws \RuntimeException
24
     */
25
    public function toPay(string $gateway, array $payload) : Collection
26
    {
27
        $payload['spbill_create_ip'] = $this->getClientIp();
28
        $payload['trade_type'] = Constant::WX_PAY_SCAN_TYPE;
29
        $gateway .= Constant::WX_PAY_PREPARE;
30
31
        Log::debug('Scan order:', [$gateway, $payload]);
32
33
        return $this->prepare($gateway, $payload, 'scan');
0 ignored issues
show
Unused Code introduced by
The call to Nilnice\Payment\Wechat\AbstractWechat::prepare() has too many arguments starting with 'scan'. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-call  annotation

33
        return $this->/** @scrutinizer ignore-call */ prepare($gateway, $payload, 'scan');

This check compares calls to functions or methods with their respective definitions. If the call has more arguments than are defined, it raises an issue.

If a function is defined several times with a different number of parameters, the check may pick up the wrong definition and report false positives. One codebase where this has been known to happen is Wordpress. Please note the @ignore annotation hint above.

Loading history...
34
    }
35
}
36