Thing related functions about forgottenmapeditor

[asamy]

• g_things.getThingNameById(id)
• g_things.getThingIdByName(name)

Should these return an ItemType (OTB Type) or ThingType (DAT Type)?

• g_map.colorThing(thing, { r, g, b, a})
• g_map.removeThingColor(thing)

These functions sound expensive, i'll try to find another approach.

[tosty33]

ItemType, they'll be used in search module :-)

[asamy]

The first one "getThingNameById" is already there:

local itemType = g_things.findItemTypeByClientId(id);
local itemName = itemType:getName()

The second one isn't, so i'm going to make it like so:

local itemType = g_things.findItemTypeByName(name);
local clientId = itemType:getClientId()
local serverId = itemType:getServerId()

(Being generic is better, because if I were not to make it generic, it would still use the same loop and everything)

[asamy]

OTClient's upstream commit edubart/otclient@5849136 implements g_things.findItemTypeByName

[asamy]

OTClient's upstream commit edubart/otclient@32647f1 implements:

• g_map.colorizeThing(thing, color)
• g_map.removeThingColor(thing)

Color can be in RGBA (a table) or a string (#ffffffff for example or just color name). Closing this now, open new issue if any bug.