From ce0c7451668317d5e871fc9d72ac83723627b962 Mon Sep 17 00:00:00 2001
From: chimp1984 <chimp1984@gmx.com>
Date: Wed, 30 Oct 2019 11:27:35 -0500
Subject: [PATCH] Add CapabilityRequiringPayload to TradeStatistics2

With v1.2.0 we changed the way how the hash is created.
To not create too heavy load for seed nodes from
requests from old nodes we use the SIGNED_ACCOUNT_AGE_WITNESS
capability to send trade statistics only to new nodes.
As trade statistics are only used for informational purpose it will
not have any critical issue for the old nodes beside that they don't see the latest trades.
---
 .../core/trade/statistics/TradeStatistics2.java    | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/core/src/main/java/bisq/core/trade/statistics/TradeStatistics2.java b/core/src/main/java/bisq/core/trade/statistics/TradeStatistics2.java
index 72e9a6fe9c5..dc2b3829703 100644
--- a/core/src/main/java/bisq/core/trade/statistics/TradeStatistics2.java
+++ b/core/src/main/java/bisq/core/trade/statistics/TradeStatistics2.java
@@ -24,9 +24,12 @@
 import bisq.core.offer.OfferPayload;
 import bisq.core.offer.OfferUtil;
 
+import bisq.network.p2p.storage.payload.CapabilityRequiringPayload;
 import bisq.network.p2p.storage.payload.LazyProcessedPayload;
 import bisq.network.p2p.storage.payload.PersistableNetworkPayload;
 
+import bisq.common.app.Capabilities;
+import bisq.common.app.Capability;
 import bisq.common.crypto.Hash;
 import bisq.common.proto.persistable.PersistableEnvelope;
 import bisq.common.util.ExtraDataMapValidator;
@@ -60,7 +63,7 @@
 
 @Slf4j
 @Value
-public final class TradeStatistics2 implements LazyProcessedPayload, PersistableNetworkPayload, PersistableEnvelope {
+public final class TradeStatistics2 implements LazyProcessedPayload, PersistableNetworkPayload, PersistableEnvelope, CapabilityRequiringPayload {
 
     //We don't support arbitrators anymore so this entry will be only for pre v1.2. trades
     @Deprecated
@@ -231,6 +234,15 @@ public boolean verifyHashSize() {
         return hash.length == 20;
     }
 
+    // With v1.2.0 we changed the way how the hash is created. To not create too heavy load for seed nodes from
+    // requests from old nodes we use the SIGNED_ACCOUNT_AGE_WITNESS capability to send trade statistics only to new
+    // nodes. As trade statistics are only used for informational purpose it will not have any critical issue for the
+    // old nodes beside that they don't see the latest trades.
+    @Override
+    public Capabilities getRequiredCapabilities() {
+        return new Capabilities(Capability.SIGNED_ACCOUNT_AGE_WITNESS);
+    }
+
 
     ///////////////////////////////////////////////////////////////////////////////////////////
     // Getters