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

Adding PhysicalCraftingRecipe class #15

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Adding PhysicalCraftingRecipe class #15

wants to merge 1 commit into from

Conversation

rmichela
Copy link
Contributor

I created a PhysicalCraftingRecipe class for triggering actions when the user lays out blocks on the ground. It supports rectangular patterns, works in any horizontal orientation, and can identify item data values. The API is very similar to the Bukkit crafting recipe class. I've been using it in GiantTrees for a while now and it's pretty quick. I thought this would be a good addition to deshtutils.

@desht
Copy link
Owner

desht commented Jul 1, 2014

Looks interesting, but what sort of applications do you have for it?

@rmichela
Copy link
Contributor Author

rmichela commented Jul 1, 2014

I use it to grow giant trees by surrounding an emerald block by two rings of saplings. This class detects the pattern on the ground.

@rmichela
Copy link
Contributor Author

rmichela commented Jul 3, 2014

Anything I need to do to make PhysicalCraftingRecipe merge worthy?

@desht
Copy link
Owner

desht commented Jul 3, 2014

Not at the moment, no - I just haven't had a proper chance to look it through yet... RL commitments plus other plugin dev stuff going on. Hopefully will get a chance early next week...

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.

2 participants