Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added exception tag for mapmaker #19

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Added exception tag for mapmaker #19

wants to merge 2 commits into from

Conversation

crux-f
Copy link

@crux-f crux-f commented Oct 12, 2022

I'm no Github expert, so forgive me if this is a little messy;
Decided to save you some time and add the enhancement myself! My Java is a little rusty, but I got there in the end.
(Closes #16 )

  • Checks if villager has scoreboard tag "cvt_trade_override"; If so, uses the vanilla trades regardless of disabled professions list
  • Allows villagers with predetermined trades in nbt to function as expected in custom map / server scenarios
  • Custom (NBT) trades are still removed at present if !plugin.isVanillaTradesAllowed(); Leaving it to Reldeam's discretion as to how that interacts

- Checks if villager has scoreboard tag "cvt_trade_override"; If so, uses the vanilla trades regardless of disabled professions list
- Allows villagers with predetermined trades in nbt to function as expected in custom map / server scenarios
- Custom (NBT) trades are still removed at present if !plugin.isVanillaTradesAllowed(); Leaving it to Reldeam's discretion as to how that interacts
- Checks if villager has scoreboard tag "cvt_trade_override"; If so, uses the vanilla trades regardless of disabled professions list
- Allows villagers with predetermined trades in nbt to function as expected in custom map / server scenarios
- Custom (NBT) trades are still removed at present if !plugin.isVanillaTradesAllowed(); Leaving it to Reldeam's discretion as to how that interacts
@crux-f
Copy link
Author

crux-f commented Oct 12, 2022

Note: I will be implementing this fork on my server and will let you know if I run into any issues, but as far as I can tell from looking over the plugin this shouldn't break anything major.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement]: Exclusion tag for mapmakers (or similar function)
1 participant