Jump to content
View in the app

A better way to browse. Learn more.

Forge Forums

A full-screen app on your home screen with push notifications, badges and more.

To install this app on iOS and iPadOS
  1. Tap the Share icon in Safari
  2. Scroll the menu and tap Add to Home Screen.
  3. Tap Add in the top-right corner.
To install this app on Android
  1. Tap the 3-dot menu (⋮) in the top-right corner of the browser.
  2. Tap Add to Home screen or Install app.
  3. Confirm by tapping Install.

Featured Replies

Posted

Ok, this is bizarre, but I'm sure there's a logical explanation (?). One of the Items in our mod acts as a Furnace, but cannot seem to smelt ores and other smeltable items from other mods. Vanilla Minecraft is fine, but all other mods items it turns into Paper (or if I install more mods, some other random Item). There is nothing fancy in the code. The only major difference I can see is that 'getSmeltingResults' is being called from an Item and not from a Container.

 

Item Code:

ItemStack itemstack = FurnaceRecipes.smelting().getSmeltingResult(s2);

entityplayer.inventory.addItemStackToInventory(new ItemStack(itemstack.getItem(), 1, 0));

 

When I print the variables to the screen (the other mod's Smelting Results), they show just fine. But when I addItemStackToInventory or  even dropItem, it makes paper.

 

However, if I change the second line to Spawn an EntityItem, it works fine (e.g.):

Entity entity = new EntityItem(worldObj, entityplayer.posX, entityplayer.posY, entityplayer.posZ, itemstack);

worldObj.spawnEntityInWorld(entity);

 

I am imagining that this is because the Item I am pretending is a Furnace is not able to access the items from other mods? Of course, it could be that I am just being stupid (there is some evidence to support this).

 

Can one of our brave moderators explain this behavior?

 

 

  • Author

Yes, you are right of course, but it didn't make a difference, still outputting paper. I doubt this will ever come up again, but thought might be interesting to chew on.

 

And thanks :)

  • Author

Ok, I played around a bit more. There were two problems.

 

Changing the way the ItemStack was instantiated did fix identification of the right smelting result from other mods (pulling the correct NBT). So, on that part you were right. However, custom smelting results were not bringing in a stackSize (stackSize was 0). For some reason, vanilla results are showing a stackSize, but custom results from other mods are showing stackSize of zero. If I'd made a custom furnace I would have used mergeItemStack which would have handled the addition of items to your inventory without using the stackSize from the recipe result. Smelting results from the Furnace are always a stackSize of 1.

 

Anyway, to fix it, I set the stackSize manually. Updated code:

 

ItemStack itemstack = FurnaceRecipes.smelting().getSmeltingResult(s2);

if (itemstack != null) {

itemstack.stackSize = 1;

        entityplayer.inventory.addItemStackToInventory(itemstack);

        s2.stackSize--;

        if (s2.stackSize == 0) {

removeItem(entityplayer, s2);

}

cookedFlag = true;

}

 

 

Thanks for the help, and thanks for pushing me to look deeper.

  • Author

Yep. That was it. Tested on multiple mods & ores. I had no idea that copying the ItemStack would make a difference. No longer need to set stackSize manually.

 

Cheers, diesieben07

 

 

  • Author

Question, though, why would this only be a problem with other mods recipes and not vanilla? Can you explain that one? That still mystifies me.

  • Author

Exactly. But it does not. That was the original conundrum and why I was so confused. There are no problems with vanilla. And with hundreds of thousands of gameplays at this point, and hours of testing these last 3 months, I am pretty confident in that statement. You can download version 1.1.2a and see for yourself. There were no reports of any issues of Furnace Pet until it started getting added to modpacks and users began to try it on other mods' ores.

 

So, what makes Vanilla recipes immune to poor ItemStack implementation? Now it's your turn to do some digging :). Or not.

 

K

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...

Important Information

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

Configure browser push notifications

Chrome (Android)
  1. Tap the lock icon next to the address bar.
  2. Tap Permissions → Notifications.
  3. Adjust your preference.
Chrome (Desktop)
  1. Click the padlock icon in the address bar.
  2. Select Site settings.
  3. Find Notifications and adjust your preference.