Changeset 60729 for trunk/tests/phpunit/tests/interactivity-api/wpInteractivityAPI.php

Timestamp:

09/11/2025 02:45:56 PM (3 months ago)

Author:

swissspidy

Message:

Code Modernization: Address reflection no-op function deprecations in PHP 8.5.

Reflection*::setAccessible() methods are no-ops since PHP 8.1. This commit adds conditional checks to only call these functions on older PHP versions.

Reference: ​PHP RFC: Deprecations for PHP 8.5: Deprecate `Reflection*::setAccessible()`.

Props rishabhwp, swissspidy.
Fixes #63956.
See #63061.

File:

• trunk/tests/phpunit/tests/interactivity-api/wpInteractivityAPI.php (modified) (8 diffs)

trunk/tests/phpunit/tests/interactivity-api/wpInteractivityAPI.php

@@ -42 +42 @@
         $interactivity   = new ReflectionClass( $this->interactivity );
         $namespace_stack = $interactivity->getProperty( 'namespace_stack' );
-        $namespace_stack->setAccessible( true );
+        if ( PHP_VERSION_ID < 80100 ) {
+            $namespace_stack->setAccessible( true );
+        }
         $namespace_stack->setValue( $this->interactivity, $stack );
     }
@@ -56 +58 @@
         $interactivity = new ReflectionClass( $this->interactivity );
         $context_stack = $interactivity->getProperty( 'context_stack' );
-        $context_stack->setAccessible( true );
+        if ( PHP_VERSION_ID < 80100 ) {
+            $context_stack->setAccessible( true );
+        }
         $context_stack->setValue( $this->interactivity, $stack );
     }
@@ -587 +591 @@
     public function test_extract_directive_value() {
         $extract_directive_value = new ReflectionMethod( $this->interactivity, 'extract_directive_value' );
-        $extract_directive_value->setAccessible( true );
+        if ( PHP_VERSION_ID < 80100 ) {
+            $extract_directive_value->setAccessible( true );
+        }
 
         $result = $extract_directive_value->invoke( $this->interactivity, 'state.foo', 'myPlugin' );
@@ -650 +656 @@
     public function test_extract_directive_value_empty_values() {
         $extract_directive_value = new ReflectionMethod( $this->interactivity, 'extract_directive_value' );
-        $extract_directive_value->setAccessible( true );
+        if ( PHP_VERSION_ID < 80100 ) {
+            $extract_directive_value->setAccessible( true );
+        }
 
         $result = $extract_directive_value->invoke( $this->interactivity, '', 'myPlugin' );
@@ -684 +692 @@
     public function test_extract_directive_value_invalid_json() {
         $extract_directive_value = new ReflectionMethod( $this->interactivity, 'extract_directive_value' );
-        $extract_directive_value->setAccessible( true );
+        if ( PHP_VERSION_ID < 80100 ) {
+            $extract_directive_value->setAccessible( true );
+        }
 
         // Invalid JSON due to missing quotes. Returns the original value.
@@ -705 +715 @@
     public function test_extract_prefix_and_suffix() {
         $extract_prefix_and_suffix = new ReflectionMethod( $this->interactivity, 'extract_prefix_and_suffix' );
-        $extract_prefix_and_suffix->setAccessible( true );
+        if ( PHP_VERSION_ID < 80100 ) {
+            $extract_prefix_and_suffix->setAccessible( true );
+        }
 
         $result = $extract_prefix_and_suffix->invoke( $this->interactivity, 'data-wp-interactive' );
@@ -979 +991 @@
 
         $evaluate = new ReflectionMethod( $this->interactivity, 'evaluate' );
-        $evaluate->setAccessible( true );
+        if ( PHP_VERSION_ID < 80100 ) {
+            $evaluate->setAccessible( true );
+        }
 
         $result = $evaluate->invokeArgs( $this->interactivity, array( $directive_value ) );
@@ -1394 +1408 @@
     public function test_kebab_to_camel_case() {
         $method = new ReflectionMethod( $this->interactivity, 'kebab_to_camel_case' );
-        $method->setAccessible( true );
+        if ( PHP_VERSION_ID < 80100 ) {
+            $method->setAccessible( true );
+        }
 
         $this->assertSame( '', $method->invoke( $this->interactivity, '' ) );