aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorLibravatar Amine Mouafik <amine@mouafik.fr>2019-11-29 19:41:09 +0700
committerLibravatar Amine Mouafik <amine@mouafik.fr>2019-11-29 19:41:09 +0700
commitb3c56428fa8fe15a35b70c5f5ca933ce2c0bba66 (patch)
tree982c30316842530af2f162fca0e5aaba96279b87
parentFix cannot read property 'args' of null (diff)
downloadferdium-app-b3c56428fa8fe15a35b70c5f5ca933ce2c0bba66.tar.gz
ferdium-app-b3c56428fa8fe15a35b70c5f5ca933ce2c0bba66.tar.zst
ferdium-app-b3c56428fa8fe15a35b70c5f5ca933ce2c0bba66.zip
Prevent Password Lock to trigger while not logged in
-rw-r--r--src/lib/Menu.js4
-rw-r--r--src/stores/SettingsStore.js4
2 files changed, 4 insertions, 4 deletions
diff --git a/src/lib/Menu.js b/src/lib/Menu.js
index 941107adb..303845e55 100644
--- a/src/lib/Menu.js
+++ b/src/lib/Menu.js
@@ -800,7 +800,7 @@ export default class FranzMenu {
800 }, { 800 }, {
801 label: intl.formatMessage(menuItems.lockFerdi), 801 label: intl.formatMessage(menuItems.lockFerdi),
802 accelerator: 'CmdOrCtrl+Shift+L', 802 accelerator: 'CmdOrCtrl+Shift+L',
803 enabled: this.stores.settings.app.lockingFeatureEnabled, 803 enabled: this.stores.user.isLoggedIn && this.stores.settings.app.lockingFeatureEnabled,
804 click() { 804 click() {
805 // Disable lock first - otherwise the application might not update correctly 805 // Disable lock first - otherwise the application might not update correctly
806 actions.settings.update({ 806 actions.settings.update({
@@ -964,7 +964,7 @@ export default class FranzMenu {
964 this.currentTemplate = tpl; 964 this.currentTemplate = tpl;
965 const menu = Menu.buildFromTemplate(tpl); 965 const menu = Menu.buildFromTemplate(tpl);
966 const lockedMenu = Menu.buildFromTemplate([]); 966 const lockedMenu = Menu.buildFromTemplate([]);
967 Menu.setApplicationMenu(this.stores.settings.app.locked ? lockedMenu : menu); 967 Menu.setApplicationMenu(this.stores.user.isLoggedIn && this.stores.settings.app.locked ? lockedMenu : menu);
968 } 968 }
969 969
970 serviceTpl() { 970 serviceTpl() {
diff --git a/src/stores/SettingsStore.js b/src/stores/SettingsStore.js
index df0fc77e9..051e86be2 100644
--- a/src/stores/SettingsStore.js
+++ b/src/stores/SettingsStore.js
@@ -68,7 +68,6 @@ export default class SettingsStore extends Store {
68 () => this.all.app.locked, 68 () => this.all.app.locked,
69 () => { 69 () => {
70 const { router } = window.ferdi.stores; 70 const { router } = window.ferdi.stores;
71
72 if (this.all.app.locked && this.all.app.lockingFeatureEnabled) { 71 if (this.all.app.locked && this.all.app.lockingFeatureEnabled) {
73 // App just got locked, redirect to unlock screen 72 // App just got locked, redirect to unlock screen
74 router.push('/auth/locked'); 73 router.push('/auth/locked');
@@ -82,7 +81,8 @@ export default class SettingsStore extends Store {
82 81
83 // Make sure to lock app on launch if locking feature is enabled 82 // Make sure to lock app on launch if locking feature is enabled
84 setTimeout(() => { 83 setTimeout(() => {
85 if (this.all.app.lockingFeatureEnabled) { 84 const isLoggedIn = Boolean(localStorage.getItem('authToken'));
85 if (isLoggedIn && this.all.app.lockingFeatureEnabled) {
86 // Disable lock first - otherwise the lock might not get activated corrently 86 // Disable lock first - otherwise the lock might not get activated corrently
87 this.actions.settings.update({ 87 this.actions.settings.update({
88 type: 'app', 88 type: 'app',