From 3f874e62dc3eece0117508ec5545540d8b479863 Mon Sep 17 00:00:00 2001 From: jotty Date: Wed, 24 Dec 2025 23:40:41 +0100 Subject: [PATCH] getting cotton seed from short grass works --- src/main/java/de/jottyfan/minecraft/Quickly.java | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/main/java/de/jottyfan/minecraft/Quickly.java b/src/main/java/de/jottyfan/minecraft/Quickly.java index 57f1925..a877312 100644 --- a/src/main/java/de/jottyfan/minecraft/Quickly.java +++ b/src/main/java/de/jottyfan/minecraft/Quickly.java @@ -10,7 +10,6 @@ import net.fabricmc.api.ModInitializer; import net.fabricmc.fabric.api.loot.v3.LootTableEvents; import net.fabricmc.fabric.api.registry.CompostingChanceRegistry; import net.minecraft.resources.Identifier; -import net.minecraft.world.level.block.Blocks; import net.minecraft.world.level.storage.loot.LootPool; import net.minecraft.world.level.storage.loot.entries.LootItem; import net.minecraft.world.level.storage.loot.predicates.LootItemRandomChanceCondition; @@ -33,15 +32,16 @@ public class Quickly implements ModInitializer { private void registerLootTableChanges() { LootTableEvents.MODIFY.register((key, tableBuilder, source, registries) -> { if (source.isBuiltin()) { - Identifier grass = Identifier.fromNamespaceAndPath("minecraft", "blocks/grass"); - Quickly.LOGGER.info("key {} ?=? grass {}", key.identifier(), grass); - if (key.identifier().equals(grass)) { + // TODO: maybe, better harvest cotton from dry grass instead? + Identifier shortGrass = Identifier.fromNamespaceAndPath("minecraft", "blocks/short_grass"); + Identifier tallGrass = Identifier.fromNamespaceAndPath("minecraft", "blocks/tall_grass"); + if (key.identifier().equals(shortGrass)) { LootPool.Builder poolBuilder = LootPool.lootPool() .setRolls(ConstantValue.exactly(1f)) .when(LootItemRandomChanceCondition.randomChance(0.1f)) .add(LootItem.lootTableItem(QuicklyItems.COTTONSEED)); tableBuilder.withPool(poolBuilder); - } else if (Blocks.TALL_GRASS.getLootTable().equals(key)) { + } else if (key.identifier().equals(tallGrass)) { // for the canola loot table block later } }