Posted November 1, 20186 yr Hi! I'm new on this forum. So that's the problem: I'm trying to add custom items to vanilla mob drops. I've got 2 problems: 1) I can't insert metadata in the dropped item: @SubscribeEvent public void parrotDrop(LootTableLoadEvent event) { if (event.getName().equals(LootTableList.ENTITIES_PARROT)) { final LootPool main = event.getTable().getPool("main"); if (main != null) { main.addEntry(new LootEntryItem(Items.DYE.getMetaData(15), 3, 1, new LootFunction[0], new LootCondition[] {new RandomChance(1F)}, "minecraft:bone")); } } } I want when killed a parrot it drops bone meal. 2)Certain animals drops can't be edited? This code ^ works with all the animals but with certains animals it not work for example silver fish. THIS WORK @SubscribeEvent public void spiderDrop(LootTableLoadEvent event) { if (event.getName().equals(LootTableList.ENTITIES_SPIDER)) { final LootPool main = event.getTable().getPool("main"); if (main != null) { main.addEntry(new LootEntryItem(Items.BONE, 3, 0, new LootFunction[0], new LootCondition[] {new RandomChance(1F)}, "minecraft:bone")); } } } THIS NOT WORK @SubscribeEvent public void ocelotDrop(LootTableLoadEvent event) { if (event.getName().equals(LootTableList.ENTITIES_OCELOT)) { final LootPool main = event.getTable().getPool("main"); if (main != null) { main.addEntry(new LootEntryItem(Items.DYE, 3, 1, new LootFunction[0], new LootCondition[] {new RandomChance(1F)}, "minecraft:bone")); } } } Thanks for help!! Sorry for my bad english Edited November 1, 20186 yr by nov4e
November 1, 20186 yr Author @dieseben07 56 minutes ago, diesieben07 said: Your code does not even compile. Item#getMetaData returns an int, you are trying to pass it where anItem is needed. Also this method does not do what you think it does. It only makes sense for items being placed as blocks. You need to pass in the item here and then pass in a loot function (class SetMetadata) to set the metadata. Also, set the entry name properly. It has nothing to do with the item. So I have to make a new int? 58 minutes ago, diesieben07 said: Ocelots do not have any loot, so they do not have a default loot pool. You need to add it first. How I can create a loot pool? Do you know a tutorial for 1.12.2? I'm new to modding.
November 2, 20186 yr Author @dieseben07 final LootPool main = event.getTable().addPool(main); This is the correct method? If I type LootTable.add, add is not a field. 10 hours ago, diesieben07 said: No. Did you even read what I said? I did not understand what you mean. EDIT: after lauching the game it crashes with this code. Edited November 2, 20186 yr by nov4e
November 2, 20186 yr Author @dieseben07 LootPool ocelot = new LootPool(); What argumets I have to put inside the ()??
November 2, 20186 yr What arguments does it ask for? About Me Spoiler My Discord - Cadiboo#8887 My Website - Cadiboo.github.io My Mods - Cadiboo.github.io/projects My Tutorials - Cadiboo.github.io/tutorials Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support. When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible. Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)
November 2, 20186 yr Author @Cadiboo Now I've put : VillagerDrops villager = new VillagerDrops(new LootEntry[] {}, new LootCondition[] {}, rollsIn, bonusRollsIn, "villager"); VillagerDrops Class: package com.nov4e.nex.misc.loots; import net.minecraft.world.storage.loot.LootEntry; import net.minecraft.world.storage.loot.LootPool; import net.minecraft.world.storage.loot.LootTable; import net.minecraft.world.storage.loot.RandomValueRange; import net.minecraft.world.storage.loot.conditions.LootCondition; public class VillagerDrops extends LootPool { public VillagerDrops(LootEntry[] lootEntriesIn, LootCondition[] poolConditionsIn, RandomValueRange rollsIn, RandomValueRange bonusRollsIn, String name) { super(lootEntriesIn, poolConditionsIn, rollsIn, bonusRollsIn, name); LootTable.EMPTY_LOOT_TABLE.addPool(this); } }
November 2, 20186 yr Author It requires: new LootEntry[] {entry}, <conditions>, <rolls>, <bonusRolls>, <name> What I have to put in entry, rools, conditions, bonusRools, name?? I've maked a json file: { "pools": [ { "name": "main", "rolls": 1, "entries": [ { "type": "entity", "name": "minecraft:silverfish", "weight": 60 }, { "type": "entity", "name": "minecraft:bat", "weight": 60 }, { "type": "entity", "name": "minecraft:villager", "weight": 60 }, { "type": "entity", "name": "minecraft:ocelor", "weight": 60 } ] } ] } Edited November 2, 20186 yr by nov4e
November 2, 20186 yr Author 1 hour ago, diesieben07 said: For what purpose did you make that JSON file? @diesieben07I maked a json file because i followed this documentation. 1 hour ago, diesieben07 said: I am not sure what is unclear about the parameters. Please ask exact questions if you have any. I don't understand what is "rools" and "bonusrools". Also what are the entries? The entities? new LootEntry[] {EntityOcelot, EntityVillager} ???? What I have to put in the name??? Edited November 2, 20186 yr by nov4e
November 2, 20186 yr Author 2 minutes ago, diesieben07 said: So, you read that documentation. Why are you still subscribing to the event? Granted, I did not know there was a way to do this from JSON, but since you knew that, why are you using the event still? I subscribed the event because also i followed that topic. Tell me what is the best way. Edited November 2, 20186 yr by nov4e
November 2, 20186 yr Author 3 minutes ago, diesieben07 said: Why are you mixing different instructions? I followed that. But 2 hours ago I tried to make a pool with the forge documentation. Edited November 2, 20186 yr by nov4e
November 2, 20186 yr Author 1 minute ago, diesieben07 said: Okay. Read the forge documentation. It has a specific section on how to modify vanilla loot. Do not follow random threads on this forum. Ok but the thread was correct. I only need to create a pool, as you told me, for add drops to mobs that not have a loot pool.
November 2, 20186 yr Author 2 minutes ago, diesieben07 said: Okay. So. If you know all that, then again: Why did you make a JSON file? Because I misunderstood the wiki. For now is all good but 16 minutes ago, nov4e said: don't understand what is "rools" and "bonusrools". Also what are the entries? The entities? new LootEntry[] {EntityOcelot, EntityVillager} ???? What I have to put in the name???
November 2, 20186 yr Author 1 minute ago, diesieben07 said: Read the article on the minecraft wiki, as suggested by the documentation to learn what rolls and bonus rolls are. The name is also explained in the documentation. LootPool customPool = new LootPool(new LootEntry[] {EntityVillager, EntityOcelot}, new LootCondition[] {}, 2, 1, "customPool"); ??
November 2, 20186 yr Author 5 minutes ago, diesieben07 said: That is not even valid Java. You need to actually pass in LootEntry instances. You already created those in your original code... Seriously, you need to learn basic Java. Can you make an example?? I can't understand what I have to put in the args!!!
November 2, 20186 yr Author RandomValueRange rolls = new RandomValueRange(1); LootPool customPool = new LootPool( new LootEntry[] { }, new LootCondition[] { new RandomChance(1F) }, rolls, null, "customPool"); This maybe work??
November 2, 20186 yr Author 1 hour ago, diesieben07 said: Your array of loot entries is empty. Why are you not doing what you did in your original code? Why are you suddenly passing that RandomChance condition? Moreover, this condition will not do anything, it's always true. Why are you passing null for bonusRolls? LootPool customPool = new LootPool ( new LootEntry[] { new LootEntryItem(Items.BONE, 3, 0, new LootFunction[0], new LootCondition[] {new RandomChance(1F)}, "minecraft:bone") }, new LootCondition[] { }, rolls, rollsBonus, "customPool" ); Thanks for all your help @diesieben07
November 2, 20186 yr Author Again not work.... @SubscribeEvent public void editVanillaLoots(LootTableLoadEvent event) { RandomValueRange rolls = new RandomValueRange(1); RandomValueRange rollsBonus = new RandomValueRange(2); LootPool customPool = new LootPool ( new LootEntry[] { new LootEntryItem(Items.BONE, 3, 0, new LootFunction[0], new LootCondition[] {new RandomChance(1F)}, "minecraft:bone") }, new LootCondition[] { }, rolls, rollsBonus, "customPool" ); LootTable.EMPTY_LOOT_TABLE.addPool(customPool); if (event.getName().equals(LootTableList.ENTITIES_VILLAGER)) { final LootPool main = event.getTable().getPool("customPool"); if (main != null) { main.addEntry(new LootEntryItem(Items.BONE, 3, 0, new LootFunction[0], new LootCondition[] {new RandomChance(1F)}, "minecraft:bone")); } } }
November 2, 20186 yr If you can use JSON, use it. About Me Spoiler My Discord - Cadiboo#8887 My Website - Cadiboo.github.io My Mods - Cadiboo.github.io/projects My Tutorials - Cadiboo.github.io/tutorials Versions below 1.14.4 are no longer supported on this forum. Use the latest version to receive support. When asking support remember to include all relevant log files (logs are found in .minecraft/logs/), code if applicable and screenshots if possible. Only download mods from trusted sites like CurseForge (minecraft.curseforge.com). A list of bad sites can be found here, with more information available at stopmodreposts.org Edit your own signature at www.minecraftforge.net/forum/settings/signature/ (Make sure to check its compatibility with the Dark Theme)
November 3, 20186 yr Author 19 hours ago, Cadiboo said: If you can use JSON, use it. ok, I have to register it??
November 8, 20186 yr Author Ok, I changed method. Now I'm using LivingDeathEvent : int quantity = 2; @SubscribeEvent public void onMobKilled(LivingDeathEvent e) { //e.getEntityLiving().dropItem(Items.BONE, quantity); e.getEntity().dropItem(Items.BONE, quantity); } But when I kill a mob it works but also drops not collectible items. ScreenShot Link: https://ibb.co/dqiVDV Edited November 8, 20186 yr by nov4e
November 8, 20186 yr 14 minutes ago, nov4e said: Ok, I changed method. Now I'm using LivingDeathEvent : No one said to do that, so why are you doing that VANILLA MINECRAFT CLASSES ARE THE BEST RESOURCES WHEN MODDING I will be posting 1.15.2 modding tutorials on this channel. If you want to be notified of it do the normal YouTube stuff like subscribing, ect. Forge and vanilla BlockState generator.
November 8, 20186 yr Author 31 minutes ago, Animefan8888 said: No one said to do that, so why are you doing that Because I followed a tutorial on a website. I think also is easier than loottables. It works well but i need to know why when I kill a mob drops 2 bones and it also drops 2 bones not collectible. 46 minutes ago, nov4e said: Edited November 8, 20186 yr by nov4e
November 8, 20186 yr 3 hours ago, nov4e said: Because I followed a tutorial on a website. I think also is easier than loottables. Just learn how to use loottables, it's not that hard. Not only are data-driven methods prefferred over code-based ones, a lot of mods rely on loottables, like JEI Mob drops addon. You also won't have any issues as a side bonus.
November 8, 20186 yr Author 15 minutes ago, V0idWa1k3r said: Just learn how to use loottables, it's not that hard. Not only are data-driven methods prefferred over code-based ones, a lot of mods rely on loottables, like JEI Mob drops addon. You also won't have any issues as a side bonus. okok but i found that way and it works. I only need to fix this bug and i got mob drops. main.addEntry(new LootEntryItem(Items.BONE, 3, 0, new LootFunction[0], new LootCondition[] {new RandomChance(1F)}, "minecraft:bone")); I tried with loot tables but if I add a new item the mob drops the item or the porkchop(in this case the pig). I want to drop porkchops and bones.
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.