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

ScanPayment::toPay()   A

Complexity

Conditions 1
Paths 1

Size

Total Lines 9
Code Lines 5

Duplication

Lines 0
Ratio 0 %

Code Coverage

Tests 0
CRAP Score 2

Importance

Changes 0
Metric Value
cc 1
eloc 5
nc 1
nop 2
dl 0
loc 9
ccs 0
cts 7
cp 0
crap 2
rs 9.6666
c 0
b 0
f 0
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