LivePersonInc\LiveEngageLaravel\LiveEngageRequest

Complexity

Total Complexity

9

Size/Duplication

Total Lines	158
Duplicated Lines	0 %

Test Coverage

Coverage

94.34%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	V2()	0	22	2
A	login()	0	27	1
A	requestClient()	0	22	1
A	__construct()	0	3	1
A	V1()	0	28	4

<?php
/**
 * LiveEngageRequest class.
 */
	
namespace LivePersonInc\LiveEngageLaravel;

use LivePersonInc\LiveEngageLaravel\Facades\LiveEngageLaravel as LiveEngage;
use GuzzleHttp\Client;
use GuzzleHttp\HandlerStack;
use GuzzleHttp\Subscriber\Oauth\Oauth1;
use LivePersonInc\LiveEngageLaravel\Models\Payload;

/**
 * LiveEngageRequest class.
 */
class LiveEngageRequest
{
	/**
	 * config
	 * 
	 * @var mixed
	 * @access private
	 */
	private $config;
	
	/**
	 * __construct function.
	 * 
	 * @access public
	 * @param mixed $config
	 * @return void
	 */
	public function __construct($config)
	{
		$this->config = $config; // @codeCoverageIgnore
	}
	
	/**
	 * login
	 * 
	 * @access public
	 * @return this
	 */
	public function login()
	{
		$le = LiveEngage::domain('agentVep');

The method domain() does not exist on LivePersonInc\LiveEngageLaravel\Facades\LiveEngageLaravel. Since you implemented __callStatic, consider adding a @method annotation.

		$domain = $le->domain;
		$account = $le->account;
		
		$consumer_key = config("{$this->config}.key");
		$consumer_secret = config("{$this->config}.secret");
		$token = config("{$this->config}.token");
		$secret = config("{$this->config}.token_secret");
		$username = config("{$this->config}.user_name");
		
		$auth = [
			'username'		  => $username,
			'appKey'			=> $consumer_key,
			'secret'			=> $consumer_secret,
			'accessToken'		=> $token,
			'accessTokenSecret' => $secret,
		];
		
		$url = "https://{$domain}/api/account/{$account}/login?v=1.3";
		
		$response = $this->V1($url, 'POST', $auth);
		
		$this->bearer = $response->bearer;

The property bearer does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

		
		return $this;
	}
	
	/**
	 * V1
	 * 
	 * @access public
	 * @param string $url
	 * @param string $method
	 * @param array $payload (default: [])
	 * @return mixed
	 */
	public function V1($url, $method, $payload = [])
	{
		$client = $this->requestClient();

		$args = [
			'auth' => 'oauth',
			'headers' => [
				'content-type' => 'application/json',
			],
			'body' => json_encode($payload)
		];

		try {
			$res = $client->request($method, $url, $args);
			$response = json_decode($res->getBody());
		} catch (\Exception $e) {
			// @codeCoverageIgnoreStart
			if ($this->retry_counter < $this->retry_limit || $this->retry_limit == -1) {

The property retry_counter does not exist on LivePersonInc\LiveEngageLaravel\LiveEngageRequest. Did you maybe forget to declare it?

The property retry_limit does not exist on LivePersonInc\LiveEngageLaravel\LiveEngageRequest. Did you maybe forget to declare it?

				usleep(1500);
				$this->retry_counter++;
				$response = $this->requestV1($url, $payload);

The method requestV1() does not exist on LivePersonInc\LiveEngageLaravel\LiveEngageRequest.

			} else {
				throw $e;
			}
			// @codeCoverageIgnoreEnd
		}

		return $response;
	}
	
	/**
	 * V2
	 * 
	 * @access public
	 * @param mixed $url
	 * @param mixed $method
	 * @param mixed $payload (default: [])
	 * @param mixed $headers (default: [])
	 * @return void
	 */
	public function V2($url, $method, $payload = [], $headers = [])
	{
		$this->login();
		
		$client = new Client();
		$args = [
			'headers' => array_merge([
				'content-type' => 'application/json',
				'Authorization' => 'Bearer ' . $this->bearer
			], $headers),
			'body' => json_encode($payload)
		];
		
		// @codeCoverageIgnoreStart
		try {
			$res = $client->request($method, $url, $args);
		} catch (\Exception $e) {
			throw $e;
		}
		// @codeCoverageIgnoreEnd
		
		return json_decode($res->getBody());
	}
	
	/**
	 * requestClient
	 * 
	 * @access private
	 * @return \GuzzleHttp\Client
	 */
	private function requestClient()
	{
		$consumer_key = config("{$this->config}.key");
		$consumer_secret = config("{$this->config}.secret");
		$token = config("{$this->config}.token");
		$secret = config("{$this->config}.token_secret");

		$stack = HandlerStack::create();
		$auth = new Oauth1([
			'consumer_key'	=> $consumer_key,
			'consumer_secret' => $consumer_secret,
			'token'		   => $token,
			'token_secret'	=> $secret,
			'signature_method'=> Oauth1::SIGNATURE_METHOD_HMAC,
		]);
		$stack->push($auth);

		$client = new Client([
			'handler' => $stack,
		]);
		
		return $client;
	}
}