Move validation after adding dispute to list

This commit is contained in:
chimp1984 2020-09-21 00:20:12 -05:00
parent 76c82631de
commit a9f10624c2
No known key found for this signature in database
GPG key ID: 9801B4EC591F90E3
2 changed files with 12 additions and 13 deletions

View file

@ -317,18 +317,6 @@ public abstract class DisputeManager<T extends DisputeList<? extends DisputeList
Contract contract = dispute.getContract(); Contract contract = dispute.getContract();
addPriceInfoMessage(dispute, 0); addPriceInfoMessage(dispute, 0);
try {
TradeDataValidation.validateDonationAddress(dispute.getDonationAddressOfDelayedPayoutTx(), daoFacade);
TradeDataValidation.testIfDisputeTriesReplay(dispute, disputeList.getList());
TradeDataValidation.validateNodeAddress(dispute, dispute.getContract().getBuyerNodeAddress(), config);
TradeDataValidation.validateNodeAddress(dispute, dispute.getContract().getSellerNodeAddress(), config);
} catch (TradeDataValidation.AddressException |
TradeDataValidation.DisputeReplayException |
TradeDataValidation.NodeAddressException e) {
log.error(e.toString());
validationExceptions.add(e);
}
PubKeyRing peersPubKeyRing = dispute.isDisputeOpenerIsBuyer() ? contract.getSellerPubKeyRing() : contract.getBuyerPubKeyRing(); PubKeyRing peersPubKeyRing = dispute.isDisputeOpenerIsBuyer() ? contract.getSellerPubKeyRing() : contract.getBuyerPubKeyRing();
if (isAgent(dispute)) { if (isAgent(dispute)) {
if (!disputeList.contains(dispute)) { if (!disputeList.contains(dispute)) {
@ -359,6 +347,18 @@ public abstract class DisputeManager<T extends DisputeList<? extends DisputeList
} }
addMediationResultMessage(dispute); addMediationResultMessage(dispute);
try {
TradeDataValidation.validateDonationAddress(dispute.getDonationAddressOfDelayedPayoutTx(), daoFacade);
TradeDataValidation.testIfDisputeTriesReplay(dispute, disputeList.getList());
TradeDataValidation.validateNodeAddress(dispute, dispute.getContract().getBuyerNodeAddress(), config);
TradeDataValidation.validateNodeAddress(dispute, dispute.getContract().getSellerNodeAddress(), config);
} catch (TradeDataValidation.AddressException |
TradeDataValidation.DisputeReplayException |
TradeDataValidation.NodeAddressException e) {
log.error(e.toString());
validationExceptions.add(e);
}
} }
// Not-dispute-requester receives that msg from dispute agent // Not-dispute-requester receives that msg from dispute agent

View file

@ -733,7 +733,6 @@ public class DisputeSummaryWindow extends Overlay<DisputeSummaryWindow> {
fee, fee,
buyerPayoutAddressString, buyerPayoutAddressString,
sellerPayoutAddressString); sellerPayoutAddressString);
log.error("transaction " + tx);
tradeWalletService.broadcastTx(tx, new TxBroadcaster.Callback() { tradeWalletService.broadcastTx(tx, new TxBroadcaster.Callback() {
@Override @Override
public void onSuccess(Transaction transaction) { public void onSuccess(Transaction transaction) {