Jump to content

GotoLink

Members
  • Posts

    2012
  • Joined

  • Last visited

  • Days Won

    1

GotoLink last won the day on January 20 2019

GotoLink had the most liked content!

Converted

  • Gender
    Undisclosed

Recent Profile Visitors

The recent visitors block is disabled and is not being shown to other users.

GotoLink's Achievements

World Shaper

World Shaper (7/8)

382

Reputation

  1. Simply make sure you are only MinecraftServer#registerTickables on server side. Besides, you have @EventHandler annotations in your proxy class. They should be restricted to your @Mod class. And please don't use EntityRegistry.findGlobalUniqueEntityId() You do not need a global entity id.
  2. Are you trying to call it on client side ? You shouldn't. Also fix this: @Override public void update() { List<EntityPlayer> players = Minecraft.getMinecraft().theWorld.playerEntities;
  3. Not if you rely on the "generated" item model. That depends on the texture itself, where each pixel is made into a cube. Which would be pain to make into fixed json model. But if your item model is already fixed, then yes, you can do the same architecture of json files as the blocks do.
  4. It does this because the constructor is missing. Explicit constructors need to be defined by the child class. See Java constructor rules.
  5. The question is about 1.8, which only got minor versions following it. Thus minor code change which shouldn't affect any mod. (Method/Field renaming is MCP noise, which ForgeGradle covers up) Though if some code changes prevent the mod from working, you could always surround the version specific code with MinecraftForge.MC_VERSION check.
  6. I would recommend [1.8,) As version range, since 1.8.0 isn't rigorously 1.8.
  7. CanHarvest is whether the block drop itself. All blocks are breakable by any item, except bedrock. If you return false here, the default "innefficient speed" value is assumed. GetStrVsBlock is the speed of breaking. Return 0 would make the item not break.
  8. You are not registering the item instance you used in the recipe. Having the same name isn't going to magically link the instances. Here the items used in recipes are of Item type, while the items added in registry are of ItemModSword type. Solve the inconsistency.
  9. If you have client values, read and used by the clients... They should stay on client side. Never into the world data, which is server property, including item NBT. Write the client stuff on the client config file. Now if the server use values, the defaults should be from the server configuration. Because there may be no client connected. If the client need to know about those values, the server would send them through packet at appropriate time. Let say there is a value such that both sides care about. Server config contains "X" as default, sends it over to client. ->X | X (both sides) Client config applies a "client modifier=A" on it: ->X | X*A (use it for its own purposes) User makes a modification "+" in GUI. ->X | X*A+A (temporary client side evaluated expression, based on client config) Client send "+" packet, (contains GUI id and other stuff for identifying source) server receives "+" packet and applies its change. -> X + 1 | X*A+A Server may need to send back an update (could be same packet as first time), just in case... -> X + 1 | X + 1 Which client config still apply client modifier on before use -> X + 1 | (X+1)*A etc. In summary: "A" is never sent over the network. It is kept by the client side data (in config file, probably). "X" is sent by the server to the client. It is kept by the server data (in world save, probably), send by packet to client. "+" is an operation known to both sides (or only server, in which case update packet is mandatory everytime). It is not data, and is thus not saved by either. Client to server packet will ask server to perform "+" operation on its value, based on mutual agreement of the code needed in the packet to identify as "+" operation, such as channel name. "*" is a client side operation. Server doesn't care nor understand.
  10. If you have 3 block instances, you don't need to have this PART property registered at any point. #getMetaFromState need to reflect #getStateFromMeta Note you use EnumRailDirection states. There are 10 of those, and you probably don't want all of them. Checkout those you don't need, like in BlockRailPowered.
  11. Please understand that the event is posted for each element type, with different rendering state. If you cancel the event with element type == ALL, NOTHING is rendered. You can cancel any number of them, but you need to choose ONE element type to render ONCE. See GuiInGameForge to check the rendering states.
  12. That is an interface. What the hell were you thinking ? Just stop using #All methods,and # All fields config. You should specify what you want to change, each field, each method; and understand what is going to happen.
  13. Problem here: IBlockState disblockstate = world.getBlockState(blockpos); world.setBlockState(blockpos, disblockstate); Those lines change nothing.
  14. Hum, that forge jar is called "custom.jar". I find it suspicious. Though this error can be due to mods not being thread safe in their packets handling.
  15. -Break statement missing -Actual generation lines missing -Swapped dimension id between End/Nether
×
×
  • Create New...

Important Information

By using this site, you agree to our Terms of Use.