- Issue created by @Marios Anagnostopoulos
- Merge request !263Make TermWeightWidgetOrderProcessor respect the full hierarchy. → (Open) created by Marios Anagnostopoulos
- Assigned to n.antonopoulou
- Status changed to Needs review
4 months ago 9:01pm 16 June 2025 - 🇺🇸United States damienmckenna NH, USA
FYI I'm getting lots of this error after trying the change:
Deprecated function: Invalid characters passed for attempted conversion, these have been ignored in Drupal\Component\Utility\Number::intToAlphadecimal() (line 80 of core/lib/Drupal/Component/Utility/Number.php).
- 🇵🇪Peru krystalcode
I think this needs a bit more work to make sure we don't harm efficiency for the various different use cases.
In my case I display hierarchy by having "Use hierarchy" enable and also "Show parents" from ✨ Add processor to show parents in hierarchical facets Active .
The Taxonomy hierarchy plugin already loads the parents and caches them statically. This MR would result in - in many normal use cases - potentially hundreds of database queries again that trigger access hooks as well, because it loads all ancestors for each term without using the cached items from the hierarchy plugin.
Maybe the best in many cases - from the point of view of reducing database queries - would be to load the entire tree (which is just one database query) in a processor plugin and make it available somehow to the sorting plugin where it can find the parents algorithmically. This may need some refactoring in the manager anyways.