bugfix: reset stores when user logs out

bugfix: ignore pending request that do not elong to current game

client/src/composables/engine/fetchUpdate.ts

@@ -21,8 +21,8 @@ type EngineResponse = {
 
 const { $t } = useI18n({
     'connection-broken': { en: 'connection to server broken.', de: 'Verbindung zum Server ist unterbrochen.' },
-    'retrying': { en: 'retrying', de: 'verbinde erneut'},
-  })
+    'retrying': { en: 'retrying', de: 'verbinde erneut' },
+})
 
 export async function fetchUpdate(this: EngineContext) {
     if (this.shouldStop || !this.isActive) {
@@ -44,6 +44,12 @@ export async function fetchUpdate(this: EngineContext) {
             throw Error('unexpected response from /api/sync')
         }
 
+        if (response.game.id != userInfoStore.gameId) {
+            // happens when user changes game and an old request is still pending
+            console.warn('response gameId does not match current gameId')
+            return
+        }
+
         this.version = parseInt(response.version)
         this.isConnected.value = true
         this.retry.value = 0

client/src/composables/useAuth.ts

@@ -1,5 +1,9 @@
-import { useUserinfoStore, Userinfo } from "@/stores/UserinfoStore"
-import useI18n from "./useI18n"
+import { useUserinfoStore, Userinfo } from '@/stores/UserinfoStore'
+import { useEngineStore } from '@/stores/EngineStore'
+import { useGameinfoStore } from '@/stores/GameinfoStore'
+import { usePlayersStore } from '@/stores/PlayersStore'
+import { useRoundStore } from '@/stores/RoundStore'
+import useI18n from './useI18n'
 import { $fetch } from 'ohmyfetch'
 
 export type AllowRole = '' | 'player' | 'gamemaster' | 'admin'
@@ -49,6 +53,11 @@ export default (): useAuth => {
 
     logout: async (): Promise<void> => {
       await $fetch('/api/logout')
+      useEngineStore().reset()
+      useGameinfoStore().reset()
+      usePlayersStore().reset()
+      useUserinfoStore().reset()
+      useRoundStore().reset()
     },
   }
 }

client/src/stores/EngineStore.ts

@@ -10,5 +10,8 @@ export const useEngineStore = defineStore('EngineStore', {
     setJson(json: any): void {
       this.json = json
     },
+    reset(): void {
+      this.json = {}
+    },
   },
 })

client/src/stores/GameinfoStore.ts

@@ -33,5 +33,13 @@ export const useGameinfoStore = defineStore('GameinfoStore', {
     setGameinfo(gameInfo: Gameinfo): void {
       this.gameInfo = gameInfo
     },
+    reset(): void {
+      this.gameInfo = {
+        id: '',
+        name: '',
+        state: '',
+        phase: '',
+      } 
+    },
   },
 })

client/src/stores/PlayersStore.ts

@@ -12,5 +12,8 @@ export const usePlayersStore = defineStore('PlayersStore', {
       players = players || []
       this.players.splice(0, this.players.length, ...players)
     },
+    reset(): void {
+      this.players.splice(0, 0)
+    },
   },
 })

client/src/stores/RoundStore.ts

@@ -24,5 +24,16 @@ export const useRoundStore = defineStore('RoundStore', {
       this.round.revelation.votes = round.revelation?.votes || {} as RevelationVotes
       this.round.revelation.sources = round.revelation?.sources || {} as RevelationSources
     },
+    reset(): void {
+      this.round = {
+        quote: '',
+        sources: [],
+        selections: {},
+        revelation: {
+          votes: {},
+          sources: {},
+        },
+      }
+    },
   },
 })

client/src/stores/UserinfoStore.ts

@@ -34,5 +34,15 @@ export const useUserinfoStore = defineStore('UserinfoStore', {
     setUserInfo(userInfo: Userinfo): void {
      this.userInfo = userInfo 
     },
+    reset(): void {
+      this.userInfo = {
+        id: '',
+        name: '',
+        role: '',
+        game: '',
+        lang: 'en',
+        isCameo: '',
+      }
+    },
   },
 })