Laravel\YandexSpeller\Application\Services\YandexSpeller\Resolvers\CurlRequestContentResolver::handle()

Complexity

Conditions	4
Paths	4

Size

Total Lines	27
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function handle(string $url, array $postData = [], array $optionsList = []): array
    {
        $curl = curl_init();
        curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);

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

        curl_setopt($curl, CURLOPT_URL, $url);

        if (!empty($postData)) {
            curl_setopt($curl, CURLOPT_POST, 1);
            curl_setopt($curl, CURLOPT_POSTFIELDS, http_build_query($postData));
        }

        if (!empty($optionsList)) {
            foreach ($optionsList as $optionKey => $optionValue) {
                curl_setopt($curl, $optionKey, $optionValue);
            }
        }

        $result = [
            'result' => curl_exec($curl),

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

            'errno' => curl_errno($curl),

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

            'error' => curl_error($curl),

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

            'http_code' => curl_getinfo($curl, CURLINFO_HTTP_CODE),

It seems like $curl 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($curl);

It seems like $curl 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 $result;
    }