Problem/Motivation

\Drupal\Tests\system\Functional\Pager\PagerTest::testMultiplePagers causes deprecations on PHP 8.1 because of query string used as NULL but later it used in string functions

See #3220021: [meta] Ensure compatibility of Drupal 9 with PHP 8.1 (as it evolves) and commit https://git.drupalcode.org/project/drupal/-/merge_requests/937/diffs?com... for credits

Steps to reproduce

Run core/modules/system/tests/src/Functional/Pager/PagerTest.php on PHP 8.1

  1x: str_replace(): Passing null to parameter #3 ($subject) of type array|string is deprecated
    1x in PagerTest::testMultiplePagers from Drupal\Tests\system\Functional\Pager

Proposed resolution

Fix the test data

Remaining tasks

review/commit

User interface changes

no

API changes

no

Data model changes

no

Release notes snippet

no

CommentFileSizeAuthor
#2 3240364-2.patch628 bytesandypost
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost created an issue. See original summary.

andypost’s picture

patch from commit

longwave’s picture

Status: Needs review » Reviewed & tested by the community
catch’s picture

Status: Reviewed & tested by the community » Fixed

Committed 7410d62 and pushed to 9.3.x. Thanks!

  • catch committed 7410d62 on 9.3.x
    Issue #3240364 by andypost: \Drupal\Tests\system\Functional\Pager\...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.