search

PocketMine / PocketMine-MP

Legacy PocketMine-MP repository. Head to https://github.com/pmmp for up to date software.
https://www.pocketmine.net/
GNU Lesser General Public License v3.0
1.25k stars 661 forks source link

Can't create Double Plant with bonemeal #3857

Open inxomnyaa opened 8 years ago

inxomnyaa commented 8 years ago

Fixed already in ImagicalMine: https://github.com/ImagicalCorp/ImagicalMine/commit/c3146346019e48cf8f9ed563ee3890dc213c2945 https://github.com/ImagicalCorp/ImagicalMine/commit/f137aaadad11c7ce424d7c4528d94bbc4b4a37be

May this be added or is the code styling wrong? If its correct, i'll PR.

jellyfishcoder commented 8 years ago

It works for me. Im using RasPi2 PHP7 (I compiled the PHP7 cause there isnt one on pocketmine.net for install wizard 😑) for MC:PE v0.14.0. Im in creative. Try to look at the top of the plants when you do it, it only works for me when i click on top not sides of plant. Also, flower pots don't work @thebigsmileXD

inxomnyaa commented 8 years ago

@jingz2007 The system doesn't matter, same core. And have you reloaded your world? After that they should be small again ("Tallgrass")

Flower pots work at several other third party builds, as example ClearSky, but there the code style is incorrect. @beito123 Made an awesome plugin for flowerpots, skulls, cauldrons + more

PEMapModder commented 8 years ago

@thebigsmileXD @jingz2007 it seems that flower pots are not relevant to this issue. Please don't use the issue tracker as a forum.

As for the changes mentioned, it is more appropriate if you create the pull request first so that it can be reviewed directly.

jellyfishcoder commented 8 years ago

@PEMapModder Ok, sorry. Also, I just tried reloading the world and the tall grass is gone. @thebigsmileXD

inxomnyaa commented 8 years ago

@PEMapModder did you know that without your comment this issue would be 3 comments smaller?