Mezon\CustomClient\CurlWrapper

Complexity

Total Complexity

3

Size/Duplication

Total Lines	46
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	sendRequest()	0	30	3

<?php
namespace Mezon\CustomClient;

/**
 * Class CurlWrapper
 *
 * @package CustomClient
 * @subpackage CurlWrapper
 * @author Dodonov A.A.
 * @version v.1.0 (2019/08/07)
 * @copyright Copyright (c) 2019, aeon.org
 */

/**
 * Wrapper for CURL routines
 */
class CurlWrapper
{

    /**
     * Method send HTTP request
     *
     * @param string $uRL
     *            URL
     * @param array $headers
     *            Headers
     * @param string $method
     *            Request HTTP Method
     * @param array $data
     *            Request data
     * @return array Response body and HTTP code
     */
    public static function sendRequest(string $uRL, array $headers, string $method, array $data = []): array
    {
        $ch = curl_init();

        $curlConfig = [
            CURLOPT_URL => $uRL,
            CURLOPT_HTTPHEADER => $headers,
            CURLOPT_POST => ($method == 'POST'),
            CURLOPT_RETURNTRANSFER => true,
            CURLOPT_SSL_VERIFYPEER => true
        ];

        if ($method == 'POST') {
            $formData = [];
            foreach ($data as $key => $value) {
                $formData[] = $key . '=' . urlencode($value);
            }
            $curlConfig[CURLOPT_POSTFIELDS] = implode('&', $formData);
        }

        curl_setopt_array($ch, $curlConfig);

It seems like $ch can also be of type false; however, parameter $ch of curl_setopt_array() does only seem to accept resource, maybe add an additional type check?

        $body = curl_exec($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

        $code = curl_getinfo($ch, CURLINFO_HTTP_CODE);

It seems like $ch can also be of type false; however, parameter $ch of curl_getinfo() does only seem to accept resource, maybe add an additional type check?

        curl_close($ch);

It seems like $ch can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

        return [
            $body,
            $code
        ];
    }
}