fix(client): updated Client class code

Author: DIOHz0r

src/Glpi/Api/Rest/Client.php

@@ -149,6 +149,9 @@ class Client {
    /** @var string Session token obtained after initSession() */
    private $sessionToken = null;
 
+   /**
+    * @deprecated this could be moved to ItemHandler
+    */
    private $simpleEndpoints = [
       'Alert',
       'APIClient',
@@ -303,16 +306,25 @@ public function initSessionByUserToken($userToken) {
     */
    public function killSession() {
       $response = $this->request('get', 'killSession');
-      if (!$response->getStatusCode() != 200) {
+      if ($response->getStatusCode() != 200) {
          throw new Exception('session_token seems invalid');
       }
       return true;
    }
 
+   /**
+    * @deprecated this could be moved to ItemHandler
+    */
    public function getSimpleEndpoints() {
       return $this->simpleEndpoints;
    }
 
+   /**
+    * @param $name
+    * @param $arguments
+    * @return \Psr\Http\Message\ResponseInterface
+    * @deprecated this will be removed, use the @method request() instead
+    */
    public function __call($name, $arguments) {
       $name = ucfirst($name);
       if (!in_array($name, $this->simpleEndpoints)) {
@@ -343,17 +355,15 @@ public function __call($name, $arguments) {
    public function request($method, $uri, array $options = []) {
       $apiToken = $this->addTokens();
       try {
-         if (!empty($uri)) {
-            $options['headers']['Content-Type'] = "application/json";
-         }
+         $options['headers']['Content-Type'] = "application/json";
          if ($apiToken) {
             $sessionHeaders = ['Session-Token' => $apiToken['Session-Token']];
             if (key_exists('App-Token', $apiToken)) {
                $sessionHeaders['App-Token'] = $apiToken['App-Token'];
             }
             $options = array_merge_recursive($options, ['headers' => $sessionHeaders]);
          }
-         $response = $this->httpClient->request($method, $this->url.$uri, $options);
+         $response = $this->httpClient->request($method, $this->url . $uri, $options);
          return $response;
       } catch (ClientException $e) {
          $response = $e->getResponse();
@@ -382,7 +392,7 @@ public function getFullSession() {
     * @return array
     */
    public function getGlpiConfig() {
-      $response = $this->request('get', 'getFullSession');
+      $response = $this->request('get', 'getGlpiConfig');
       return ['statusCode' => $response->getStatusCode(), 'body' => $response->getBody()->getContents()];
    }
 
@@ -396,6 +406,7 @@ public function getGlpiConfig() {
     * @throws Exception
     *
     * @return \Psr\Http\Message\ResponseInterface the response sent by the server
+    * @deprecated this will be removed, use the @method request() instead
     */
    protected function doHttpRequest($verb = "get", $relative_uri = "", $params = []) {
       if (!empty($relative_uri)) {