App\Service\Braintree\ForwardApiService::directTokenization()

Complexity

Conditions	3
Paths	12

Size

Total Lines	84
Code Lines	59

Duplication

Lines	0
Ratio	0 %

Importance

Changes	3
Bugs	0	Features	0

<?php
    public function directTokenization(string $paymentNonce, string $deviceData): ?array
    {
        try {
            $ch = curl_init();
            curl_setopt($ch, CURLOPT_URL, $this->ENDPOINT);
            curl_setopt($ch, CURLOPT_HTTPHEADER, ['Content-Type:application/json']);
            curl_setopt(
                $ch,
                CURLOPT_USERPWD,
                $this->gateway->config->getPublicKey() . ':' . $this->gateway->config->getPrivateKey()
            );
            curl_setopt($ch, CURLOPT_CUSTOMREQUEST, 'POST');
            curl_setopt($ch, CURLOPT_POSTFIELDS, json_encode([
                'debug_transformations' => true,
                'name' => 'default',
                'merchant_id' => $this->gateway->config->getMerchantId(),
                'payment_method_nonce' => $paymentNonce,
                'device_data' => $deviceData,
                'method' => 'POST',
                'data' => [
                    'card-id' => 'custom_card_id_1',
                    'open-id-token' => sha1(time()),
                ],
                'url' => 'https://test.com',
                'config' => [
                    'name' => 'default',
                    'url' => '^https://test.com',
                    'methods' => ['POST'],
                    'types' => ['NetworkTokenizedCard'],
                    'request_format' => [
                        'body' => 'json'
                    ],
                    'transformations' => [
                        [
                            'path' => '/body/card/pan',
                            'value' => '$number'
                        ],
                        [
                            'path' => '/body/card/card-cvv',
                            'value' => '$cvv'
                        ],
                        [
                            'path' => '/body/card/expiration-month',
                            'value' => '$expiration_month'
                        ],
                        [
                            'path' => '/body/card/expiration-year',
                            'value' => '$expiration_year'
                        ],
                        [
                            'path' => '/body/custom-fields/card-id',
                            'value' => '$card-id'
                        ],
                        [
                            'path' => '/header/Authorization',
                            'value' => '$open-id-token'
                        ]
                    ]

                ]
            ], true));

true of type true is incompatible with the type integer expected by parameter $flags of json_encode().

            curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
            curl_setopt($ch, CURLOPT_HEADERFUNCTION, function ($curl, $header) use (&$headers) {
                $len = strlen($header);
                $header = explode(':', $header, 2);
                if (count($header) < 2) {
                    return $len;
                }
                $headers[(trim($header[0]))] = trim($header[1]);
                return $len;
            });
            $result = curl_exec($ch);
            $statusCode = curl_getinfo($ch, CURLINFO_HTTP_CODE);

            curl_close($ch);
        } catch (Exception $e) {
            $this->logger->error('Error on ForwardAPI::'.$this->ENDPOINT.' = ' . $e->getMessage());
            return null;
        }
        return ([
            'headers' => (object) $headers,
            'body' => (object) json_decode($result),

It seems like $result can also be of type true; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

            'statusCode' => $statusCode
        ]);