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.
Follow up to #2044435: Convert pager.inc to a service We left an unused RequestStack property on \Drupal\views\Plugin\views\pager\SqlBase when switching to using PagerParameters.
We need to remove the unused property:
/**
* The request stack.
*
* @var \Symfony\Component\HttpFoundation\RequestStack
*/
protected $requestStack;
Comments
Comment #2
andypostRemoval is 9.0 target
Comment #3
plachLet's try to get this in before beta, otherwise we might have to provide some sort of BC. Also, I think this should still target D8, since it's not a D9-only change.
Comment #4
kim.pepperLgtm
Comment #5
alexpottCommitted and pushed 3ccf26469e to 9.0.x and 31dfdce5ca to 8.9.x and d37edf9f12 to 8.8.x. Thanks!
Backported to 8.8.x as this code was only added in 8.8.x so let's never release a Drupal with this mistake...
Comment #9
xjm