Skip to content

Added exception tag for mapmaker#19

Open
crux-f wants to merge 2 commits into
Reldeam:mainfrom
crux-f:main
Open

Added exception tag for mapmaker#19
crux-f wants to merge 2 commits into
Reldeam:mainfrom
crux-f:main

Conversation

@crux-f

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

Copy link
Copy Markdown

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

crux-f commented Oct 12, 2022

Copy link
Copy Markdown
Author

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