kilarin / fractured Goto Github PK

ATM, wildweapons assumes, every entity has the fleshy armour group of 100. The correct damage calculation is stated here https://github.com/minetest/minetest/blob/master/doc/lua_api.txt#L1212
This also respects the full punch interval, so you damage less when you wait too short.

Additionally, this mod only overrides tools that have been registered before it loads. To get all tools, you would have to override minetest.register_tool with something like this:

local old_register_tool = minetest.register_tool
minetest.register_tool = function(name, item_definition)
           local old_on_use = item_definition.on_use
           item_definition.on_use = function(itemstack, user, pointed_thing)
            wildweapon_onuse(itemstack, user, pointed_thing)
                if old_on_use then
                  old_on_use(itemstack, user, pointed_thing)
                end
            end
           old_register_tool(name, item_definition)
end

I haven't tested it yet, but I think it should work.

on_use returns a modified itemstack, so you would set itemstack to the return value of old_on_use(...) and return itemstack at the end.
But since there are other, more severe problems, this isn't as urgent. Maybe even not necessary since the whole system doesn't seem to work like it should (well, it does for weapons, but only for them)

Since wildweapons overwrites the default behaviour of tools with the fleshy damage group, every tool with this group becomes useless for digging. But in default, every(?) tool has the fleshy damage group, so you can't dig with a shovel any more.