add more typehints

Signed-off-by: Robin Appelman <robin@icewind.nl>

5 files changed, 16 insertions, 0 deletions

diff --git a/lib/private/Remote/Api/ApiBase.php b/lib/private/Remote/Api/ApiBase.php
index 64153a9311f..70292a977f4 100644
--- a/lib/private/Remote/Api/ApiBase.php
+++ b/lib/private/Remote/Api/ApiBase.php
@@ -57,6 +57,7 @@ class ApiBase {
 	 * @param array $query
 	 * @param array $headers
 	 * @return resource|string
+	 * @throws \InvalidArgumentException
 	 */
 	protected function request($method, $url, array $body = [], array $query = [], array $headers = []) {
 		$fullUrl = trim($this->instance->getFullUrl(), '/') . '/' . $url;
diff --git a/lib/private/Remote/Api/ApiCollection.php b/lib/private/Remote/Api/ApiCollection.php
index 41b1bac0e08..5ce97621dbb 100644
--- a/lib/private/Remote/Api/ApiCollection.php
+++ b/lib/private/Remote/Api/ApiCollection.php
@@ -28,8 +28,11 @@ use OCP\Remote\ICredentials;
 use OCP\Remote\IInstance;
 
 class ApiCollection implements IApiCollection {
+	/** @var IInstance */
 	private $instance;
+	/** @var ICredentials */
 	private $credentials;
+	/** @var IClientService */
 	private $clientService;
 
 	public function __construct(IInstance $instance, ICredentials $credentials, IClientService $clientService) {
diff --git a/lib/private/Remote/Api/ApiFactory.php b/lib/private/Remote/Api/ApiFactory.php
index ea084c188f1..19b8e8eb50c 100644
--- a/lib/private/Remote/Api/ApiFactory.php
+++ b/lib/private/Remote/Api/ApiFactory.php
@@ -28,6 +28,7 @@ use OCP\Remote\ICredentials;
 use OCP\Remote\IInstance;
 
 class ApiFactory implements IApiFactory {
+	/** @var IClientService */
 	private $clientService;
 
 	public function __construct(IClientService $clientService) {
diff --git a/lib/private/Remote/Instance.php b/lib/private/Remote/Instance.php
index ab0081d86cf..0ed301ae868 100644
--- a/lib/private/Remote/Instance.php
+++ b/lib/private/Remote/Instance.php
@@ -39,6 +39,7 @@ class Instance implements IInstance {
 	/** @var IClientService */
 	private $clientService;
 
+	/** @var array|null */
 	private $status;
 
 	/**
@@ -93,6 +94,10 @@ class Instance implements IInstance {
 		return $status['installed'] && !$status['maintenance'];
 	}
 
+	/**
+	 * @return array
+	 * @throws NotFoundException
+	 */
 	private function getStatus() {
 		if ($this->status) {
 			return $this->status;
@@ -120,6 +125,10 @@ class Instance implements IInstance {
 		return $status;
 	}
 
+	/**
+	 * @param string $url
+	 * @return bool|string
+	 */
 	private function downloadStatus($url) {
 		try {
 			$request = $this->clientService->newClient()->get($url);
diff --git a/lib/private/Remote/InstanceFactory.php b/lib/private/Remote/InstanceFactory.php
index 3b99bc61825..72baa433615 100644
--- a/lib/private/Remote/InstanceFactory.php
+++ b/lib/private/Remote/InstanceFactory.php
@@ -27,7 +27,9 @@ use OCP\ICache;
 use OCP\Remote\IInstanceFactory;
 
 class InstanceFactory implements IInstanceFactory {
+	/** @var ICache */
 	private $cache;
+	/** @var IClientService */
 	private $clientService;
 
 	public function __construct(ICache $cache, IClientService $clientService) {