search

ObsidianBox / Obsidian

Obsidian is the implementation of Magma that extends Forge to deliver server-side content on demand.
http://obsidianbox.org
MIT License
12 stars 3 forks source link

Phase out @SideOnly #15

Closed Zidane closed 10 years ago

Zidane commented 10 years ago

@modwizcode brought up a good point that @SideOnly is a legacy construct in FML/Forge and could be removed in the future. We need to begin the transition to doing SidedProxies.

Lunaphied commented 10 years ago

Also we can use world.isLocal (I think that's the method/parameter) when we have a world context.

Zidane commented 10 years ago

Its world.isRemote, someone changed it to world.isLocal on Lex and he flipped.

Lunaphied commented 10 years ago

Oh ha funny.

Lunaphied commented 10 years ago

For now this issue isn't really solvable, unless we want double the classes but only one method in those extra classes. This isn't an issue with an alternative that meets all demands. For now, lets just make sure we know what exactly we're doing when we use @SideOnly

Zidane commented 10 years ago

I agree, closed