This function accepts $load_entities as argument so returning EntityNG now #1818560: Convert taxonomy entities to the new Entity Field API or just stdObject from db_select() fetch
Proposed
Clean-up usage of taxonomy_get_tree() to return array of stdObject and separate function to load entities based on result
Comments
Comment #1
pcambraIf we return stdclass instead of the entity, we need to change (at least) the entity reference plugin for them (TermSelection.php) that uses $term->id() for $term->tid
Comment #2
berdirWe'd need to change a ton of things. For example, whenever you need the term name in the correct translation, you *need* the loaded term object and call label(). That's why the load_entities argument was introduced initially.
Comment #3
jibrantaxonomy_get_tree()isTermStorage::loadTree()now. I think it is not a bug nothing is broken. It's an api change without BC break so I think we can move it to 8.1.xComment #4
jibranNext Generation of entities has arrived.
Comment #20
smustgrave commentedThank you for creating this issue to improve Drupal.
We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.
Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.
Comment #21
smustgrave commentedSince there's been no follow up going to close this one out. If still valid task please re-open
Thanks all