From 001aaa4dcdd78c47b7980840c2beb82f834d0184 Mon Sep 17 00:00:00 2001
From: Jakob <jakob.roehrl@web.de>
Date: Tue, 27 Aug 2019 11:24:37 +0200
Subject: [PATCH] format entries, do not load on end, loading by click on
 another card/board

Signed-off-by: Jakob <jakob.roehrl@web.de>
---
 src/components/board/TimelineTabSidebard.vue |  9 ++++---
 src/components/card/CardSidebar.vue          | 26 +++++++++++++++++---
 src/store/main.js                            | 20 ++++++++++++---
 3 files changed, 43 insertions(+), 12 deletions(-)

diff --git a/src/components/board/TimelineTabSidebard.vue b/src/components/board/TimelineTabSidebard.vue
index f5daf4d2d..1ce6a4420 100644
--- a/src/components/board/TimelineTabSidebard.vue
+++ b/src/components/board/TimelineTabSidebard.vue
@@ -7,12 +7,12 @@
 			<span v-html="parseMessage(entry)" />
 			<span> {{ getTime(entry.datetime) }} </span>
 		</div>
-		<button @click="loadMore">Load More</button>
+		<button v-if="activityLoadMore" @click="loadMore">Load More</button>
 	</div>
 </template>
 
 <script>
-import { mapGetters } from 'vuex'
+import { mapState } from 'vuex'
 
 export default {
 	name: 'TimelineTabSidebard',
@@ -36,8 +36,9 @@ export default {
 		}
 	},
 	computed: {
-		...mapGetters({
-			boardActivity: 'activity'
+		...mapState({
+			boardActivity: 'activity',
+			activityLoadMore: 'activityLoadMore'
 		})
 	},
 	created() {
diff --git a/src/components/card/CardSidebar.vue b/src/components/card/CardSidebar.vue
index a5199fecc..23e30f8e4 100644
--- a/src/components/card/CardSidebar.vue
+++ b/src/components/card/CardSidebar.vue
@@ -81,10 +81,10 @@
 			<div v-if="isLoading" class="icon icon-loading" />
 			<div v-for="entry in cardActivity" v-else :key="entry.activity_id">
 				<img :src="entry.icon">
-				{{ entry.subject }}
+				<span v-html="parseMessage(entry)" />
 				{{ getTime(entry.datetime) }}
 			</div>
-			<button @click="loadMore">Load More</button>
+			<button v-if="activityLoadMore" @click="loadMore">Load More</button>
 		</AppSidebarTab>
 	</app-sidebar>
 </template>
@@ -142,7 +142,8 @@ export default {
 		...mapState({
 			currentBoard: state => state.currentBoard,
 			assignableUsers: state => state.assignableUsers,
-			cardActivity: 'activity'
+			cardActivity: 'activity',
+			activityLoadMore: 'activityLoadMore'
 		}),
 		currentCard() {
 			return this.$store.getters.cardById(this.id)
@@ -175,8 +176,16 @@ export default {
 			handler() {
 				this.copiedCard = JSON.parse(JSON.stringify(this.currentCard))
 				this.allLabels = this.currentCard.labels
-				this.assignedUsers = this.currentCard.assignedUsers.map((item) => item.participant)
+
+				if (this.currentCard.assignedUsers.length > 0) {
+					this.assignedUsers = this.currentCard.assignedUsers.map((item) => item.participant)
+				}
+
+				this.desc = this.currentCard.description
 				this.updateRelativeTimestamps()
+
+				this.params.object_id = this.id
+				this.loadCardActivity()
 			}
 		},
 
@@ -261,6 +270,15 @@ export default {
 			this.params.since = aId
 			this.loadCardActivity()
 		},
+		parseMessage(activity) {
+			let subject = activity.subject_rich[0]
+			let parameters = activity.subject_rich[1]
+			if (parameters.after && typeof parameters.after.id === 'string' && parameters.after.id.startsWith('dt:')) {
+				let dateTime = parameters.after.id.substr(3)
+				parameters.after.name = window.moment(dateTime).format('L LTS')
+			}
+			return OCA.Activity.RichObjectStringParser.parseMessage(subject, parameters)
+		},
 		clickAddNewAttachmment() {
 
 		}
diff --git a/src/store/main.js b/src/store/main.js
index 2da17cd07..93aaba30b 100644
--- a/src/store/main.js
+++ b/src/store/main.js
@@ -55,7 +55,8 @@ export default new Vuex.Store({
 		sharees: [],
 		assignableUsers: [],
 		boardFilter: BOARD_FILTERS.ALL,
-		activity: []
+		activity: [],
+		activityLoadMore: true
 	},
 	getters: {
 		boards: state => {
@@ -64,9 +65,6 @@ export default new Vuex.Store({
 		sharees: state => {
 			return state.sharees
 		},
-		activity: state => {
-			return state.activity
-		},
 		noneArchivedBoards: state => {
 			return state.boards.filter(board => {
 				return board.archived === false && !board.deletedAt
@@ -152,6 +150,12 @@ export default new Vuex.Store({
 			})
 
 		},
+		clearActivity(state) {
+			state.activity = []
+		},
+		setActivityLoadMore(state, value) {
+			state.activityLoadMore = value
+		},
 		setAssignableUsers(state, users) {
 			state.assignableUsers = users
 		},
@@ -288,12 +292,20 @@ export default new Vuex.Store({
 			params.append('object_type', obj.object_type)
 			params.append('object_id', obj.object_id)
 
+			if (obj.since === 0) {
+				commit('clearActivity')
+			}
+
 			let keyword = 'deck'
 			if (obj.type === 'filter') {
 				keyword = 'filter'
 			}
 			axios.get(OC.linkToOCS('apps/activity/api/v2/activity') + keyword, { params }).then((response) => {
 				commit('setActivity', response.data.ocs.data)
+				commit('setActivityLoadMore', true)
+				if (response.data.ocs.meta.statuscode === 304) {
+					commit('setActivityLoadMore', false)
+				}
 			})
 		},