Weird bug on server

[Thor597]

I have a block which changes the time, but when I choose one of the options and on client it changes the time, on server. Either it doesnt do anything, or it gives me an NPE on the if() lines, code:

 

public void actionPerformed(GuiButton guibutton)
{
	if(guibutton.id == 1)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(7000);
		}

	}

	if(guibutton.id == 2)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(17000);
		}

	}

	if(guibutton.id == 3)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(0);
		}

	}

	if(guibutton.id == 4)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(13000);
		}

	}

}    

[SanAndreaP]

  On 10/8/2012 at 6:15 PM, Thor597 said:

I have a block which changes the time, but when I choose one of the options and on client it changes the time, on server. Either it doesnt do anything, or it gives me an NPE on the if() lines, code:

 

public void actionPerformed(GuiButton guibutton)
{
	if(guibutton.id == 1)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(7000);
		}

	}

	if(guibutton.id == 2)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(17000);
		}

	}

	if(guibutton.id == 3)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(0);
		}

	}

	if(guibutton.id == 4)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(13000);
		}

	}

}    

 

Because the GUI classes are only called client-side. If you call MinecraftServer.getServer() in a client, it gives you only the InternalServer. If you are connected to a dedi server, it gives you a null object (unless you call this method on the dedi server).

I would recommend to send packets for this one.

[Thor597]

  On 10/8/2012 at 7:21 PM, SanAndreasP said:

  Quote

I have a block which changes the time, but when I choose one of the options and on client it changes the time, on server. Either it doesnt do anything, or it gives me an NPE on the if() lines, code:

 

public void actionPerformed(GuiButton guibutton)
{
	if(guibutton.id == 1)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(7000);
		}

	}

	if(guibutton.id == 2)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(17000);
		}

	}

	if(guibutton.id == 3)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(0);
		}

	}

	if(guibutton.id == 4)
	{
		for (int var3 = 0; var3 < MinecraftServer.getServer().theWorldServer.length; ++var3)
		{
			MinecraftServer.getServer().theWorldServer[var3].setTime(13000);
		}

	}

}    

 

Because the GUI classes are only called client-side. If you call MinecraftServer.getServer() in a client, it gives you only the InternalServer. If you are connected to a dedi server, it gives you a null object (unless you call this method on the dedi server).

I would recommend to send packets for this one.

 

oh gawd packets