Merge pull request #2595 from moreal/hotfix-transfer-ncg-histories

Deserialize only transfer-asset actions
planetarium · Oct 2, 2024 · 26f0f7b · 26f0f7b
2 parents 97965e0 + 4c1a232
commit 26f0f7b
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/NineChronicles.Headless/GraphTypes/StandaloneQuery.cs b/NineChronicles.Headless/GraphTypes/StandaloneQuery.cs
@@ -238,17 +238,20 @@ public StandaloneQuery(StandaloneContext standaloneContext, IKeyStore keyStore,
 
                     var filtered = block.Transactions
                         .Where(tx => tx.Actions.Count == 1)
+                        .Where(tx =>
+                            tx.Actions[0] is Dictionary dictionary && dictionary.ContainsKey("type_id") &&
+                            dictionary["type_id"] is Text typeId && typeId == TransferAsset.TypeIdentifier)
                         .Select(tx =>
                         (
                             transactionRepository.GetTxExecution(blockHash, tx.Id) ??
-                                throw new InvalidOperationException($"TxExecution {tx.Id} not found."),
+                            throw new InvalidOperationException($"TxExecution {tx.Id} not found."),
                             ToAction(tx.Actions[0])
                         ))
                         .Where(pair => pair.Item2 is ITransferAsset)
                         .Select(pair => (pair.Item1!, (ITransferAsset)pair.Item2))
                         .Where(pair => !pair.Item1.Fail &&
-                            (!recipient.HasValue || pair.Item2.Recipient == recipient) &&
-                            pair.Item2.Amount.Currency.Ticker == "NCG");
+                                       (!recipient.HasValue || pair.Item2.Recipient == recipient) &&
+                                       pair.Item2.Amount.Currency.Ticker == "NCG");
 
                     var histories = filtered.Select(pair =>
                         new TransferNCGHistory(