From 3232df7a8c0e7e203b9df610412130530a80be79 Mon Sep 17 00:00:00 2001
From: Maxence Lange <maxence@artificial-owl.com>
Date: Tue, 4 Dec 2018 23:37:08 -0100
Subject: [PATCH 1/2] exception when following yourself

Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
---
 lib/Exceptions/FollowSameAccountException.php | 8 ++++++++
 lib/Service/ActivityPub/FollowService.php     | 6 ++++++
 2 files changed, 14 insertions(+)
 create mode 100644 lib/Exceptions/FollowSameAccountException.php

diff --git a/lib/Exceptions/FollowSameAccountException.php b/lib/Exceptions/FollowSameAccountException.php
new file mode 100644
index 00000000..cdf5140a
--- /dev/null
+++ b/lib/Exceptions/FollowSameAccountException.php
@@ -0,0 +1,8 @@
+<?php
+
+namespace OCA\Social\Exceptions;
+
+class FollowSameAccountException extends \Exception {
+
+}
+
diff --git a/lib/Service/ActivityPub/FollowService.php b/lib/Service/ActivityPub/FollowService.php
index 7e1c35f8..d12bf936 100644
--- a/lib/Service/ActivityPub/FollowService.php
+++ b/lib/Service/ActivityPub/FollowService.php
@@ -36,6 +36,7 @@ use OCA\Social\Db\FollowsRequest;
 use OCA\Social\Exceptions\ActorDoesNotExistException;
 use OCA\Social\Exceptions\CacheActorDoesNotExistException;
 use OCA\Social\Exceptions\FollowDoesNotExistException;
+use OCA\Social\Exceptions\FollowSameAccountException;
 use OCA\Social\Exceptions\InvalidResourceException;
 use OCA\Social\Exceptions\RequestException;
 use OCA\Social\Exceptions\SocialAppConfigException;
@@ -121,9 +122,14 @@ class FollowService implements ICoreService {
 	 * @throws CacheActorDoesNotExistException
 	 * @throws InvalidResourceException
 	 * @throws UrlCloudException
+	 * @throws FollowSameAccountException
 	 */
 	public function followAccount(Person $actor, string $account) {
 		$remoteActor = $this->personService->getFromAccount($account);
+		if ($remoteActor->getId() === $actor->getId()) {
+			throw new FollowSameAccountException("Don't follow yourself, be your own lead");
+		}
+
 		$follow = new Follow();
 		$follow->setUrlCloud($this->configService->getCloudAddress());
 		$follow->generateUniqueId();

From 8184dbd70583e79686322f5b551167168e283392 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Julius=20H=C3=A4rtl?= <jus@bitgrid.net>
Date: Wed, 5 Dec 2018 09:48:02 +0100
Subject: [PATCH 2/2] Hide follow actions for own account
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Signed-off-by: Julius Härtl <jus@bitgrid.net>
---
 src/components/ProfileInfo.vue |  8 ++++++--
 src/components/UserEntry.vue   | 18 +++++++++++-------
 src/mixins/currentUserMixin.js |  9 ++++++---
 src/mixins/follow.js           |  4 ++--
 4 files changed, 25 insertions(+), 14 deletions(-)

diff --git a/src/components/ProfileInfo.vue b/src/components/ProfileInfo.vue
index 983a7fe1..9a4ba0f1 100644
--- a/src/components/ProfileInfo.vue
+++ b/src/components/ProfileInfo.vue
@@ -27,7 +27,7 @@
 			<h2>{{ displayName }}</h2>
 			<p>{{ accountInfo.account }}</p>
 			<p v-if="accountInfo.website">Website: <a :href="accountInfo.website.value">{{ accountInfo.website.value }}</a></p>
-			<template v-if="currentUser.uid !== accountInfo.preferredUsername">
+			<template v-if="cloudId !== accountInfo.account">
 				<button v-if="accountInfo.details && accountInfo.details.following" :class="{'icon-loading-small': followLoading}"
 					@click="unfollow()"
 					@mouseover="followingText=t('social', 'Unfollow')" @mouseleave="followingText=t('social', 'Following')">
@@ -100,7 +100,11 @@ export default {
 	components: {
 		Avatar
 	},
-	mixins: [serverData, currentUser, follow],
+	mixins: [
+		serverData,
+		currentUser,
+		follow
+	],
 	props: {
 		uid: {
 			type: String,
diff --git a/src/components/UserEntry.vue b/src/components/UserEntry.vue
index 95a94c4a..56220887 100644
--- a/src/components/UserEntry.vue
+++ b/src/components/UserEntry.vue
@@ -36,12 +36,14 @@
 				<!-- TODO check where the html is coming from to avoid security issues -->
 				<p v-html="item.summary" />
 			</div>
-			<button v-if="item.details && item.details.following" :class="{'icon-loading-small': followLoading}"
-				@click="unfollow()"
-				@mouseover="followingText=t('social', 'Unfollow')" @mouseleave="followingText=t('social', 'Following')">
-			<span><span class="icon-checkmark" />{{ followingText }}</span></button>
-			<button v-else-if="item.details" :class="{'icon-loading-small': followLoading}" class="primary"
-				@click="follow"><span>{{ t('social', 'Follow') }}</span></button>
+			<template v-if="cloudId !== item.account">
+				<button v-if="item.details && item.details.following" :class="{'icon-loading-small': followLoading}"
+					@click="unfollow()"
+					@mouseover="followingText=t('social', 'Unfollow')" @mouseleave="followingText=t('social', 'Following')">
+				<span><span class="icon-checkmark" />{{ followingText }}</span></button>
+				<button v-else-if="item.details" :class="{'icon-loading-small': followLoading}" class="primary"
+					@click="follow"><span>{{ t('social', 'Follow') }}</span></button>
+			</template>
 		</div>
 	</div>
 </template>
@@ -49,6 +51,7 @@
 <script>
 import { Avatar } from 'nextcloud-vue'
 import follow from '../mixins/follow'
+import currentUser from '../mixins/currentUserMixin'
 
 export default {
 	name: 'UserEntry',
@@ -56,7 +59,8 @@ export default {
 		Avatar
 	},
 	mixins: [
-		follow
+		follow,
+		currentUser
 	],
 	props: {
 		item: { type: Object, default: () => {} }
diff --git a/src/mixins/currentUserMixin.js b/src/mixins/currentUserMixin.js
index 8bc5adfa..1678b3f3 100644
--- a/src/mixins/currentUserMixin.js
+++ b/src/mixins/currentUserMixin.js
@@ -26,13 +26,16 @@ export default {
 		serverData
 	],
 	computed: {
-		currentUser: function() {
+		currentUser() {
 			return OC.getCurrentUser()
 		},
-		socialId: function() {
+		socialId() {
+			return '@' + this.cloudId
+		},
+		cloudId() {
 			const url = document.createElement('a')
 			url.setAttribute('href', this.serverData.cloudAddress)
-			return '@' + OC.getCurrentUser().uid + '@' + url.hostname
+			return this.currentUser.uid + '@' + url.hostname
 		}
 	}
 }
diff --git a/src/mixins/follow.js b/src/mixins/follow.js
index 2a9a3aa9..43e5e541 100644
--- a/src/mixins/follow.js
+++ b/src/mixins/follow.js
@@ -48,7 +48,7 @@ export default {
 			}).catch((error) => {
 				this.followLoading = false
 				OC.Notification.showTemporary(`Failed to follow user ${this.item.account}`)
-				console.error(`Failed to follow user ${this.item.account}`, error)
+				console.error(`Failed to follow user ${this.item.account}`, error.response.data)
 			})
 
 		},
@@ -63,7 +63,7 @@ export default {
 			}).catch((error) => {
 				this.followLoading = false
 				OC.Notification.showTemporary(`Failed to unfollow user ${this.item.account}`)
-				console.error(`Failed to unfollow user ${this.item.account}`, error)
+				console.error(`Failed to unfollow user ${this.item.account}`, error.response.data)
 			})
 		}
 	}