Merge pull request #71 from WoWAnalyzer/push-vwowmtnrwurs

emallson · web-flow · commit 1c0b22eb7b56 · 2025-10-25T14:12:05.000-04:00
don't crash when the blizzard api dumps undefined talent data
diff --git a/src/route/blizzard/index.ts b/src/route/blizzard/index.ts
@@ -107,6 +107,7 @@ async function fetchCharacter(
           code: activeTree.talent_loadout_code,
           nodes: activeTree.selected_class_talents
             .concat(activeTree.selected_spec_talents)
+            .filter(Boolean) // sometimes the blizzard api does terrible, terrible things
             .map(({ id, rank }) => ({ id, rank })),
         }
       : undefined,

Original file line number	Diff line number	Diff line change
`@@ -107,6 +107,7 @@ async function fetchCharacter(`
`107`	`107`	`code: activeTree.talent_loadout_code,`
`108`	`108`	`nodes: activeTree.selected_class_talents`
`109`	`109`	`.concat(activeTree.selected_spec_talents)`
	`110`	`+ .filter(Boolean) // sometimes the blizzard api does terrible, terrible things`
`110`	`111`	`.map(({ id, rank }) => ({ id, rank })),`
`111`	`112`	`}`
`112`	`113`	`: undefined,`