EcoSkills icon indicating copy to clipboard operation
EcoSkills copied to clipboard

Require Pumpkins & Melons to not be player placed

Open n1ckoates opened this issue 7 months ago • 0 comments

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!

n1ckoates avatar Jun 28 '24 03:06 n1ckoates