[1.7.10] @SideOnly(Side.CLIENT) Not use it anymore?

[American2050 11]

I can't remember where but I think I read that @SideOnly(Side.CLIENT) shouldn't be used anymore.

 

My question is, wherever I have things like this:

	@Override
@SideOnly(Side.CLIENT)
public void registerIcons(IIconRegister iconRegister)
{
itemIcon = iconRegister.registerIcon(this.getUnlocalizedName().substring(this.getUnlocalizedName().indexOf(".") + 1));
}

 

I should modify them like

 

	if(!world.isRemote)
	{
		itemIcon = iconRegister.registerIcon(this.getUnlocalizedName().substring(this.getUnlocalizedName().indexOf(".") + 1));
	}

 

But what happens when I don't have "world" available to use, like in this case?

 

 

[Failender 119]

http://www.minecraftforge.net/forum/index.php/topic,22764.0.html

 

EDIT: thats the general answer for it. but there are things that are already client only, like icons, so keep using things like vanilla did

[American2050 11]

Thanks going to read it

Hopefully that clarifies my confusion

[WitherBoss2000 2]

I am pretty sure you cant have 2 annotations with the @SideOnly, you  dont need the overide. Maybe put the override below the @sideonly for some strange reason. This happens to me too. Trust me, it still works in 1.8

 

[Choonster 1651]

I am pretty sure you cant have 2 annotations with the @SideOnly, you  dont need the overide. Maybe put the override below the @sideonly for some strange reason. This happens to me too. Trust me, it still works in 1.8

You can use as many annotations as you want. You should always use

@Override

when overriding a method and

@SideOnly

when overriding a method with

@SideOnly

.