From 43071b45a96d2f298d1cdc49d076fd54ddfbfc40 Mon Sep 17 00:00:00 2001
From: "Torsten Schulz (local)" <tsschulz@gmx.net>
Date: Tue, 21 Oct 2025 14:19:30 +0200
Subject: [PATCH] Replace composable with Pinia store for persistent auth state

---
 components/Footer.vue     | 12 ++++----
 components/Navigation.vue |  8 ++++--
 composables/useAuth.js    | 48 --------------------------------
 nuxt.config.js            |  2 +-
 package-lock.json         | 47 +++++++++++++++++++++++++++++++
 package.json              |  2 ++
 pages/login.vue           | 13 ++-------
 server/data/sessions.json | 14 ++++++++++
 server/data/users.json    |  2 +-
 stores/auth.js            | 58 +++++++++++++++++++++++++++++++++++++++
 10 files changed, 137 insertions(+), 69 deletions(-)
 delete mode 100644 composables/useAuth.js
 create mode 100644 stores/auth.js

diff --git a/components/Footer.vue b/components/Footer.vue
index 56238c4..a9e3576 100644
--- a/components/Footer.vue
+++ b/components/Footer.vue
@@ -93,30 +93,32 @@
 </template>
 
 <script setup>
-import { ref, onMounted, onUnmounted } from 'vue'
+import { ref, computed, onMounted, onUnmounted } from 'vue'
 import { useRouter } from 'vue-router'
 import { User, ChevronUp } from 'lucide-vue-next'
 
 const router = useRouter()
+const authStore = useAuthStore()
 const currentYear = new Date().getFullYear()
 const isMemberMenuOpen = ref(false)
 
-// Use global auth state
-const { isLoggedIn, isAdmin, logout: authLogout, checkAuth } = useAuth()
+// Reactive auth state from store
+const isLoggedIn = computed(() => authStore.isLoggedIn)
+const isAdmin = computed(() => authStore.isAdmin)
 
 const toggleMemberMenu = () => {
   isMemberMenuOpen.value = !isMemberMenuOpen.value
 }
 
 const handleLogout = async () => {
-  await authLogout()
+  await authStore.logout()
   isMemberMenuOpen.value = false
   router.push('/')
 }
 
 // Check auth status on mount
 onMounted(() => {
-  checkAuth()
+  authStore.checkAuth()
 })
 
 // Close menu when clicking outside
diff --git a/components/Navigation.vue b/components/Navigation.vue
index d091369..6d892ae 100644
--- a/components/Navigation.vue
+++ b/components/Navigation.vue
@@ -396,13 +396,15 @@ import { useRoute } from 'vue-router'
 import { Menu, X, ChevronDown } from 'lucide-vue-next'
 
 const route = useRoute()
+const authStore = useAuthStore()
 const isMobileMenuOpen = ref(false)
 const mobileSubmenu = ref(null)
 const mannschaften = ref([])
 const hasGalleryImages = ref(false)
 
-// Use global auth state
-const { isLoggedIn, userRole, isAdmin, checkAuth } = useAuth()
+// Reactive auth state from store
+const isLoggedIn = computed(() => authStore.isLoggedIn)
+const isAdmin = computed(() => authStore.isAdmin)
 
        // Automatisches Setzen des Submenus basierend auf der Route
        const currentSubmenu = computed(() => {
@@ -486,7 +488,7 @@ const checkGalleryImages = async () => {
 onMounted(() => {
   loadMannschaften()
   checkGalleryImages()
-  checkAuth()
+  authStore.checkAuth()
 })
 
        const toggleSubmenu = (menu) => {
diff --git a/composables/useAuth.js b/composables/useAuth.js
deleted file mode 100644
index 0d01719..0000000
--- a/composables/useAuth.js
+++ /dev/null
@@ -1,48 +0,0 @@
-import { ref, computed } from 'vue'
-
-// Global state (shared across all components)
-const isLoggedIn = ref(false)
-const user = ref(null)
-const userRole = ref(null)
-
-export const useAuth = () => {
-  const isAdmin = computed(() => {
-    return userRole.value === 'admin' || userRole.value === 'vorstand'
-  })
-
-  const checkAuth = async () => {
-    try {
-      const response = await $fetch('/api/auth/status')
-      isLoggedIn.value = response.isLoggedIn
-      user.value = response.user
-      userRole.value = response.role
-      return response
-    } catch (error) {
-      isLoggedIn.value = false
-      user.value = null
-      userRole.value = null
-      return { isLoggedIn: false }
-    }
-  }
-
-  const logout = async () => {
-    try {
-      await $fetch('/api/auth/logout', { method: 'POST' })
-      isLoggedIn.value = false
-      user.value = null
-      userRole.value = null
-    } catch (error) {
-      console.error('Logout fehlgeschlagen:', error)
-    }
-  }
-
-  return {
-    isLoggedIn,
-    user,
-    userRole,
-    isAdmin,
-    checkAuth,
-    logout
-  }
-}
-
diff --git a/nuxt.config.js b/nuxt.config.js
index 26023c8..0d569c4 100644
--- a/nuxt.config.js
+++ b/nuxt.config.js
@@ -2,7 +2,7 @@
 export default defineNuxtConfig({
   devtools: { enabled: true },
   
-  modules: ['@nuxtjs/tailwindcss'],
+  modules: ['@nuxtjs/tailwindcss', '@pinia/nuxt'],
   
   nitro: {
     preset: 'node-server',
diff --git a/package-lock.json b/package-lock.json
index 40b0875..83987f4 100644
--- a/package-lock.json
+++ b/package-lock.json
@@ -9,10 +9,12 @@
       "version": "1.0.0",
       "hasInstallScript": true,
       "dependencies": {
+        "@pinia/nuxt": "^0.11.2",
         "bcryptjs": "^2.4.3",
         "jsonwebtoken": "^9.0.2",
         "nodemailer": "^7.0.9",
         "nuxt": "^4.1.3",
+        "pinia": "^3.0.3",
         "vue": "^3.5.22"
       },
       "devDependencies": {
@@ -2530,6 +2532,21 @@
         "url": "https://opencollective.com/parcel"
       }
     },
+    "node_modules/@pinia/nuxt": {
+      "version": "0.11.2",
+      "resolved": "https://registry.npmjs.org/@pinia/nuxt/-/nuxt-0.11.2.tgz",
+      "integrity": "sha512-CgvSWpbktxxWBV7ModhAcsExsQZqpPq6vMYEe9DexmmY6959ev8ukL4iFhr/qov2Nb9cQAWd7niFDnaWkN+FHg==",
+      "license": "MIT",
+      "dependencies": {
+        "@nuxt/kit": "^3.9.0"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/posva"
+      },
+      "peerDependencies": {
+        "pinia": "^3.0.3"
+      }
+    },
     "node_modules/@pkgjs/parseargs": {
       "version": "0.11.0",
       "resolved": "https://registry.npmjs.org/@pkgjs/parseargs/-/parseargs-0.11.0.tgz",
@@ -8123,6 +8140,36 @@
         "node": ">=0.10.0"
       }
     },
+    "node_modules/pinia": {
+      "version": "3.0.3",
+      "resolved": "https://registry.npmjs.org/pinia/-/pinia-3.0.3.tgz",
+      "integrity": "sha512-ttXO/InUULUXkMHpTdp9Fj4hLpD/2AoJdmAbAeW2yu1iy1k+pkFekQXw5VpC0/5p51IOR/jDaDRfRWRnMMsGOA==",
+      "license": "MIT",
+      "dependencies": {
+        "@vue/devtools-api": "^7.7.2"
+      },
+      "funding": {
+        "url": "https://github.com/sponsors/posva"
+      },
+      "peerDependencies": {
+        "typescript": ">=4.4.4",
+        "vue": "^2.7.0 || ^3.5.11"
+      },
+      "peerDependenciesMeta": {
+        "typescript": {
+          "optional": true
+        }
+      }
+    },
+    "node_modules/pinia/node_modules/@vue/devtools-api": {
+      "version": "7.7.7",
+      "resolved": "https://registry.npmjs.org/@vue/devtools-api/-/devtools-api-7.7.7.tgz",
+      "integrity": "sha512-lwOnNBH2e7x1fIIbVT7yF5D+YWhqELm55/4ZKf45R9T8r9dE2AIOy8HKjfqzGsoTHFbWbr337O4E0A0QADnjBg==",
+      "license": "MIT",
+      "dependencies": {
+        "@vue/devtools-kit": "^7.7.7"
+      }
+    },
     "node_modules/pirates": {
       "version": "4.0.7",
       "resolved": "https://registry.npmjs.org/pirates/-/pirates-4.0.7.tgz",
diff --git a/package.json b/package.json
index 5e7435f..2dbda58 100644
--- a/package.json
+++ b/package.json
@@ -13,10 +13,12 @@
     "postinstall": "nuxt prepare"
   },
   "dependencies": {
+    "@pinia/nuxt": "^0.11.2",
     "bcryptjs": "^2.4.3",
     "jsonwebtoken": "^9.0.2",
     "nodemailer": "^7.0.9",
     "nuxt": "^4.1.3",
+    "pinia": "^3.0.3",
     "vue": "^3.5.22"
   },
   "devDependencies": {
diff --git a/pages/login.vue b/pages/login.vue
index 4ac843f..db5a8de 100644
--- a/pages/login.vue
+++ b/pages/login.vue
@@ -101,7 +101,7 @@ import { useRouter } from 'vue-router'
 import { AlertCircle, Check, Loader2, Lock } from 'lucide-vue-next'
 
 const router = useRouter()
-const { checkAuth } = useAuth()
+const authStore = useAuthStore()
 
 const formData = ref({
   email: '',
@@ -118,20 +118,11 @@ const handleLogin = async () => {
   successMessage.value = ''
 
   try {
-    const response = await $fetch('/api/auth/login', {
-      method: 'POST',
-      body: {
-        email: formData.value.email,
-        password: formData.value.password
-      }
-    })
+    const response = await authStore.login(formData.value.email, formData.value.password)
 
     if (response.success) {
       successMessage.value = 'Anmeldung erfolgreich! Sie werden weitergeleitet...'
       
-      // Update global auth state
-      await checkAuth()
-      
       // Redirect based on role
       setTimeout(() => {
         if (response.user.role === 'admin' || response.user.role === 'vorstand') {
diff --git a/server/data/sessions.json b/server/data/sessions.json
index 98d6d68..90a8433 100644
--- a/server/data/sessions.json
+++ b/server/data/sessions.json
@@ -40,5 +40,19 @@
     "token": "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6IjEiLCJlbWFpbCI6ImFkbWluQGhhcmhlaW1lcnRjLmRlIiwicm9sZSI6ImFkbWluIiwiaWF0IjoxNzYxMDQ4NTgwLCJleHAiOjE3NjE2NTMzODB9._ZEOavin4scWHMI9ofk3MBNdx98K7Q_JossQYxUX4rQ",
     "createdAt": "2025-10-21T12:09:40.103Z",
     "expiresAt": "2025-10-28T12:09:40.103Z"
+  },
+  {
+    "id": "1761049028598",
+    "userId": "1",
+    "token": "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6IjEiLCJlbWFpbCI6ImFkbWluQGhhcmhlaW1lcnRjLmRlIiwicm9sZSI6ImFkbWluIiwiaWF0IjoxNzYxMDQ5MDI4LCJleHAiOjE3NjE2NTM4Mjh9.FsC7CJpxVYVJ824vEPcZol4lKsCKU2I7-L644yeZuMw",
+    "createdAt": "2025-10-21T12:17:08.598Z",
+    "expiresAt": "2025-10-28T12:17:08.598Z"
+  },
+  {
+    "id": "1761049044973",
+    "userId": "1",
+    "token": "eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpZCI6IjEiLCJlbWFpbCI6ImFkbWluQGhhcmhlaW1lcnRjLmRlIiwicm9sZSI6ImFkbWluIiwiaWF0IjoxNzYxMDQ5MDQ0LCJleHAiOjE3NjE2NTM4NDR9.fhaWO80YN8qKy9zBCK5nauGWBT29hmr3KxAK_Vlab4E",
+    "createdAt": "2025-10-21T12:17:24.973Z",
+    "expiresAt": "2025-10-28T12:17:24.973Z"
   }
 ]
\ No newline at end of file
diff --git a/server/data/users.json b/server/data/users.json
index 7ad4b01..cd24eee 100644
--- a/server/data/users.json
+++ b/server/data/users.json
@@ -8,6 +8,6 @@
     "phone": "",
     "active": true,
     "created": "2025-10-21T00:00:00.000Z",
-    "lastLogin": "2025-10-21T12:09:40.103Z"
+    "lastLogin": "2025-10-21T12:17:24.973Z"
   }
 ]
\ No newline at end of file
diff --git a/stores/auth.js b/stores/auth.js
new file mode 100644
index 0000000..6fcb78e
--- /dev/null
+++ b/stores/auth.js
@@ -0,0 +1,58 @@
+import { defineStore } from 'pinia'
+
+export const useAuthStore = defineStore('auth', {
+  state: () => ({
+    isLoggedIn: false,
+    user: null,
+    role: null
+  }),
+
+  getters: {
+    isAdmin: (state) => {
+      return state.role === 'admin' || state.role === 'vorstand'
+    }
+  },
+
+  actions: {
+    async checkAuth() {
+      try {
+        const response = await $fetch('/api/auth/status')
+        this.isLoggedIn = response.isLoggedIn
+        this.user = response.user
+        this.role = response.role
+        return response
+      } catch (error) {
+        this.isLoggedIn = false
+        this.user = null
+        this.role = null
+        return { isLoggedIn: false }
+      }
+    },
+
+    async login(email, password) {
+      const response = await $fetch('/api/auth/login', {
+        method: 'POST',
+        body: { email, password }
+      })
+      
+      if (response.success) {
+        await this.checkAuth()
+      }
+      
+      return response
+    },
+
+    async logout() {
+      try {
+        await $fetch('/api/auth/logout', { method: 'POST' })
+        this.isLoggedIn = false
+        this.user = null
+        this.role = null
+      } catch (error) {
+        console.error('Logout fehlgeschlagen:', error)
+        throw error
+      }
+    }
+  }
+})
+