refactor(pinia): refactor and rename store methods

Signed-off-by: Maksim Sukharev <[email protected]>
nextcloud · Mar 22, 2024 · 358a262 · 358a262
1 parent 786e04e
commit 358a262
Show file tree

Hide file tree

Showing 9 changed files with 139 additions and 104 deletions.
diff --git a/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue b/src/components/BreakoutRoomsEditor/BreakoutRoomsEditor.vue
@@ -156,7 +156,7 @@ export default {
 	methods: {
 		async handleCreateRooms() {
 			try {
-				await this.breakoutRoomsStore.configureBreakoutRoomsAction({
+				await this.breakoutRoomsStore.configureBreakoutRooms({
 					token: this.token,
 					mode: this.mode,
 					amount: this.amount,

diff --git a/src/components/BreakoutRoomsEditor/BreakoutRoomsParticipantsEditor.vue b/src/components/BreakoutRoomsEditor/BreakoutRoomsParticipantsEditor.vue
@@ -323,7 +323,7 @@ export default {
 		},
 
 		createRooms() {
-			this.breakoutRoomsStore.configureBreakoutRoomsAction({
+			this.breakoutRoomsStore.configureBreakoutRooms({
 				token: this.token,
 				mode: 2,
 				amount: this.roomNumber,
@@ -333,7 +333,7 @@ export default {
 		},
 
 		reorganizeAttendees() {
-			this.breakoutRoomsStore.reorganizeAttendeesAction({
+			this.breakoutRoomsStore.reorganizeAttendees({
 				token: this.token,
 				attendeeMap: this.createAttendeeMap(),
 			})
@@ -345,7 +345,7 @@ export default {
 		},
 
 		deleteBreakoutRooms() {
-			this.breakoutRoomsStore.deleteBreakoutRoomsAction({ token: this.token })
+			this.breakoutRoomsStore.deleteBreakoutRooms(this.token)
 		},
 	},
 }

diff --git a/src/components/NewMessage/NewMessage.vue b/src/components/NewMessage/NewMessage.vue
@@ -718,7 +718,7 @@ export default {
 		// Broadcast message to all breakout rooms
 		async broadcastMessage(token, message) {
 			try {
-				await this.breakoutRoomsStore.broadcastMessageToBreakoutRoomsAction({ token, message })
+				await this.breakoutRoomsStore.broadcastMessageToBreakoutRooms({ token, message })
 				this.$emit('sent')
 			} catch {
 				this.$emit('failure')

diff --git a/src/components/RightSidebar/BreakoutRooms/BreakoutRoomItem.vue b/src/components/RightSidebar/BreakoutRooms/BreakoutRoomItem.vue
@@ -224,7 +224,7 @@ export default {
 		},
 
 		dismissRequestAssistance() {
-			this.breakoutRoomsStore.resetRequestAssistanceAction({ token: this.roomToken })
+			this.breakoutRoomsStore.dismissRequestAssistance(this.roomToken)
 		},
 
 		async joinRoom() {
@@ -235,7 +235,7 @@ export default {
 			} else {
 				try {
 					if (this.mainConversation.breakoutRoomMode === CONVERSATION.BREAKOUT_ROOM_MODE.FREE) {
-						await this.breakoutRoomsStore.switchToBreakoutRoomAction({
+						await this.breakoutRoomsStore.switchToBreakoutRoom({
 							token: this.breakoutRoomsStore.getParentRoomToken(this.roomToken),
 							target: this.roomToken,
 						})

diff --git a/src/components/RightSidebar/BreakoutRooms/BreakoutRoomsActions.vue b/src/components/RightSidebar/BreakoutRooms/BreakoutRoomsActions.vue
@@ -238,11 +238,11 @@ export default {
 
 	methods: {
 		startBreakoutRooms() {
-			this.breakoutRoomsStore.startBreakoutRoomsAction(this.mainToken)
+			this.breakoutRoomsStore.startBreakoutRooms(this.mainToken)
 		},
 
 		stopBreakoutRooms() {
-			this.breakoutRoomsStore.stopBreakoutRoomsAction(this.mainToken)
+			this.breakoutRoomsStore.stopBreakoutRooms(this.mainToken)
 		},
 
 		openSendMessageDialog() {

diff --git a/src/components/RightSidebar/BreakoutRooms/BreakoutRoomsTab.vue b/src/components/RightSidebar/BreakoutRooms/BreakoutRoomsTab.vue
@@ -146,7 +146,9 @@ export default {
 
 	mounted() {
 		// Get the breakout room every time the tab is mounted
-		this.getBreakoutRooms()
+		if (this.breakoutRoomsConfigured) {
+			this.breakoutRoomsStore.getBreakoutRooms(this.mainToken)
+		}
 	},
 
 	beforeDestroy() {
@@ -155,19 +157,9 @@ export default {
 	},
 
 	methods: {
-		getBreakoutRooms() {
-			if (this.breakoutRoomsConfigured) {
-				this.breakoutRoomsStore.getBreakoutRoomsAction({
-					token: this.mainToken,
-				})
-			}
-		},
-
 		getParticipants() {
 			if (this.breakoutRoomsConfigured) {
-				this.breakoutRoomsStore.getBreakoutRoomsParticipantsAction({
-					token: this.mainToken,
-				})
+				this.breakoutRoomsStore.fetchBreakoutRoomsParticipants(this.mainToken)
 			}
 		},
 	},

diff --git a/src/components/TopBar/TopBarMenu.vue b/src/components/TopBar/TopBarMenu.vue
@@ -478,9 +478,9 @@ export default {
 				}
 				const hasAssistanceRequested = this.conversation.breakoutRoomStatus === CONVERSATION.BREAKOUT_ROOM_STATUS.STATUS_ASSISTANCE_REQUESTED
 				if (newState && !hasAssistanceRequested) {
-					this.breakoutRoomsStore.requestAssistanceAction({ token: this.token })
+					this.breakoutRoomsStore.requestAssistance(this.token)
 				} else if (!newState && hasAssistanceRequested) {
-					this.breakoutRoomsStore.resetRequestAssistanceAction({ token: this.token })
+					this.breakoutRoomsStore.dismissRequestAssistance(this.token)
 				}
 			}
 		},

diff --git a/src/store/conversationsStore.js b/src/store/conversationsStore.js
@@ -394,10 +394,7 @@ const actions = {
 			}
 
 			if (newConversation.objectType === CONVERSATION.OBJECT_TYPE.BREAKOUT_ROOM) {
-				breakoutRoomsStore.addBreakoutRoom({
-					parentRoomToken: newConversation.objectId,
-					breakoutRoom: newConversation,
-				})
+				breakoutRoomsStore.addBreakoutRoom(newConversation.objectId, newConversation)
 			}
 		}