Problem/Motivation

As of 8.2 callables changed https://wiki.php.net/rfc/deprecate_partially_supported_callables

Mostly all places been fixed in #3299327: Replace deprecated static::method() calls for PHP 8.2 except one place

Steps to reproduce

See #3295821-91: Ignore: patch testing issue for PHP 8.2 attributes

core/modules/views/src/Views.php
-        $views = call_user_func("static::get{$filter}Views");
+        $views = call_user_func(static::class . "::get{$filter}Views");

Proposed resolution

Change syntax to supported PHP versions

Remaining tasks

- collect all changes from testing issue #3295821-91: Ignore: patch testing issue for PHP 8.2 attributes
- make sure tests pass on PHP 8.2
- review/commit

User interface changes

no

API changes

no

Data model changes

no

Release notes snippet

no

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

andypost created an issue. See original summary.

Berdir’s picture

We already had an issue for most of these but we missed this one.

andypost’s picture

IIRC the issue been fixed but with follow-up to improve via rector - will dig later today

andypost’s picture

andypost’s picture

Title: Fix callable syntax for PHP 8.2 » Fix callable syntax for PHP 8.2 in Views.php

re-title to point the scope

Berdir’s picture

Status: Needs review » Reviewed & tested by the community

Yes, lets do just do this for now.

  • catch committed 31365cd on 10.0.x
    Issue #3309750 by andypost, Berdir: Fix callable syntax for PHP 8.2 in...
  • catch committed 9ff2c84 on 10.1.x
    Issue #3309750 by andypost, Berdir: Fix callable syntax for PHP 8.2 in...
  • catch committed b3b6c07 on 9.5.x
    Issue #3309750 by andypost, Berdir: Fix callable syntax for PHP 8.2 in...
catch’s picture

Version: 10.0.x-dev » 9.5.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 10.1.x, cherry-picked to 10.0.x and 9.5.x, thanks!

Status: Fixed » Closed (fixed)

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