[wp-trac] [WordPress Trac] #58673: WordPress database error when installing PHPUnit tests

WordPress Trac noreply at wordpress.org
Thu Jul 13 11:32:35 UTC 2023


#58673: WordPress database error when installing PHPUnit tests
--------------------------------------------+---------------------------
 Reporter:  Chouby                          |       Owner:  spacedmonkey
     Type:  defect (bug)                    |      Status:  closed
 Priority:  high                            |   Milestone:  6.3
Component:  Editor                          |     Version:  trunk
 Severity:  normal                          |  Resolution:  fixed
 Keywords:  has-patch needs-testing commit  |     Focuses:  performance
--------------------------------------------+---------------------------
Changes (by spacedmonkey):

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


Comment:

 In [changeset:"56226" 56226]:
 {{{
 #!CommitTicketReference repository="" revision="56226"
 Editor: Lazily load Duotone settings only when needed.

 Introduced in [56101] the `WP_Duotone` class, hooks into the `wp_loaded`
 action to load duotone style data from global styles. Hooking in early in
 the bootstrap process caused a number of problems. This hook, triggered an
 error on installing, as this lookup for global styles, would result in a
 global post trying to be created, even before the table existed.
 Additionally, this implementation caused a severe performance regression,
 as duotone styling data was loaded unnecessarily for requests that did not
 require such data, such as REST API calls or actions within the wp-admin
 interface.

 In this change, refactor the `WP_Duotone` to lazily load the global styles
 and theme.json data, only when a block that supports duotone is
 encountered. The method `render_duotone_support` was change to take a
 third parameter to reuse the existing `WP_Block_Type` object passed to the
 filter, to save it being looked up again. The code has also got improved
 type checking and the use of the util function  `block_has_support`.
 Furthermore, the code's readability has been improved, along with
 enhancements to the documentation blocks.

 Props Chouby, spacedmonkey, SergeyBiryukov, swissspidy, costdev,
 joemcgill, flixos90, mukesh27, nazmul111, ajlende, isabel_brison.
 Fixes #58673.
 }}}

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


More information about the wp-trac mailing list