Jump to content

Recommended Posts

Posted

Hello,

I just wanted to know if it exists a way of generating programmatically a loot when you have the loot table.

In my case I've got the loot table from a resouce location but I don't understand how I need to use the LootContext ect ...

Here is my code : 

ResourceLocation loc = new ResourceLocation(resourcePath);

LootTables ltManager = this.getLevel().getServer().getLootTables();
LootTable lt = ltManager.get(loc);
LootContextParamSet ltps = lt.getParamSet();
               
LootContext.Builder builder = new LootContext.Builder((ServerLevel) this.level);
List<ItemStack> generated =  lt.getRandomItems(builder.create(ltps));

 

Posted (edited)

TLDR, in order to generate loot from a loot table you need to pass in all of the relevant context for that kind of loot table, because "I just need cow loot" requires knowing if the cow was on fire. And if the player was holding a sword with the looting enchantment.

Etc etc etc.

So you need to specify where the loot is being dropped, who cause it to drop, what is being dropped, and so on. All of that it put into the loot context builder.

Look for existing usages of LootContextParamSets

(Oh and be sure to pass the result through ForgeHooks.modifyLoot)

Edited by Draco18s
  • Like 1

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

Hello,

Thank you for your anwsers, I finally succeeded. I have just one more question : 

Why in the code below, the parameter TOOL is not considered ? Indeed, when I try to generate loot from a cow, it generates beef whereas I added a parameter TOOL with a diamond sword with fire aspect, so It should generate Steak.

ResourceLocation loc = new ResourceLocation(resourcePath);

LootTables ltManager = this.getLevel().getServer().getLootTables();
LootTable lt = ltManager.get(loc);

ItemStack sword = new ItemStack(Items.DIAMOND_SWORD);
sword.enchant(Enchantments.MOB_LOOTING, 3);
sword.enchant(Enchantments.FIRE_ASPECT, 1);


Vec3 position = new Vec3(getBlockPos().getX(), getBlockPos().getY(), getBlockPos().getZ());
LootContext.Builder builder = (new LootContext.Builder((ServerLevel) this.level))
                        .withParameter(LootContextParams.TOOL, sword)
                        .withParameter(LootContextParams.ORIGIN, position);

LootContext ctx = builder.create(LootContextParamSets.FISHING);
List<ItemStack> generated = lt.getRandomItems(ctx);

 

Posted
19 minutes ago, Sweetmimike said:

Hello,

Thank you for your anwsers, I finally succeeded. I have just one more question : 

Why in the code below, the parameter TOOL is not considered ? Indeed, when I try to generate loot from a cow, it generates beef whereas I added a parameter TOOL with a diamond sword with fire aspect, so It should generate Steak.

If you look at the loot table for cows, you'll see that it checks whether the entity is on fire to determine whether to drop steak; not whether the tool has Fire Aspect.

Please don't PM me to ask for help. Asking your question in a public thread preserves it for people who are having the same problem in the future.

Posted
11 minutes ago, Choonster said:

If you look at the loot table for cows, you'll see that it checks whether the entity is on fire to determine whether to drop steak; not whether the tool has Fire Aspect.

Yes indeed :)

And for the looting enchant ? It seems to not be considered

Posted
11 hours ago, diesieben07 said:

No, LootTable#getRandomItems already does that.

👍

Haven't traced all the calls lately, as I haven't been doing any active modding.

33 minutes ago, Sweetmimike said:

And for the looting enchant ? It seems to not be considered

I believe that is a separate context entry.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted
1 hour ago, Draco18s said:

I believe that is a separate context entry.

What do you mean about that ? Do you think it is possible to apply a looting effect to the LootContext ?

Posted

Yes, but like I said, it's been a while since I looked at the relevant code.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

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.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.