It is being frequently states that the default trees are apple trees. However, they are not named/id'd as such - the tree trunk is default:tree and the leaves are default:leaves. If they are truly supposed to be apple trees, it would make sense that the nodes are id'd default:tree_apple and default:leaves_apple or something like that.
I am working on a mod which adds different types of tool handles (but it isn't the main feature) - I call the handle made with default:wood "toolhandle_applewood", which when observed does not make sense
because the names of default:tree and default:leaves are just "Tree" and "Leaves", respectively. If they are truly supposed to be apple trees, the names should reflect that, but currently they do not. So, either apples need to be moved to a new tree that is actually called "Apple" outside of a single-line comment somewhere in the code, and the "regular" trees left without apples, - or, the nodes need to be re-id'd and re-named such that they are default:tree_apple and default:leaves_apple, and the names being "Apple Tree" and "Apple Leaves" respectively.
Leaving it the way it is now is wrong, because the current implementation is self-contradictory, even without the mod I am working on. If nothing is done, it will continue to be wrong.
Don't say that renaming the nodes will cause problems - that's what minetest.register_alias() is for.
EDIT:
Wuzzy wrote:If the “default” tree is supposed to be an apple tree, why not simply renaming the nodes?
- “Tree” → “Apple Tree” (actually should be “Apple tree trunk” but for some reason tree trunks are called “trees” in Minetest Game)
- “Leaves” → “Apple Tree Leaves”
and so on.
I didn't see this one yet, that basically sums up what I just said, except for the part about tool handles and the part about leaving "regular trees" without apples