Storage Drawers: Biomes O' Plenty Pack

Storage Drawers: Biomes O' Plenty Pack

6M Downloads

Conversion Upgrade isn't working

DanteAyagawe opened this issue ยท 16 comments

commented

There's an issue I've been having with the Auto-convert item, because as I place a different type of ingot and then another in a compacting drawer with the auto-convert upgrade, it doesn't auto convert the other to the other. I was searching this issue myself and got myself nowhere, I also checked the config file and didn't see anything about manually doing the auto-convert code myself, so yeah I'm really at a lost here.

The version is 1.16.4 and the drawer mod is the most updated ;-; halp

commented

Same issue as above with Tinker's, Thermal, and Mekanism installed, which add some of the same metals. Most notably copper, lead, tin, and bronze.

Minecraft 1.16.5
Forge 36.2.2
Storage Drawers 8.3.0
Tinker's Construct 3.1.1.252
Thermal Foundation 1.3.2
Mekanism 10.0.21.448

commented

The conversion upgrade is being checked for...

for (ItemStack stack : upgrades) {
Item item = stack.getItem();
if (item == ModItems.ONE_STACK_UPGRADE)
hasOneStack = true;
else if (item == ModItems.VOID_UPGRADE)
hasVoid = true;
else if (item == ModItems.CONVERSION_UPGRADE)
hasConversion = true;
else if (item == ModItems.CREATIVE_STORAGE_UPGRADE)
hasUnlimited = true;
else if (item == ModItems.CREATIVE_VENDING_UPGRADE)
hasVending = true;
else if (item == ModItems.ILLUMINATION_UPGRADE)
hasIllumination = true;
}
attrs.setIsVoid(hasVoid);
attrs.setIsDictConvertible(hasConversion);
attrs.setIsUnlimitedStorage(hasUnlimited);
attrs.setIsUnlimitedVending(hasVending);

public boolean hasConversionUpgrade () {
return hasConversion;
}

@Override
public boolean isDictConvertible () {
return isConversion;
}
@Override
public boolean setIsDictConvertible (boolean state) {
if (isConversion != state) {
isConversion = state;
onAttributeChanged();
}
return true;
}

...but this isn't actually doing anything in practice.

/*package com.jaquadro.minecraft.storagedrawers.util;
import com.jaquadro.minecraft.storagedrawers.StorageDrawers;
import net.minecraft.item.ItemStack;
import net.minecraftforge.oredict.OreDictionary;
import javax.annotation.Nonnull;
import java.util.ArrayList;
import java.util.List;
public class ItemStackOreMatcher extends ItemStackMatcher
{

//if (attrs.isDictConvertible())
// matcher = new ItemStackOreMatcher(protoStack);
//else
matcher = new ItemStackMatcher(protoStack);

private void populateSlots (@Nonnull ItemStack itemPrototype) {
World world = group.getWorld();
if (world == null) {
protoStack[0] = itemPrototype;
convRate[0] = 1;
matchers[0] = new ItemStackMatcher(protoStack[0]);
//matchers[0] = attrs.isDictConvertible()
// ? new ItemStackOreMatcher(protoStack[0])
// : new ItemStackMatcher(protoStack[0]);
return;
}

private void populateRawSlot (int slot, @Nonnull ItemStack itemPrototype, int rate) {
protoStack[slot] = itemPrototype;
convRate[slot] = rate;
matchers[slot] = new ItemStackMatcher(protoStack[slot]);
//matchers[slot] = attrs.isDictConvertible()
// ? new ItemStackOreMatcher(protoStack[slot])
// : new ItemStackMatcher(protoStack[slot]);
}

Functionality for this upgrade is disabled, and apparently has been for a long time.

commented

It's been the better part of a year and nothing has happened on this issue at all.

commented

It's been the better part of a year and nothing has happened on this issue at all.

oof... I gave up on the upgrade and just went for alternatives, but damn this would be fucking useful ;-;

commented

up

commented

@jaquadro you should probably either fix this or disable the conversion upgrade completely. In it's current state it's very misleading.

commented

Include the conversion upgrade in the compacting drawers recipe or remove it please, I litteraly googled for 20mins what this upgrade does

commented

bump

commented

also bump

commented

bump

commented

Bump

commented

A year and a half of nothing from the mod author isn't very encouraging.

commented

this is a dumb issue, shouldnt be one and is a really easy fix. just remove it and stop misleading people

commented

I'm also having this issue and I would love to have it fixed, I have no way I know of to ore-dictionary convert items.

commented

Playing the Direwolf20 1.16 pack and ran into this issue myself. Tetra Geodes currently gives different versions of different nuggets. The only way I've found to resolve this is to manually craft them into ingots as I also tried the conversion upgrade and found it didn't work. The ingot recipe works, but the conversion upgrade does not. Compacting drawers were useful for this.

commented

Same issue. Conversion upgrade is in place but mekanism and tinkers copper are not converting.