linting, unit tests, comment fixes

Signed-off-by: leith abdulla <online-nextcloud@eleith.com>
nextcloud · Nov 2, 2020 · da321c1 · da321c1
1 parent 296ebf7
commit da321c1
Show file tree

Hide file tree

Showing 11 changed files with 238 additions and 143 deletions.
diff --git a/lib/Service/Social/CompositeSocialProvider.php b/lib/Service/Social/CompositeSocialProvider.php
@@ -65,10 +65,9 @@ public function getSupportedNetworks() : array {
 	/**
 	 * generate download url for a social entry
 	 *
-	 * @param array contact all social data from the contact
 	 * @param String network the choice which network to use
 	 *
-	 * @returns ISocialProvider if provider of 'network' is found, otherwise null
+	 * @return ISocialProvider if provider of 'network' is found, otherwise null
 	 */
 	public function getSocialConnector(string $network) : ?ISocialProvider {
 		$connector = null;
@@ -82,8 +81,6 @@ public function getSocialConnector(string $network) : ?ISocialProvider {
 	/**
 	 * generate download url for a social entry
 	 *
-	 * @param array contact all social data from the contact
-	 *
 	 * @return ISocialProvider[] all social providers
 	 */
 	public function getSocialConnectors() : array {

diff --git a/lib/Service/Social/DiasporaProvider.php b/lib/Service/Social/DiasporaProvider.php
@@ -33,8 +33,8 @@ class DiasporaProvider implements ISocialProvider {
 	/** @var bool */
 	private $looping;
 
-  /** @var string */
-  public $name = "diaspora";
+	/** @var string */
+	public $name = "diaspora";
 
 	public function __construct(IClientService $httpClient) {
 		$this->httpClient = $httpClient->NewClient();
@@ -49,41 +49,41 @@ public function __construct(IClientService $httpClient) {
 	 * @return bool
 	 */
 	public function supportsContact(array $contact):bool {
-    $socialprofiles = $contact['X-SOCIALPROFILE'];
-    $supports = false;
-    if(isset($socialprofiles)) {
-      foreach($socialprofiles as $profile) {
-        if ($profile['type'] == $this->name) {
-          $supports = true;
-          break;
-        }
-      }
-    }
-    return $supports;
+		$socialprofiles = $contact['X-SOCIALPROFILE'];
+		$supports = false;
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
+				if ($profile['type'] == $this->name) {
+					$supports = true;
+					break;
+				}
+			}
+		}
+		return $supports;
 	}
 
 	/**
 	 * Returns all possible profile-picture urls
 	 *
 	 * @param {array} contact information
 	 *
-	 * @return array 
+	 * @return array
 	 */
 	public function getImageUrls(array $contact):array {
-    $profileIds = $this->getProfileIds($contact);
-    $urls = array();
+		$profileIds = $this->getProfileIds($contact);
+		$urls = [];
 
-    foreach($profileIds as $profileId) {
-      $url = $this->getImageUrl($profileId);
-      if (isset($url)) {
-        $urls[] = $url;
-      }
-    }
+		foreach ($profileIds as $profileId) {
+			$url = $this->getImageUrl($profileId);
+			if (isset($url)) {
+				$urls[] = $url;
+			}
+		}
 
-    return $urls;
+		return $urls;
 	}
 
-  /**
+	/**
 	 * Returns the profile-picture url
 	 *
 	 * @param {string} profileId the profile-id
@@ -113,29 +113,29 @@ protected function getImageUrl(string $profileUrl):?string {
 		}
 	}
 
-  /**
-	 * Returns all possible profile ids for contact 
+	/**
+	 * Returns all possible profile ids for contact
 	 *
 	 * @param {array} contact information
 	 *
-	 * @return array 
+	 * @return array
 	 */
-  protected function getProfileIds($contact):array {
-    $socialprofiles = $contact['X-SOCIALPROFILE'];
-    $profileIds = array();
+	protected function getProfileIds($contact):array {
+		$socialprofiles = $contact['X-SOCIALPROFILE'];
+		$profileIds = [];
 
-    if(isset($socialprofiles)) {
-      foreach($socialprofiles as $profile) {
-        if (strtolower($profile['type']) == $this->name) {
-          $profileId = $this->cleanupId($profile['value']);
-          if (isset($profileId)) {
-            $profileIds[] = $profileId;
-          }
-        }
-      }
-    }
-    return $profileIds;
-  }
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
+				if (strtolower($profile['type']) == $this->name) {
+					$profileId = $this->cleanupId($profile['value']);
+					if (isset($profileId)) {
+						$profileIds[] = $profileId;
+					}
+				}
+			}
+		}
+		return $profileIds;
+	}
 
 	/**
 	 * Returns the profile-id

diff --git a/lib/Service/Social/FacebookProvider.php b/lib/Service/Social/FacebookProvider.php
@@ -47,8 +47,8 @@ public function __construct(IClientService $httpClient) {
 	public function supportsContact(array $contact):bool {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
 		$supports = false;
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$supports = true;
 					break;
@@ -67,8 +67,8 @@ public function supportsContact(array $contact):bool {
 	 */
 	public function getImageUrls(array $contact):array {
 		$profileIds = $this->getProfileIds($contact);
-		$urls = array();
-		foreach($profileIds as $profileId) {
+		$urls = [];
+		foreach ($profileIds as $profileId) {
 			$recipe = 'https://graph.facebook.com/{socialId}/picture?width=720';
 			$connector = str_replace("{socialId}", $profileId, $recipe);
 			$urls[] = $connector;
@@ -100,9 +100,9 @@ protected function cleanupId(string $candidate):string {
 	 */
 	protected function getProfileIds($contact):array {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
-		$profileIds = array();
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		$profileIds = [];
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$profileIds[] = $this->cleanupId($profile['value']);
 				}

diff --git a/lib/Service/Social/GravatarProvider.php b/lib/Service/Social/GravatarProvider.php
@@ -54,8 +54,8 @@ public function supportsContact(array $contact):bool {
 	 */
 	public function getImageUrls(array $contact):array {
 		$emails = $this->getProfileIds($contact);
-		$urls = array();
-		foreach($emails as $email) {
+		$urls = [];
+		foreach ($emails as $email) {
 			$hash = md5(strtolower(trim($email['value'])));
 			$recipe = 'https://www.gravatar.com/avatar/{hash}?s=720&d=404';
 			$connector = str_replace("{hash}", $hash, $recipe);
@@ -76,6 +76,6 @@ protected function getProfileIds(array $contact):array {
 		if (isset($emails)) {
 			return $emails;
 		}
-		return array();
+		return [];
 	}
 }
diff --git a/lib/Service/Social/InstagramProvider.php b/lib/Service/Social/InstagramProvider.php
@@ -47,8 +47,8 @@ public function __construct(IClientService $httpClient) {
 	public function supportsContact(array $contact):bool {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
 		$supports = false;
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$supports = true;
 					break;
@@ -67,8 +67,8 @@ public function supportsContact(array $contact):bool {
 	 */
 	public function getImageUrls(array $contact):array {
 		$profileIds = $this->getProfileIds($contact);
-		$urls = array();
-		foreach($profileIds as $profileId) {
+		$urls = [];
+		foreach ($profileIds as $profileId) {
 			$recipe = 'https://www.instagram.com/{socialId}/?__a=1';
 			$connector = str_replace("{socialId}", $profileId, $recipe);
 			$connector = $this->getFromJson($connector, 'graphql->user->profile_pic_url_hd');
@@ -98,9 +98,9 @@ protected function cleanupId(string $candidate):string {
 	 */
 	protected function getProfileIds($contact):array {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
-		$profileIds = array();
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		$profileIds = [];
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$profileIds[] = $this->cleanupId($profile['value']);
 				}

diff --git a/lib/Service/Social/MastodonProvider.php b/lib/Service/Social/MastodonProvider.php
@@ -30,7 +30,7 @@ class MastodonProvider implements ISocialProvider {
 	/** @var IClientService */
 	private $httpClient;
 
-		/** @var string */
+	/** @var string */
 	public $name = "mastodon";
 
 	public function __construct(IClientService $httpClient) {
@@ -47,8 +47,8 @@ public function __construct(IClientService $httpClient) {
 	public function supportsContact(array $contact):bool {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
 		$supports = false;
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$supports = true;
 					break;
@@ -67,9 +67,9 @@ public function supportsContact(array $contact):bool {
 	 */
 	public function getImageUrls(array $contact):array {
 		$profileIds = $this->getProfileIds($contact);
-		$urls = array();
+		$urls = [];
 
-		foreach($profileIds as $profileId) {
+		foreach ($profileIds as $profileId) {
 			$url = $this->getImageUrl($profileId);
 			if (isset($url)) {
 				$urls[] = $url;
@@ -110,12 +110,12 @@ public function getImageUrl(string $profileUrl):?string {
 	 */
 	protected function getProfileIds($contact):array {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
-		$profileIds = array();
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		$profileIds = [];
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$profileId = $this->cleanupId($profile['value']);
-					if(isset($profileId)) {
+					if (isset($profileId)) {
 						$profileIds[] = $profileId;
 					}
 				}

diff --git a/lib/Service/Social/TumblrProvider.php b/lib/Service/Social/TumblrProvider.php
@@ -40,8 +40,8 @@ public function __construct() {
 	public function supportsContact(array $contact):bool {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
 		$supports = false;
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$supports = true;
 					break;
@@ -60,8 +60,8 @@ public function supportsContact(array $contact):bool {
 	 */
 	public function getImageUrls(array $contact):array {
 		$profileIds = $this->getProfileIds($contact);
-		$urls = array();
-		foreach($profileIds as $profileId) {
+		$urls = [];
+		foreach ($profileIds as $profileId) {
 			$recipe = 'https://api.tumblr.com/v2/blog/{socialId}/avatar/512';
 			$connector = str_replace("{socialId}", $profileId, $recipe);
 			$urls[] = $connector;
@@ -94,9 +94,9 @@ protected function cleanupId(string $candidate):?string {
 	 */
 	protected function getProfileIds($contact):array {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
-		$profileIds = array();
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		$profileIds = [];
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$profileIds[] = $this->cleanupId($profile['value']);
 				}

diff --git a/lib/Service/Social/TwitterProvider.php b/lib/Service/Social/TwitterProvider.php
@@ -45,8 +45,8 @@ public function __construct(IClientService $httpClient) {
 	 */
 	public function supportsContact(array $contact):bool {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					return true;
 				}
@@ -64,8 +64,8 @@ public function supportsContact(array $contact):bool {
 	 */
 	public function getImageUrls(array $contact):array {
 		$profileIds = $this->getProfileIds($contact);
-		$urls = array();
-		foreach($profileIds as $profileId) {
+		$urls = [];
+		foreach ($profileIds as $profileId) {
 			$recipe = 'https://mobile.twitter.com/{socialId}';
 			$connector = str_replace("{socialId}", $profileId, $recipe);
 			$connector = $this->getFromHtml($connector, '_normal');
@@ -98,9 +98,9 @@ protected function cleanupId(string $candidate):string {
 	 */
 	protected function getProfileIds($contact):array {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
-		$profileIds = array();
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		$profileIds = [];
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$profileIds[] = $this->cleanupId($profile['value']);
 				}

diff --git a/lib/Service/Social/XingProvider.php b/lib/Service/Social/XingProvider.php
@@ -48,8 +48,8 @@ public function __construct(IClientService $httpClient) {
 	public function supportsContact(array $contact):bool {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
 		$supports = false;
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$supports = true;
 					break;
@@ -68,9 +68,9 @@ public function supportsContact(array $contact):bool {
 	 */
 	public function getImageUrls(array $contact):array {
 		$profileIds = $this->getProfileIds($contact);
-		$urls = array();
+		$urls = [];
 
-		foreach($profileIds as $profileId) {
+		foreach ($profileIds as $profileId) {
 			$url = $this->getImageUrl($profileId);
 			if (isset($url)) {
 				$urls[] = $url;
@@ -131,12 +131,12 @@ protected function cleanupId(string $candidate):?string {
 	 */
 	protected function getProfileIds($contact):array {
 		$socialprofiles = $contact['X-SOCIALPROFILE'];
-		$profileIds = array();
-		if(isset($socialprofiles)) {
-			foreach($socialprofiles as $profile) {
+		$profileIds = [];
+		if (isset($socialprofiles)) {
+			foreach ($socialprofiles as $profile) {
 				if (strtolower($profile['type']) == $this->name) {
 					$profileId = $this->cleanupId($profile['value']);
-					if(isset($profileId)) {
+					if (isset($profileId)) {
 						$profileIds[] = $profileId;
 					}
 				}