Changeset 58320

Timestamp:

06/04/2024 06:56:25 AM (2 months ago)

Author:

gziolo

Message:

Interactivity API: Some property access does not work well in server directives

Ensures property access in PHP works for object properties or associative array values correctly when processing Interactivity API directives.

Props narenin, cbravobernal, jonsurrell, gziolo, czapla.
Fixes #61039.

Location:

trunk

Files:

• src/wp-includes/interactivity-api/class-wp-interactivity-api.php (modified) (1 diff)
• tests/phpunit/tests/interactivity-api/wpInteractivityAPI.php (modified) (3 diffs)

trunk/src/wp-includes/interactivity-api/class-wp-interactivity-api.php

@@ -424 +424 @@
         $current       = $store;
         foreach ( $path_segments as $path_segment ) {
-            if ( isset( $current[ $path_segment ] ) ) {
+            if ( isset( $current[ $path_segment ] ) ) {
                 $current = $current[ $path_segment ];
+
+
             } else {
                 return null;

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

@@ -817 +817 @@
     private function evaluate( $directive_value ) {
         $generate_state = function ( $name ) {
+
+
             return array(
-                'key'    => $name,
-                'nested' => array( 'key' => $name . '-nested' ),
+                'key'       => $name,
+                'nested'    => array( 'key' => $name . '-nested' ),
+                'obj'       => $obj,
+                'arrAccess' => new class() implements ArrayAccess {
+                    #[\ReturnTypeWillChange]
+                    public function offsetExists( $offset ) {
+                        return true;
+                    }
+
+                    public function offsetGet( $offset ): string {
+                        return $offset;
+                    }
+
+                    public function offsetSet( $offset, $value ): void {}
+
+                    public function offsetUnset( $offset ): void {}
+                },
             );
         };
@@ -827 +844 @@
             'myPlugin'    => $generate_state( 'myPlugin-context' ),
             'otherPlugin' => $generate_state( 'otherPlugin-context' ),
+
         );
         $evaluate = new ReflectionMethod( $this->interactivity, 'evaluate' );
@@ -852 +870 @@
         $result = $this->evaluate( 'otherPlugin::context.key' );
         $this->assertEquals( 'otherPlugin-context', $result );
+
+
+
+
+
+
     }