Skip to content

Fix get_transaction with include_reversible=True

Krzysztof Leśniak requested to merge 322-fix-include_reversible-in-get_transaction into develop

get_transaction was trying to return transaction that was still pending and that triggered assertion full_txs.size() > txInBlock. This was because account_history_rocksdb_plugin::impl::on_pre_apply_operation was creating a volatile_operation_object for a pending transaction, which it shouldn't.

The fix is to exit early from on_pre_apply_operation in this case, so that volatile_operation_object is not created.

Closes #322 (closed)

Merge request reports