Skip to content
This repository has been archived by the owner on Nov 9, 2017. It is now read-only.

Add flower pot state #48

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add flower pot state #48

wants to merge 2 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Oct 23, 2014

In this pull request the interface for a flower pot state is added, most importantly.

Modifications
Create a new extension of BlockState for flower pots with a getter and setter for the flower pot's contents. This is required for the post-1.7 way of handling flower pots, which is done with tile entities. To indicate this new option as preferable to the old option of handling flower pot contents, I deprecated the old manner. Why exactly is the new option preferable, you ask? Tile entities support all flowers, while the block data way doesn't; and changing the block data of flower pots that use tile entities, could result in unknown behaviour.

Result
A present and future proof way to handle flower pot contents.


Edited by turt2live

Related links:

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant