cgiacofei/TandoorRecipes
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

prevent auto sync from running mulitple time

Browse Source
This commit is contained in:
vabene1111 2024-01-19 19:36:34 +08:00
parent 62e1d860a9
commit eae409da67
1 changed files with 14 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
vue/src/apps/ShoppingListView/ShoppingListView.vue
View File

@ -34,8 +34,9 @@
<!-- shopping list tab --> <!-- shopping list tab -->
<b-tab active> <b-tab active>
<template #title> <template #title>
<i v-if="!shopping_list_store.currently_updating" <i v-if="!shopping_list_store.currently_updating && useShoppingListStore().autosync_has_focus"
class="fas fa-shopping-cart fa-fw"></i> class="fas fa-shopping-cart fa-fw"></i>
<i v-if="!shopping_list_store.currently_updating && !useShoppingListStore().autosync_has_focus" class="fas fa-eye-slash"></i>
<b-spinner v-if="shopping_list_store.currently_updating" type="border" small <b-spinner v-if="shopping_list_store.currently_updating" type="border" small
style="width: 1.25em!important; height: 1.25em!important;"></b-spinner> style="width: 1.25em!important; height: 1.25em!important;"></b-spinner>
<span class="d-none d-md-inline-block ml-1"> <span class="d-none d-md-inline-block ml-1">
@ -574,7 +575,7 @@ export default {
this.shopping_list_store.refreshFromAPI() this.shopping_list_store.refreshFromAPI()
useUserPreferenceStore().loadUserSettings() useUserPreferenceStore().loadUserSettings()
useUserPreferenceStore().loadDeviceSettings() useUserPreferenceStore().loadDeviceSettings()
this.setupAutoSync() this.autoSyncLoop()
this.setupFocusMonitor() this.setupFocusMonitor()
}, },
@ -582,25 +583,29 @@ export default {
useUserPreferenceStore, useUserPreferenceStore,
useShoppingListStore, useShoppingListStore,
/**
* setup interval checking for focus every 1000ms and updating the store variable
*/
setupFocusMonitor: function () { setupFocusMonitor: function () {
setInterval(() => { setInterval(() => {
useShoppingListStore().autosync_has_focus = document.hasFocus() useShoppingListStore().autosync_has_focus = document.hasFocus()
}, 1000); }, 1000);
}, },
/**
setupAutoSync: function () { * recursive function calling autosync after set amount of time has passed
// prevent setting up multiple loops on accident */
// TODO should this just raise an error? autoSyncLoop: function () {
clearInterval(this.autosync_id) // this should not happen in production but sometimes in development with HMR
clearTimeout(this.autosync_id)
this.autosync_id = undefined this.autosync_id = undefined
let timeout = Math.max(this.user_preference_store.user_settings.shopping_auto_sync, 1) * 1000 // if disabled (shopping_auto_sync=0) check again after 1 second if enabled let timeout = Math.max(this.user_preference_store.user_settings.shopping_auto_sync, 1) * 1000 // if disabled (shopping_auto_sync=0) check again after 1 second if enabled
this.autosync_id = setInterval(() => { //TODO does setInterval automatically loop (because it kind of did) this.autosync_id = setTimeout(() => {
if (this.online && this.user_preference_store.user_settings.shopping_auto_sync > 0) { if (this.online && this.user_preference_store.user_settings.shopping_auto_sync > 0) {
this.shopping_list_store.autosync() this.shopping_list_store.autosync()
this.setupAutoSync() this.autoSyncLoop()
} }
}, timeout) }, timeout)
}, },