Jump to content

Custom Crafting Bench bug


Heltrato

Recommended Posts

So im porting my mod to 1.12.2 fixed the errors most of it but bugs are everywhere since no reworks.

 

Right now im trying to fix my custom crafting bench that has a clickable button (craft) if the following requiredStack slot is met,

 

but two things i find out 

1.) it allows to craft even if the required quantity is not me

2.) upon finish crafting it just consumes on how many quantity the requiredstack is (even if it exceed)

 

Here is my class that has a related on it

 

https://github.com/Guild-Hall/MHFC/blob/1.12.2/src/main/java/mhfc/net/common/tile/TileHunterBench.java

 

https://github.com/Guild-Hall/MHFC/blob/1.12.2/src/main/java/mhfc/net/common/block/container/BlockHunterBench.java

 

P.S on 1.11.2 this methods are all the same, 

Link to comment
Share on other sites

Quote

public class TileHunterBench extends TileEntity implements ..., IInventory, ...

Don't ever implement IInventory. Use capabilities instead.

 

Quote

extends BlockContainer

Don't extend BlockContainer. It overrides a bunch of methods you do not need, like the rendering method. Just override Block#hasTileEntity and Block#createTileEntity.

 

26 minutes ago, Heltrato said:

it allows to craft even if the required quantity is not me

IIngredient by default doesn't care for the stack size since it is only used for vanilla crafting bench and that one only operates in stack sizes of one. You need to check the stack size yourself.

 

26 minutes ago, Heltrato said:

upon finish crafting it just consumes on how many quantity the requiredstack is (even if it exceed)

Well, why wouldn't it? That's what you are telling it to do:

private void finishRecipe() {
		...
		resetInputStacks();
		...
}

private void resetInputStacks() {
		Arrays.fill(inputStacks, ItemStack.EMPTY);
	}

You are telling your TE to fill all inputs with an empty stack when it is done.

 

Unrelated

Quote

package mhfc.net

Do you own mhfc.net? Because if you don't you can't name your package like that. And if you do - fix the security warning, I can't even access the site.

 

Link to comment
Share on other sites

6 minutes ago, V0idWa1k3r said:

Don't ever implement IInventory. Use capabilities instead.

 

Don't extend BlockContainer. It overrides a bunch of methods you do not need, like the rendering method. Just override Block#hasTileEntity and Block#createTileEntity.

 

IIngredient by default doesn't care for the stack size since it is only used for vanilla crafting bench and that one only operates in stack sizes of one. You need to check the stack size yourself.

 

Well, why wouldn't it? That's what you are telling it to do:


private void finishRecipe() {
		...
		resetInputStacks();
		...
}

private void resetInputStacks() {
		Arrays.fill(inputStacks, ItemStack.EMPTY);
	}

You are telling your TE to fill all inputs with an empty stack when it is done.

 

Unrelated

Do you own mhfc.net? Because if you don't you can't name your package like that. And if you do - fix the security warning, I can't even access the site.

 

Is there any code related that reduces the StackSize of the inputStacks? I haven't really found out one since then :/ thou this method works fine during 1.11.2

 

Oh yeah, mhfc.net was our package since we started from 1.4.7 i'll have to rename this soon.

Edited by Heltrato
Link to comment
Share on other sites

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.



×
×
  • Create New...

Important Information

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