nextcloud / server

lib/private/Remote/Api/ApiBase.php

Doc Comments +1 added lines, -1 removed lines

@@ -56,7 +56,7 @@
 	 * @param array $body
 	 * @param array $query
 	 * @param array $headers
-	 * @return resource|string
+	 * @return string
 	 */
 	protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
 		$fullUrl = trim($this->instance->getFullUrl(), '/') . '/' . $url;

Indentation +60 added lines, -60 removed lines

@@ -26,71 +26,71 @@
 use OCP\Http\Client\IClientService;
 
 class ApiBase {
-	/** @var Instance */
-	private $instance;
-	/** @var Credentials */
-	private $credentials;
-	/** @var IClientService */
-	private $clientService;
+    /** @var Instance */
+    private $instance;
+    /** @var Credentials */
+    private $credentials;
+    /** @var IClientService */
+    private $clientService;
 
-	public function __construct(Instance $instance, Credentials $credentials, IClientService $clientService) {
-		$this->instance = $instance;
-		$this->credentials = $credentials;
-		$this->clientService = $clientService;
-	}
+    public function __construct(Instance $instance, Credentials $credentials, IClientService $clientService) {
+        $this->instance = $instance;
+        $this->credentials = $credentials;
+        $this->clientService = $clientService;
+    }
 
-	protected function getHttpClient() {
-		return $this->clientService->newClient();
-	}
+    protected function getHttpClient() {
+        return $this->clientService->newClient();
+    }
 
-	protected function addDefaultHeaders(array $headers) {
-		return array_merge([
-			'OCS-APIREQUEST' => 'true',
-			'Accept' => 'application/json'
-		], $headers);
-	}
+    protected function addDefaultHeaders(array $headers) {
+        return array_merge([
+            'OCS-APIREQUEST' => 'true',
+            'Accept' => 'application/json'
+        ], $headers);
+    }
 
-	/**
-	 * @param string $method
-	 * @param string $url
-	 * @param array $body
-	 * @param array $query
-	 * @param array $headers
-	 * @return resource|string
-	 */
-	protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
-		$fullUrl = trim($this->instance->getFullUrl(), '/') . '/' . $url;
-		$options = [
-			'query' => $query,
-			'headers' => $this->addDefaultHeaders($headers),
-			'auth' => [$this->credentials->getUsername(), $this->credentials->getPassword()]
-		];
-		if ($body) {
-			$options['body'] = $body;
-		}
+    /**
+     * @param string $method
+     * @param string $url
+     * @param array $body
+     * @param array $query
+     * @param array $headers
+     * @return resource|string
+     */
+    protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
+        $fullUrl = trim($this->instance->getFullUrl(), '/') . '/' . $url;
+        $options = [
+            'query' => $query,
+            'headers' => $this->addDefaultHeaders($headers),
+            'auth' => [$this->credentials->getUsername(), $this->credentials->getPassword()]
+        ];
+        if ($body) {
+            $options['body'] = $body;
+        }
 
-		$client = $this->getHttpClient();
+        $client = $this->getHttpClient();
 
-		switch ($method) {
-			case 'get':
-				$response = $client->get($fullUrl, $options);
-				break;
-			case 'post':
-				$response = $client->post($fullUrl, $options);
-				break;
-			case 'put':
-				$response = $client->put($fullUrl, $options);
-				break;
-			case 'delete':
-				$response = $client->delete($fullUrl, $options);
-				break;
-			case 'options':
-				$response = $client->options($fullUrl, $options);
-				break;
-			default:
-				throw new \InvalidArgumentException('Invalid method ' . $method);
-		}
+        switch ($method) {
+            case 'get':
+                $response = $client->get($fullUrl, $options);
+                break;
+            case 'post':
+                $response = $client->post($fullUrl, $options);
+                break;
+            case 'put':
+                $response = $client->put($fullUrl, $options);
+                break;
+            case 'delete':
+                $response = $client->delete($fullUrl, $options);
+                break;
+            case 'options':
+                $response = $client->options($fullUrl, $options);
+                break;
+            default:
+                throw new \InvalidArgumentException('Invalid method ' . $method);
+        }
 
-		return $response->getBody();
-	}
+        return $response->getBody();
+    }
 }

Spacing +2 added lines, -2 removed lines

@@ -59,7 +59,7 @@ 
 	 * @return resource|string
 	 */
 	protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
-		$fullUrl = trim($this->instance->getFullUrl(), '/') . '/' . $url;
+		$fullUrl = trim($this->instance->getFullUrl(), '/').'/'.$url;
 		$options = [
 			'query' => $query,
 			'headers' => $this->addDefaultHeaders($headers),
@@ -88,7 +88,7 @@ 
 				$response = $client->options($fullUrl, $options);
 				break;
 			default:
-				throw new \InvalidArgumentException('Invalid method ' . $method);
+				throw new \InvalidArgumentException('Invalid method '.$method);
 		}
 
 		return $response->getBody();

lib/private/Remote/Instance.php

Doc Comments +3 added lines

@@ -116,6 +116,9 @@
 		return $status;
 	}
 
+	/**
+	 * @param string $url
+	 */
 	private function downloadStatus($url) {
 		try {
 			$request = $this->clientService->newClient()->get($url);

Indentation +85 added lines, -85 removed lines

@@ -28,100 +28,100 @@
  * Provides some basic info about a remote Nextcloud instance
  */
 class Instance {
-	/** @var string */
-	private $url;
+    /** @var string */
+    private $url;
 
-	/** @var ICache */
-	private $cache;
+    /** @var ICache */
+    private $cache;
 
-	/** @var IClientService */
-	private $clientService;
+    /** @var IClientService */
+    private $clientService;
 
-	private $status;
+    private $status;
 
-	/**
-	 * @param string $url
-	 * @param ICache $cache
-	 * @param IClientService $clientService
-	 */
-	public function __construct($url, ICache $cache, IClientService $clientService) {
-		$url = str_replace('https://', '', $url);
-		$this->url = str_replace('http://', '', $url);
-		$this->cache = $cache;
-		$this->clientService;
-	}
+    /**
+     * @param string $url
+     * @param ICache $cache
+     * @param IClientService $clientService
+     */
+    public function __construct($url, ICache $cache, IClientService $clientService) {
+        $url = str_replace('https://', '', $url);
+        $this->url = str_replace('http://', '', $url);
+        $this->cache = $cache;
+        $this->clientService;
+    }
 
-	/**
-	 * @return string The url of the remote server without protocol
-	 */
-	public function getUrl() {
-		return $this->url;
-	}
+    /**
+     * @return string The url of the remote server without protocol
+     */
+    public function getUrl() {
+        return $this->url;
+    }
 
-	/**
-	 * @return string The of of the remote server with protocol
-	 */
-	public function getFullUrl() {
-		return $this->getProtocol() . '://' . $this->getUrl();
-	}
+    /**
+     * @return string The of of the remote server with protocol
+     */
+    public function getFullUrl() {
+        return $this->getProtocol() . '://' . $this->getUrl();
+    }
 
-	/**
-	 * @return string The full version string in '13.1.2.3' format
-	 */
-	public function getVersion() {
-		$status = $this->getStatus();
-		return $status['version'];
-	}
+    /**
+     * @return string The full version string in '13.1.2.3' format
+     */
+    public function getVersion() {
+        $status = $this->getStatus();
+        return $status['version'];
+    }
 
-	/**
-	 * @return string 'http' or 'https'
-	 */
-	public function getProtocol() {
-		$status = $this->getStatus();
-		return $status['protocol'];
-	}
+    /**
+     * @return string 'http' or 'https'
+     */
+    public function getProtocol() {
+        $status = $this->getStatus();
+        return $status['protocol'];
+    }
 
-	/**
-	 * Check that the remote server is installed and not in maintenance mode
-	 *
-	 * @return bool
-	 */
-	public function isActive() {
-		$status = $this->getStatus();
-		return $status['installed'] && !$status['maintenance'];
-	}
+    /**
+     * Check that the remote server is installed and not in maintenance mode
+     *
+     * @return bool
+     */
+    public function isActive() {
+        $status = $this->getStatus();
+        return $status['installed'] && !$status['maintenance'];
+    }
 
-	private function getStatus() {
-		if ($this->status) {
-			return $this->status;
-		}
-		$key = 'remote/' . $this->url . '/status';
-		$status = $this->cache->get($key);
-		if (!$status) {
-			$response = $this->downloadStatus('https://' . $this->getUrl() . '/status.php');
-			$protocol = 'https';
-			if (!$response) {
-				$response = $this->downloadStatus('http://' . $this->getUrl() . '/status.php');
-				$protocol = 'http';
-			}
-			$status = json_decode($response, true);
-			if ($status) {
-				$status['protocol'] = $protocol;
-			}
-			if ($status) {
-				$this->cache->set($key, $status, 5 * 60);
-				$this->status = $status;
-			}
-		}
-		return $status;
-	}
+    private function getStatus() {
+        if ($this->status) {
+            return $this->status;
+        }
+        $key = 'remote/' . $this->url . '/status';
+        $status = $this->cache->get($key);
+        if (!$status) {
+            $response = $this->downloadStatus('https://' . $this->getUrl() . '/status.php');
+            $protocol = 'https';
+            if (!$response) {
+                $response = $this->downloadStatus('http://' . $this->getUrl() . '/status.php');
+                $protocol = 'http';
+            }
+            $status = json_decode($response, true);
+            if ($status) {
+                $status['protocol'] = $protocol;
+            }
+            if ($status) {
+                $this->cache->set($key, $status, 5 * 60);
+                $this->status = $status;
+            }
+        }
+        return $status;
+    }
 
-	private function downloadStatus($url) {
-		try {
-			$request = $this->clientService->newClient()->get($url);
-			return $request->getBody();
-		} catch (\Exception $e) {
-			return false;
-		}
-	}
+    private function downloadStatus($url) {
+        try {
+            $request = $this->clientService->newClient()->get($url);
+            return $request->getBody();
+        } catch (\Exception $e) {
+            return false;
+        }
+    }
 }

Spacing +4 added lines, -4 removed lines

@@ -62,7 +62,7 @@ 
 	 * @return string The of of the remote server with protocol
 	 */
 	public function getFullUrl() {
-		return $this->getProtocol() . '://' . $this->getUrl();
+		return $this->getProtocol().'://'.$this->getUrl();
 	}
 
 	/**
@@ -95,13 +95,13 @@ 
 		if ($this->status) {
 			return $this->status;
 		}
-		$key = 'remote/' . $this->url . '/status';
+		$key = 'remote/'.$this->url.'/status';
 		$status = $this->cache->get($key);
 		if (!$status) {
-			$response = $this->downloadStatus('https://' . $this->getUrl() . '/status.php');
+			$response = $this->downloadStatus('https://'.$this->getUrl().'/status.php');
 			$protocol = 'https';
 			if (!$response) {
-				$response = $this->downloadStatus('http://' . $this->getUrl() . '/status.php');
+				$response = $this->downloadStatus('http://'.$this->getUrl().'/status.php');
 				$protocol = 'http';
 			}
 			$status = json_decode($response, true);

lib/private/Remote/User.php

Indentation +98 added lines, -98 removed lines

@@ -23,102 +23,102 @@
 
 
 class User {
-	/** @var array */
-	private $data;
-
-	public function __construct(array $data) {
-		$this->data = $data;
-	}
-
-
-	/**
-	 * @return string
-	 */
-	public function getUserId() {
-		return $this->data['id'];
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getEmail() {
-		return $this->data['email'];
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getDisplayName() {
-		return $this->data['displayname'];
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getPhone() {
-		return $this->data['phone'];
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getAddress() {
-		return $this->data['address'];
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getWebsite() {
-		return $this->data['website'];
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getTwitter() {
-		return $this->data['twitter'];
-	}
-
-	/**
-	 * @return string[]
-	 */
-	public function getGroups() {
-		return $this->data['groups'];
-	}
-
-	/**
-	 * @return string
-	 */
-	public function getLanguage() {
-		return $this->data['language'];
-	}
-
-	/**
-	 * @return int
-	 */
-	public function getUsedSpace() {
-		return $this->data['quota']['used'];
-	}
-
-	/**
-	 * @return int
-	 */
-	public function getFreeSpace() {
-		return $this->data['quota']['free'];
-	}
-
-	/**
-	 * @return int
-	 */
-	public function getTotalSpace() {
-		return $this->data['quota']['total'];
-	}
-
-	/**
-	 * @return int
-	 */
-	public function getQuota() {
-		return $this->data['quota']['quota'];
-	}
+    /** @var array */
+    private $data;
+
+    public function __construct(array $data) {
+        $this->data = $data;
+    }
+
+
+    /**
+     * @return string
+     */
+    public function getUserId() {
+        return $this->data['id'];
+    }
+
+    /**
+     * @return string
+     */
+    public function getEmail() {
+        return $this->data['email'];
+    }
+
+    /**
+     * @return string
+     */
+    public function getDisplayName() {
+        return $this->data['displayname'];
+    }
+
+    /**
+     * @return string
+     */
+    public function getPhone() {
+        return $this->data['phone'];
+    }
+
+    /**
+     * @return string
+     */
+    public function getAddress() {
+        return $this->data['address'];
+    }
+
+    /**
+     * @return string
+     */
+    public function getWebsite() {
+        return $this->data['website'];
+    }
+
+    /**
+     * @return string
+     */
+    public function getTwitter() {
+        return $this->data['twitter'];
+    }
+
+    /**
+     * @return string[]
+     */
+    public function getGroups() {
+        return $this->data['groups'];
+    }
+
+    /**
+     * @return string
+     */
+    public function getLanguage() {
+        return $this->data['language'];
+    }
+
+    /**
+     * @return int
+     */
+    public function getUsedSpace() {
+        return $this->data['quota']['used'];
+    }
+
+    /**
+     * @return int
+     */
+    public function getFreeSpace() {
+        return $this->data['quota']['free'];
+    }
+
+    /**
+     * @return int
+     */
+    public function getTotalSpace() {
+        return $this->data['quota']['total'];
+    }
+
+    /**
+     * @return int
+     */
+    public function getQuota() {
+        return $this->data['quota']['quota'];
+    }
 }

lib/private/Remote/Api/OCS.php

Indentation +26 added lines, -26 removed lines

@@ -27,33 +27,33 @@
 use OCP\API;
 
 class OCS extends ApiBase {
-	/**
-	 * @param string $method
-	 * @param string $url
-	 * @param array $body
-	 * @param array $query
-	 * @param array $headers
-	 * @return array
-	 * @throws ForbiddenException
-	 * @throws NotFoundException
-	 */
-	protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
-		$response = json_decode(parent::request($method, $url, $body, $query, $headers), true);
-		if ($response['ocs']['meta']['status'] === API::RESPOND_UNAUTHORISED) {
-			throw new ForbiddenException();
-		}
-		if ($response['ocs']['meta']['status'] === API::RESPOND_NOT_FOUND) {
-			throw new NotFoundException();
-		}
+    /**
+     * @param string $method
+     * @param string $url
+     * @param array $body
+     * @param array $query
+     * @param array $headers
+     * @return array
+     * @throws ForbiddenException
+     * @throws NotFoundException
+     */
+    protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
+        $response = json_decode(parent::request($method, $url, $body, $query, $headers), true);
+        if ($response['ocs']['meta']['status'] === API::RESPOND_UNAUTHORISED) {
+            throw new ForbiddenException();
+        }
+        if ($response['ocs']['meta']['status'] === API::RESPOND_NOT_FOUND) {
+            throw new NotFoundException();
+        }
 
-		return $response['ocs']['data'];
-	}
+        return $response['ocs']['data'];
+    }
 
-	public function getUser($userId) {
-		return new User($this->request('get', '/ocs/v1.php/cloud/users/' . $userId));
-	}
+    public function getUser($userId) {
+        return new User($this->request('get', '/ocs/v1.php/cloud/users/' . $userId));
+    }
 
-	public function getCapabilities() {
-		return $this->request('get', '/ocs/v1.php/cloud/capabilities');
-	}
+    public function getCapabilities() {
+        return $this->request('get', '/ocs/v1.php/cloud/capabilities');
+    }
 }

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
 	}
 
 	public function getUser($userId) {
-		return new User($this->request('get', '/ocs/v1.php/cloud/users/' . $userId));
+		return new User($this->request('get', '/ocs/v1.php/cloud/users/'.$userId));
 	}
 
 	public function getCapabilities() {

lib/private/Remote/Credentials.php

Indentation +24 added lines, -24 removed lines

@@ -23,31 +23,31 @@
 
 
 class Credentials {
-	/** @var string */
-	private $user;
-	/** @var string */
-	private $password;
+    /** @var string */
+    private $user;
+    /** @var string */
+    private $password;
 
-	/**
-	 * @param string $user
-	 * @param string $password
-	 */
-	public function __construct($user, $password) {
-		$this->user = $user;
-		$this->password = $password;
-	}
+    /**
+     * @param string $user
+     * @param string $password
+     */
+    public function __construct($user, $password) {
+        $this->user = $user;
+        $this->password = $password;
+    }
 
-	/**
-	 * @return string
-	 */
-	public function getUsername() {
-		return $this->user;
-	}
+    /**
+     * @return string
+     */
+    public function getUsername() {
+        return $this->user;
+    }
 
-	/**
-	 * @return string
-	 */
-	public function getPassword() {
-		return $this->password;
-	}
+    /**
+     * @return string
+     */
+    public function getPassword() {
+        return $this->password;
+    }
 }