Skip to content

Commit 71d4597

Browse files
mangonuiclaude
andcommitted
fix(proxy): restore hasLogEvents check in checkIfMoveBalanceNotarized
Forward-port from upstream PR multiversx#511 (commit 2176395 "process status fixes") that was inadvertently reverted by the earlier Dharitri RWA development snapshot. NewArc/Fixes/mx-chain-proxy-go carries this upstream improvement correctly; dev/rwa has been missing it since the snapshot was applied on top of the older PR multiversx#506 base. Without this check, a move-balance transaction that produces log events (e.g. an inner ESDT transfer wrapped as a move-balance) was classified as a plain move-balance and the proxy missed the log-event signal in its processing-status decision. The function body now matches NewArc/Fixes/mx-chain-proxy-go process/transactionProcessor.go exactly. All ./process/... tests pass. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
1 parent a81fc47 commit 71d4597

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

‎process/transactionProcessor.go‎

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -627,7 +627,9 @@ func checkIfMoveBalanceNotarized(tx *transaction.ApiTransactionResult) bool {
627627
if !isNotarized {
628628
return false
629629
}
630-
isMoveBalance := tx.ProcessingTypeOnSource == moveBalanceDescriptor && tx.ProcessingTypeOnDestination == moveBalanceDescriptor
630+
hasLogEvents := tx.Logs != nil && len(tx.Logs.Events) > 0
631+
isMoveBalance := tx.ProcessingTypeOnSource == moveBalanceDescriptor &&
632+
tx.ProcessingTypeOnDestination == moveBalanceDescriptor && !hasLogEvents
631633

632634
return isMoveBalance
633635
}

0 commit comments

Comments
 (0)