Completed
Push — master ( 189007...259fd8 )
by Dmitry
08:51
created

PaxumMerchant   A

Complexity

Total Complexity 3

Size/Duplication

Total Lines 55
Duplicated Lines 0 %

Coupling/Cohesion

Components 2
Dependencies 8

Test Coverage

Coverage 0%

Importance

Changes 0
Metric Value
wmc 3
lcom 2
cbo 8
dl 0
loc 55
ccs 0
cts 18
cp 0
rs 10
c 0
b 0
f 0

3 Methods

Rating   Name   Duplication   Size   Complexity  
A createGateway() 0 7 1
A requestPurchase() 0 17 1
A completePurchase() 0 13 1
1
<?php
2
3
namespace hiqdev\php\merchant\merchants\paxum;
4
5
use hiqdev\php\merchant\InvoiceInterface;
6
use hiqdev\php\merchant\merchants\AbstractMerchant;
7
use hiqdev\php\merchant\response\CompletePurchaseResponse;
8
use hiqdev\php\merchant\response\RedirectPurchaseResponse;
9
use Omnipay\WebMoney\Gateway;
10
11
/**
12
 * Class PaxumMerchant
13
 *
14
 * @author Dmytro Naumenko <[email protected]>
15
 */
16
class PaxumMerchant extends AbstractMerchant
17
{
18
    /**
19
     * @var Gateway
20
     */
21
    protected $gateway;
22
23
    protected function createGateway()
24
    {
25
        return $this->gatewayFactory->build('Paxum', [
26
            'purse' => $this->credentials->getPurse(),
27
            'secret'  => $this->credentials->getKey1(),
28
        ]);
29
    }
30
31
    /**
32
     * @param InvoiceInterface $invoice
33
     * @return RedirectPurchaseResponse
34
     */
35
    public function requestPurchase(InvoiceInterface $invoice)
36
    {
37
        /**
38
         * @var \Omnipay\Paxum\Message\PurchaseResponse $response
39
         */
40
        $response = $this->gateway->purchase([
41
            'transactionId' => $invoice->getId(),
42
            'description' => $invoice->getDescription(),
43
            'amount' => $this->moneyFormatter->format($invoice->getAmount()),
44
            'currency' => $invoice->getCurrency()->getCode(),
45
            'returnUrl' => $invoice->getReturnUrl(),
46
            'notifyUrl' => $invoice->getNotifyUrl(),
47
            'cancelUrl' => $invoice->getCancelUrl(),
48
        ])->send();
49
50
        return new RedirectPurchaseResponse($response->getRedirectUrl(), $response->getRedirectData());
51
    }
52
53
    /**
54
     * @param array $data
55
     * @return CompletePurchaseResponse
56
     */
57
    public function completePurchase($data)
58
    {
59
        /** @var \Omnipay\Paxum\Message\CompletePurchaseResponse $response */
60
        $response = $this->gateway->completePurchase($data)->send();
61
62
        return (new CompletePurchaseResponse())
63
            ->setIsSuccessful($response->isSuccessful())
0 ignored issues
show
Documentation introduced by
$response->isSuccessful() is of type boolean, but the function expects a string.

It seems like the type of the argument is not accepted by the function/method which you are calling.

In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug.

We suggest to add an explicit type cast like in the following example:

function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
Loading history...
64
            ->setAmount($this->moneyParser->parse($response->getAmount(), $response->getCurrency()))
65
            ->setTransactionReference($response->getTransactionReference())
66
            ->setTransactionId($response->getTransactionId())
67
            ->setPayer($response->getPayer())
68
            ->setTime(new \DateTime($response->getTime()));
69
    }
70
}
71