Saplings don't grow

Question

Saplings don't grow

kurisubrooks opened this issue 8 years ago · 10 comments

kurisubrooks commented 8 years ago

In the latest build, saplings don't appear to grow, not normally nor with bone meal.
This causes quite a problem for both survival and creative players using this plugin.

Barteks2x · Answer 1 · 2016-09-30T13:42:44.000Z

Are you sure there is enough space for them to grow? Are there any errors in console?

If it's neighter, then something bust have changed in bukkit.

kurisubrooks · Answer 2 · 2016-09-30T14:19:06.000Z

Yes, there's plenty of room. I even tried on a block in the middle of the sky. Even after 2 stacks of bonemeal there was no growth at all.

It was not working in the 1.8 build also, so i'm guessing it's definitely on Bukkit's end.

My friend managed to get them working again with a modified build he made from your repo, he told me to tell you

in the place where you check if it's LEAVES or AIR, add an OR clause for SAPLING

He would submit a PR, but he couldn't get Spruce saplings to work, and his repo is too dirty

cc: @justin97530

Barteks2x · Answer 3 · 2016-09-30T15:06:25.000Z

That was something I didn't expect. Beta version just checked if the block is transparent but that also makes trees replace glass and a few other transparent blocks which is not good and I tried to avoid it. I guess I tried too hard.

kurisubrooks · Answer 4 · 2016-09-30T15:11:51.000Z

👏 Your efforts are greatly appreciated regardless of any bugs that may exist

Barteks2x · Answer 5 · 2016-09-30T16:46:52.000Z

I will debug and fix it tomorrow.

kurisubrooks · Answer 6 · 2016-10-09T06:36:00.000Z

Any progress? 😄

Barteks2x · Answer 7 · 2016-10-09T08:37:19.000Z

Still working on it, I had to change quite a bit of code to get trees to generate the right way.

I also found something interesting: beta 1.7.3 doesn't need to check for sapling in tree generation code because when attempting to generate tree, the sapling is temporarily removed: https://github.com/Bukkit/mc-dev/blob/1a792ed860ebe2c6d4c40c52f3bc7b9e0789ca23/net/minecraft/server/BlockSapling.java#L37-L53

kurisubrooks · Answer 8 · 2016-10-09T08:47:31.000Z

Huh. Interesting logic on Bukkit's behalf. Glad to see you're working on
the bug fix though 🙂

On Sun., 9 Oct. 2016, 7:37 pm Bartosz Skrzypczak, [email protected]
wrote:

Still working on it, I has to change quite a bit of code to get trees to
generate the right way.

I also found something interesting: beta 1.7.3 doesn't need to check for
sapling in tree generation code because when attempting to generate tree,
the sapling is temporarily removed:
https://github.com/Bukkit/mc-dev/blob/1a792ed860ebe2c6d4c40c52f3bc7b9e0789ca23/net/minecraft/server/BlockSapling.java#L37-L53

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#13 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGAAC1bCqmEDJGM4mhS71zaF9aJUNKBOks5qyKe_gaJpZM4KKgvJ
.

kurisubrooks · Answer 9 · 2016-10-09T09:36:27.000Z

I'll be able to test in about an hour. Thanks so much!

On Sun., 9 Oct. 2016, 8:35 pm Bartosz Skrzypczak, [email protected]
wrote:

It should work now, also the bukkit API to generate trees is now used
correctly (or at least I hope so).

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#13 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGAAC8S9DytsdZCH4ZzOi5LwTRv0p85Iks5qyLVVgaJpZM4KKgvJ
.

Barteks2x · Answer 10 · 2016-10-09T09:35:16.000Z

It should work now, also the bukkit API to generate trees is now used correctly (or at least I hope so).

Share to