Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Problem/Motivation
We moved token tree builder functions to a service in #2641076: Move token builder functions to a service. We didn't remove the functions in that issue to check for more effects. Let's do that in this follow-up.
Proposed resolution
Remove functions:
- token_build_tree
Remaining tasks
Patch, Review and Commit
User interface changes
None
API changes
The deprecated functions may not be used anymore. Callers should directly use the @token.tree_builder service.
Data model changes
None
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | remove_deprecated_tree_builder-2641386-6.patch | 3.31 KB | hussainweb |
| |||
Comments
Comment #2
hussainwebThis is almost certainly going to fail, but let's worry about this issue after we close a few other pressing issues first. :)
I am attaching the patch for later reference.
Comment #3
hussainwebWow, it passed. It shouldn't really. There are quite a few usages of token_tree_build. #2641136: Write tests for token tree builder will help.
Comment #5
BerdirYay test fails! ;)
Comment #6
hussainwebYay! Let's fix them now. Skipping interdiff as the previous change was quite small.
Comment #7
hussainwebBTW, I manually tested various pages (help, autocomplete, token browser dialog). All work fine.
Comment #8
BerdirThank you, committed.