Require Pumpkins & Melons to not be player placed #159
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
With the current default configuration, players can repeatedly place and break a pumpkin (or a melon with silk touch) and get farming XP each time. This fixes that by requiring pumpkins and melons to not be player placed.
I also moved melon to the same category as pumpkin; given that they require the same effort to grow, it makes sense that they give the same XP. Melons and pumpkins don't have growth stages either (their stems do), so
fully_grown
isn't necessary. It also means less config lines!As a side note, I noticed the default configuration doesn't actually give XP for glow berries, cactuses, or sugar cane, despite them being in the config because they don't have growth stages. This is something you probably want to address by having a trigger for picking berries and for breaking cactuses/sugar cane (e.g. breaking a 3 high sugar cane at the bottom gives XP for each block).
Thanks!