[wp-trac] [WordPress Trac] #59193: Remove misleading comment in class-wp-rest-blocks-controller.php

WordPress Trac noreply at wordpress.org
Wed Oct 4 19:39:30 UTC 2023


#59193: Remove misleading comment in class-wp-rest-blocks-controller.php
------------------------------------------+-------------------------
 Reporter:  kadamwhite                    |       Owner:  kadamwhite
     Type:  defect (bug)                  |      Status:  closed
 Priority:  low                           |   Milestone:  6.3.2
Component:  REST API                      |     Version:  6.3
 Severity:  normal                        |  Resolution:  fixed
 Keywords:  has-patch commit fixed-major  |     Focuses:  rest-api
------------------------------------------+-------------------------
Changes (by audrasjb):

 * status:  reopened => closed
 * resolution:   => fixed


Comment:

 In [changeset:"56781" 56781]:
 {{{
 #!CommitTicketReference repository="" revision="56781"
 REST API: Remove misleading comment in
 `WP_REST_Blocks_Controller->get_item_schema`.

 In r56093 schema caching was added above a comment instructing developers
 not to cache that controller's schema. However, there is no obvious
 penalty for re-caching
 schema that is partially derived from a parent.

 Caching schema in the same way in every controller is beneficial
 consistency, and discussion at WCUS2023 contributor day concluded we could
 remove this comment.

 Props ahardyjpl, davidbinda, johnjamesjacoby, TimothyBlynJacobs,
 kadamwhite.
 Fixes #59193.
 See #58657.
 }}}

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/59193#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list