From 22421e893b866d8815607b0ea8e364e7eaaa0008 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Wed, 11 Mar 2026 21:52:50 +0100
Subject: [PATCH 1/3] fix routing to 404

Signed-off-by: dartcafe <github@dartcafe.de>
---
 src/components/Public/PublicRegisterModal.vue | 11 +++++++----
 src/views/Vote.vue                            |  5 +++--
 2 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/components/Public/PublicRegisterModal.vue b/src/components/Public/PublicRegisterModal.vue
index 07d4ed31a7..6e87ea3648 100644
--- a/src/components/Public/PublicRegisterModal.vue
+++ b/src/components/Public/PublicRegisterModal.vue
@@ -147,7 +147,8 @@ const emailAddressHint = computed(() => {
 
 const validateEmailAddress = debounce(async function (): Promise<void> {
 	if (emailAddress.value.length < 1) {
-		checkStatus.value.email = sessionStore.share.publicPollEmail === 'mandatory' ? 'missing': 'empty'
+		checkStatus.value.email =
+			sessionStore.share.publicPollEmail === 'mandatory' ? 'missing' : 'empty'
 		return
 	}
 
@@ -181,7 +182,7 @@ function routeToPersonalShare(token: string): void {
 		// if share was not a public share, but a personal share
 		// (i.e. email shares allow to change personal data by fist entering of the poll),
 		// just refresh the session and load the poll
-		sessionStore.load()
+		sessionStore.loadSession()
 		pollStore.load()
 	} else {
 		// in case of a public share, redirect to the generated share
@@ -246,8 +247,10 @@ onMounted(() => {
 	if (route.name === 'publicVote') {
 		// TODO: remove displayName at controller side
 		// this ist just a quick fix
-		sessionStore.currentUser.displayName = route.query.displayName as string || ''
-		sessionStore.currentUser.emailAddress = route.query.emailAddress as string || ''
+		sessionStore.currentUser.displayName =
+			(route.query.displayName as string) || ''
+		sessionStore.currentUser.emailAddress =
+			(route.query.emailAddress as string) || ''
 	}
 
 	// pre-fill input field with existing data
diff --git a/src/views/Vote.vue b/src/views/Vote.vue
index d4aeea0b88..12d973d3b1 100644
--- a/src/views/Vote.vue
+++ b/src/views/Vote.vue
@@ -5,7 +5,7 @@
 
 <script setup lang="ts">
 import { computed, nextTick, onMounted, onUnmounted, ref, watch } from 'vue'
-import { onBeforeRouteLeave, onBeforeRouteUpdate } from 'vue-router'
+import { onBeforeRouteLeave, onBeforeRouteUpdate, useRouter } from 'vue-router'
 
 import { showError } from '@nextcloud/dialogs'
 import { emit } from '@nextcloud/event-bus'
@@ -42,6 +42,7 @@ import type { CollapsibleProps } from '../components/Base/modules/Collapsible.vu
 import { Event } from '../Types'
 import { loadContext } from '../composables/context'
 
+const router = useRouter()
 const pollStore = usePollStore()
 const optionsStore = useOptionsStore()
 const preferencesStore = usePreferencesStore()
@@ -142,10 +143,10 @@ async function loadPoll(isChanging: boolean = false) {
 	try {
 		await pollStore.load(isChanging)
 	} catch {
-		showError(t('polls', 'Error loading poll'))
 		votesStore.$reset()
 		optionsStore.$reset()
 		subscriptionStore.$reset()
+		router.replace({ name: 'notfound' })
 		return
 	}
 

From 3195c47a038f4d5be89056eb31d33e963c5b5183 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Wed, 11 Mar 2026 23:07:14 +0100
Subject: [PATCH 2/3] centralize store loading to router

Signed-off-by: dartcafe <github@dartcafe.de>
---
 src/App.vue                 | 11 ++++++-
 src/router.ts               | 58 +++++++++++++++++++++++++++++++++++++
 src/stores/session.ts       |  1 +
 src/stores/session.types.ts |  4 ++-
 src/views/PollList.vue      |  4 +--
 src/views/Vote.vue          | 54 +---------------------------------
 6 files changed, 74 insertions(+), 58 deletions(-)

diff --git a/src/App.vue b/src/App.vue
index 081e1245a4..9146dd5b26 100644
--- a/src/App.vue
+++ b/src/App.vue
@@ -14,11 +14,13 @@ import {
 } from 'vue'
 import debounce from 'lodash/debounce'
 import { subscribe, unsubscribe } from '@nextcloud/event-bus'
+import { t } from '@nextcloud/l10n'
 
 import NcContent from '@nextcloud/vue/components/NcContent'
 
 // import UserSettingsDlg from './components/Settings/UserSettingsDlg.vue'
 
+import LoadingOverlay from './components/Base/modules/LoadingOverlay.vue'
 import { usePollWatcher } from './composables/usePollWatcher'
 
 import { useSessionStore } from './stores/session'
@@ -86,7 +88,10 @@ function transitionsOff(delay: number) {
 	}
 }
 
-const debouncedNotify = debounce(async function (payload: { store: string; message: string }) {
+const debouncedNotify = debounce(async function (payload: {
+	store: string
+	message: string
+}) {
 	if (payload.store === 'poll') {
 		showSuccess(payload.message)
 	}
@@ -123,6 +128,10 @@ onUnmounted(() => {
 		<router-view />
 		<router-view v-if="useSidebar" name="sidebar" />
 		<UserSettingsDlg />
+		<LoadingOverlay
+			:show="sessionStore.navigationStatus === 'loading'"
+			:name="t('polls', 'Loading…')"
+			teleport-to="#content" />
 	</NcContent>
 </template>
 
diff --git a/src/router.ts b/src/router.ts
index fe9d389012..5a4ecee6dd 100644
--- a/src/router.ts
+++ b/src/router.ts
@@ -14,10 +14,17 @@ import { generateUrl } from '@nextcloud/router'
 import { getCookieValue, setCookie } from './helpers/modules/cookieHelper'
 import { Logger } from './helpers/modules/logger'
 import { loadContext } from './composables/context'
+import { emit } from '@nextcloud/event-bus'
+import { Event } from './Types'
 
 import Navigation from './views/Navigation.vue'
 
 import { useSessionStore } from './stores/session'
+import { usePollStore } from './stores/poll'
+import { usePollsStore } from './stores/polls'
+import { useVotesStore } from './stores/votes'
+import { useOptionsStore } from './stores/options'
+import { useSubscriptionStore } from './stores/subscription'
 
 async function validateToken(to: RouteLocationNormalized) {
 	const sessionStore = useSessionStore()
@@ -199,6 +206,20 @@ const router = createRouter({
 router.beforeEach(
 	async (to: RouteLocationNormalized, from: RouteLocationNormalized) => {
 		const sessionStore = useSessionStore()
+		const pollStore = usePollStore()
+		const votesStore = useVotesStore()
+		const optionsStore = useOptionsStore()
+		const subscriptionStore = useSubscriptionStore()
+
+		sessionStore.navigationStatus = 'loading'
+		emit(Event.TransitionsOff, 0)
+
+		if (from.meta.votePage) {
+			pollStore.$reset()
+			votesStore.$reset()
+			optionsStore.$reset()
+			subscriptionStore.$reset()
+		}
 
 		// if the previous and the requested routes have the same name and
 		// the watcher is active, we can do a cheap loading
@@ -227,4 +248,41 @@ router.beforeEach(
 	},
 )
 
+router.beforeResolve(async (to: RouteLocationNormalized) => {
+	const sessionStore = useSessionStore()
+	const watcherActive =
+		sessionStore.watcher.mode !== 'noPolling'
+		&& sessionStore.watcher.status !== 'stopped'
+
+	if (to.meta.listPage) {
+		const pollsStore = usePollsStore()
+		await pollsStore.load(!watcherActive)
+	}
+
+	if (to.meta.votePage) {
+		const pollStore = usePollStore()
+		const votesStore = useVotesStore()
+		const optionsStore = useOptionsStore()
+		const subscriptionStore = useSubscriptionStore()
+
+		try {
+			await pollStore.load()
+		} catch {
+			return { name: 'notfound' }
+		}
+
+		await Promise.allSettled([
+			votesStore.load(),
+			optionsStore.load(),
+			subscriptionStore.load(),
+		])
+	}
+})
+
+router.afterEach(() => {
+	const sessionStore = useSessionStore()
+	sessionStore.navigationStatus = 'idle'
+	emit(Event.TransitionsOn, null)
+})
+
 export { router }
diff --git a/src/stores/session.ts b/src/stores/session.ts
index a329f5ef1c..9ae684d80c 100644
--- a/src/stores/session.ts
+++ b/src/stores/session.ts
@@ -114,6 +114,7 @@ export const useSessionStore = defineStore('session', {
 		token: null,
 		currentUser: defaultUser,
 		share: defaultShare,
+		navigationStatus: 'idle',
 	}),
 
 	getters: {
diff --git a/src/stores/session.types.ts b/src/stores/session.types.ts
index e9b8ae7f84..3ca81985cb 100644
--- a/src/stores/session.types.ts
+++ b/src/stores/session.types.ts
@@ -75,4 +75,6 @@ export type Session = {
 	watcher: Watcher
 }
 
-export type SessionStore = Session
+export type SessionStore = Session & {
+	navigationStatus: 'idle' | 'loading'
+}
diff --git a/src/views/PollList.vue b/src/views/PollList.vue
index 01a5473031..905af9bfd2 100644
--- a/src/views/PollList.vue
+++ b/src/views/PollList.vue
@@ -122,9 +122,7 @@ async function loadMore() {
 	}
 }
 
-onMounted(() => {
-	pollsStore.load(false)
-})
+
 </script>
 
 <template>
diff --git a/src/views/Vote.vue b/src/views/Vote.vue
index 12d973d3b1..796e2646e9 100644
--- a/src/views/Vote.vue
+++ b/src/views/Vote.vue
@@ -4,8 +4,7 @@
 -->
 
 <script setup lang="ts">
-import { computed, nextTick, onMounted, onUnmounted, ref, watch } from 'vue'
-import { onBeforeRouteLeave, onBeforeRouteUpdate, useRouter } from 'vue-router'
+import { computed, nextTick, onMounted, ref, watch } from 'vue'
 
 import { showError } from '@nextcloud/dialogs'
 import { emit } from '@nextcloud/event-bus'
@@ -36,18 +35,13 @@ import { usePollStore } from '../stores/poll'
 import { useOptionsStore } from '../stores/options'
 import { usePreferencesStore } from '../stores/preferences'
 import { useVotesStore } from '../stores/votes'
-import { useSubscriptionStore } from '../stores/subscription'
-
 import type { CollapsibleProps } from '../components/Base/modules/Collapsible.vue'
 import { Event } from '../Types'
-import { loadContext } from '../composables/context'
 
-const router = useRouter()
 const pollStore = usePollStore()
 const optionsStore = useOptionsStore()
 const preferencesStore = usePreferencesStore()
 const votesStore = useVotesStore()
-const subscriptionStore = useSubscriptionStore()
 
 const voteMainId = 'vote-view'
 const topObserverVisible = ref(false)
@@ -139,56 +133,10 @@ const showBottomObserver = computed(
 		&& pollStore.viewMode === 'table-view',
 )
 
-async function loadPoll(isChanging: boolean = false) {
-	try {
-		await pollStore.load(isChanging)
-	} catch {
-		votesStore.$reset()
-		optionsStore.$reset()
-		subscriptionStore.$reset()
-		router.replace({ name: 'notfound' })
-		return
-	}
-
-	const results = await Promise.allSettled([
-		votesStore.load(),
-		optionsStore.load(),
-		subscriptionStore.load(),
-	])
-
-	if (results.some((result) => result.status === 'rejected')) {
-		showError(t('polls', 'Error loading poll data'))
-	}
-}
-
-async function resetPoll() {
-	pollStore.$reset()
-	votesStore.$reset()
-	optionsStore.$reset()
-	subscriptionStore.$reset()
-}
-
-onBeforeRouteUpdate(async (to, from) => {
-	if (to.name === 'publicVote' && to.params.token !== from.params.token) {
-		loadContext(to)
-		// Same poll, no need to reload
-	}
-	loadPoll(true)
-})
-
-onBeforeRouteLeave(() => {
-	resetPoll()
-})
-
 onMounted(() => {
-	loadPoll(true)
 	emit(Event.TransitionsOff, 500)
 })
 
-onUnmounted(() => {
-	resetPoll()
-})
-
 watch(
 	[
 		() => pollStore.permissions.seeUsernames,

From 679eeb21f32ade3fe0d239aba4df10b2405d3a13 Mon Sep 17 00:00:00 2001
From: dartcafe <github@dartcafe.de>
Date: Wed, 11 Mar 2026 23:10:28 +0100
Subject: [PATCH 3/3] lint-fix

Signed-off-by: dartcafe <github@dartcafe.de>
---
 src/views/PollList.vue | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/views/PollList.vue b/src/views/PollList.vue
index 905af9bfd2..1718f88aa0 100644
--- a/src/views/PollList.vue
+++ b/src/views/PollList.vue
@@ -4,7 +4,7 @@
 -->
 
 <script setup lang="ts">
-import { computed, onMounted } from 'vue'
+import { computed } from 'vue'
 import { useRouter, useRoute } from 'vue-router'
 
 import { showError } from '@nextcloud/dialogs'
@@ -121,8 +121,6 @@ async function loadMore() {
 		showError(t('polls', 'Error loading more polls'))
 	}
 }
-
-
 </script>
 
 <template>