minetest.colorspec_to_colorstring used from MT 5.5

[dennisjenkins75]

Server: MultiCraft 2.0.0 (Linux)
(Hosted on EdgyNet)

Possibly due to commit c46e246 (2022-01-20, ~2d ago). No crash like this seen before in my logs until now. I was not in-game when this happened, and do not yet know what the trigger or repro is.

2023-01-23 02:53:57: ERROR[Main]: ServerError: AsyncErr: ServerThread::run Lua: Runtime error from mod 'drawers' in callback LuaLBM::trigger(): /home/1hit/worlds/world/worldmods/drawers/lua/helpers.lua:65: attempt to call field 'colorspec_to_colorstring' (a nil value)
2023-01-23 02:53:57: ERROR[Main]: stack traceback:
2023-01-23 02:53:57: ERROR[Main]:       /home/1hit/worlds/world/worldmods/drawers/lua/helpers.lua:65: in function 'tile_to_image'
2023-01-23 02:53:57: ERROR[Main]:       /home/1hit/worlds/world/worldmods/drawers/lua/helpers.lua:83: in function 'get_inv_image'
2023-01-23 02:53:57: ERROR[Main]:       /home/1hit/worlds/world/worldmods/drawers/lua/helpers.lua:184: in function 'spawn_visuals'
2023-01-23 02:53:57: ERROR[Main]:       /home/1hit/worlds/world/worldmods/drawers/lua/visual.lua:461: in function </home/1hit/worlds/world/worldmods/drawers/lua/visual.lua:435>

[dennisjenkins75]

Now that I checked my logs, this crash is FREQUENT. I will likely rollback this update on my server. Suggest that mod author review the suspect commit.

[OgelGames]

MultiCraft doesn't have the function minetest.colorspec_to_colorstring, which was added in Minetest 5.5.0.

But the cause is that commit, because currently drawers's min_minetest_version is 5.0, so that function shouldn't have been used.

drawers/mod.conf

Line 4 in c46e246

min_minetest_version = 5.0