Fixed up NEI handler

src/main/java/com/cricketcraft/chisel/carving/Carving.java

@@ -1,12 +1,11 @@
 package com.cricketcraft.chisel.carving;
 
 import java.util.ArrayList;
-import java.util.Collection;
 import java.util.Collections;
 import java.util.HashMap;
+import java.util.List;
 
 import net.minecraft.block.Block;
-import net.minecraft.init.Blocks;
 import net.minecraft.item.Item;
 import net.minecraft.item.ItemStack;
 import net.minecraftforge.oredict.OreDictionary;
@@ -121,7 +120,7 @@ public class Carving {
 		return null;
 	}
 
-	CarvingGroup getGroup(String name) {
+	public CarvingGroup getGroup(String name) {
 		CarvingGroup group = carvingGroupsByName.get(name);
 		if (group == null) {
 			group = new CarvingGroup(name);
@@ -178,7 +177,12 @@ public class Carving {
 		return blockGroup.sound;
 	}
 
-	public Collection<CarvingGroup> getGroups() {
-		return carvingGroupsByVariation.values();
+	public List<String> getSortedGroupNames() {
+		List<String> names = new ArrayList<String>();
+		for (String s : carvingGroupsByName.keySet()) {
+			names.add(s);
+		}
+		Collections.sort(names);
+		return names;
 	}
 }

src/main/java/com/cricketcraft/chisel/carving/CarvingGroup.java

@@ -17,8 +17,17 @@ public class CarvingGroup {
 	String oreName;
 
 	List<CarvingVariation> variations = new ArrayList<CarvingVariation>();
-	
+
 	public List<CarvingVariation> getVariations() {
 		return ImmutableList.copyOf(variations);
 	}
-}
+
+	public String getName() {
+		return name;
+	}
+
+	public String getOreName() {
+		return oreName;
+	}
+
+}

src/main/java/com/cricketcraft/chisel/carving/CarvingVariation.java

@@ -1,6 +1,7 @@
 package com.cricketcraft.chisel.carving;
 
 import net.minecraft.block.Block;
+import net.minecraft.item.ItemStack;
 
 public class CarvingVariation implements Comparable<CarvingVariation> {
 
@@ -20,4 +21,9 @@ public class CarvingVariation implements Comparable<CarvingVariation> {
 	public Block block;
 	public int meta;
 	public int damage;
+
+	public ItemStack getStack() {
+		return new ItemStack(block, 1, damage);
+	}
+
 }

src/main/java/com/cricketcraft/chisel/compat/nei/NEIChiselConfig.java

@@ -0,0 +1,27 @@
+package com.cricketcraft.chisel.compat.nei;
+
+import com.cricketcraft.chisel.Chisel;
+
+import codechicken.nei.api.API;
+import codechicken.nei.api.IConfigureNEI;
+
+public class NEIChiselConfig implements IConfigureNEI {
+
+	@Override
+	public String getName() {
+		return Chisel.MOD_NAME;
+	}
+
+	@Override
+	public String getVersion() {
+		return Chisel.VERSION;
+	}
+
+	@Override
+	public void loadConfig() {
+		RecipeHandlerChisel handler = new RecipeHandlerChisel();
+		API.registerRecipeHandler(handler);
+		API.registerUsageHandler(handler);
+	}
+
+}

src/main/java/com/cricketcraft/chisel/compat/nei/RecipeHandlerChisel.java

@@ -1,105 +1,186 @@
 package com.cricketcraft.chisel.compat.nei;
 
+import java.awt.Point;
 import java.awt.Rectangle;
+import java.util.ArrayList;
 import java.util.List;
 
-import net.minecraft.client.gui.inventory.GuiContainer;
+import net.minecraft.init.Items;
+import net.minecraft.item.Item;
 import net.minecraft.item.ItemStack;
+import net.minecraftforge.oredict.OreDictionary;
 
 import org.lwjgl.opengl.GL11;
 
 import codechicken.lib.gui.GuiDraw;
+import codechicken.nei.NEIServerUtils;
 import codechicken.nei.PositionedStack;
 import codechicken.nei.recipe.TemplateRecipeHandler;
 
 import com.cricketcraft.chisel.carving.Carving;
 import com.cricketcraft.chisel.carving.CarvingGroup;
 import com.cricketcraft.chisel.carving.CarvingVariation;
-import com.cricketcraft.chisel.client.gui.GuiChisel;
-import com.google.common.collect.Lists;
 
 public class RecipeHandlerChisel extends TemplateRecipeHandler {
 
+	private static final int SLOTS_PER_PAGE = 45;
+
+	public class CachedChiselRecipe extends CachedRecipe {
+
+		private List<PositionedStack> input = new ArrayList<PositionedStack>();
+		private List<PositionedStack> outputs = new ArrayList<PositionedStack>();
+		public Point focus;
+
+		public CachedChiselRecipe(List<ItemStack> variations, ItemStack base, ItemStack focus) {
+			PositionedStack pStack = new PositionedStack(base != null ? base : variations, 74, 4);
+			pStack.setMaxSize(1);
+			this.input.add(pStack);
+
+			int row = 0;
+			int col = 0;
+			for (ItemStack v : variations) {
+				this.outputs.add(new PositionedStack(v, 3 + 18 * col, 37 + 18 * row));
+
+				if (focus != null && NEIServerUtils.areStacksSameTypeCrafting(focus, v)) {
+					this.focus = new Point(2 + 18 * col, 36 + 18 * row);
+				}
+
+				col++;
+				if (col > 8) {
+					col = 0;
+					row++;
+				}
+			}
+		}
+
+		public CachedChiselRecipe(List<ItemStack> variations) {
+			this(variations, null, null);
+		}
+
+		@Override
+		public List<PositionedStack> getIngredients() {
+			return this.getCycledIngredients(cycleticks / 20, this.input);
+		}
+
+		@Override
+		public List<PositionedStack> getOtherStacks() {
+			return this.outputs;
+		}
+
+		@Override
+		public PositionedStack getResult() {
+			return null;
+		}
+
+	}
+
 	@Override
 	public String getRecipeName() {
-		return "Chiseling";
+		return "Chisel";
 	}
 
 	@Override
 	public String getGuiTexture() {
-		return "Chisel:textures/chisel2Gui.png";
-	}
-
-	public PositionedStack getResult() {
-		return null;
-	}
-
-	@Override
-	public Class<? extends GuiContainer> getGuiClass() {
-		return GuiChisel.class;
-	}
-
-	@Override
-	public String getOverlayIdentifier() {
-		return "EnderIOSagMill";
-	}
-
-	@Override
-	public void loadTransferRects() {
-		transferRects.add(new TemplateRecipeHandler.RecipeTransferRect(new Rectangle(149, 32, 16, 16), "ChiselRecipes", new Object[0]));
-	}
-
-	public void loadCraftingRecipes(String outputId, Object... results) {
-		// is this right? idk
-		for (CarvingGroup g : Carving.chisel.getGroups()) {
-			arecipes.add(new CachedChiselRecipe(g));
-		}
-	}
-	
-	@Override
-	public void loadCraftingRecipes(final ItemStack result) {
-		// unfinished
-	}
-
-	@Override
-	public void loadUsageRecipes(ItemStack ingredient) {
-		// unfinished
+		return "Chisel:textures/chiselNEI.png";
 	}
 
 	@Override
 	public void drawBackground(int recipeIndex) {
 		GL11.glColor4f(1.0F, 1.0F, 1.0F, 1.0F);
 		GuiDraw.changeTexture(getGuiTexture());
-		GuiDraw.drawTexturedModalRect(0, 0, 0, 0, 166, 65);
+		GuiDraw.drawTexturedModalRect(0, 0, 0, 0, 166, 130);
+
+		Point focus = ((CachedChiselRecipe) this.arecipes.get(recipeIndex)).focus;
+		if (focus != null) {
+			GuiDraw.drawTexturedModalRect(focus.x, focus.y, 166, 0, 18, 18);
+		}
 	}
 
-	public class CachedChiselRecipe extends TemplateRecipeHandler.CachedRecipe {
+	@Override
+	public int recipiesPerPage() {
+		return 1;
+	}
 
-		private PositionedStack input;
-		private List<PositionedStack> outputs;
+	@Override
+	public void loadTransferRects() {
+		this.transferRects.add(new TemplateRecipeHandler.RecipeTransferRect(new Rectangle(75, 22, 15, 13), "chisel2.chisel", new Object[0]));
+	}
 
-		@Override
-		public List<PositionedStack> getIngredients() {
-			return Lists.newArrayList(input);
-		}
-
-		@Override
-		public PositionedStack getResult() {
-			return outputs.size() > 0 ? outputs.get(0) : null;
-		}
-		
-		public CachedChiselRecipe(CarvingGroup group) {
-			this.outputs = Lists.newArrayList();
-			for (CarvingVariation v : group.getVariations()) {
-				if (this.input == null) {
-					this.input = new PositionedStack(stack(v), 10, 10);
+	public void loadCraftingRecipes(String outputId, Object... results) {
+		if (outputId.equals("chisel2.chisel")) {
+			for (String name : Carving.chisel.getSortedGroupNames()) {
+				CarvingGroup g = Carving.chisel.getGroup(name);
+				if (!g.getVariations().isEmpty()) {
+					addCached(getVariationStacks(g));
+				}
+			}
+		} else {
+			super.loadCraftingRecipes(outputId, results);
+		}
+	}
+
+	@Override
+	public void loadCraftingRecipes(ItemStack result) {
+		for (String name : Carving.chisel.getSortedGroupNames()) {
+			CarvingGroup g = Carving.chisel.getGroup(name);
+			List<ItemStack> variations = getVariationStacks(g);
+			for (ItemStack stack : variations) {
+				if (NEIServerUtils.areStacksSameTypeCrafting(stack, result)) {
+					addCached(variations, null, result);
 				}
-				outputs.add(new PositionedStack(stack(v), 50, 10));
 			}
 		}
-		
-		private ItemStack stack(CarvingVariation v) {
-			return new ItemStack(v.block, 1, v.damage);
+	}
+
+	@Override
+	public void loadUsageRecipes(ItemStack ingredient) {
+		for (String name : Carving.chisel.getSortedGroupNames()) {
+			CarvingGroup g = Carving.chisel.getGroup(name);
+			List<ItemStack> variations = getVariationStacks(g);
+			for (ItemStack stack : variations) {
+				if (NEIServerUtils.areStacksSameTypeCrafting(stack, ingredient)) {
+					addCached(variations, ingredient, null);
+				}
+			}
 		}
 	}
-}
+
+	private void addCached(List<ItemStack> variations, ItemStack base, ItemStack focus) {
+		if (variations.size() > SLOTS_PER_PAGE) {
+			List<List<ItemStack>> parts = new ArrayList<List<ItemStack>>();
+			int size = variations.size();
+			for (int i = 0; i < size; i += SLOTS_PER_PAGE) {
+				parts.add(new ArrayList<ItemStack>(variations.subList(i, Math.min(size, i + SLOTS_PER_PAGE))));
+			}
+			for (List<ItemStack> part : parts) {
+				this.arecipes.add(new CachedChiselRecipe(part, base, focus));
+			}
+		} else {
+			this.arecipes.add(new CachedChiselRecipe(variations, base, focus));
+		}
+	}
+
+	private void addCached(List<ItemStack> variations) {
+		addCached(variations, null, null);
+	}
+
+	private static List<ItemStack> getVariationStacks(CarvingGroup g) {
+		List<ItemStack> stacks = new ArrayList<ItemStack>();
+		for (CarvingVariation v : g.getVariations()) {
+			stacks.add(v.getStack());
+		}
+		String oreName = g.getOreName();
+		if (oreName != null) {
+			check: for (ItemStack ore : OreDictionary.getOres(oreName)) {
+				for (ItemStack stack : stacks) {
+					if (NEIServerUtils.areStacksSameTypeCrafting(stack, ore)) {
+						continue check;
+					}
+				}
+				stacks.add(ore.copy());
+			}
+		}
+		return stacks;
+	}
+
+}