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

Players can still craft pumpkin seeds with shears #324

Open
Nanpure opened this issue Mar 23, 2024 · 7 comments
Open

Players can still craft pumpkin seeds with shears #324

Nanpure opened this issue Mar 23, 2024 · 7 comments

Comments

@Nanpure
Copy link

Nanpure commented Mar 23, 2024

I have the following enabled in my config:

Cant Craft Melonseeds: true

Players can still craft pumpkin seeds by using shears on pumpkin blocks.

@RoboMWM
Copy link
Member

RoboMWM commented Mar 28, 2024

are pumpkin seeds crafted or merely obtained from the pumpkin blocks?

If the latter, then I guess this would technically be a new feature request to implement. (Though should be relatively easy to implement.)

@Nanpure
Copy link
Author

Nanpure commented Mar 29, 2024

They are obtained from placed pumpkin blocks. When you use shears on them to make a jack o lantern, their seeds pop out as an entity that can be picked up.

@RoboMWM
Copy link
Member

RoboMWM commented Mar 29, 2024

mk then that means it'd be a separate event handler.

However, not sure if this is intended behavior. (Maybe the intent was to require shears?) Checking docs...

@RoboMWM
Copy link
Member

RoboMWM commented Mar 29, 2024

hmm, I guess to help provide context, when did pumpkin seeds drop, and I guess what benefit do pumpkins bring... can they be used for food?

(Yes I haven't played much Minecraft, my knowledge is still mostly from beta lol.)

@Nanpure
Copy link
Author

Nanpure commented Mar 29, 2024

The pumpkin seeds pop out as an entity that can be picked up when shears are right clicked on a placed pumpkin block (this is how you make a jack o lantern). Our server uses Aurelium/Aura Skills, which gets EXP from farming. Farming melons and pumpkins was deemed too easy, because players could create huge towers of them and break the blocks easily without having to replant them for exp. Therefore we decided to make seeds scarce.

But because pumpkin seeds can still be obtained, people are now making pumpkin farms instead of melon/pumpkin farms to grind. We want to nerf all melon and pumpkin seeds so that they can only be obtained from chests.

@RoboMWM
Copy link
Member

RoboMWM commented Apr 6, 2024

Cool, thanks for the info.

As per the readme, I only give feature request support to patrons. Since this seems like a relatively simple and easy feature, and it kind of exists between the bug/feature line, I may get around to implementing this feature soon.

@RoboMWM
Copy link
Member

RoboMWM commented Jul 17, 2024

Looked into this, spending a bit of time to setup an environment to test. There appears to be no specific event fired for shearing a block. EventDebug reveals an interact event with the pumpkin, and ItemSpawnEvent for the seeds that drop. Will take more work than expected to get this to work right, effort equivalent to adding a new feature, estimating at medium complexity. As such I'll work on this for patrons only, but anyone else is free to PR this or provide assistance or info.

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

No branches or pull requests

2 participants