Ticket #53646: 53646.diff

src/wp-includes/rest-api/endpoints/class-wp-rest-sidebars-controller.php

@@ -281 +281 @@
 
                         $sidebar['status']        = 'active';
                         $sidebar['name']          = isset( $registered_sidebar['name'] ) ? $registered_sidebar['name'] : '';
-                        $sidebar['description']   = isset( $registered_sidebar['description'] ) ? $registered_sidebar['description'] : '';
+                        $sidebar['description']   = isset( $registered_sidebar['description'] ) ? wp_sidebar_description( $id ) : '';
                         $sidebar['class']         = isset( $registered_sidebar['class'] ) ? $registered_sidebar['class'] : '';
                         $sidebar['before_widget'] = isset( $registered_sidebar['before_widget'] ) ? $registered_sidebar['before_widget'] : '';
                         $sidebar['after_widget']  = isset( $registered_sidebar['after_widget'] ) ? $registered_sidebar['after_widget'] : '';

tests/phpunit/tests/rest-api/rest-sidebars-controller.php

@@ -313 +313 @@
         }
 
         /**
+         * @ticket 53646
+         */
+        public function test_get_items_when_descriptions_have_markup() {
+                register_sidebar(
+                        array(
+                                'name'          => 'New Sidebar',
+                                'id'            => 'new-sidebar',
+                                'description'   => '<iframe></iframe>This is a <b>description</b> with some <a href="#">markup</a>.<script></script>',
+                                'before_widget' => '',
+                                'after_widget'  => '',
+                                'before_title'  => '',
+                                'after_title'   => '',
+                        )
+                );
+
+                $request  = new WP_REST_Request( 'GET', '/wp/v2/sidebars' );
+                $response = rest_get_server()->dispatch( $request );
+                $data     = $response->get_data();
+                $data     = $this->remove_links( $data );
+                $this->assertSame(
+                        array(
+                                array(
+                                        'id'            => 'wp_inactive_widgets',
+                                        'name'          => 'Inactive widgets',
+                                        'description'   => '',
+                                        'class'         => '',
+                                        'before_widget' => '',
+                                        'after_widget'  => '',
+                                        'before_title'  => '',
+                                        'after_title'   => '',
+                                        'status'        => 'inactive',
+                                        'widgets'       => array(),
+                                ),
+                                array(
+                                        'id'            => 'new-sidebar',
+                                        'name'          => 'New Sidebar',
+                                        'description'   => 'This is a <b>description</b> with some <a href="#">markup</a>.',
+                                        'class'         => '',
+                                        'before_widget' => '',
+                                        'after_widget'  => '',
+                                        'before_title'  => '',
+                                        'after_title'   => '',
+                                        'status'        => 'active',
+                                        'widgets'       => array(),
+                                ),
+                        ),
+                        $data
+                );
+        }
+
+        /**
          * @ticket 41683
          */
         public function test_get_item() {