Whisper\HetznerCloud\Clients\HetznerCloud

Complexity

Total Complexity

4

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	getClient()	0	2	2
A	__callStatic()	0	3	1
A	setClient()	0	10	1

<?php
/**
 * Created by PhpStorm.
 * User: sergio.rodenas
 * Date: 29/7/18
 * Time: 18:43
 */

namespace Whisper\HetznerCloud\Clients;

use Zttp\PendingZttpRequest;

class HetznerCloud
{
	public static $baseHost = "https://api.hetzner.cloud/v1/";
	public static $authenticationToken;

	private static $client;

	public static function __callStatic($name, $arguments)
	{
		return (new static())->getClient()->{$name}(...$arguments)->json();
	}

	private function getClient(){
		return (static::$client) ? static::$client : static::setClient();

Since $client is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $client to at least protected.

The method Whisper\HetznerCloud\Clients\HetznerCloud::setClient() is not static, but was called statically.

	}

	private function setClient(){
		$client = (new PendingZttpRequest())->withHeaders([
			'Authorization' => 'Bearer '.static::$authenticationToken
		]);

		$client->options += [
			'base_uri' => static::$baseHost
		];

		return static::$client = $client;

Since $client is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $client to at least protected.

	}
}