remove S1066 [skip changelog]

planetarium · Dec 14, 2019 · 9cd3212 · 9cd3212
1 parent f5b3aa4
commit 9cd3212
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 7 deletions.
diff --git a/Libplanet.ruleset b/Libplanet.ruleset
@@ -108,8 +108,6 @@
     other handling the asynchronous code. -->
     <Rule Id="S4457" Action="None" />
 
-    <!-- Merge this if statement with the enclosing one. -->
-    <Rule Id="S1066" Action="None" />
     <!-- Remove the unnecessary Boolean literal(s). -->
     <Rule Id="S1125" Action="None" />
     <!-- Remove the field '_bucketSize' and declare it as a local variable

diff --git a/Libplanet/Store/DefaultStore.cs b/Libplanet/Store/DefaultStore.cs
@@ -660,12 +660,9 @@ public override IEnumerable<KeyValuePair<Address, long>> ListTxNonces(Guid chain
                 if (doc.TryGetValue("_id", out BsonValue id) && id.IsBinary)
                 {
                     var address = new Address(id.AsBinary);
-                    if (doc.TryGetValue("v", out BsonValue v) && v.IsInt64)
+                    if (doc.TryGetValue("v", out BsonValue v) && v.IsInt64 && v.AsInt64 > 0)
                     {
-                        if (v.AsInt64 > 0)
-                        {
-                            yield return new KeyValuePair<Address, long>(address, v.AsInt64);
-                        }
+                        yield return new KeyValuePair<Address, long>(address, v.AsInt64);
                     }
                 }
             }