package sobbaches.tuto;

import net.minecraftforge.fml.common.Mod;

@Mod(Tuto.MODID)
public class Tuto {
  public static final String MODID = "tuto";
}

package sobbaches.tuto;

import static sobbaches.tuto.Items.items;

import net.minecraft.item.Item;
import net.minecraft.item.ItemGroup;
import net.minecraft.item.Rarity;
import net.minecraftforge.common.MinecraftForge;
import net.minecraftforge.event.RegistryEvent;
import net.minecraftforge.eventbus.api.SubscribeEvent;
import net.minecraftforge.fml.common.Mod;
import net.minecraftforge.registries.IForgeRegistry;

@Mod.EventBusSubscriber(modid = Tuto.MODID, bus = Mod.EventBusSubscriber.Bus.MOD)
public class TutoEventSubscriber {
  @SubscribeEvent
  public static void onRegisterItem(RegistryEvent.Register<Item> event) {
    items.forEach(item -> item.getCreativeTabs().add(ItemGroup.COMBAT));
    items.forEach(event.getRegistry()::registerAll);
  }
}

package sobbaches.tuto;

import net.minecraft.item.Item;
import net.minecraft.item.ItemGroup;
import net.minecraft.item.Rarity;
import net.minecraft.util.ResourceLocation;
import java.util.ArrayList;
import java.util.List;

public class Items {
  public static final List<Item> items;

  static {
    items = new ArrayList<>();
    items.add(new Item(new Item.Properties().rarity(Rarity.EPIC))
        .setRegistryName(new ResourceLocation(Tuto.MODID, "test")));
  }
}

 

  @SubscribeEvent
  public static void onRegisterItem(RegistryEvent.Register<Item> event) {
    Item item = new Item(new Item.Properties().rarity(Rarity.EPIC));
    item.getCreativeTabs().add(ItemGroup.COMBAT);
    item.setRegistryName("test");
    event.getRegistry().registerAll(item);
  }

I just created item in registry method, but i've got another error.

Problem is solved. The main reason is ItemGroup and I don't have any thoughts about why does it occur the problem

OK how should I do it? 

2 minutes ago, diesieben07 said:

Item.Properties#group.

Thanks

1 hour ago, no namezzzz said:

items.forEach(event.getRegistry()::registerAll);

event.getRegistry().registerAll(items)

 

There is no need to for-each to pass each one individually to a method that accepts a list.

Edited December 9, 20195 yr by Draco18s