Jump to content

Recommended Posts

Posted (edited)

I am interested in adding a unique item, which could be found with some probability in one of several ways (nether chest, fishing, mob drop). I can save a 'global' if item exists, but having trouble with checking against it to see if a new one should be created.

 

I can trap the LootTableLoadEvent and add the item to say, CHESTS_NETHER_BRIDGE or where ever I wish, with what ever probability I want.  However, I only want the item to occur once.  So, like if it occurs, I'd want to remove it from the loot tables. However, after the LootTableLoadEvent, they appear to be 'frozen' (unable to alter items in table). Similarly, I know how to do it from a entity death event and add item there if possible, but doesn't help me with chests or fishing.  The PlayerContainerEvent.Open potentially could be used, but I don't think there is a way to prevent the player from creating their own chest and finding the item (only want in MC generated chests).

 

I found Draco18s Artifacts on GitHub which looks sweet, but trying to puruse the code... looks like it uses chest hooks and a single item which then behaves like many other items...not quite what I want and I don't see the compatibility functions (hooks,etc) in 1.11.2. So, I am not sure that is the way I should go for this in 1.11.2. 

 

Further, I want the item to be found, so I don't want to simply turn down the probability in the loot tables, just want one, and then turn the probability off.

 

Trying to trap the created event on the item itself (to possible destroy itself if item already exists globally) wasn't getting anywhere. onCreated looks like a crafting event, not when MC creates it.

 

Any thoughts on best course to pursue?

Thanks,

Tony

 

Edited by aw_wolfe
solved
Posted

Loot is done by Loot Tables now.

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

diesieben07:

 

Thanks a bunch.  I had the global data save working, but was adding a new LootEntryItem to the pool tables in LootTableLoadEvent instead of creating a new condition and LootEntry.

@SubscribeEvent
    public void onLootTableLoad(LootTableLoadEvent event)
    {
		if (event.getName().equals(LootTableList.CHESTS_NETHER_BRIDGE))
		{
			
			 LootPool main = event.getTable().getPool("main");
			 lootpools.add(main);	//ArrayList I was storing to try and change/alter table later in game
			 
	            if (main != null)
	            {
	                main.addEntry(new LootEntryItem(ModItems.staffBlazeKing, 10, 0, new LootFunction[0], new LootCondition[0], "uniqueitemsmod:staff_blaze_king")); //works, but wrong, can't change the condition or remove chance (how I was doing it)
	            }
		}
		else if(event.getName().equals(LootTableList.GAMEPLAY_FISHING)){
			LootPool main = event.getTable().getPool("main");
			
            if (main != null)
            {
            	//as per diesieben07, much better, can tweek the condition to zero (or whatever) at any time after
            	main.addEntry(new UniqueEntry(10,1,new LootCondition[] {new UniqueLoot()},"uniqueitemsmod:uniqueloot")); 
            }

 

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



  • Recently Browsing

    • No registered users viewing this page.
  • Posts

    • inicie un mundo donde instale Croptopia y Farmer's Delight, entonces instale el addon Croptopia Delight pero no funciona. es la version 1.18.2
    • Hello all. I'm currently grappling with the updateShape method in a custom class extending Block.  My code currently looks like this: The conditionals in CheckState are there to switch blockstate properties, which is working fine, as it functions correctly every time in getStateForPlacement.  The problem I'm running into is that when I update a state, the blocks seem to call CheckState with the position of the block which was changed updated last.  If I build a wall I can see the same change propagate across. My question thus is this: is updateShape sending its return to the neighbouring block?  Is each block not independently executing the updateShape method, thus inserting its own current position?  The first statement appears to be true, and the second false (each block is not independently executing the method). I have tried to fix this by saving the block's own position to a variable myPos at inception, and then feeding this in as CheckState(myPos) but this causes a worse outcome, where all blocks take the update of the first modified block, rather than just their neighbour.  This raises more questions than it answers, obviously: how is a different instance's variable propagating here?  I also tried changing it so that CheckState did not take a BlockPos, but had myPos built into the body - same problem. I have previously looked at neighbourUpdate and onNeighbourUpdate, but could not find a way to get this to work at all.  One post on here about updatePostPlacement and other methods has proven itself long superceded.  All other sources on the net seem to be out of date. Many thanks in advance for any help you might offer me, it's been several days now of trying to get this work and several weeks of generally trying to get round this roadblock.  - Sandermall
    • sorry, I might be stupid, but how do I open it? because the only options I have are too X out, copy it, which doesn't work and send crash report, which doesn't show it to me, also, sorry for taking so long.
    • Can you reproduce this with version 55.0.21? A whole lot of plant placement issues were just fixed in this PR.
  • Topics

×
×
  • Create New...

Important Information

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