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
Follow-up to #3209617: [Symfony 6] Symfony\Component\HttpFoundation\RequestStack::getMasterRequest() is deprecated, use getMainRequest() instead discovered in #3265356: Deprecate Symfony 4 RequestStack BC shim for removal in D11
The deprecation message should follow standard https://www.drupal.org/about/core/policies/core-change-policies/drupal-d...
Proposed resolution
Set message to required with link to change record https://www.drupal.org/node/3253744 and fix test
Remaining tasks
review/commit
User interface changes
no
API changes
no
Data model changes
no
Release notes snippet
no
Comment | File | Size | Author |
---|---|---|---|
#2 | 3265419-2.patch | 1.84 KB | andypost |
Comments
Comment #2
andypostComment #3
SpokjeMolding the deprecation message to fit into the standard and adding a link to a CR always looks good to me.
Comment #4
alexpottCommitted and pushed 3c7c0fdfc3d to 9.4.x and 31d2dc98dfa to 9.3.x. Thanks!
Backported to 9.3.x since it is helpful there too.