HTTP\CURL::request()

Complexity

Conditions	12
Paths	32

Size

Total Lines	53
Code Lines	43

Duplication

Lines	20
Ratio	37.74 %

Importance

Changes	1
Bugs	0	Features	0

<?php
  public function request(HTTP\Request $request){
    $http_method = strtoupper($method);

The variable $method does not exist. Did you forget to declare it?

    $ch  = curl_init($url);

The variable $url does not exist. Did you forget to declare it?

    $opt = [
      CURLOPT_CUSTOMREQUEST   => $http_method,
      CURLOPT_SSL_VERIFYHOST  => false,
      CURLOPT_CONNECTTIMEOUT  => 10,
      CURLOPT_RETURNTRANSFER  => true,
      CURLOPT_USERAGENT       => static::$UA,
      CURLOPT_HEADER          => false,
      CURLOPT_MAXREDIRS       => 10,
      CURLOPT_FOLLOWLOCATION  => true,
      CURLOPT_ENCODING        => '',
    ];

    if($username && $password) {
      $opt[CURLOPT_USERPWD] = "$username:$password";

The variable $username does not exist. Did you forget to declare it?

The variable $password does not exist. Did you forget to declare it?

    }

    $headers = array_merge($headers,static::$headers);

The variable $headers seems only to be defined at a later point. Did you maybe move this code here without moving the variable definition?

    if($http_method == 'GET'){
        if($data && is_array($data)){
          $tmp                       = [];
          $queried_url               = $url;
          foreach($data as $key=>$val) $tmp[] = $key.'='.$val;

The variable $data does not exist. Did you forget to declare it?

          $queried_url               .= (strpos($queried_url,'?') === false) ? '?' : '&';
          $queried_url               .= implode('&',$tmp);
          $opt[CURLOPT_URL]          = $queried_url;
          $opt[CURLOPT_HTTPGET]      = true;
          unset($opt[CURLOPT_CUSTOMREQUEST]);
        }
    } else {
        $opt[CURLOPT_CUSTOMREQUEST]  = $http_method;
        if($data_as_json or is_object($data)){

The variable $data_as_json does not exist. Did you forget to declare it?

Using logical operators such as or instead of || is generally not recommended.

          $headers['Content-Type']   = 'application/json';
          $opt[CURLOPT_POSTFIELDS]   = json_encode($data);
        } else {
          $opt[CURLOPT_POSTFIELDS]   = http_build_query($data);
        }
    }

    curl_setopt_array($ch,$opt);
    $_harr = [];
    foreach($headers as $key=>$val)  $_harr[] = $key.': '.$val;
    curl_setopt($ch, CURLOPT_HTTPHEADER, $_harr);
    $result = curl_exec($ch);
    $contentType = strtolower(curl_getinfo($ch, CURLINFO_CONTENT_TYPE));
    static::$last_info = curl_getinfo($ch);
    if(false !== strpos($contentType,'json')) $result = json_decode($result);
    curl_close($ch);
    return $result;
  }