LuaEntity::get_recipe() subclasses

[Therenas]

https://lua-api.factorio.com/latest/Lua ... get_recipe

Works on a ghost of a crafting machine. Not sure if it is intentional or not (there is no other obvious way to check the recipe on a ghost), but it works. get_recipe() and previous_recipe don't do anything on a blueprint of a furnace, which makes sense.

Is this more of a comment? What do you think should be changed?

[DaveMcW]

Can only be used if this is CraftingMachine, or EntityGhost of CraftingMachine.

[Therenas]

Doesn't CraftingMachine already imply that?

[DaveMcW]

Not unless you know the LuaEntity inheritance tree. If you want to document how a LuaEntity can be two types at once, that would be great too!

[Therenas]

You're right, probably best to be explicit. Fixed for the next release.

The whole 'Can only be used if' system is kind of bad in general though, it's due for a re-do at some point I feel like.

[Honktown]

Doesn't CraftingMachine already imply that?

Ghosts can't craft and machines of them are an entity-ghost, so from a modder perspective, no. How I viewed it, hence the request. Similarly, seemingly far more important attributes, ghost_name, ghost_prototype, ghost_type, e.g. are separate. No ghost_get_recipe() or ghost_set_recipe()

[eradicator]

Doesn't CraftingMachine already imply that?

Intuitively and without evidence and very personal opinion: No.
I generally assume that SomethingGhost behaves entirely different than the real thing.