fix: use null for missing fields

Signed-off-by: Nikita Lebedev <nikita.lebedev@limechain.tech>
hashgraph · Dec 12, 2023 · 03be5d0 · 03be5d0
1 parent 1c250da
commit 03be5d0
Showing 1 changed file with 8 additions and 14 deletions.
diff --git a/...dera-app/src/main/java/com/hedera/node/app/workflows/handle/record/RecordListBuilder.java b/...dera-app/src/main/java/com/hedera/node/app/workflows/handle/record/RecordListBuilder.java
@@ -22,14 +22,8 @@
 import static java.util.Collections.unmodifiableList;
 import static java.util.Objects.requireNonNull;
 
-import com.hedera.hapi.node.base.AccountID;
-import com.hedera.hapi.node.base.ContractID;
-import com.hedera.hapi.node.base.FileID;
 import com.hedera.hapi.node.base.ResponseCodeEnum;
-import com.hedera.hapi.node.base.ScheduleID;
-import com.hedera.hapi.node.base.TokenID;
 import com.hedera.hapi.node.base.TransactionID;
-import com.hedera.hapi.node.contract.ContractFunctionResult;
 import com.hedera.node.app.spi.workflows.HandleContext;
 import com.hedera.node.app.spi.workflows.HandleContext.TransactionCategory;
 import com.hedera.node.app.spi.workflows.HandleException;
@@ -384,12 +378,12 @@ public void revertChildrenOf(@NonNull final SingleTransactionRecordBuilderImpl r
             } else {
                 if (child.reversingBehavior() == ReversingBehavior.REVERSIBLE && SUCCESSES.contains(child.status())) {
                     // clear "side effect" fields
-                    child.accountID(AccountID.DEFAULT);
-                    child.contractID(ContractID.DEFAULT);
-                    child.fileID(FileID.DEFAULT);
-                    child.tokenID(TokenID.DEFAULT);
-                    child.scheduleID(ScheduleID.DEFAULT);
-                    child.scheduledTransactionID(TransactionID.DEFAULT);
+                    child.accountID(null);
+                    child.contractID(null);
+                    child.fileID(null);
+                    child.tokenID(null);
+                    child.scheduleID(null);
+                    child.scheduledTransactionID(null);
                     child.serialNumbers().clear();
                     child.topicRunningHash(Bytes.EMPTY);
                     child.newTotalSupply(-1L);
@@ -401,8 +395,8 @@ public void revertChildrenOf(@NonNull final SingleTransactionRecordBuilderImpl r
                         child.transferList().accountAmounts().clear();
                     }
                     child.paidStakingRewards().clear();
-                    child.contractCreateResult(ContractFunctionResult.DEFAULT);
-                    child.scheduleRef(ScheduleID.DEFAULT);
+                    child.contractCreateResult(null);
+                    child.scheduleRef(null);
                     child.assessedCustomFees().clear();
                     child.alias(Bytes.EMPTY);
                     child.ethereumHash(Bytes.EMPTY);