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

WordPress Trac noreply at wordpress.org
Thu Aug 24 20:21:50 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  |     Focuses:  rest-api
------------------------------+-------------------------
Changes (by kadamwhite):

 * owner:  (none) => kadamwhite
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"56459" 56459]:
 {{{
 #!CommitTicketReference repository="" revision="56459"
 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.

 Fixes #59193. See #58657.
 }}}

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


More information about the wp-trac mailing list