From 9485ccbd975e8f175e296edbcc039572d728ff81 Mon Sep 17 00:00:00 2001 From: fnuesse Date: Thu, 6 Dec 2018 00:07:01 +0100 Subject: [PATCH] limit initializeNavbar calls to one time per controllerinstance Signed-off-by: fnuesse --- js/app/controllers/menu.js | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/js/app/controllers/menu.js b/js/app/controllers/menu.js index 36844df3..1506bd1a 100644 --- a/js/app/controllers/menu.js +++ b/js/app/controllers/menu.js @@ -193,18 +193,23 @@ }); }; - - $scope.legacyNavbarDefault=true; + $scope.legacyNavbarAlreadyInitialized=false; + if (typeof $scope.legacyNavbar === 'undefined') { $scope.legacyNavbar = $scope.legacyNavbarDefault; } + $scope.$watch('legacyNavbar', function(newValue, oldValue) { VaultService.setVaultSetting("vaultNavBarLegacy",newValue); }); $scope.initializeNavbar = function () { + if($scope.legacyNavbarAlreadyInitialized){ + return; + } $scope.legacyNavbar = VaultService.getVaultSetting('vaultNavBarLegacy',$scope.legacyNavbarDefault); + $scope.legacyNavbarAlreadyInitialized=true; }; $rootScope.$on('credentials_loaded', function () {