Changeset 58320

Timestamp:

06/04/2024 06:56:25 AM (2 years 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 ( ( is_array( $current ) || $current instanceof ArrayAccess ) && isset( $current[ $path_segment ] ) ) {
                                 $current = $current[ $path_segment ];
+                        } elseif ( is_object( $current ) && 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 ) {
+                        $obj       = new stdClass();
+                        $obj->prop = $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' ),
+                        'obj'         => new stdClass(),
                 );
                 $evaluate = new ReflectionMethod( $this->interactivity, 'evaluate' );
@@ -852 +870 @@
                 $result = $this->evaluate( 'otherPlugin::context.key' );
                 $this->assertEquals( 'otherPlugin-context', $result );
+
+                $result = $this->evaluate( 'state.obj.prop' );
+                $this->assertSame( 'myPlugin-state', $result );
+
+                $result = $this->evaluate( 'state.arrAccess.1' );
+                $this->assertSame( '1', $result );
         }