Completed
Push — master ( 576b56...533c2b )
by Dmitry
14:42
created

src/merchants/freekassa/FreeKassaMerchant.php (1 issue)

Upgrade to new PHP Analysis Engine

These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more

1
<?php
2
/**
3
 * Generalization over Omnipay and Payum
4
 *
5
 * @link      https://github.com/hiqdev/php-merchant
6
 * @package   php-merchant
7
 * @license   BSD-3-Clause
8
 * @copyright Copyright (c) 2015-2018, HiQDev (http://hiqdev.com/)
9
 */
10
11
namespace hiqdev\php\merchant\merchants\freekassa;
12
13
use hiqdev\php\merchant\InvoiceInterface;
14
use hiqdev\php\merchant\merchants\AbstractMerchant;
15
use hiqdev\php\merchant\response\CompletePurchaseResponse;
16
use hiqdev\php\merchant\response\RedirectPurchaseResponse;
17
use Omnipay\FreeKassa\Gateway;
18
19
/**
20
 * Class FreeKassaMerchant.
21
 *
22
 * @author Dmytro Naumenko <[email protected]>
23
 */
24
class FreeKassaMerchant extends AbstractMerchant
25
{
26
    /**
27
     * @var Gateway
28
     */
29
    protected $gateway;
30
31 3
    protected function createGateway()
32
    {
33 3
        return $this->gatewayFactory->build('FreeKassa', [
34 3
            'purse' => $this->credentials->getPurse(),
35 3
            'secretKey' => $this->credentials->getKey1(),
36 3
            'secretKey2' => $this->credentials->getKey2(),
37
        ]);
38
    }
39
40
    /**
41
     * @param InvoiceInterface $invoice
42
     * @return RedirectPurchaseResponse
43
     */
44 1
    public function requestPurchase(InvoiceInterface $invoice)
45
    {
46
        /**
47
         * @var \Omnipay\FreeKassa\Message\PurchaseResponse
48
         */
49 1
        $response = $this->gateway->purchase([
50 1
            'transaction_id' => $invoice->getId(),
51 1
            'currency' => $invoice->getCurrency()->getCode(),
52 1
            'amount' => $this->moneyFormatter->format($invoice->getAmount()),
53 1
            'client' => $invoice->getClient(),
54 1
        ])->send();
55
56 1
        return new RedirectPurchaseResponse($response->getRedirectUrl(), $response->getRedirectData());
57
    }
58
59
    /**
60
     * @param array $data
61
     * @return CompletePurchaseResponse
62
     */
63 1
    public function completePurchase($data)
64
    {
65
        /** @var \Omnipay\FreeKassa\Message\CompletePurchaseResponse $response */
66 1
        $response = $this->gateway->completePurchase($data)->send();
67
68 1
        return (new CompletePurchaseResponse())
69 1
            ->setIsSuccessful($response->isSuccessful())
0 ignored issues
show
$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...
70
            // TODO: !(>_<)! FreeKassa does not indicate currency.
71 1
            ->setAmount($this->moneyParser->parse($response->getAmount(), 'RUB'))
72 1
            ->setTransactionReference($response->getTransactionReference())
73 1
            ->setTransactionId($response->getTransactionId())
74 1
            ->setPayer($response->getPayer())
75 1
            ->setTime((new \DateTime($response->getTime()))->setTimezone(new \DateTimeZone('UTC')));
76
    }
77
}
78