Hidden Mining menu when there is no wallet (#405)

* Hidden Mining menu when there is no wallet * Use walletManager instead of walletSyncManager * Clean Codes Fields should start with lowercase initial character, properties with uppercase.
block-core · Jun 7, 2022 · 58d4540 · 58d4540
1 parent bcb7e44
commit 58d4540
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/src/Features/Blockcore.Features.ColdStaking/UI/NavItem.cs b/src/Features/Blockcore.Features.ColdStaking/UI/NavItem.cs
@@ -6,17 +6,17 @@ namespace Blockcore.Features.Wallet.UI
 {
     public class ColdStakingNavigationItem : INavigationItem
     {
-        private readonly IWalletManager WalletManager;
+        private readonly IWalletManager walletManager;
 
-        public ColdStakingNavigationItem(IWalletManager WalletManager)
+        public ColdStakingNavigationItem(IWalletManager walletManager)
         {
-            this.WalletManager = WalletManager;
+            this.walletManager = walletManager;
         }
 
         public string Name => "Cold Staking";
         public string Navigation => "ColdStaking";
         public string Icon => "oi-pulse";
-        public bool IsVisible => this.WalletManager?.ContainsWallets ?? false;
+        public bool IsVisible => this.walletManager?.ContainsWallets ?? false;
         public int NavOrder => 30;
 
     }

diff --git a/src/Features/Blockcore.Features.Miner/UI/NavItem.cs b/src/Features/Blockcore.Features.Miner/UI/NavItem.cs
@@ -7,11 +7,11 @@ namespace Blockcore.Features.Miner.UI
     public class MineNavigationItem : INavigationItem
     {
         private readonly Network network;
-        private readonly IWalletSyncManager walletSyncManager;
-        public MineNavigationItem(Network network, IWalletSyncManager walletSyncManager)
+        private readonly IWalletManager walletManager;
+        public MineNavigationItem(Network network, IWalletManager walletManager)
         {
             this.network = network;
-            this.walletSyncManager = walletSyncManager;
+            this.walletManager = walletManager;
         }
 
         public string Name => "Mining";
@@ -21,26 +21,26 @@ public MineNavigationItem(Network network, IWalletSyncManager walletSyncManager)
         public int NavOrder => 15;
         private bool CheckIsVisible()
         {
-            if (this.network.Consensus.IsProofOfStake && (this.walletSyncManager.WalletTip.Height > this.network.Consensus.LastPOWBlock))
+            if (this.network.Consensus.IsProofOfStake && (this.walletManager?.WalletTipHeight > this.network.Consensus.LastPOWBlock))
             {
                 return false;
             }
-            return true;
+            return this.walletManager?.ContainsWallets ?? false;
         }
     }
     public class StakeNavigationItem : INavigationItem
     {
-        private readonly IWalletManager WalletManager;
+        private readonly IWalletManager walletManager;
 
-        public StakeNavigationItem(IWalletManager WalletManager)
+        public StakeNavigationItem(IWalletManager walletManager)
         {
-            this.WalletManager = WalletManager;
+            this.walletManager = walletManager;
         }
 
         public string Name => "Staking";
         public string Navigation => "Stake";
         public string Icon => "oi-bolt";
-        public bool IsVisible => this.WalletManager?.ContainsWallets ?? false;
+        public bool IsVisible => this.walletManager?.ContainsWallets ?? false;
         public int NavOrder => 20;
     }
 }