App\Service\Paypal\VaultService::getDataUserIdToken()

Complexity

Conditions	3
Paths	12

Size

Total Lines	27
Code Lines	21

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function getDataUserIdToken(string $clientId)
    {
        try {
            $ch = curl_init();
            curl_setopt($ch, CURLOPT_URL, 'https://api-m.sandbox.paypal.com/v1/oauth2/token');
            curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
            curl_setopt($ch, CURLOPT_POST, 1);
            curl_setopt(
                $ch,
                CURLOPT_POSTFIELDS,
                "grant_type=client_credentials&response_type=id_token&target_customer_id=" . $clientId
            );
            curl_setopt($ch, CURLOPT_USERPWD, $this->clientId . ':' . $this->clientSecret);
            $headers[] = 'Content-Type: application/x-www-form-urlencoded';

$headers was never initialized. Although not strictly required by PHP, it is generally a good practice to add $headers = array(); before regardless.

            curl_setopt($ch, CURLOPT_HTTPHEADER, $headers);
            $result = curl_exec($ch);
            curl_close($ch);
        } catch (Exception $e) {
            $this->logger->error('Error on PayPal::getDataUserIdToken = ' . $e->getMessage());
            return null;
        }
        $result = json_decode($result, true);

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?

        if (array_key_exists('id_token', $result)) {
            return $result['id_token'];
        }
        return null;
    }