Jump to content

Recommended Posts

Posted

Hello!

I've been playing around with the idea of creating a "plugin like" chest based menu system, implemented on the server side ONLY, so no code runs on the client. I have gotten so far as to create a generic container and have the player open it, but several issues remain that I can't solve on my own, #1 Is setting the contents of said generic container, so the player sees items and can interact with the menu, and then #2 is actually interacting with the container, how do I detect when a player picks up an item from the container (and cancel that, so they can't take items from it) and then do some other action, like opening another more different container or running some logic or things of that nature.

This is what is running currently in my mod:
 

        INamedContainerProvider provider = new INamedContainerProvider() {
            @Override
            public ITextComponent getDisplayName() {
                return new TextComponent() {
                    @Override
                    public String getUnformattedComponentText() {
                        return "name";
                    }

                    @Override
                    public ITextComponent shallowCopy() {
                        return null;
                    }
                };
            }

            @Nullable
            @Override
            public Container createMenu(int p_createMenu_1_, PlayerInventory p_createMenu_2_, PlayerEntity p_createMenu_3_) {
                Container c = new Container(ContainerType.GENERIC_9X2, 1) {
                    @Override
                    public boolean canInteractWith(PlayerEntity playerIn) {
                        return true;
                    }
                };

				// Both of these line straight up crash the game when they are ran
                //c.getSlot(0).inventory.setInventorySlotContents(0, new ItemStack(Blocks.DIRT));
                //c.getInventory().set(0, new ItemStack(Blocks.DIRT)); 

                return c;
            }
        };

        event.getPlayer().openContainer(provider);


Thanks for the help!

Posted

I see, why does it have to be a ChestContainer, the inventory still opens like this, I just cant put items in from my code, but wouldnt that be fixed if were to put an inventory behind it and use that? ChestContainer has a constructor that takes in a PlayerInventory an IInventory and the number of rows(?). Do I need some sort of wrapper around my IItemHandler to pass it into that IInventory param?

Posted

Okay I'm lost now, I want to use IInventory because that is what ChestContainer uses, but how do I create one for it, if I understand correctly, right now you are saying I use IInventory of the chest and manage it with an IItemHandler, provided by InvWrapper right?
But how do I provide the chest container with an IInventory in the first place?
Can you give an example on how I create the ChestContainer from start to finish? Or I need to implement a custom IInventory?

Posted

This is what I am trying currently, but this doesnt work, it fails to compile.

		IInventory inv = new Inventory(18);
                
                InvWrapper wrapper = new InvWrapper(inv);
                wrapper.insertItem(0, new ItemStack(Blocks.DIRT), false);

                ChestContainer c = new ChestContainer(ContainerType.GENERIC_9X2, event.getPlayer().inventory, inv, 2);
'ChestContainer(net.minecraft.inventory.container.ContainerType<?>, int, net.minecraft.entity.player.PlayerInventory, int)' has private access in 'net.minecraft.inventory.container.ChestContainer'

 

Posted

Ah okay I got it, I missed the `id` param. It works now for the first part.
And as for detecting and canceling interaction, how does one do that?

Posted

So, I will populate the slots of the opened "chest" with items that I dont want to be in game items, rather option in a menu, so when a player tries to take one of them I can run some logic because they clicked an "option", instead of the taking the item I want to run a function to do something.

Example, the player opens a virtual chest they see their level by the size of the green wool stack, if they try to take the stack, instead of taking 13 green wool, they would get taken to another "chest" with more detailed information about their character.

Or a shop interface, where them clicking an item would buy it, put it into their inventory and remove some balance from their account.

So I need a mechanism to detect them trying to take an item, and to know what item it is, and one mechanism to prevent them from actually taking the item.

I hope it makes sense

Posted

So essentially I create a new class, extend from Container maybe and start implementing things, and maybe go off the implementation of ChestContainer.
Will this approach not break the server side only criteria? Does the client not need to know about the custom container?

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

    • 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.
    • Necro'ing that thread to ask if you found a solution ? I'm encountering the same crash on loading the world. I created the world in Creative to test my MP, went into survival to test combat, died, crashed on respawn and since then crash on loading the world. Deactivating Oculus isn't fixing it either, and I don't have Optifine (Twilight forest is incompatible)
  • Topics

×
×
  • Create New...

Important Information

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