feat: use store getters to check auth (#731)

Co-authored-by: Dominik Pschenitschni <mail@celement.de>
Reviewed-on: https://kolaente.dev/vikunja/frontend/pulls/731
Reviewed-by: konrad <k@knt.li>
Co-authored-by: dpschen <dpschen@noreply.kolaente.de>
Co-committed-by: dpschen <dpschen@noreply.kolaente.de>

src/App.vue

@@ -23,11 +23,9 @@
 </template>
 
 <script>
-import {mapState} from 'vuex'
+import {mapState, mapGetters} from 'vuex'
 import isTouchDevice from 'is-touch-device'
 
-import authTypes from './models/authTypes'
-
 import Notification from './components/misc/notification'
 import {KEYBOARD_SHORTCUTS_ACTIVE, ONLINE} from './store/mutation-types'
 import KeyboardShortcuts from './components/misc/keyboard-shortcuts'
@@ -74,11 +72,13 @@ export default {
 			return isTouchDevice()
 		},
 		...mapState({
-			authUser: state => state.auth.authenticated && (state.auth.info && state.auth.info.type === authTypes.USER),
-			authLinkShare: state => state.auth.authenticated && (state.auth.info && state.auth.info.type === authTypes.LINK_SHARE),
 			online: ONLINE,
 			keyboardShortcutsActive: KEYBOARD_SHORTCUTS_ACTIVE,
 		}),
+		...mapGetters('auth', [
+			'authUser',
+			'authLinkShare',
+		]),
 	},
 	methods: {
 		setupOnlineStatus() {

src/models/authTypes.json

@@ -1,5 +0,0 @@
-{
-  "UNKNOWN": 0,
-  "USER": 1,
-  "LINK_SHARE": 2
-}

src/store/modules/auth.js

@@ -3,6 +3,12 @@ import {ERROR_MESSAGE, LOADING} from '../mutation-types'
 import UserModel from '../../models/user'
 import {getToken, refreshToken, removeToken, saveToken} from '@/helpers/auth'
 
+const AUTH_TYPES = {
+	'UNKNOWN': 0,
+	'USER': 1,
+	'LINK_SHARE': 2,
+}
+
 const defaultSettings = settings => {
 	if (typeof settings.weekStart === 'undefined' || settings.weekStart === '') {
 		settings.weekStart = 0
@@ -21,6 +27,20 @@ export default {
 		lastUserInfoRefresh: null,
 		settings: {},
 	}),
+	getters: {
+		authUser(state) {
+			return state.authenticated && (
+				state.info &&
+				state.info.type === AUTH_TYPES.USER
+			)
+		},
+		authLinkShare(state) {
+			return state.authenticated && (
+				state.info &&
+				state.info.type === AUTH_TYPES.LINK_SHARE
+			)
+		},
+	},
 	mutations: {
 		info(state, info) {
 			state.info = info

src/views/sharing/LinkSharingAuth.vue

@@ -33,8 +33,7 @@
 </template>
 
 <script>
-import {mapState} from 'vuex'
+import {mapGetters} from 'vuex'
-import authTypes from '@/models/authTypes.json'
 
 export default {
 	name: 'LinkSharingAuth',
@@ -54,9 +53,9 @@ export default {
 	mounted() {
 		this.setTitle(this.$t('sharing.authenticating'))
 	},
-	computed: mapState({
+	computed: mapGetters('auth', [
-		authLinkShare: state => state.auth.authenticated && (state.auth.info && state.auth.info.type === authTypes.LINK_SHARE),
+		'authLinkShare',
-	}),
+	]),
 	methods: {
 		auth() {
 			this.errorMessage = ''