feat: allow filtering ba gambles from loot notifier

CHANGELOG.md

@@ -1,5 +1,6 @@
 ## Unreleased
 
+- Minor: Allow excluding Barbarian Assault high gambles from loot notifications to avoid duplication with the gambles notifier. (#657)
 - Minor: Treat `SPAM` messages as matchable game messages for the chat notifier. (#653)
 
 ## 1.10.21

src/main/java/dinkplugin/DinkPluginConfig.java

@@ -987,6 +987,17 @@ default boolean lootIncludeClueScrolls() {
         return true;
     }
 
+    @ConfigItem(
+        keyName = "lootIncludeGambles",
+        name = "Include BA Gambles",
+        description = "Allow notifications for barbarian assault high gambles",
+        position = 36,
+        section = lootSection
+    )
+    default boolean lootIncludeGambles() {
+        return false;
+    }
+
     @ConfigItem(
         keyName = "lootItemAllowlist",
         name = "Item Allowlist",

src/main/java/dinkplugin/notifiers/LootNotifier.java

@@ -131,6 +131,11 @@ public void onLootReceived(LootReceived lootReceived) {
 
         // only consider non-NPC and non-PK loot
         if (lootReceived.getType() == LootRecordType.EVENT || lootReceived.getType() == LootRecordType.PICKPOCKET) {
+            if ("Barbarian Assault high gamble".equals(lootReceived.getName()) && !config.lootIncludeGambles()) {
+                // skip ba gambles, depending on config (since we have GambleNotifier)
+                return;
+            }
+
             if (!config.lootIncludeClueScrolls() && StringUtils.startsWithIgnoreCase(lootReceived.getName(), "Clue Scroll")) {
                 // skip clue scroll loot, depending on config
                 return;