[Bug] Removing discipline (and sub-discipline) scrolls from loot when added to WhiteLists for New Players
KnightCa opened this issue ยท 2 comments
Is there an existing issue for this?
- I have searched the existing issues
Observed behaviour
Discipline scrolls and Sub-Discipline scrolls matching the "Whitelist For Main Disciplines Assigned To New Players Joining World" are found in treasure chests.
Sub-Discipline scrolls matching the "Whitelist For Sub Disciplines Assigned To New Players Joining World" are found in treasure chests.
Expected behaviour
Discipline scrolls and Sub-Discipline scrolls matching the "Whitelist For Main Disciplines Assigned To New Players Joining World" are not found in treasure chests.
Sub-Discipline scrolls matching the "Whitelist For Sub Disciplines Assigned To New Players Joining World" are not found in treasure chests.
Steps to reproduce
- Setup a server config so that the "Whitelist For Main Disciplines Assigned To New Players Joining World" includes both "Ancient" and "Sorcery".
- Discover a rare Discipline or Sub-Discipline scroll in a treasure chest
- Disappointingly realize that the scroll is one of the two Disiciplines you automatically start with, and is thus not helpful for anyone.
Crashlog
No response
Environment (Singleplayer/Server, etc.)
Issue present on both sides
Mod version
2.2.1
Forge version
1.12.2 latest version
Other mods
No response
If I add loot-tables from other mods (such as Ancient Spellcraft & Wizardry Tales structures) to "Loot Inject Locations", do I need to modify each of those loot-tables with LootTweaker to remove Discipline scrolls that correspond to ones given by Whitelist to all players?
Here is the compiled script file I created with LootTweaker to resolve these issues for my server. Might be helpful for others.
Wizardry Fates config
- Added these loot-tables to config "Discipline Scroll Loot Inject Locations":
ancientspellcraft:chests/ancient_vault
ancientspellcraft:chests/sage_camp
ancientspellcraft:chests/battlemage_camp
ebwizardry:chests/obelisk
ebwizardry:chests/shrine
ebwizardry:chests/wizard_tower
w_tweaks:chests/novice_house
w_tweaks:chests/shrine
wizardry_tales:chests/novice_house
wizardry_tales:chests/shrine
minecraft:chests/desert_pyramid
minecraft:chests/end_city_treasure
minecraft:chests/igloo_chest
minecraft:chests/jungle_temple
minecraft:chests/stronghold_corridor
minecraft:chests/stronghold_crossing
minecraft:chests/stronghold_library
minecraft:chests/woodland_mansion
mod_lavacow:chests/cemetery_chest
mod_lavacow:chests/desert_tomb_chest
iceandfire:fire_dragon_female_cave
iceandfire:fire_dragon_male_cave
iceandfire:ice_dragon_female_cave
iceandfire:ice_dragon_male_cave
iceandfire:lightning_dragon_female_cave
iceandfire:lightning_dragon_male_cave
iceandfire:hydra_cave
iceandfire:mausoleum_chest
iceandfire:myrmex_loot_chest
illagers_plus:structure/illager_centre
illagers_plus:structure/illager_tower_rare
illagers_plus:structure/illager_tower_researcher
illagers_plus:structure/illager_tower_uncommon
twilightforest:structures/tower_room/uncommon
twilightforest:structures/tower_room/rare
twilightforest:structures/tower_room/ultrarare
twilightforest:structures/tower_library/uncommon
twilightforest:structures/tower_library/rare
twilightforest:structures/tower_library/ultrarare
LootTweaker Script
- Removed Discipline scrolls (Ancient and Sorcery) from Loot-Tables
- Removed Discipline and Sub-Discipline scrolls (Ancient and Sorcery) from JEI
- Added remaining Sub-Discipline scrolls into Loot-Tables with a weight of 5 (vs 1 for Discipline scrolls)