From 46eb872774f5ac9df848bdc263dd086ca2b20a52 Mon Sep 17 00:00:00 2001 From: sampleblood <361042631@qq.com> Date: Fri, 11 Feb 2022 01:28:42 +0800 Subject: [PATCH] fix: handle AffectedNodes edge case in settings.ts (#1907) AffectedNodes may have not only ModifiedNode, but also CreatedNode and DeletedNode. https://xrpl.org/transaction-metadata.html Co-authored-by: Mayukha Vadari --- src/ledger/parse/settings.ts | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/ledger/parse/settings.ts b/src/ledger/parse/settings.ts index 564ff6ba..e18f1ec2 100644 --- a/src/ledger/parse/settings.ts +++ b/src/ledger/parse/settings.ts @@ -5,9 +5,7 @@ const AccountFlags = constants.AccountFlags import parseFields from './fields' function getAccountRootModifiedNode(tx: any) { - const modifiedNodes = tx.meta.AffectedNodes.filter( - (node) => node.ModifiedNode.LedgerEntryType === 'AccountRoot' - ) + const modifiedNodes = tx.meta.AffectedNodes.filter(node => node.ModifiedNode?.LedgerEntryType === 'AccountRoot'); assert.ok(modifiedNodes.length === 1) return modifiedNodes[0].ModifiedNode }