Gidkom\OpenFireRestApi\RestClient

Complexity

Total Complexity

8

Size/Duplication

Total Lines	71
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	geti()	0	4	1
B	doRequest()	0	34	6

<?php

namespace Gidkom\OpenFireRestApi;

use GuzzleHttp\Client;
/**
* 
*/
class RestClient
{
	public $host = 'localhost';
    public $port = '9090';
    public $plugin = '/plugins/restapi/v1';
    public $secret = 'SuperSecret';
    public $useSSL = false;
    protected $params  = array();
    private $client;
    public $bcastRoles = array();
    public $useBasicAuth = false;
    public $basicUser = 'admin';
    public $basicPwd = '1234';

	function __construct()

It is generally recommended to explicitly declare the visibility for methods.

	{
		$this->client = new Client();
	}

	public function geti()
	{
		return $this->host. '  '. $this->secret;
	}



    /**
     * Make the request and analyze the result
     *
     * @param   string          $type           Request method
     * @param   string          $endpoint       Api request endpoint
     * @param   array           $params         Parameters
     * @return  array|false                     Array with data or error, or False when something went fully wrong
     */
	protected function doRequest($type, $endpoint, $params=[])
    {
    	$base = ($this->useSSL) ? "https" : "http";
    	$url = $base . "://" . $this->host . ":" .$this->port.$this->plugin.$endpoint;
	    
		if ($this->useBasicAuth)
            $auth = 'Basic ' . base64_encode($this->basicUser . ':' . $this->basicPwd);
        else
            $auth = $this->secret;
	    
    	$headers = array(
  			'Accept' => 'application/json',
  			'Authorization' => $auth
  		);

        $body = json_encode($params);
        // $headers += ['Content-Type'=>'application/json'];

59% of this comment could be valid code. Did you maybe forget this after debugging?

        

        try {
        	$result = $this->client->request($type, $url, compact('headers','body'));
        } catch (Exception $e) {

The class Gidkom\OpenFireRestApi\Exception does not exist. Did you forget a USE statement, or did you not list all dependencies?

        	$result = $e->message;
        }
	        


        
        if ($result->getStatusCode() == 200 || $result->getStatusCode() == 201) {
            return array('status'=>true, 'message'=>json_decode($result->getBody()));
        }
        return array('status'=>false, 'message'=>json_decode($result->getBody()));
    	
    }
    

}