Jump to content

Recommended Posts

Posted (edited)

I have no idea how to make it work, I tried playing with the codes for a couples of hours now. Here is my codes: 

package com.SupremeMarshal.MoreToolsTierMod.util.handlers;

import com.SupremeMarshal.MoreToolsTierMod.init.ModBlocks;
import com.SupremeMarshal.MoreToolsTierMod.init.ModItems;

import net.minecraft.item.ItemStack;
import net.minecraftforge.fml.common.registry.GameRegistry;

public class RecipeHandler
{
	public static void registerSmelting()
	{
		GameRegistry.addSmelting(new ItemStack(ModBlocks.RUNE_ORECOAL_BLOCK ,1 ,0), new ItemStack(ModItems.RUNE_INGOT, 1), 1.0f); 
	}

}

Or is it possible to create a Smelting recipe with JSON file? I hope I get some help!

Also, do I need to do a Register thing for this class?
(I'm still new in coding Java)

Edited by supreme marshal
Posted (edited)
36 minutes ago, larsgerrits said:

Are you calling RecipeHandler.registerSmelting?

On RegistryHandler? I have but I'm not sure I did it correctly.

 

package com.SupremeMarshal.MoreToolsTierMod.util.handlers;

import com.SupremeMarshal.MoreToolsTierMod.init.ModBlocks;
import com.SupremeMarshal.MoreToolsTierMod.init.ModItems;
import com.SupremeMarshal.MoreToolsTierMod.util.IHasModel;

import net.minecraft.block.Block;
import net.minecraft.item.Item;
import net.minecraft.item.crafting.CraftingManager;
import net.minecraft.world.storage.loot.functions.Smelt;
import net.minecraftforge.client.event.ModelRegistryEvent;
import net.minecraftforge.event.RegistryEvent;
import net.minecraftforge.fml.common.Mod.EventBusSubscriber;
import net.minecraftforge.fml.common.Mod.EventHandler;
import net.minecraftforge.fml.common.event.FMLInitializationEvent;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;

@EventBusSubscriber
public class RegistryHandler 
{
	
	//Calling RecipeHandler.registerSmelting
	public static void Client()
	{
		RecipeHandler.registerSmelting();
	}
	
	
	@SubscribeEvent
	public static void onItemRegister(RegistryEvent.Register<Item> event)
	{
		event.getRegistry().registerAll(ModItems.ITEMS.toArray(new Item[0]));
	}
	
	
	@SubscribeEvent
	public static void onBlockRegister(RegistryEvent.Register<Block> event)
	{
		event.getRegistry().registerAll(ModBlocks.BLOCKS.toArray(new Block[0]));
	}

I think that is what you mean by calling RecipeHandler.registerSmelting? 

Edited by supreme marshal
Posted

Why are you registering your recipes only on the client?

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted
9 minutes ago, supreme marshal said:

I get some error log when I test the game:

 

[17:35:23] [main/ERROR] [FML]: Parsing error loading recipe ms:rune_ingot_smelting
com.google.gson.JsonSyntaxException: Unknown recipe type: minecraft:smelting

Smelting does not use the JSON system

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted
51 minutes ago, supreme marshal said:

I only play on client version anyways. So can you help me for my problem?

Even for single player you need to register it on the server thread.

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted (edited)
	public static void Client()
	{
		RecipeHandler.registerSmelting();
	}
	
	public static void Common()
	{
		RecipeHandler.registerSmelting();
	}

Like this? I feel like it should be a bit more complicated than this. Is there something I need to implement in my proxy class? ClientProxy and CommonProxy.

Edited by supreme marshal
Posted

This is my whole RegistryHandler class:

 

package com.SupremeMarshal.MoreToolsTierMod.util.handlers;

import com.SupremeMarshal.MoreToolsTierMod.init.ModBlocks;
import com.SupremeMarshal.MoreToolsTierMod.init.ModItems;
import com.SupremeMarshal.MoreToolsTierMod.util.IHasModel;

import net.minecraft.block.Block;
import net.minecraft.item.Item;
import net.minecraft.item.crafting.CraftingManager;
import net.minecraft.world.storage.loot.functions.Smelt;
import net.minecraftforge.client.event.ModelRegistryEvent;
import net.minecraftforge.event.RegistryEvent;
import net.minecraftforge.fml.common.Mod.EventBusSubscriber;
import net.minecraftforge.fml.common.Mod.EventHandler;
import net.minecraftforge.fml.common.event.FMLInitializationEvent;
import net.minecraftforge.fml.common.eventhandler.SubscribeEvent;

@EventBusSubscriber
public class RegistryHandler 
{
	
	
	public static void Client()
	{
	RecipeHandler.registerSmelting();
	}
	
	public static void Common()
	{
	RecipeHandler.registerSmelting();
	}

	
	@SubscribeEvent
	public static void onItemRegister(RegistryEvent.Register<Item> event)
	{
		event.getRegistry().registerAll(ModItems.ITEMS.toArray(new Item[0]));
	}
	
	
	@SubscribeEvent
	public static void onBlockRegister(RegistryEvent.Register<Block> event)
	{
		event.getRegistry().registerAll(ModBlocks.BLOCKS.toArray(new Block[0]));
	}
	

	@SubscribeEvent
	public static void onModelRegister(ModelRegistryEvent event)
	{
		for(Item item : ModItems.ITEMS)	
		{
			if(item instanceof IHasModel)
			{
				((IHasModel)item).registerModels();
			}
		}
		
		for(Block block : ModBlocks.BLOCKS)	
		{
			if(block instanceof IHasModel)
			{
				((IHasModel)block).registerModels();
			}
		}
	}
}

 

Posted

You realize "common" is both sides, right?

Apparently I'm a complete and utter jerk and come to this forum just like to make fun of people, be confrontational, and make your personal life miserable.  If you think this is the case, JUST REPORT ME.  Otherwise you're just going to get reported when you reply to my posts and point it out, because odds are, I was trying to be nice.

 

Exception: If you do not understand Java, I WILL NOT HELP YOU and your thread will get locked.

 

DO NOT PM ME WITH PROBLEMS. No help will be given.

Posted

The way I register my smelting recipes is I have this in my RegistryHandler

	public static void initRegistries()
	{
		SmeltingRecipes.init();
	}

for you instead of "SmeltingRecipes.init();" it would be "RecipeHandler.registerSmelting();"

and then in my main mod class I call

RegistryHandler.initRegistries();

in my initialization event

Posted
19 minutes ago, Hazmatik said:

The way I register my smelting recipes is I have this in my RegistryHandler


	public static void initRegistries()
	{
		SmeltingRecipes.init();
	}

for you instead of "SmeltingRecipes.init();" it would be "RecipeHandler.registerSmelting();"

and then in my main mod class I call


RegistryHandler.initRegistries();

in my initialization event

OMFG THANK YOU IT WORKED!

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Announcements



×
×
  • Create New...

Important Information

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