TarfinLabs\Netgsm\NetgsmApiClient::callApi()

Complexity

Conditions	6
Paths	12

Size

Total Lines	28
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	1

<?php
    protected function callApi($method, $url, $params = null, $headers = [])
    {
        $options = [
            'query' => [
                'usercode' => $this->credentials['user_code'],
                'password' => $this->credentials['secret'],
            ],
        ];

        if ($method == 'POST') {
            if (is_array($params)) {

The condition is_array($params) is always false.

                $options['form_params'] = $params;
            } else {
                $options['body'] = $params;
            }
        }

        if ($method == 'GET' && is_array($params)) {

The condition is_array($params) is always false.

            $options['query'] = array_merge($options['query'], $params);
        }

        if ($headers) {

The expression $headers of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            $options['headers'] = $headers;
        }

        $response = $this->client->request($method, $url, $options);

        return $response->getBody()->getContents();
    }