Flowerpot griefing

[x1p]

It is possible to steal flowers out of the pot inside claims.

Griefprevention 16.11.3
WorldGuard v7.0.0-SNAPSHOT;1749-a6261b9

[x1p]

Exists too in GP 16.11.4

[RoboMWM]

This is a client glitch as I'm sure I already fixed this a while ago. If you relog into the server you'll find that the flower pot hasn't been griefed.

Unless of course spigot 1.13 changed that stuff, which they may have. So I'm going to need all the ticket info that you deleted.

[x1p]

No it's not a client glitch. The item is indeed in the inventory and is droppable.

[x1p]

https://www.youtube.com/watch?v=AFinGEj4M1Y

[mirolm]

It is indeed possible to do that. I just tried it on my server with latest GP.

@RoboMWM Note that it is impossible to break the pot, but possible to pickup the sappling / flower from the pot. You cannot return it back tho, because it says you need administrator / claim owner rights.

It does not prevent you to get the item from the pot, but blocks you from returning it back. Maybe a check on pickup from pot is missing.

This server is running CraftBukkit version git-Spigot-f6a273b-7a2f486 (MC: 1.13.1)
[GriefPrevention] Loading GriefPrevention v16.11.4.dev

[x1p]

thank you for confirming

[RoboMWM]

Ok, thank you mirolm for providing reproduction and details. x1p you can't just delete the issue template, it exists for a reason.

[112RG]

The bug seems to be happening because when right-clicking the flower in the pot, the material isn't FLOWER_POT. The material is now POTTED_{wood type}_SAPPLING

[RoboMWM]

Thanks. I'm guessing a Tag could be used here? Or better yet an event (but I doubt there is such)?

[112RG]

I have a PR adding all the materials. Is there a better way todo it than that?

[RoboMWM]

Probably using this tag. https://hub.spigotmc.org/javadocs/bukkit/org/bukkit/Tag.html#FLOWER_POTS