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
There's functions deprecated for removal in core 10
Proposed resolution
remove the usage, make sure no mentions left
Remaining tasks
review/commit
User interface changes
no
API changes
no
Data model changes
no
Release notes snippet
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | 3261243-2.patch | 4.95 KB | andypost |
| |||
Comments
Comment #2
andypostComment #3
andypostComment #4
longwaveNo deprecated methods remain in comment.module, so this is good to go.
One thing I did spot that is marked legacy but that (as far as I know) we have no issue to remove is the
comment.node_redirectroute, which is the old/comment/{node}/replypath for nodes only which predates multiple comment fields. Should we discuss removing this or shall we just leave it to handle old URLs forever?Comment #5
andypostGood point, needs separate issue to deprecate and remove the route
Comment #7
catchCommitted a1a9297 and pushed to 10.0.x. Thanks!
Let's open a follow-up to discuss the route deprecation.
Comment #8
andypostThank you, filed follow-up #3262903: Deprecate comment.node_redirect route