Merge pull request #10721 from FlaminSarge/dupeperks

Make is:dupeperks avoid seeing different-class armor as dupeperks
DestinyItemManager · Sep 9, 2024 · afbddf5 · afbddf5
2 parents 951df1f + b3e2aea
commit afbddf5
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/src/app/search/items/search-filters/dupes.ts b/src/app/search/items/search-filters/dupes.ts
@@ -211,15 +211,16 @@ const dupeFilters: ItemFilterDefinition[] = [
       const duplicates = new Map<string, PerksSet>();
       for (const i of allItems) {
         if (i.sockets?.allSockets.some((s) => s.isPerk && s.socketDefinition.defaultVisible)) {
-          if (!duplicates.has(i.typeName)) {
-            duplicates.set(i.typeName, new PerksSet());
+          const dupeId = i.classType + i.type;
+          if (!duplicates.has(dupeId)) {
+            duplicates.set(dupeId, new PerksSet());
           }
-          duplicates.get(i.typeName)!.insert(i);
+          duplicates.get(dupeId)!.insert(i);
         }
       }
       return (item) =>
         item.sockets?.allSockets.some((s) => s.isPerk && s.socketDefinition.defaultVisible) &&
-        Boolean(duplicates.get(item.typeName)?.hasPerkDupes(item));
+        Boolean(duplicates.get(item.classType + item.type)?.hasPerkDupes(item));
     },
   },
 ];