Differences

This shows you the differences between two versions of the page.

--- tutorial:adding_to_loot_tables [2019/08/12 23:06] – fixed parenthesis yanis48
+++ tutorial:adding_to_loot_tables [2021/06/10 13:00] – update to current yarn naming juuz
@@ Line 9: / Line 9: @@
 ===== Listening to loot table loading =====
-Fabric API has an event that’s fired when loot tables are loaded, ''%%LootTableLoadingCallback%%''. You can register an event listener for it in your initializer. Let’s also check that the current loot table is ''%%minecraft:blocks/coal_ore%%''.
+Fabric API has an event that’s fired when loot tables are loaded, ''%%LootTableLoadingCallback%%''. You can register an event listener for it in your initializer. Let’s also check that the current loot table is the coal ore loot table.
 <code java>
-LootTableLoadingCallback.EVENT.register((resourceManager, lootManager, id, supplier, setter) -> {
+// No magic constants!
-    if ("minecraft:blocks/coal_ore".equals(id.toString())) {
+private static final Identifier COAL_ORE_LOOT_TABLE_ID = Blocks.COAL_ORE.getLootTableId();
+// Actual code
+LootTableLoadingCallback.EVENT.register((resourceManager, lootManager, id, table, setter) -> {
+    if (COAL_ORE_LOOT_TABLE_ID.equals(id)) {
         // Our code will go here
     }
@@ Line 26: / Line 31: @@
 <code java>
-LootTableLoadingCallback.EVENT.register((resourceManager, lootManager, id, supplier, setter) -> {
+LootTableLoadingCallback.EVENT.register((resourceManager, lootManager, id, table, setter) -> {
-    if ("minecraft:blocks/coal_ore".equals(id.toString())) {
+    if (COAL_ORE_LOOT_TABLE_ID.equals(id)) {
         FabricLootPoolBuilder poolBuilder = FabricLootPoolBuilder.builder()
-                .withRolls(ConstantLootTableRange.create(1)); // Same as "rolls": 1 in the loot table json
+                .rolls(ConstantLootTableRange.create(1)); // Same as "rolls": 1 in the loot table json
-        supplier.withPool(poolBuilder);
+        table.pool(poolBuilder);
     }
 });
@@ Line 38: / Line 43: @@
 <code java>
-LootTableLoadingCallback.EVENT.register((resourceManager, lootManager, id, supplier, setter) -> {
+LootTableLoadingCallback.EVENT.register((resourceManager, lootManager, id, table, setter) -> {
-    if ("minecraft:blocks/coal_ore".equals(id.toString())) {
+    if (COAL_ORE_LOOT_TABLE_ID.equals(id)) {
         FabricLootPoolBuilder poolBuilder = FabricLootPoolBuilder.builder()
-                .withRolls(ConstantLootTableRange.create(1))
+                .rolls(ConstantLootTableRange.create(1))
-                .withEntry(ItemEntry.builder(Items.EGG));
+                .with(ItemEntry.builder(Items.EGG));
-        supplier.withPool(poolBuilder);
+        table.pool(poolBuilder);
     }
 });