mrkubax10/industrialtest
1
3
Fork 3
Code Issues 2 Pull Requests Packages Projects Releases 7 Wiki Activity

Hopper compatibility and iron furnace fix #11

Merged
mrkubax10 merged 4 commits from HandfulOfFrogs/industrialtest:master into master 2024-05-10 21:38:37 +02:00
Conversation 5 Commits 4 Files Changed 2 +21 -1
1 changed files with 21 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 9c0668b996 - Show all commits

22
machines/common.lua
View File

@@ -357,9 +357,29 @@ function industrialtest.internal.registerMachine(config)
if config.sounds=="metal" then if config.sounds=="metal" then
definition.sounds=mcl_sounds.node_sound_metal_defaults() definition.sounds=mcl_sounds.node_sound_metal_defaults()
end end
definition.groups={pickaxey=1} definition.groups={
pickaxey=1,
container=2
}
definition._mcl_blast_resistance=3.5 definition._mcl_blast_resistance=3.5
definition._mcl_hardness=3.9 definition._mcl_hardness=3.9
definition._mcl_hoppers_on_try_pull=function(pos, hop_pos, hop_inv, hop_list)
mrkubax10 marked this conversation as resolved Outdated
mrkubax10 commented 2024-05-10 15:15:24 +02:00
Outdated
Review
Copy Link

Problem with this code is that not every machine has dst or src slots. For example Canning Machine has target slot instead of dst.

Problem with this code is that not every machine has `dst` or `src` slots. For example Canning Machine has `target` slot instead of `dst`.
HandfulOfFrogs commented 2024-05-10 15:20:22 +02:00
Outdated
Review
Copy Link

I see, maybe this can be moved into registerSimpleElectricItemProcessor specifically?

I see, maybe this can be moved into `registerSimpleElectricItemProcessor` specifically?
HandfulOfFrogs commented 2024-05-10 15:21:06 +02:00
Outdated
Review
Copy Link

Or, rather, have canning machine use dst slot too

Or, rather, have canning machine use dst slot too
mrkubax10 commented 2024-05-10 15:44:42 +02:00
Outdated
Review
Copy Link

Yeah this seems like a better solution though fact that src and dst are "special" should be documented somehow.

Yeah this seems like a better solution though fact that `src` and `dst` are "special" should be documented somehow.
mrkubax10 commented 2024-05-10 19:13:02 +02:00
Outdated
Review
Copy Link

Alright I renamed inventory list names to src and dst where it was making sense in 2669b7bc27.

Alright I renamed inventory list names to `src` and `dst` where it was making sense in 2669b7bc27.
HandfulOfFrogs commented 2024-05-10 20:46:01 +02:00
Outdated
Review
Copy Link

Oh, great!

Oh, great!
local meta = minetest.get_meta(pos)
local inv = meta:get_inventory()
local stack = inv:get_stack("dst", 1)
if not stack:is_empty() and hop_inv:room_for_item(hop_list, stack) then
return inv, "dst", 1
end
return nil, nil, nil
end
definition._mcl_hoppers_on_try_push=function(pos, hop_pos, hop_inv, hop_list)
local meta = minetest.get_meta(pos)
local inv = meta:get_inventory()
return inv, "src", mcl_util.select_stack(hop_inv, hop_list, inv, "src")
end
definition._mcl_hoppers_on_after_push=function(pos)
minetest.get_node_timer(pos):start(1.0)
mrkubax10 marked this conversation as resolved Outdated
mrkubax10 commented 2024-05-10 14:43:44 +02:00
Outdated
Review
Copy Link

Use industrialtest.updateDelay instead of hardcoding 1.0.

Use `industrialtest.updateDelay` instead of hardcoding `1.0`.
end
end end
definition.groups._industrialtest_wrenchUnmountable=1 definition.groups._industrialtest_wrenchUnmountable=1
if config.requiresWrench then if config.requiresWrench then