EcoSkills
EcoSkills copied to clipboard
Require Pumpkins & Melons to not be player placed
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!