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

[FEATURE] Is it a house or a tree? #10

Open
voigtman opened this issue Jun 17, 2021 · 2 comments
Open

[FEATURE] Is it a house or a tree? #10

voigtman opened this issue Jun 17, 2021 · 2 comments
Labels
Enhancement A feature or suggestion Priority: Optional

Comments

@voigtman
Copy link

voigtman commented Jun 17, 2021

Is your feature request related to a problem? Please describe.

Kinda,
The plugin does not differentiate between a house (generated or built) and a tree. This means with one mis-swing of an axe an entire building may come crumbling down.

Describe the solution you'd like

The plugin could look at connected blocks to determine if it is a house or a tree by looking to see if a connected block is a form of cobble, plank, brick, door, glass, etc
If these blocks are detected an axe would only break a single block, regardless if the player is sneaking or not.
This feature could be enable or disabled with a simple true/false in the config.

Detect Buildings = 'true'

AND/OR

The plugin could possibly do this task in another manner, by looking to see if there are leaves attached to the wood column. Going this route might also open the possibility to allow the leaves to break with the rest of the tree as well. There are older datapacks that do this. When a tree is felled they then drop the saplings, sticks, apples, etc as drops along with the wood.

OR

The solution could be more complex and allow the server admin to be responsible for inputing the types of blocks to look for, additionally this list could be flagged as either a whitelist or blacklist allowing the configuration to be created one way or the other.

Describe alternatives you've considered

Additional context
Add any other context or screenshots about the feature request here.

@voigtman voigtman added Enhancement A feature or suggestion Priority: Optional labels Jun 17, 2021
@Lauriichan
Copy link
Member

So detecting if smth is a tree or not is even more performance heavy than the plugin is rn
There are already ppl who are complaining about performance so there is not really a way of adding smth like that into ST atm.

The best solution to prevent this atm is to install for example LogBlock or CoreProtect, those plugin log every block a player breaked / placed. This information is used by SmoothTimber, if a block is placed by a player then the plugin will only break the first block and then stop.

@Lauriichan
Copy link
Member

The SmoothTimber rewrite might fix this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement A feature or suggestion Priority: Optional
Projects
None yet
Development

No branches or pull requests

2 participants