Animefan8888

You don't need this at all. Yes, but now you need to give it a value by initializing it.

Shift clicking has a different method entirely. Container#transferStackInSlot.

This is a method. Not a field. Create a field, an instance, a variable. Then use its constructor to initialize it.

Obsidian_shard_block.json does not equal obsidian_shard_block.json

You don't. I never said anything like that. When I said this I was talking about RecipeManager#getRecipe(the method you are having a problem with).

Use its constructor...

You need to do the first part I said. You need to have a RecipeWrapper field in your TileEntity to use in getRecipe because it requires an IInventory.

Look for the class called IRenderFactory then look at where it is used and you'll find RenderingRegistry.registerEntityRenderingHandler or something close to that.

Post all of your current code. Preferably as a github repo. But on here works too.

No no no no no. Why are you changing that mods code? Do you have permission to do that?

Take a look at the class and you'll find out. I didnt mean change the whole thing just the thing that was bolded...

EntityPlayerMP#sendContainerToPlayer after every click you change the default behavior of.

Show a screenshot of your package explorer.

This is a terrible modid. You have 64 characters use them. The reason it is terrible is what if another mod maker uses the same modid because they abbreviated some other words. Your mods are now incompatible. Does this file exist at assets/modid/blockstates/obsidian_shard_block.json in your IDE? Did you refresh your IDE's package explorer?

Well I cant think of any reason why you wouldn't register it, it's also needed in the Container class so I'm not sure that's entirely accurate. That all looks good. Though unless you are sending specific data to the client like a TileEntity's BlockPos(which you are not) you dont need to worry about the PacketBuffer Parameter. You can also have one constructor and deserialize the data in the lambda.

Well obviously it isn't. They can't be fine and produce these errors... Why haven't you posted your model, blockstate, and console? It's impossible to debug your problem without them....

What version are you using? Is it limited to only your block placed or is the item like that too? What do your blockstate and model jsons look like? Is your console spitting out an error(s)?

Extend ContainerScreen instead. This will solve the E button problem. PlayerEntity#openContainer assuming your container is the one open on the client side. Show what you are doing now. They don't. The only reason to have a PacketBuffer parameter in your constructor is to pass extra data over from the server. I dont think yours needs that. I'm not at my IDE at the moment so I cant give you definitive answers with code, but I think I can point you in the proper direction, where to look, ect.

Your method is also static it needs to not be.

No it does not create an instance. It provides the mod developer with the instance of there @Mod class. This won't work for reasons I've stated above you might as well make it a final field and do new ClassName()

I dont believe so. I believe there is a method to provide you with an ID that hasn't been used in 1.12 and in 1.14 I dont think it uses integer ids for anything other than saving to disk.

Yes 100% more. It provides an instance of the @Mod class it refers to.

The better question is why is your dimension ID configurable?

What type of crash? Could it be that you never assign a value to FogEventHandler.instance?

There are several ways here is the documentation on events.