Remove the ability to use PropertyDelegateHolders on blocks

Blocks can't have global state, and creating property delegates based on block states
isn't a good idea in my opinion. Property changes would involve state changes, and
menus sync the properties automatically so it could cause a lot of pointless
client-side updates.

Closes #56.

1 files changed, 1 insertions, 6 deletions

diff --git a/src/main/java/io/github/cottonmc/cotton/gui/CottonInventoryController.java b/src/main/java/io/github/cottonmc/cotton/gui/CottonInventoryController.java
index cbefb7a..f5153e7 100644
--- a/src/main/java/io/github/cottonmc/cotton/gui/CottonInventoryController.java
+++ b/src/main/java/io/github/cottonmc/cotton/gui/CottonInventoryController.java
@@ -405,18 +405,13 @@ public class CottonInventoryController extends ScreenHandler implements GuiDescr
 	 *
 	 * <p>If no property delegate is found, returns an empty property delegate with no properties.
 	 *
-	 * <p>Searches for blocks and block entities implementing {@link PropertyDelegateHolder}.
+	 * <p>Searches for block entities implementing {@link PropertyDelegateHolder}.
 	 *
 	 * @param ctx the context
 	 * @return the found property delegate
 	 */
 	public static PropertyDelegate getBlockPropertyDelegate(ScreenHandlerContext ctx) {
 		return ctx.run((world, pos) -> {
-			BlockState state = world.getBlockState(pos);
-			Block block = state.getBlock();
-			if (block instanceof PropertyDelegateHolder) {
-				return ((PropertyDelegateHolder)block).getPropertyDelegate();
-			}
 			BlockEntity be = world.getBlockEntity(pos);
 			if (be!=null && be instanceof PropertyDelegateHolder) {
 				return ((PropertyDelegateHolder)be).getPropertyDelegate();