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

Added Tree tool to grow various trees #26

Merged
merged 2 commits into from
Jan 16, 2013
Merged

Added Tree tool to grow various trees #26

merged 2 commits into from
Jan 16, 2013

Conversation

mileippert
Copy link
Contributor

Item for the tool is a Sapling.
Left-Click to cycle through the tree types and Right-Click to place the tree.

Implements: http://dev.bukkit.org/server-mods/toolbelt/?comment=19

@buildhive
Copy link

Peter Olson » ToolBelt #13 SUCCESS
This pull request looks good
(what's this?)

@Peter200lx
Copy link
Owner

Thanks for this! I've currently pulled this into my local branch and am cleaning up the code style so it matches the rest of the repository.

I also realized that before I release a new version this will need to support WorldGuard. My current best guess for doing so involves learning BlockChangeDelegate from:
http://jd.bukkit.org/doxygen/dd/daa/interfaceorg_1_1bukkit_1_1World.html#a2b3cc146bf319e9cb87f11b17f1c3d9d

@Peter200lx Peter200lx merged commit 53f4af3 into Peter200lx:master Jan 16, 2013
@Peter200lx
Copy link
Owner

Sorry if I played with your code too much, I just got started on tweaking and got carried away. Thanks much for the idea and base code.

I'm tracking the need for WorldGuard support with issue #27

A couple of other ideas that I'd like to implement:

  • Error message if generateTree failed. (Done)
  • Support for administrator defined possible tree types. (Added with 994da88)
  • Investigate if sapling is best default for new tool. (Changed to Gold Hoe in 6c8013a)
  • Support WorldGuard. (Added with 61d67a3)

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.

None yet

3 participants