I'm having a problem with my event

It's spamming like crazy

 

package com.neosup.VampZ.handler;

import net.minecraftforge.event.entity.EntityJoinWorldEvent;
import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import cpw.mods.fml.relauncher.Side;

public class Playerjoin {

@SubscribeEvent
public void Playerjoins(EntityJoinWorldEvent event){
	if(event.world.isRemote){	
		event.world.setWorldTime(1600000);	
		System.out.println("Time has changed");
	}		
}
}

Plus, do you want to do time changing on server instead? (world.isRemote() = client...)

Now it doesn'twork anymore

package com.neosup.VampZ.handler;

import cpw.mods.fml.common.eventhandler.SubscribeEvent;
import cpw.mods.fml.common.gameevent.PlayerEvent.PlayerLoggedInEvent;

public class Playerjoin {

@SubscribeEvent
public void Playerjoins(PlayerLoggedInEvent event){
	if(event.player.worldObj.isRemote && event.player.worldObj.isDaytime()){
			event.player.worldObj.setWorldTime(160000);
			System.out.println("Time has changed");				
	}
}		
}

Plus, do you want to do time changing on server instead? (world.isRemote() = client...)

It seems logic to have it only for a singleplayer, because on a server it would be annoying(Every time somebody joins it becomes night)

a) What elix said.

b) Did you register to the FML EventBus? PlayerLoggedInEvent is an FML event.

a) No

b) Yep

 

I fixed it with removing the IsRemote and i added getTotalWorldTime

It seems logic to have it only for a singleplayer

Singleplayer is exactly the same as a multiplayer server. The server is just running on the same machine.

Thanks for the info

Plus, do you want to do time changing on server instead? (world.isRemote() = client...)

It seems logic to have it only for a singleplayer, because on a server it would be annoying(Every time somebody joins it becomes night)

 

Checking whether you're in single player is different than checking if world is remote. Even in single player there is a server running. I think there is some other way to detect if you're in single player (and you still want to do the time change on the server in that case).

Plus, do you want to do time changing on server instead? (world.isRemote() = client...)

It seems logic to have it only for a singleplayer, because on a server it would be annoying(Every time somebody joins it becomes night)

 

Checking whether you're in single player is different than checking if world is remote. Even in single player there is a server running. I think there is some other way to detect if you're in single player (and you still want to do the time change on the server in that case).

I wanted that if you spawn in the world it would be night, but not that everytime you join it would become night. I fixed it with using getTotalWorldTime. So the problem with single/multiplayer is fixed