detain / cloudlinux-licensing

src/Cloudlinux.php

Doc Comments +5 added lines, -2 removed lines

@@ -107,6 +107,9 @@ 
 		return $ret;
 	}
 
+	/**
+	 * @param string $level
+	 */
 	public function log($level, $text, $line = '', $file = '') {
 		if (function_exists('myadmin_log'))
 			myadmin_log('cloudlinux', $level, $text, $line, $file);
@@ -243,8 +246,8 @@ 
 	 * or
 	 * Remove IP licenses with specified type for customer. Also un­registers from CLN server associated with IP.
 	 * @param string         $ip   ip address to remove
-	 * @param bool|false|int $type optional parameter to specify the type of license to remove (1,2, or 16)
-	 * @return integer 0 on success, -1 on error, Error will be returned also if account have no licenses for provided IP.
+	 * @param integer $type optional parameter to specify the type of license to remove (1,2, or 16)
+	 * @return boolean 0 on success, -1 on error, Error will be returned also if account have no licenses for provided IP.
 	 */
 	public function remove_license($ip, $type = false) {
 		$this->log('info', "Calling CLoudLinux->xml_client->remove_license({$this->authToken()}, {$ip}, {$type})", __LINE__, __FILE__);

Indentation +278 added lines, -278 removed lines

@@ -30,301 +30,301 @@
  */
 class Cloudlinux
 {
-	private $login = '';
-	private $key = '';
-	private $authToken;
-	public $prefix = 'registration.';
-	public $encoding = 'utf-8'; // utf-8 / UTF-8
-	public $api_type = 'rest';
-	public $sslverify = false;
-	public $xml_options = [];
-	public $xml_url = 'https://cln.cloudlinux.com/clweb/xmlrpc';
-	public $rest_url = 'https://cln.cloudlinux.com/api/';
-	public $rest_options = [];
-	/**
-	 * @var \XML_RPC2_Client
-	 */
-	public $xml_client;
-	public $response;
+    private $login = '';
+    private $key = '';
+    private $authToken;
+    public $prefix = 'registration.';
+    public $encoding = 'utf-8'; // utf-8 / UTF-8
+    public $api_type = 'rest';
+    public $sslverify = false;
+    public $xml_options = [];
+    public $xml_url = 'https://cln.cloudlinux.com/clweb/xmlrpc';
+    public $rest_url = 'https://cln.cloudlinux.com/api/';
+    public $rest_options = [];
+    /**
+     * @var \XML_RPC2_Client
+     */
+    public $xml_client;
+    public $response;
 
-	/**
-	 * Cloudlinux::__construct()
-	 *
-	 * @param string $login API Login Name
-	 * @param string $key API Key
-	 * @param string $type API type to use, can be 'rest' or 'xml'
-	 */
-	public function __construct($login, $key, $type = 'rest') {
-		$this->login = $login;
-		$this->key = $key;
-		if (!isset($GLOBALS['HTTP_RAW_POST_DATA']))
-			$GLOBALS['HTTP_RAW_POST_DATA'] = file_get_contents('php://input');
-		//if ($api_type == 'xml') {
-			include_once('XML/RPC2/Client.php');
-			$this->xml_options['prefix'] = $this->prefix;
-			$this->xml_options['encoding'] = $this->encoding;
-			$this->xml_options['sslverify'] = $this->sslverify;
-			$this->xml_client = \XML_RPC2_Client::create($this->xml_url, $this->xml_options);
-		//} elseif ($this->api_type == 'rest') {
-			//$this->rest_options[CURLOPT_SSL_VERIFYHOST] = $this->sslverify;
-		//}
-	}
+    /**
+     * Cloudlinux::__construct()
+     *
+     * @param string $login API Login Name
+     * @param string $key API Key
+     * @param string $type API type to use, can be 'rest' or 'xml'
+     */
+    public function __construct($login, $key, $type = 'rest') {
+        $this->login = $login;
+        $this->key = $key;
+        if (!isset($GLOBALS['HTTP_RAW_POST_DATA']))
+            $GLOBALS['HTTP_RAW_POST_DATA'] = file_get_contents('php://input');
+        //if ($api_type == 'xml') {
+            include_once('XML/RPC2/Client.php');
+            $this->xml_options['prefix'] = $this->prefix;
+            $this->xml_options['encoding'] = $this->encoding;
+            $this->xml_options['sslverify'] = $this->sslverify;
+            $this->xml_client = \XML_RPC2_Client::create($this->xml_url, $this->xml_options);
+        //} elseif ($this->api_type == 'rest') {
+            //$this->rest_options[CURLOPT_SSL_VERIFYHOST] = $this->sslverify;
+        //}
+    }
 
-	/**
-	 * getcurlpage()
-	 * gets a webpage via curl and returns the response.
-	 * also it sets a mozilla type agent.
-	 * @param string $url        the url of the page you want
-	 * @param string $postfields postfields in the format of "v1=10&v2=20&v3=30"
-	 * @param string $options
-	 * @return string the webpage
-	 */
-	public function getcurlpage($url, $postfields = '', $options = '') {
-		//myadmin_log('myadmin', 'info', "Get Url Page $url", __LINE__, __FILE__);
-		$agent = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2790.0 Safari/537.36';
-		$ch = curl_init($url);
-		curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
-		curl_setopt($ch, CURLOPT_USERAGENT, $agent);
-		if (is_array($postfields) || $postfields != '') {
-			if (is_array($postfields)) {
-				$postdata = [];
-				foreach ($postfields as $field => $value) {
-					$postdata[] = $field . '=' . urlencode($value);
-				}
-				curl_setopt($ch, CURLOPT_POSTFIELDS, implode('&', $postdata));
-			} else {
-				curl_setopt($ch, CURLOPT_POSTFIELDS, $postfields);
-			}
-		}
-		if (is_array($options) && sizeof($options) > 0) {
-			foreach ($options as $key => $value) {
-				curl_setopt($ch, $key, $value);
-			}
-		}
-		$tmp = curl_exec($ch);
-		curl_close($ch);
-		$ret = $tmp;
-		return $ret;
-	}
+    /**
+     * getcurlpage()
+     * gets a webpage via curl and returns the response.
+     * also it sets a mozilla type agent.
+     * @param string $url        the url of the page you want
+     * @param string $postfields postfields in the format of "v1=10&v2=20&v3=30"
+     * @param string $options
+     * @return string the webpage
+     */
+    public function getcurlpage($url, $postfields = '', $options = '') {
+        //myadmin_log('myadmin', 'info', "Get Url Page $url", __LINE__, __FILE__);
+        $agent = 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2790.0 Safari/537.36';
+        $ch = curl_init($url);
+        curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
+        curl_setopt($ch, CURLOPT_USERAGENT, $agent);
+        if (is_array($postfields) || $postfields != '') {
+            if (is_array($postfields)) {
+                $postdata = [];
+                foreach ($postfields as $field => $value) {
+                    $postdata[] = $field . '=' . urlencode($value);
+                }
+                curl_setopt($ch, CURLOPT_POSTFIELDS, implode('&', $postdata));
+            } else {
+                curl_setopt($ch, CURLOPT_POSTFIELDS, $postfields);
+            }
+        }
+        if (is_array($options) && sizeof($options) > 0) {
+            foreach ($options as $key => $value) {
+                curl_setopt($ch, $key, $value);
+            }
+        }
+        $tmp = curl_exec($ch);
+        curl_close($ch);
+        $ret = $tmp;
+        return $ret;
+    }
 
-	public function log($level, $text, $line = '', $file = '') {
-		if (function_exists('myadmin_log'))
-			myadmin_log('cloudlinux', $level, $text, $line, $file);
-		else
-			error_log($text);
-	}
+    public function log($level, $text, $line = '', $file = '') {
+        if (function_exists('myadmin_log'))
+            myadmin_log('cloudlinux', $level, $text, $line, $file);
+        else
+            error_log($text);
+    }
 
-	/**
-	 * Return system information about several Cloudlinux services
-	 *
-	 * @return array array of system information
-	 */
-	public function status() {
-		$this->response = $this->getcurlpage($this->rest_url.'status.json', '', $this->rest_options);
-		return json_decode($this->response, true);
-	}
+    /**
+     * Return system information about several Cloudlinux services
+     *
+     * @return array array of system information
+     */
+    public function status() {
+        $this->response = $this->getcurlpage($this->rest_url.'status.json', '', $this->rest_options);
+        return json_decode($this->response, true);
+    }
 
-	/**
-	 * Will return information about what kind of license types are available for registration and what types are already used by current account.
-	 * @param string $ip ip address to check
-	 * @return array returns an array with  available(int[]) ­ list of types that can be used to register new IP license, and owned(int[]) ­ list of types that already registered(owned) by this account
-	 */
-	public function availability($ip) {
-		$this->response = $this->getcurlpage($this->rest_url.'ipl/availability.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
-		return json_decode($this->response, true);
-	}
+    /**
+     * Will return information about what kind of license types are available for registration and what types are already used by current account.
+     * @param string $ip ip address to check
+     * @return array returns an array with  available(int[]) ­ list of types that can be used to register new IP license, and owned(int[]) ­ list of types that already registered(owned) by this account
+     */
+    public function availability($ip) {
+        $this->response = $this->getcurlpage($this->rest_url.'ipl/availability.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
+        return json_decode($this->response, true);
+    }
 
-	/**
-	 * Check if IP license is registered by any customer.
-	 *
-	 * @param string $ip ip address to check
-	 * @return array Will return list of registered license types or empty list if provided IP is not registered yet.
-	 */
-	public function check($ip) {
-		$this->response = $this->getcurlpage($this->rest_url.'ipl/check.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
-		$response = json_decode($this->response, true);
-		if ($response['success'] == 1)
-			return $response['data'];
-		else
-			return false;
-	}
+    /**
+     * Check if IP license is registered by any customer.
+     *
+     * @param string $ip ip address to check
+     * @return array Will return list of registered license types or empty list if provided IP is not registered yet.
+     */
+    public function check($ip) {
+        $this->response = $this->getcurlpage($this->rest_url.'ipl/check.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
+        $response = json_decode($this->response, true);
+        if ($response['success'] == 1)
+            return $response['data'];
+        else
+            return false;
+    }
 
-	/**
-	 * Will register IP based license for authorized user.
-	 *
-	 * @param string $ip ip address to registger
-	 * @param int $type IP license type (1,2 or 16)
-	 * @return array On success response returns information about created or already registered license.   ip(string)    type(int) ­ license type (1,2,16)   registered(boolean) ­ true if server was registered in CLN with this license (CLN licenses only).     created(string) ­ license creation time
-	 */
-	public function register($ip, $type) {
-		$this->response = $this->getcurlpage($this->rest_url.'ipl/register.json?ip='.$ip.'&type='.$type.'&token='.$this->authToken(), '', $this->rest_options);
-		return json_decode($this->response, true);
-	}
+    /**
+     * Will register IP based license for authorized user.
+     *
+     * @param string $ip ip address to registger
+     * @param int $type IP license type (1,2 or 16)
+     * @return array On success response returns information about created or already registered license.   ip(string)    type(int) ­ license type (1,2,16)   registered(boolean) ­ true if server was registered in CLN with this license (CLN licenses only).     created(string) ­ license creation time
+     */
+    public function register($ip, $type) {
+        $this->response = $this->getcurlpage($this->rest_url.'ipl/register.json?ip='.$ip.'&type='.$type.'&token='.$this->authToken(), '', $this->rest_options);
+        return json_decode($this->response, true);
+    }
 
-	/**
-	 * Will remove IP based license from authorized user licenses.
-	 *
-	 * @param string $ip ip address to remove licenses on
-	 * @param int $type optional license type. If empty, will remove licenses with all types
-	 * @return bool
-	 */
-	public function rest_remove($ip, $type = 0) {
-		if ($type != 0)
-			$this->response = $this->getcurlpage($this->rest_url.'ipl/remove.json?ip='.$ip.'&type='.$type.'&token='.$this->authToken(), '', $this->rest_options);
-		else
-			$this->response = $this->getcurlpage($this->rest_url.'ipl/remove.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
-		return json_decode($this->response, true);
-	}
+    /**
+     * Will remove IP based license from authorized user licenses.
+     *
+     * @param string $ip ip address to remove licenses on
+     * @param int $type optional license type. If empty, will remove licenses with all types
+     * @return bool
+     */
+    public function rest_remove($ip, $type = 0) {
+        if ($type != 0)
+            $this->response = $this->getcurlpage($this->rest_url.'ipl/remove.json?ip='.$ip.'&type='.$type.'&token='.$this->authToken(), '', $this->rest_options);
+        else
+            $this->response = $this->getcurlpage($this->rest_url.'ipl/remove.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
+        return json_decode($this->response, true);
+    }
 
-	/**
-	 * Will remove IP based license from authorized user licenses.
-	 *
-	 * @param string $ip ip address to remove licenses on
-	 * @param int $type optional license type. If empty or 0, will remove licenses with all types
-	 * @return bool
-	 */
-	public function remove($ip, $type = 0) {
-		if ($this->api_type == 'xml')
-			return $this->remove_license($ip, $type);
-		else
-			return $this->rest_remove($ip, $type);
-	}
+    /**
+     * Will remove IP based license from authorized user licenses.
+     *
+     * @param string $ip ip address to remove licenses on
+     * @param int $type optional license type. If empty or 0, will remove licenses with all types
+     * @return bool
+     */
+    public function remove($ip, $type = 0) {
+        if ($this->api_type == 'xml')
+            return $this->remove_license($ip, $type);
+        else
+            return $this->rest_remove($ip, $type);
+    }
 
-	/**
-	 * Return all IP licenses owned by authorized user.
-	 *
-	 * @return array an array of licenses each one containing these fields: ip(string)   ype(int) ­ license type (1,2,16)   registered(boolean) ­ true if server was registered in CLN with this license (CLN licenses only).    created(string) ­ license creation time
-	 */
-	public function rest_list() {
-		$this->response = $this->getcurlpage($this->rest_url.'ipl/list.json?token='.$this->authToken(), '', $this->rest_options);
-		return json_decode($this->response, true);
-	}
+    /**
+     * Return all IP licenses owned by authorized user.
+     *
+     * @return array an array of licenses each one containing these fields: ip(string)   ype(int) ­ license type (1,2,16)   registered(boolean) ­ true if server was registered in CLN with this license (CLN licenses only).    created(string) ­ license creation time
+     */
+    public function rest_list() {
+        $this->response = $this->getcurlpage($this->rest_url.'ipl/list.json?token='.$this->authToken(), '', $this->rest_options);
+        return json_decode($this->response, true);
+    }
 
-	/**
-	 * automatic authToken generator
-	 *
-	 * @return string the authToken
-	 */
-	public function authToken() {
-		$time = time();
-		try {
-			return $this->login . '|' . $time . '|' . sha1($this->key . $time);
-		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
-			return false;
-		}
-	}
+    /**
+     * automatic authToken generator
+     *
+     * @return string the authToken
+     */
+    public function authToken() {
+        $time = time();
+        try {
+            return $this->login . '|' . $time . '|' . sha1($this->key . $time);
+        } catch (Exception $e) {
+            $this->log('error', 'Caught exception code: ' . $e->getCode());
+            $this->log('error', 'Caught exception message: ' . $e->getMessage());
+            return false;
+        }
+    }
 
-	/**
-	 * Register new IP license.
-	 *
-	 * @param string $ip IP Address
-	 * @param integer $type license type (1,2 or 16)
-	 * @throws XmlRpcException for critical errors
-	 * @return integer 0 on success, -1 on error
-	 */
-	public function license($ip, $type) {
-		$type = (int)$type;
-		try {
-			$this->log('error', 'Calling License(' . $this->authToken() . ',' . $ip . ',' . $type . ')');
-			$this->response = $this->xml_client->license($this->authToken(), $ip, $type);
-			$this->log('error', 'Response: ' . var_export($this->response, true));
-			return $this->response;
-		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
-			return false;
-		}
-	}
+    /**
+     * Register new IP license.
+     *
+     * @param string $ip IP Address
+     * @param integer $type license type (1,2 or 16)
+     * @throws XmlRpcException for critical errors
+     * @return integer 0 on success, -1 on error
+     */
+    public function license($ip, $type) {
+        $type = (int)$type;
+        try {
+            $this->log('error', 'Calling License(' . $this->authToken() . ',' . $ip . ',' . $type . ')');
+            $this->response = $this->xml_client->license($this->authToken(), $ip, $type);
+            $this->log('error', 'Response: ' . var_export($this->response, true));
+            return $this->response;
+        } catch (Exception $e) {
+            $this->log('error', 'Caught exception code: ' . $e->getCode());
+            $this->log('error', 'Caught exception message: ' . $e->getMessage());
+            return false;
+        }
+    }
 
-	/**
-	 * Remove IP licenses with specified type for customer. Also un­registers from CLN server associated with IP.
-	 * or
-	 * Remove IP licenses with specified type for customer. Also un­registers from CLN server associated with IP.
-	 * @param string         $ip   ip address to remove
-	 * @param bool|false|int $type optional parameter to specify the type of license to remove (1,2, or 16)
-	 * @return integer 0 on success, -1 on error, Error will be returned also if account have no licenses for provided IP.
-	 */
-	public function remove_license($ip, $type = false) {
-		$this->log('info', "Calling CLoudLinux->xml_client->remove_license({$this->authToken()}, {$ip}, {$type})", __LINE__, __FILE__);
-		try {
-			if ($type === false)
-				return $this->response = $this->remove($ip);
-			else
-				return $this->response = $this->remove($ip, $type);
-		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
-			return false;
-		}
-	}
+    /**
+     * Remove IP licenses with specified type for customer. Also un­registers from CLN server associated with IP.
+     * or
+     * Remove IP licenses with specified type for customer. Also un­registers from CLN server associated with IP.
+     * @param string         $ip   ip address to remove
+     * @param bool|false|int $type optional parameter to specify the type of license to remove (1,2, or 16)
+     * @return integer 0 on success, -1 on error, Error will be returned also if account have no licenses for provided IP.
+     */
+    public function remove_license($ip, $type = false) {
+        $this->log('info', "Calling CLoudLinux->xml_client->remove_license({$this->authToken()}, {$ip}, {$type})", __LINE__, __FILE__);
+        try {
+            if ($type === false)
+                return $this->response = $this->remove($ip);
+            else
+                return $this->response = $this->remove($ip, $type);
+        } catch (Exception $e) {
+            $this->log('error', 'Caught exception code: ' . $e->getCode());
+            $this->log('error', 'Caught exception message: ' . $e->getMessage());
+            return false;
+        }
+    }
 
-	/**
-	 * Check if IP license was registered by any customer. Arguments:
-	 *
-	 * @param string $ip ip address to remove
-	 * @param bool $checkAll True will search for any type of license. False ­ only for types 1 or 2
-	 * @throws XmlRpcException for critical errors
-	 * @return array (list<int>): List of registered license types or empty list if no license found
-	 */
-	public function is_licensed($ip, $checkAll = true) {
-		if ($this->api_type == 'xml')
-			return $this->xml_is_licensed($ip, $checkAll);
-		else
-			return $this->check($ip, $checkAll);
-	}
-	/**
-	 * Check if IP license was registered by any customer. Arguments:
-	 *
-	 * @param string $ip ip address to remove
-	 * @param bool $checkAll True will search for any type of license. False ­ only for types 1 or 2
-	 * @throws XmlRpcException for critical errors
-	 * @return array (list<int>): List of registered license types or empty list if no license found
-	 */
-	public function xml_is_licensed($ip, $checkAll = true) {
-		try {
-			return $this->response = $this->xml_client->is_licensed($this->authToken(), $ip, $checkAll);
-		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
-			return false;
-		}
-	}
+    /**
+     * Check if IP license was registered by any customer. Arguments:
+     *
+     * @param string $ip ip address to remove
+     * @param bool $checkAll True will search for any type of license. False ­ only for types 1 or 2
+     * @throws XmlRpcException for critical errors
+     * @return array (list<int>): List of registered license types or empty list if no license found
+     */
+    public function is_licensed($ip, $checkAll = true) {
+        if ($this->api_type == 'xml')
+            return $this->xml_is_licensed($ip, $checkAll);
+        else
+            return $this->check($ip, $checkAll);
+    }
+    /**
+     * Check if IP license was registered by any customer. Arguments:
+     *
+     * @param string $ip ip address to remove
+     * @param bool $checkAll True will search for any type of license. False ­ only for types 1 or 2
+     * @throws XmlRpcException for critical errors
+     * @return array (list<int>): List of registered license types or empty list if no license found
+     */
+    public function xml_is_licensed($ip, $checkAll = true) {
+        try {
+            return $this->response = $this->xml_client->is_licensed($this->authToken(), $ip, $checkAll);
+        } catch (Exception $e) {
+            $this->log('error', 'Caught exception code: ' . $e->getCode());
+            $this->log('error', 'Caught exception message: ' . $e->getMessage());
+            return false;
+        }
+    }
 
-	/**
-	 * @return bool|mixed
-	 */
-	public function license_list() {
-		try {
-			return json_decode($this->getcurlpage($this->rest_url.'ipl/list.json?token=' . $this->authToken()));
-		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
-			return false;
-		}
-	}
+    /**
+     * @return bool|mixed
+     */
+    public function license_list() {
+        try {
+            return json_decode($this->getcurlpage($this->rest_url.'ipl/list.json?token=' . $this->authToken()));
+        } catch (Exception $e) {
+            $this->log('error', 'Caught exception code: ' . $e->getCode());
+            $this->log('error', 'Caught exception message: ' . $e->getMessage());
+            return false;
+        }
+    }
 
-	/**
-	 * Return list of all IP licenses owned by authorized user
-	 *
-	 * @throws XmlRpcException for critical errors
-	 * @return array (list<structure>): List of structures or empty list. Each structure contains keys:
-	 * 	IP(string)
-	 * 	TYPE(int) ­ license type
-	 * 	REGISTERED(boolean) ­ True if server was registered in CLN with this license
-	 */
-	public function reconcile() {
-		try {
-			return $this->response = $this->xml_client->reconcile($this->authToken());
-		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
-			return false;
-		}
-	}
+    /**
+     * Return list of all IP licenses owned by authorized user
+     *
+     * @throws XmlRpcException for critical errors
+     * @return array (list<structure>): List of structures or empty list. Each structure contains keys:
+     * 	IP(string)
+     * 	TYPE(int) ­ license type
+     * 	REGISTERED(boolean) ­ True if server was registered in CLN with this license
+     */
+    public function reconcile() {
+        try {
+            return $this->response = $this->xml_client->reconcile($this->authToken());
+        } catch (Exception $e) {
+            $this->log('error', 'Caught exception code: ' . $e->getCode());
+            $this->log('error', 'Caught exception message: ' . $e->getMessage());
+            return false;
+        }
+    }
 
-	/*
+    /*
 	public function getKeyInfo($Key) {
 	$this->response = $this->xml->__call('partner10.getKeyInfo', array(
 	$this->AuthInfo(),

Spacing +18 added lines, -18 removed lines

@@ -89,7 +89,7 @@ 
 			if (is_array($postfields)) {
 				$postdata = [];
 				foreach ($postfields as $field => $value) {
-					$postdata[] = $field . '=' . urlencode($value);
+					$postdata[] = $field.'='.urlencode($value);
 				}
 				curl_setopt($ch, CURLOPT_POSTFIELDS, implode('&', $postdata));
 			} else {
@@ -208,10 +208,10 @@ 
 	public function authToken() {
 		$time = time();
 		try {
-			return $this->login . '|' . $time . '|' . sha1($this->key . $time);
+			return $this->login.'|'.$time.'|'.sha1($this->key.$time);
 		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
+			$this->log('error', 'Caught exception code: '.$e->getCode());
+			$this->log('error', 'Caught exception message: '.$e->getMessage());
 			return false;
 		}
 	}
@@ -225,15 +225,15 @@ 
 	 * @return integer 0 on success, -1 on error
 	 */
 	public function license($ip, $type) {
-		$type = (int)$type;
+		$type = (int) $type;
 		try {
-			$this->log('error', 'Calling License(' . $this->authToken() . ',' . $ip . ',' . $type . ')');
+			$this->log('error', 'Calling License('.$this->authToken().','.$ip.','.$type.')');
 			$this->response = $this->xml_client->license($this->authToken(), $ip, $type);
-			$this->log('error', 'Response: ' . var_export($this->response, true));
+			$this->log('error', 'Response: '.var_export($this->response, true));
 			return $this->response;
 		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
+			$this->log('error', 'Caught exception code: '.$e->getCode());
+			$this->log('error', 'Caught exception message: '.$e->getMessage());
 			return false;
 		}
 	}
@@ -254,8 +254,8 @@ 
 			else
 				return $this->response = $this->remove($ip, $type);
 		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
+			$this->log('error', 'Caught exception code: '.$e->getCode());
+			$this->log('error', 'Caught exception message: '.$e->getMessage());
 			return false;
 		}
 	}
@@ -286,8 +286,8 @@ 
 		try {
 			return $this->response = $this->xml_client->is_licensed($this->authToken(), $ip, $checkAll);
 		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
+			$this->log('error', 'Caught exception code: '.$e->getCode());
+			$this->log('error', 'Caught exception message: '.$e->getMessage());
 			return false;
 		}
 	}
@@ -297,10 +297,10 @@ 
 	 */
 	public function license_list() {
 		try {
-			return json_decode($this->getcurlpage($this->rest_url.'ipl/list.json?token=' . $this->authToken()));
+			return json_decode($this->getcurlpage($this->rest_url.'ipl/list.json?token='.$this->authToken()));
 		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
+			$this->log('error', 'Caught exception code: '.$e->getCode());
+			$this->log('error', 'Caught exception message: '.$e->getMessage());
 			return false;
 		}
 	}
@@ -318,8 +318,8 @@ 
 		try {
 			return $this->response = $this->xml_client->reconcile($this->authToken());
 		} catch (Exception $e) {
-			$this->log('error', 'Caught exception code: ' . $e->getCode());
-			$this->log('error', 'Caught exception message: ' . $e->getMessage());
+			$this->log('error', 'Caught exception code: '.$e->getCode());
+			$this->log('error', 'Caught exception message: '.$e->getMessage());
 			return false;
 		}
 	}

Braces +33 added lines, -26 removed lines

@@ -57,8 +57,9 @@ 
 	public function __construct($login, $key, $type = 'rest') {
 		$this->login = $login;
 		$this->key = $key;
-		if (!isset($GLOBALS['HTTP_RAW_POST_DATA']))
-			$GLOBALS['HTTP_RAW_POST_DATA'] = file_get_contents('php://input');
+		if (!isset($GLOBALS['HTTP_RAW_POST_DATA'])) {
+					$GLOBALS['HTTP_RAW_POST_DATA'] = file_get_contents('php://input');
+		}
 		//if ($api_type == 'xml') {
 			include_once('XML/RPC2/Client.php');
 			$this->xml_options['prefix'] = $this->prefix;
@@ -108,10 +109,11 @@ 
 	}
 
 	public function log($level, $text, $line = '', $file = '') {
-		if (function_exists('myadmin_log'))
-			myadmin_log('cloudlinux', $level, $text, $line, $file);
-		else
-			error_log($text);
+		if (function_exists('myadmin_log')) {
+					myadmin_log('cloudlinux', $level, $text, $line, $file);
+		} else {
+					error_log($text);
+		}
 	}
 
 	/**
@@ -143,10 +145,11 @@ 
 	public function check($ip) {
 		$this->response = $this->getcurlpage($this->rest_url.'ipl/check.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
 		$response = json_decode($this->response, true);
-		if ($response['success'] == 1)
-			return $response['data'];
-		else
-			return false;
+		if ($response['success'] == 1) {
+					return $response['data'];
+		} else {
+					return false;
+		}
 	}
 
 	/**
@@ -169,10 +172,11 @@ 
 	 * @return bool
 	 */
 	public function rest_remove($ip, $type = 0) {
-		if ($type != 0)
-			$this->response = $this->getcurlpage($this->rest_url.'ipl/remove.json?ip='.$ip.'&type='.$type.'&token='.$this->authToken(), '', $this->rest_options);
-		else
-			$this->response = $this->getcurlpage($this->rest_url.'ipl/remove.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
+		if ($type != 0) {
+					$this->response = $this->getcurlpage($this->rest_url.'ipl/remove.json?ip='.$ip.'&type='.$type.'&token='.$this->authToken(), '', $this->rest_options);
+		} else {
+					$this->response = $this->getcurlpage($this->rest_url.'ipl/remove.json?ip='.$ip.'&token='.$this->authToken(), '', $this->rest_options);
+		}
 		return json_decode($this->response, true);
 	}
 
@@ -184,10 +188,11 @@ 
 	 * @return bool
 	 */
 	public function remove($ip, $type = 0) {
-		if ($this->api_type == 'xml')
-			return $this->remove_license($ip, $type);
-		else
-			return $this->rest_remove($ip, $type);
+		if ($this->api_type == 'xml') {
+					return $this->remove_license($ip, $type);
+		} else {
+					return $this->rest_remove($ip, $type);
+		}
 	}
 
 	/**
@@ -249,10 +254,11 @@ 
 	public function remove_license($ip, $type = false) {
 		$this->log('info', "Calling CLoudLinux->xml_client->remove_license({$this->authToken()}, {$ip}, {$type})", __LINE__, __FILE__);
 		try {
-			if ($type === false)
-				return $this->response = $this->remove($ip);
-			else
-				return $this->response = $this->remove($ip, $type);
+			if ($type === false) {
+							return $this->response = $this->remove($ip);
+			} else {
+							return $this->response = $this->remove($ip, $type);
+			}
 		} catch (Exception $e) {
 			$this->log('error', 'Caught exception code: ' . $e->getCode());
 			$this->log('error', 'Caught exception message: ' . $e->getMessage());
@@ -269,10 +275,11 @@ 
 	 * @return array (list<int>): List of registered license types or empty list if no license found
 	 */
 	public function is_licensed($ip, $checkAll = true) {
-		if ($this->api_type == 'xml')
-			return $this->xml_is_licensed($ip, $checkAll);
-		else
-			return $this->check($ip, $checkAll);
+		if ($this->api_type == 'xml') {
+					return $this->xml_is_licensed($ip, $checkAll);
+		} else {
+					return $this->check($ip, $checkAll);
+		}
 	}
 	/**
 	 * Check if IP license was registered by any customer. Arguments:

tests/bootstrap.php

Spacing +1 added lines, -1 removed lines

@@ -1,2 +1,2 @@
 <?php
-require_once(__DIR__ . '/../vendor/autoload.php');
+require_once(__DIR__.'/../vendor/autoload.php');

tests/CloudlinuxTest.php

Indentation +209 added lines, -209 removed lines

@@ -9,213 +9,213 @@
  */
 class CloudlinuxTest extends \PHPUnit_Framework_TestCase
 {
-	/**
-	 * @var Cloudlinux
-	 */
-	protected $object;
-
-	/**
-	 * Sets up the fixture, for example, opens a network connection.
-	 * This method is called before a test is executed.
-	 */
-	protected function setUp()
-	{
-		if (file_exists(__DIR__.'/.env')) {
-			$dotenv = new Dotenv\Dotenv(__DIR__);
-			$dotenv->load();
-		}
-		$this->object = new Cloudlinux(getenv('CLOUDLINUX_LOGIN'), getenv('CLOUDLINUX_KEY'));
-	}
-
-	/**
-	 * Tears down the fixture, for example, closes a network connection.
-	 * This method is called after a test is executed.
-	 */
-	protected function tearDown()
-	{
-	}
-
-	public function testTestEnvironment() {
-		$CLOUDLINUX_LOGIN = getenv('CLOUDLINUX_LOGIN');
-		$this->assertNotEmpty($CLOUDLINUX_LOGIN, 'No environment variables! Copy .env.example -> .env and fill out your account details.');
-		$this->assertInstanceOf('\Detain\Cloudlinux\Cloudlinux', $this->object);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::getcurlpage
-	 */
-	public function testGetcurlpage()
-	{
-		$this->assertRegExp('/<html/i', $this->object->getcurlpage('https://cln.cloudlinux.com'));
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::status
-	 */
-	public function testStatus()
-	{
-		//Array( [success] => 1  [data] => Array([db_rhn_online] => 1  [db_clweb_connected] => 1   [db_clweb_online] => 1  [ip_server_reg] => 1  [xmlrpc] => 1  [rhn_overloaded] =>   [db_rhn_connected] => 1  ) )
-		// Remove the following lines when you implement this test.
-		$status = $this->object->status();
-		$this->assertTrue(is_array($status));
-		$this->assertArrayHasKey('success', $status, 'Missing success status in response');
-		$this->assertArrayHasKey('data', $status, 'Missing data in response');
-		$this->assertEquals($status['data']['ip_server_reg'], 1, 'IP Server is not up');
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::availability
-	 * @todo   Implement testAvailability().
-	 */
-	public function testAvailability()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::check
-	 * @todo   Implement testCheck().
-	 */
-	public function testCheck()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::register
-	 * @todo   Implement testRegister().
-	 */
-	public function testRegister()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::rest_remove
-	 * @todo   Implement testRest_remove().
-	 */
-	public function testRest_remove()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::remove
-	 * @todo   Implement testRemove().
-	 */
-	public function testRemove()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::rest_list
-	 * @todo   Implement testRest_list().
-	 */
-	public function testRest_list()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::authToken
-	 * @todo   Implement testAuthToken().
-	 */
-	public function testAuthToken()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::license
-	 * @todo   Implement testLicense().
-	 */
-	public function testLicense()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::remove_license
-	 * @todo   Implement testRemove_license().
-	 */
-	public function testRemove_license()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::is_licensed
-	 * @todo   Implement testIs_licensed().
-	 */
-	public function testIs_licensed()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::xml_is_licensed
-	 * @todo   Implement testXml_is_licensed().
-	 */
-	public function testXml_is_licensed()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::license_list
-	 * @todo   Implement testLicense_list().
-	 */
-	public function testLicense_list()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
-
-	/**
-	 * @covers Detain\Cloudlinux\Cloudlinux::reconcile
-	 * @todo   Implement testReconcile().
-	 */
-	public function testReconcile()
-	{
-		// Remove the following lines when you implement this test.
-		$this->markTestIncomplete(
-			'This test has not been implemented yet.'
-		);
-	}
+    /**
+     * @var Cloudlinux
+     */
+    protected $object;
+
+    /**
+     * Sets up the fixture, for example, opens a network connection.
+     * This method is called before a test is executed.
+     */
+    protected function setUp()
+    {
+        if (file_exists(__DIR__.'/.env')) {
+            $dotenv = new Dotenv\Dotenv(__DIR__);
+            $dotenv->load();
+        }
+        $this->object = new Cloudlinux(getenv('CLOUDLINUX_LOGIN'), getenv('CLOUDLINUX_KEY'));
+    }
+
+    /**
+     * Tears down the fixture, for example, closes a network connection.
+     * This method is called after a test is executed.
+     */
+    protected function tearDown()
+    {
+    }
+
+    public function testTestEnvironment() {
+        $CLOUDLINUX_LOGIN = getenv('CLOUDLINUX_LOGIN');
+        $this->assertNotEmpty($CLOUDLINUX_LOGIN, 'No environment variables! Copy .env.example -> .env and fill out your account details.');
+        $this->assertInstanceOf('\Detain\Cloudlinux\Cloudlinux', $this->object);
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::getcurlpage
+     */
+    public function testGetcurlpage()
+    {
+        $this->assertRegExp('/<html/i', $this->object->getcurlpage('https://cln.cloudlinux.com'));
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::status
+     */
+    public function testStatus()
+    {
+        //Array( [success] => 1  [data] => Array([db_rhn_online] => 1  [db_clweb_connected] => 1   [db_clweb_online] => 1  [ip_server_reg] => 1  [xmlrpc] => 1  [rhn_overloaded] =>   [db_rhn_connected] => 1  ) )
+        // Remove the following lines when you implement this test.
+        $status = $this->object->status();
+        $this->assertTrue(is_array($status));
+        $this->assertArrayHasKey('success', $status, 'Missing success status in response');
+        $this->assertArrayHasKey('data', $status, 'Missing data in response');
+        $this->assertEquals($status['data']['ip_server_reg'], 1, 'IP Server is not up');
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::availability
+     * @todo   Implement testAvailability().
+     */
+    public function testAvailability()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::check
+     * @todo   Implement testCheck().
+     */
+    public function testCheck()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::register
+     * @todo   Implement testRegister().
+     */
+    public function testRegister()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::rest_remove
+     * @todo   Implement testRest_remove().
+     */
+    public function testRest_remove()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::remove
+     * @todo   Implement testRemove().
+     */
+    public function testRemove()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::rest_list
+     * @todo   Implement testRest_list().
+     */
+    public function testRest_list()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::authToken
+     * @todo   Implement testAuthToken().
+     */
+    public function testAuthToken()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::license
+     * @todo   Implement testLicense().
+     */
+    public function testLicense()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::remove_license
+     * @todo   Implement testRemove_license().
+     */
+    public function testRemove_license()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::is_licensed
+     * @todo   Implement testIs_licensed().
+     */
+    public function testIs_licensed()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::xml_is_licensed
+     * @todo   Implement testXml_is_licensed().
+     */
+    public function testXml_is_licensed()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::license_list
+     * @todo   Implement testLicense_list().
+     */
+    public function testLicense_list()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
+
+    /**
+     * @covers Detain\Cloudlinux\Cloudlinux::reconcile
+     * @todo   Implement testReconcile().
+     */
+    public function testReconcile()
+    {
+        // Remove the following lines when you implement this test.
+        $this->markTestIncomplete(
+            'This test has not been implemented yet.'
+        );
+    }
 }