Hey so I've encountered another problem with my GUI button. I've successfully fixed my issue earlier where I couldn't get my button to remove the poison effect on the player. I looked into packets and got it to work. Now, when the player clicks on the button, it removes the poison effect but I want the button to be disabled temporarily. After some time passes the button re-enables and the player can use it again.

 

This is what I've got so far but it just disables the button and doesn't re-enable it unless I close the GUI and re-open it.

public void actionPerformed(GuiButton button)
{

	switch(button.id)
	{
		case 0: 
		button.enabled = true;
		MineGrades.packetPipeline.sendToServer(new RemovePoisonPacket(Potion.poison.id));
		long time = this.mc.theWorld.getWorldInfo().getWorldTime();
		for(long i = time; i < (time + 1000); i++)
		{
			button.enabled = false;
		}

	}
}

 

This whole setup is probably terrible so feel free to correct me and thanks for any help in advance.

Don't use for loops to count time.

Ok so I've changed it so that when you click the button it removes the poison effect, deletes the item used to open the GUI, and closes the GUI. The only problem now is that it deletes the item but it just reappears if you click in the slot it was in. I imagine I have to use a packet to send to the server and tell it to delete the item?

Yeah my plan is to use a packet. Right now though this is all I could come up with and it doesn't work.

 

public class DeleteItemPacket extends AbstractPacket{



public DeleteItemPacket()
{

}

@Override
public void encodeInto(ChannelHandlerContext ctx, ByteBuf buffer) 
{

}

@Override
public void decodeInto(ChannelHandlerContext ctx, ByteBuf buffer)
{

}

@Override
public void handleClientSide(EntityPlayer player)
{

}

@Override
public void handlerServerSide(EntityPlayer player)
{
	player.destroyCurrentEquippedItem();
}

}

Ok I looked into it and it seems easy enough. Ill code it in and see if I can get it working, thanks for input and help everyone.

Ok so I looked into Simple Network Wrapper and I thought I had it sorted out but when it try to remove a poison potion effect via GUI button it just kicks me out of the world.

 

Here is my code for the Simple Network Wrapper. Can someone please tell me what I'm doing wrong.

 

Main Mod Class

network.registerMessage(RemovePoison.Handler.class, RemovePoison.class, 0, Side.SERVER);

 

GUI class

public void actionPerformed(GuiButton button)
{

	switch(button.id)
	{
		case 0: 
		MineGrades.network.sendToServer(new RemovePoison(4));


	}
}

 

Simple Network Wrapper Remove Poison Class

package NetworkWrapper;

import net.minecraft.potion.Potion;
import io.netty.buffer.ByteBuf;
import cpw.mods.fml.common.network.simpleimpl.IMessage;
import cpw.mods.fml.common.network.simpleimpl.IMessageHandler;
import cpw.mods.fml.common.network.simpleimpl.MessageContext;

public class RemovePoison implements IMessage
{

private static int id;

public RemovePoison()
{

}

public RemovePoison(int id)
{
	this.id = id;
}

@Override
public void fromBytes(ByteBuf buf) 
{
	buf.writeByte(id);
}

@Override
public void toBytes(ByteBuf buf) 
{
	id = buf.readByte();
}


public static class Handler implements IMessageHandler<RemovePoison, IMessage> 
{
    
    @Override
    public IMessage onMessage(RemovePoison poison, MessageContext ctx) 
    {
        ctx.getServerHandler().playerEntity.removePotionEffect(id);
        	return null;
    	}
}

}

Oops the static was me just trying something out, ill remove it. Also, how would I go about having the server remove the potion effect?