Sweenus / SimplySwords

Other
41 stars 23 forks source link

[Compatibility] Lootr Loot Tables #144

Open Snugs opened 7 months ago

Snugs commented 7 months ago

Hiya, just curious if it would be possible to get compatibility with Lootr chests? I understand in the Config files we can add various loot tables to be ignored when injecting Unique Weapons, however this list does not take effect when Lootr is installed as well, and due to the way Lootr works I don't think it's possible to simply add a Lootr loot table to that config without it affecting EVERY Lootr chest of that type (chest, barrel, trapped chest, etc).

My suggestion would be to incorporate some kind of dependency mixin that if Lootr is installed then check which loot table the Lootr chest is trying to draw it's values from, and if that loot table matches any from the Config file, reject the weapon injection. I'm not sure how else to go about fixing this issue, so please let me know your thoughts, and if you'd like assistance I'd be happy to help ^_^

Sweenus commented 4 months ago

Hi there, thank you for the report. I was not aware that there was an issue with the blacklist system when paired with Lootr, but this would explain some strange issues I've noticed in the past.

I will look into it 🙂