From 66756b205a12f884ef425cc953bde00a9fa61e67 Mon Sep 17 00:00:00 2001 From: woodser Date: Mon, 18 Dec 2023 12:43:54 -0500 Subject: [PATCH] replace Plaform.runLater with UserThread.execute to fix concurrency --- .../main/portfolio/pendingtrades/PendingTradesView.java | 6 +----- .../main/portfolio/pendingtrades/steps/TradeStepView.java | 7 +------ 2 files changed, 2 insertions(+), 11 deletions(-) diff --git a/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/PendingTradesView.java b/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/PendingTradesView.java index b309de20a3e..65f4dde199b 100644 --- a/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/PendingTradesView.java +++ b/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/PendingTradesView.java @@ -931,11 +931,7 @@ public void updateItem(PendingTradesListItem newItem, boolean empty) { super.updateItem(newItem, empty); if (!empty && newItem != null) { trade = newItem.getTrade(); - listener = (observable, oldValue, newValue) -> Platform.runLater(new Runnable() { - @Override public void run() { - update(); - } - }); + listener = (observable, oldValue, newValue) -> UserThread.execute(() -> update()); trade.stateProperty().addListener(listener); update(); } else { diff --git a/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/steps/TradeStepView.java b/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/steps/TradeStepView.java index d236a6c373a..163a3cf0841 100644 --- a/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/steps/TradeStepView.java +++ b/desktop/src/main/java/haveno/desktop/main/portfolio/pendingtrades/steps/TradeStepView.java @@ -424,12 +424,7 @@ private void updateTimeLeft() { log.warn("deactivating TradeStepView because model's trade is null"); // schedule deactivation to avoid concurrent modification of clock listeners - Platform.runLater(new Runnable() { - @Override - public void run() { - deactivate(); - } - }); + UserThread.execute(() -> deactivate()); return; }