Fix wrong initial value.

For myGenesisOutputNames we check for null not Optional.empty as empty is a valid case and we do not want to wrap the optional field into another Optional. Also myCompensationRequestNames is using a null check and gets reset to null instead of an empty set as the empty set is a valid case as well. Signed-off-by: HenrikJannsen <[email protected]>
HenrikJannsen · Dec 9, 2022 · e8df364 · e8df364
1 parent 5dbd291
commit e8df364
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/core/src/main/java/bisq/core/dao/burningman/BurningManPresentationService.java b/core/src/main/java/bisq/core/dao/burningman/BurningManPresentationService.java
@@ -55,8 +55,8 @@ public class BurningManPresentationService implements DaoStateListener {
     private final Map<String, BurningManCandidate> burningManCandidatesByName = new HashMap<>();
     private final Set<ReimbursementModel> reimbursements = new HashSet<>();
     private Optional<Long> averageDistributionPerCycle = Optional.empty();
-    private Set<String> myCompensationRequestNames;
-    private Optional<Set<String>> myGenesisOutputNames = Optional.empty();
+    private Set<String> myCompensationRequestNames = null;
+    private Optional<Set<String>> myGenesisOutputNames = null;
 
     @Inject
     public BurningManPresentationService(DaoStateService daoStateService,