longbowrocks / ExplorersDepot

1 stars 0 forks source link

[REQ] Additional Functionality #3

Open ExcitedName opened 5 years ago

ExcitedName commented 5 years ago

Functionality with furnace, minecart with both chest and hopper, and ender chest if possible.

longbowrocks commented 5 years ago

Furnace: Pretty sure I already support furnace, but it places items in the fuel slot, so only fuel items work. Are you looking for support for the other input (ores/smeltable), or are you looking for something else entirely?

Minecart chest, minecart hopper: How do you see yourself using this? Get close to a minecart chest, press Z, and then press a button to make the minecart chest go somewhere else?

Ender chest: Ender chests are a pain, but it's an interesting idea. I'll look into it.

ExcitedName commented 5 years ago

Sorry for the delayed response. I spent a lot of time testing your mod yesterday. It looks like most of the issues are with the force compatibility. These are my findings.

MINECARTS: Yes, exactly! Maybe not a button but a pressure plate in the middle of a minecart station that hits all of the carts. Whether it be sending off storage in a chest back to base or sending charcoal in a hopper for fuel elsewhere. Personally I don't really use minecarts other than for player transport but I know there are a lot of players out there who do that might like the feature. If you don't want to no worries, please at least remove the pencil button from the gui when Force Compatibility=false as to not confuse users thinking it is broken.

FURNACE: [CONFIG SETTINGS: Force Compatibility=false, Pickup Keeps Config=true, Storage Range: 10] -Pencil button not showing up, refer to first image.

[CONFIG SETTINGS: Force Compatibility=true, Pickup Keeps Config=true, Storage Range: 10] [AUTOCONFIG SETTINGS: Tested Accept everything, from contents and Intelligently] -Items are in fact inserted the furnace. However, all items are inserted into the smelting input, even the charcoal.

ADDITIONAL BUGS: -The "Accept Everything" autoconfig button functionally works but the button icon and tooltips revert to off after closing the chest -When an item is selected from the search bar
-Force Compatibility=true is causing the pencil button to be added main creative inventory for all tabs and the player. -Force Compatibility=true seems to be hit and miss with mods.

CONSOLE: -Getting these every time I save their configuration. ORAGNGE - [16:56:40] [Netty Server IO #3/WARN] [exdepot]: That's weird. We have a GUI open for a EntityMinecartChest['Minecart with Chest'/32659, l='Proving Grounds', x=724.50, y=22.06, z=157.50] RED - [16:56:40] [Netty Server IO #3/ERROR] [exdepot]: StorageConfig requested for net.minecraft.inventory.ContainerChest@4d4ba9db, which can't have StorageConfig. ORAGNGE - [16:56:43] [Server thread/WARN] [exdepot]: That's weird. We have a GUI open for a EntityMinecartChest['Minecart with Chest'/32659, l='Proving Grounds', x=724.50, y=22.06, z=157.50] RED - [17:10:41] [Netty Server IO #3/ERROR] [exdepot]: StorageConfig requested for net.minecraft.inventory.ContainerHopper@30903dd1, which can't have StorageConfig.

SUGGESTIONS: -Change the chat from "Stored x items to x chest/s" to "Stored x items to x inventories" since it works with more than chests. -RClick in search bar to clear text. -Refine search. If I search "Pick" for pickaxe nothing shows up. Must be preceded by material type.

ADDITIONAL REQUESTS: -Brewing Stand: Functionality for the inputs of the blaze powder and ingedients, not bottles, to top it off when not using an fully automated system. -Filter more than just players inserting into inventories. Like from hoppers/pipes into chests or from hoppers/pipes into hoppers.

QUESTIONS: -I want confirm my observation of the sorting priority of inventories. It seems like the last placed and configured gets priority unless it is already full or out of range.

longbowrocks commented 5 years ago

Suggestions:

Additional requests:

Questions:

ExcitedName commented 5 years ago

Clearing Text:

Additional Request:

Questions:

Thanks for you time.