From 1f90ef1c88293c9e6c8e3e6ac35f24d5b97dce56 Mon Sep 17 00:00:00 2001
From: woodser <woodser@protonmail.com>
Date: Sun, 14 Jan 2024 13:51:33 -0500
Subject: [PATCH] fix warning about multiple acks on payment sent

---
 .../portfolio/pendingtrades/PendingTradesViewModel.java  | 9 +++++----
 .../pendingtrades/steps/buyer/BuyerStep2View.java        | 9 +++++----
 2 files changed, 10 insertions(+), 8 deletions(-)

diff --git a/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/PendingTradesViewModel.java b/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/PendingTradesViewModel.java
index 73db3c3541..0a68b04afa 100644
--- a/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/PendingTradesViewModel.java
+++ b/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/PendingTradesViewModel.java
@@ -194,15 +194,16 @@ public class PendingTradesViewModel extends ActivatableWithDataModel<PendingTrad
         }
     }
 
-    public void setMessageStateProperty(MessageState messageState) {
+    public void setMessageStatePropertyIfNotAcked(MessageState messageState) {
+
+        // skip if already acked
         if (messageStateProperty.get() == MessageState.ACKNOWLEDGED) {
-            log.warn("We have already an ACKNOWLEDGED message received. " +
-                    "We would not expect any other message after that. Received messageState={}", messageState);
             return;
         }
 
-        if (trade != null)
+        if (trade != null) {
             trade.getProcessModel().setPaymentSentMessageState(messageState);
+        }
     }
 
     private void onMessageStateChanged(MessageState messageState) {
diff --git a/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/steps/buyer/BuyerStep2View.java b/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/steps/buyer/BuyerStep2View.java
index 6990d40343..b090ee9e35 100644
--- a/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/steps/buyer/BuyerStep2View.java
+++ b/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/steps/buyer/BuyerStep2View.java
@@ -22,6 +22,7 @@ import haveno.common.UserThread;
 import haveno.common.app.DevEnv;
 import haveno.common.util.Tuple4;
 import haveno.core.locale.Res;
+import haveno.core.network.MessageState;
 import haveno.core.offer.Offer;
 import haveno.core.payment.PaymentAccount;
 import haveno.core.payment.PaymentAccountUtil;
@@ -157,7 +158,7 @@ public class BuyerStep2View extends TradeStepView {
                         case BUYER_SAW_ARRIVED_PAYMENT_SENT_MSG:
                             busyAnimation.play();
                             statusLabel.setText(Res.get("shared.sendingConfirmation"));
-                            model.setMessageStateProperty(trade.getPaymentSentMessageState());
+                            model.setMessageStatePropertyIfNotAcked(MessageState.SENT);
                             timeoutTimer = UserThread.runAfter(() -> {
                                 busyAnimation.stop();
                                 statusLabel.setText(Res.get("shared.sendingConfirmationAgain"));
@@ -166,18 +167,18 @@ public class BuyerStep2View extends TradeStepView {
                         case BUYER_STORED_IN_MAILBOX_PAYMENT_SENT_MSG:
                             busyAnimation.stop();
                             statusLabel.setText(Res.get("shared.messageStoredInMailbox"));
-                            model.setMessageStateProperty(trade.getPaymentSentMessageState());
+                            model.setMessageStatePropertyIfNotAcked(MessageState.STORED_IN_MAILBOX);
                             break;
                         case SELLER_RECEIVED_PAYMENT_SENT_MSG:
                             busyAnimation.stop();
                             statusLabel.setText(Res.get("shared.messageArrived"));
-                            model.setMessageStateProperty(trade.getPaymentSentMessageState());
+                            model.setMessageStatePropertyIfNotAcked(MessageState.ARRIVED);
                             break;
                         case BUYER_SEND_FAILED_PAYMENT_SENT_MSG:
                             // We get a popup and the trade closed, so we dont need to show anything here
                             busyAnimation.stop();
                             statusLabel.setText("");
-                            model.setMessageStateProperty(trade.getPaymentSentMessageState());
+                            model.setMessageStatePropertyIfNotAcked(MessageState.FAILED);
                             break;
                         default:
                             log.warn("Unexpected case: State={}, tradeId={} ", state.name(), trade.getId());