Posted September 21, 20178 yr Hello! I currently working on a mod called TooManyOres mod that depends on a library mod called HexianCore. The mod worked properly and i could compile it without any problem but recently, I encountered some pretty strange compilation error sthat only occurs when I'm trying to build the project (works fine in dev environnement). It says that I didn't implemented the method GuiContainer::drawGuiContainerBackgroundLayer even though it is implemented the the GuiConainerBase class. So I almost tried everthing and I could not find a solution to this problem, so if someone knows what i did wrong, I would be very grateful ! GuiContainerBase: Spoiler package misterpemodder.hc.main.client.gui; import java.awt.Dimension; import java.awt.Point; import java.io.IOException; import java.util.ArrayList; import java.util.Arrays; import java.util.List; import org.apache.commons.lang3.tuple.MutablePair; import org.apache.commons.lang3.tuple.Pair; import misterpemodder.hc.main.client.gui.tabs.TabArmorInventory; import misterpemodder.hc.main.client.gui.tabs.TabBase; import misterpemodder.hc.main.client.gui.tabs.TabBase.TabPos; import misterpemodder.hc.main.client.gui.tabs.TabBase.TabTexture; import misterpemodder.hc.main.client.gui.tabs.TabMain; import misterpemodder.hc.main.client.gui.tabs.TabPlayerInventory; import misterpemodder.hc.main.compat.craftingtweaks.CraftingTweaksCompat; import misterpemodder.hc.main.compat.jei.JEICompat; import misterpemodder.hc.main.compat.jei.JEIPluginHC; import misterpemodder.hc.main.inventory.ContainerBase; import misterpemodder.hc.main.tileentity.TileEntityContainerBase; import misterpemodder.hc.main.utils.StringUtils; import net.minecraft.client.Minecraft; import net.minecraft.client.gui.FontRenderer; import net.minecraft.client.gui.Gui; import net.minecraft.client.gui.GuiButton; import net.minecraft.client.gui.inventory.GuiContainer; import net.minecraft.client.renderer.GlStateManager; import net.minecraft.client.renderer.RenderHelper; import net.minecraft.init.SoundEvents; import net.minecraft.item.ItemStack; import net.minecraft.util.text.TextFormatting; import net.minecraftforge.fml.client.config.GuiUtils; public abstract class GuiContainerBase<C extends ContainerBase<TE>, TE extends TileEntityContainerBase> extends GuiContainer { public static final int TAB_OFFSET = 4; private static List<String> hoveringText; private static int hoveringTextMaxWidth; public C container; protected MutablePair<TabBase<C, TE>, TabBase<C, TE>> selectedTabs; protected List<TabBase<C, TE>> tabs; protected List<GuiButton> selectedButtonsLeft; protected List<GuiButton> selectedButtonsRight; private short ctButtonsState = -1; private List<GuiButton> ctButtons = new ArrayList<>(); public GuiContainerBase(C container) { super(container); this.container = container; this.tabs = registerTabs(); if(tabs.size() < 2) throw new IllegalArgumentException("There must be at least 2 tabs!"); initTabs(); Pair<TabBase<C, TE>, TabBase<C, TE>> p = getDefaultPair(); this.selectedTabs = MutablePair.of(p.getLeft(), p.getRight()); Dimension dl = selectedTabs.left.getTexture().dim; Dimension dr = selectedTabs.right.getTexture().dim; this.xSize = (Math.max(dl.width, dr.width) + TabBase.WIDTH); this.ySize = dl.height + dr.height; this.selectedButtonsLeft = new ArrayList<>(); this.selectedButtonsRight = new ArrayList<>(); if(CraftingTweaksCompat.isModLoaded() && CraftingTweaksCompat.guiTweakButtonClass != null) { ctButtonsState = 0; } } private void initTabs() { for(TabBase<C, TE> tab : tabs) { tab.setGuiContainer(this); } } @Override public void initGui() { super.initGui(); for(TabBase<C, TE> tab : tabs) { tab.initButtons(this.guiLeft, this.guiTop); } } @Override public void setWorldAndResolution(Minecraft mc, int width, int height) { tabs.forEach(t -> t.getButtonsList().clear()); super.setWorldAndResolution(mc, width, height); } protected Pair<TabBase<C, TE>, TabBase<C, TE>> getDefaultPair() { MutablePair<TabBase<C, TE>, TabBase<C, TE>> p = new MutablePair<>(); for(TabBase<C, TE> tab : tabs) { if(tab instanceof TabMain) { p.setLeft(tab); } else if(tab instanceof TabPlayerInventory) { p.setRight(tab); } } if(p.getLeft() == null || p.getRight() == null) { p.setLeft(tabs.get(0)); p.setRight(tabs.get(1)); } return p; } public abstract List<TabBase<C, TE>> registerTabs(); @Override public void drawScreen(int mouseX, int mouseY, float partialTicks) { super.drawScreen(mouseX, mouseY, partialTicks); this.selectedTabs.left.drawScreen(mouseX, mouseY, partialTicks); this.selectedTabs.right.drawScreen(mouseX, mouseY, partialTicks); addButtons(); } @Override protected void drawGuiContainerBackgroundLayer(float partialTicks, int mouseX, int mouseY) { //Disabled Tabs GlStateManager.pushMatrix(); GlStateManager.color(1.0f, 1.0f, 1.0f, 1.0f); this.drawDisabledTabs(); //Top Part TabTexture lt = selectedTabs.getLeft().getTexture(); this.mc.getTextureManager().bindTexture(lt.screenTexture); Gui.drawModalRectWithCustomSizedTexture(this.guiLeft, this.guiTop, 0, 0, lt.dim.width, lt.dim.height, lt.textureSize.width, lt.textureSize.height); //Bottom part TabTexture rt = selectedTabs.getRight().getTexture(); this.mc.getTextureManager().bindTexture(rt.screenTexture); Gui.drawModalRectWithCustomSizedTexture(this.guiLeft, getBottomPartPos(), 0, 0, rt.dim.width, rt.dim.height, rt.textureSize.width, rt.textureSize.height); //Enabled Tabs this.drawTab(selectedTabs.getLeft(), true); selectedTabs.getLeft().drawGuiContainerBackgroundLayer(partialTicks, mouseX, mouseY); this.drawTab(selectedTabs.getRight(), true); selectedTabs.getRight().drawGuiContainerBackgroundLayer(partialTicks, mouseX, mouseY); GlStateManager.popMatrix(); } protected void drawTab(TabBase<C, TE> tab, boolean enabled) { TabTexture texture = tab.getTexture(); Point uv = enabled? texture.enabledCoords : texture.disabledCoords; Point coords = tab.getPos(); this.mc.getTextureManager().bindTexture(tab.getTexture().tabTexture); Gui.drawModalRectWithCustomSizedTexture(this.getGuiLeft()+coords.x , this.getGuiTop()+coords.y, uv.x, uv.y, TabBase.WIDTH, TabBase.HEIGHT, 128F, 128F); } @Override protected void drawGuiContainerForegroundLayer(int mouseX, int mouseY) { selectedTabs.getLeft().updateButtons(); selectedTabs.getRight().updateButtons(); selectedTabs.getLeft().drawGuiContainerForegroundLayer(mouseX, mouseY); selectedTabs.getRight().drawGuiContainerForegroundLayer(mouseX, mouseY); boolean flag = true; for(TabBase<C, TE> tab : tabs) { RenderHelper.enableGUIStandardItemLighting(); ItemStack stack = tab.getItemStack(); Point tabPos = tab.getPos(); this.itemRender.renderItemIntoGUI(stack, tabPos.x + 8, tabPos.y + 6); this.itemRender.renderItemOverlays(this.fontRendererObj, stack, tabPos.x + 8, tabPos.y + 6); RenderHelper.disableStandardItemLighting(); if(isPointInRegion(tab.getPos().x+1, tab.getPos().y, TabBase.WIDTH, TabBase.HEIGHT-1, mouseX, mouseY)) { flag = false; addHoveringText(TextFormatting.GRAY + "" + TextFormatting.ITALIC + StringUtils.translate(tab.getUnlocalizedName())); } } if(flag && JEICompat.isModLoaded()) { List<RecipeClickableAreaHC> list = getRecipeClickableAreas(); for(RecipeClickableAreaHC r : list) { if(r.checkHover(mouseX, mouseY)) { addHoveringText(StringUtils.translate("jei.tooltip.show.recipes")); } } } if(hoveringText != null && !hoveringText.isEmpty()) { GuiUtils.drawHoveringText(hoveringText, mouseX-this.getGuiLeft(), mouseY-this.getGuiTop(), this.width, this.height, hoveringTextMaxWidth, fontRendererObj); hoveringText = null; } } private List<RecipeClickableAreaHC> getRecipeClickableAreas() { List<RecipeClickableAreaHC> l = new ArrayList<>(); if(selectedTabs.getLeft().hasRecipeClickableAreas()) { l.addAll(Arrays.asList(selectedTabs.getLeft().getRecipeClickableAreas())); } if(selectedTabs.getRight().hasRecipeClickableAreas()) { l.addAll(Arrays.asList(selectedTabs.getRight().getRecipeClickableAreas())); } return l; } @Override protected void mouseClicked(int mouseX, int mouseY, int mouseButton) throws IOException { this.selectedTabs.left.mouseClicked(mouseX, mouseY, mouseButton); this.selectedTabs.right.mouseClicked(mouseX, mouseY, mouseButton); for(TabBase<C, TE> tab : tabs) { Point pos = tab.getPos(); if(isPointInRegion(pos.x+1, pos.y, TabBase.WIDTH, TabBase.HEIGHT-1, mouseX, mouseY)) { if(tab.isEnabled() && selectedTabs.getLeft() != tab && selectedTabs.getRight() != tab) { this.mc.player.playSound(SoundEvents.UI_BUTTON_CLICK, 1.0F, 1.0F); TabPos tabPos = tab.getTabPos(); if(tabPos == TabPos.TOP_LEFT || tabPos == TabPos.TOP_RIGHT) { selectedTabs.setLeft(tab); } else { selectedTabs.setRight(tab); } selectedTabs = tab.forceTabConfig(); setContainerTabs(); this.container.hideSlots(); } } else if(JEIPluginHC.hasJEIRuntime()) { List<RecipeClickableAreaHC> list = getRecipeClickableAreas(); for(RecipeClickableAreaHC r : list) { if(r.checkHover(mouseX, mouseY)) { JEIPluginHC.getJEIRuntime().getRecipesGui().showCategories(r.getRecipeCategoryUids()); this.onGuiClosed(); } } } } super.mouseClicked(mouseX, mouseY, mouseButton); } @SuppressWarnings({ "unchecked", "rawtypes" }) public void setContainerTabs() { this.container.setSelectedTabs((Pair)selectedTabs); } public void updateScreen() { super.updateScreen(); TE te = this.container.getTileEntity(); if(te == null || !te.hasWorld() || te.getWorld().getBlockState(te.getPos()).getBlock() != te.getBlockType()) { this.onGuiClosed(); this.mc.player.closeScreen(); } } public boolean isPointInTheRegion(int rectX, int rectY, int rectWidth, int rectHeight, int pointX, int pointY) { return this.isPointInRegion(rectX, rectY, rectWidth, rectHeight, pointX, pointY); } @Override public void onGuiClosed() { this.selectedTabs.left.onGuiClosed(); this.selectedTabs.right.onGuiClosed(); tabs.forEach(t -> t.getButtonsList().clear()); super.onGuiClosed(); } protected void keyTyped(char typedChar, int keyCode) throws IOException { if(!selectedTabs.left.keyTyped(typedChar, keyCode) && !selectedTabs.right.keyTyped(typedChar, keyCode)) { super.keyTyped(typedChar, keyCode); } } private void addButtons() { List<GuiButton> leftButtons = this.selectedTabs.left.getButtonsList(); List<GuiButton> rightButtons = this.selectedTabs.right.getButtonsList(); if(leftButtons == null || leftButtons.isEmpty()) { selectedButtonsLeft.clear(); } else if(!selectedButtonsLeft.containsAll(leftButtons)) { selectedButtonsLeft.clear(); selectedButtonsLeft.addAll(leftButtons); } if(rightButtons == null || rightButtons.isEmpty()) { selectedButtonsRight.clear(); } else if(!selectedButtonsRight.containsAll(rightButtons)) { selectedButtonsRight.clear(); selectedButtonsRight.addAll(rightButtons); } if(ctButtonsState == 0 && !buttonList.isEmpty()) { for(GuiButton b : buttonList) { if(CraftingTweaksCompat.guiTweakButtonClass.isAssignableFrom(b.getClass())) { this.ctButtonsState = 1; this.ctButtons.add(b); } } } try { if(ctButtonsState == 1 && !buttonList.isEmpty() && !ctButtons.isEmpty()) { List<GuiButton> blist = new ArrayList<>(buttonList); for(GuiButton b1 : blist) { for(GuiButton b2 : ctButtons) { if(b1 == b2) { buttonList.remove(b1); } } } } } catch(Exception e) { e.printStackTrace(); } this.buttonList.clear(); this.buttonList.addAll(selectedButtonsLeft); this.buttonList.addAll(selectedButtonsRight); if(this.selectedTabs.right instanceof TabArmorInventory && !ctButtons.isEmpty()) { buttonList.addAll(ctButtons); } } @Override protected void actionPerformed(GuiButton button) throws IOException { if(selectedButtonsLeft.contains(button)) { this.selectedTabs.left.onButtonClicked(button); } else if(selectedButtonsRight.contains(button)) { this.selectedTabs.right.onButtonClicked(button); } } public static void addHoveringText(String str) { addHoveringText(Arrays.asList(str), -1); } public static void addHoveringText(List<String> strs, int maxWidth) { if(strs != null && !strs.isEmpty()) { hoveringText = strs; hoveringTextMaxWidth = maxWidth; } } public FontRenderer getFontRenderer() { return this.fontRendererObj; } public int getBottomPartPos() { return this.getGuiTop() + selectedTabs.getLeft().getTexture().dim.height; } protected void drawDisabledTabs() { for(TabBase<C, TE> tab : tabs) { if(tab != selectedTabs.getLeft() && tab != selectedTabs.getRight()) { this.drawTab(tab, false); } } } @Override public boolean isPointInRegion(int rectX, int rectY, int rectWidth, int rectHeight, int pointX, int pointY) { return super.isPointInRegion(rectX, rectY, rectWidth, rectHeight, pointX, pointY); } public MutablePair<TabBase<C, TE>, TabBase<C, TE>> getSelectedTabs() { return this.selectedTabs; } public List<TabBase<C, TE>> getRegisteredTabs() { return this.tabs; } public Minecraft getMinecraft() { return this.mc; } } GuiContainerDestabilizer: Spoiler package misterpemodder.tmo.main.client.gui; import java.util.ArrayList; import java.util.List; import org.objectweb.asm.commons.Method; import misterpemodder.hc.main.client.gui.GuiContainerBase; import misterpemodder.hc.main.client.gui.tabs.TabBase; import misterpemodder.hc.main.client.gui.tabs.TabPlayerInventory; import misterpemodder.tmo.main.client.gui.tabs.TabArmorInventoryTMO; import misterpemodder.tmo.main.client.gui.tabs.TabIO; import misterpemodder.tmo.main.client.gui.tabs.TabInfo; import misterpemodder.tmo.main.client.gui.tabs.TabMainDestabilizer; import misterpemodder.tmo.main.inventory.ContainerDestabilizer; import misterpemodder.tmo.main.tileentity.TileEntityDestabilizer; import net.minecraft.entity.player.InventoryPlayer; import net.minecraftforge.fml.relauncher.ReflectionHelper; public class GuiContainerDestabilizer extends GuiContainerBase<ContainerDestabilizer, TileEntityDestabilizer> { public GuiContainerDestabilizer(InventoryPlayer playerInv, TileEntityDestabilizer te) { super(new ContainerDestabilizer(te, playerInv)); } @Override public List<TabBase<ContainerDestabilizer, TileEntityDestabilizer>> registerTabs() { List<TabBase<ContainerDestabilizer, TileEntityDestabilizer>> list = new ArrayList<>(); list.add(new TabMainDestabilizer()); list.add(new TabInfo<ContainerDestabilizer, TileEntityDestabilizer>(false)); list.add(new TabPlayerInventory<ContainerDestabilizer, TileEntityDestabilizer>()); list.add(new TabArmorInventoryTMO<ContainerDestabilizer, TileEntityDestabilizer>()); list.add(new TabIO<>(container.getTileEntity().getIoConfigHandler())); return list; } } Gradle log: Spoiler This mapping 'snapshot_20161220' was designed for MC 1.11! Use at your own peril.This mapping 'snapshot_20161220' was designed for MC 1.11! Use at your own peril. ################################################# ForgeGradle 2.2-SNAPSHOT-d13a6c8 https://github.com/MinecraftForge/ForgeGradle ################################################# Powered by MCP unknown http://modcoderpack.com by: Searge, ProfMobius, Fesh0r, R4wk, ZeuX, IngisKahn, bspkrs ################################################# :extractMcpMappings SKIPPED :deobfCompileDeobfDepTask0 :deobfCompileDeobfDepTask1 SKIPPED :deobfCompileRemapDepSourcesTask2 SKIPPED :deobfCompileRemapDepSourcesTask3 SKIPPED :deobfCompileDummyTask :deobfProvidedDummyTask :sourceApiJava UP-TO-DATE :compileApiJava :processApiResources UP-TO-DATE :apiClasses :sourceMainJava :compileJavaC:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\tileentity\TileEntityTitaniumChest.java:5: error: TileEntityTitaniumChest is not abstract and does not override abstract method update() in ITickable public class TileEntityTitaniumChest extends TileEntityCustomChest { ^ C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\tileentity\TileEntityStrongPiston.java:70: error: no suitable method found for findMethod(Class<CAP#1>,<null>,String[],Class<?>[]) return ReflectionHelper.findMethod(clazz, null, methodNames, methodTypes); ^ method ReflectionHelper.<E>findMethod(Class<? super E>,E,String[],Class<?>...) is not applicable (cannot infer type-variable(s) E (argument mismatch; Class<CAP#1> cannot be converted to Class<? super E>)) method ReflectionHelper.findMethod(Class<?>,String,String,Class<?>...) is not applicable (argument mismatch; String[] cannot be converted to String) where E is a type-variable: E extends Object declared in method <E>findMethod(Class<? super E>,E,String[],Class<?>...) where CAP#1 is a fresh type-variable: CAP#1 extends Object from capture of ? C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerDestabilizer.java:20: error: GuiContainerDestabilizer is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer public class GuiContainerDestabilizer extends GuiContainerBase<ContainerDestabilizer, TileEntityDestabilizer> { ^ C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerInjector.java:17: error: GuiContainerInjector is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer public class GuiContainerInjector extends GuiContainerBase<ContainerInjector, TileEntityInjector> { ^ C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerThermoelectricGenerator.java:17: error: GuiContainerThermoelectricGenerator is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer public class GuiContainerThermoelectricGenerator extends GuiContainerBase<ContainerThermoelectricGenerator, TileEntityThemoelectricGenerator> { ^ C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerTitaniumAnvil.java:16: error: GuiContainerTitaniumAnvil is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer public class GuiContainerTitaniumAnvil extends GuiContainerBase<ContainerTitaniumAnvil, TileEntityTitaniumAnvil>{ ^ C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\client\gui\GuiContainerTitaniumChest.java:18: error: GuiContainerTitaniumChest is not abstract and does not override abstract method drawGuiContainerBackgroundLayer(float,int,int) in GuiContainer public class GuiContainerTitaniumChest extends GuiContainerBase<ContainerTitaniumChest, TileEntityTitaniumChest>{ ^ Note: Some input files use or override a deprecated API. Note: Recompile with -Xlint:deprecation for details. Note: C:\Users\Patricia\Desktop\Modding\Workspace\TooManyOres\build\sources\main\java\misterpemodder\tmo\main\items\TMOItemVariants.java uses unchecked or unsafe operations. Note: Recompile with -Xlint:unchecked for details. 7 errors FAILED FAILURE: Build failed with an exception. * What went wrong: Execution failed for task ':compileJava'. > Compilation failed; see the compiler error output for details. * Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. BUILD FAILED Total time: 1 mins 44.976 secs PS: sorry for my bad english Edited September 21, 20178 yr by MisterPeModder PS: sorry for my bad english :)
September 22, 20178 yr Could you post the compiler error log as well? Edited September 22, 20178 yr by MDW01
September 22, 20178 yr Author HexianCore: https://github.com/MisterPeModder44/HexianCore/tree/1.11.2/ TooManyOres: https://github.com/MisterPeModder44/TooManyOres3/tree/1.11.2
September 22, 20178 yr It may take me a while to find out whats wrong. But in the mean time you could check when you last successfully compiled and what changes you have made since then that could be effecting this.
September 22, 20178 yr Author Hmm... the last time I compiled this mod I wasn't using hexian core as a dependency, what may have caused this issue is the fact that i moved most of the abstract classes/interfaces such as GuiContainerBase to my library.
September 22, 20178 yr Author No but i added it as a jar file in the libs folder so it should work (I built my other mods this way so that is propably what's causing the problem)
September 22, 20178 yr Yea with my core mod I usually don't start running into issues until I compile. So I would try adding it as a dependency in gradle.
September 22, 20178 yr Neither do I but in the end it works and thats all that matters. Thankfully Forge has this nice article about it. https://github.com/MinecraftForge/ForgeGradle/wiki/Dependencies
September 22, 20178 yr Author Well thanks i didn't knew about that! ...But sadly this give me the exact same errors as previously.
September 22, 20178 yr I can look at it more tomorrow. I tend to think better when I'm not half asleep.
September 22, 20178 yr 1) Your reflection is broken you need to add the srg names as well. The findMethod you are using is deprecated and switching to the other one will make it compile. I believe that the problem is the null instance 2) All the other errors are that you are not implementing things in the classes. You need to implement the missing methods. Once these things are fixed it compiles Edited September 22, 20178 yr by CoderAtParadise grammer Did you really need to know?
September 22, 20178 yr Author 1) oh I didn't know I had to include srg names! 2)well I know that, but the problem is that I DID implemented all of these methods
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.