Ender IO Zoo

Ender IO Zoo

968k Downloads

[Suggestion] Allow to use Metallurgy tools by default in Farming Stations

hron84 opened this issue ยท 8 comments

commented

I know the config options, however Metallurgy has a dozen type of tools (one set for almost all metals), and I do not want to add them one-by-one to my config. Instead, it would be good if FarmingStations can accept them by default.

commented

We'd be happy to add those to the default config. If you'd provide an item list?

commented

But.... Metallurgy axes are look like an axe because they're inheriting from ItemAxe.
Update: also they have a harvest level.

commented

@HenryLoenwind Why not include everything that looks like an axe/hoe and provide a blacklist for banned items?

commented

Also, I wonder if ThermalFoundation shears are supported

commented

We already do that. The problem are those tools that do not "look" like an axe or a hoe. Unless you wanted to suggest to utilize image recognition on the the icon? ;)

Update: We take all shears that "look" like shears, all items that have a harvest level as an "axe", the IC2 tree tap as a specific item and the list of whitelisted hoes as specific items. The test if something looks like a hoe was disabled because there were too many modded hoes that gave problems (e.g. breaking when they are supposed to be unbreakable, crashing when being used by a machine...).

commented

@HenryLoenwind now, it smells like a bug...

commented

Then someone will have to investigate why https://github.com/SleepyTrousers/EnderIO/blob/master/src/main/java/crazypants/enderio/machine/farm/TileFarmStation.java#L57 doesn't catch them.

(I'm short on time this week)

commented

@CrazyPants thanks for solving this issue. Will it be ported back to 1.7.10 too?