[wp-trac] [WordPress Trac] #58898: Fix WP_Text_Diff_Renderer_Table magic methods for PHP 8.2 dynamic properties

WordPress Trac noreply at wordpress.org
Sun Oct 15 14:53:27 UTC 2023


#58898: Fix WP_Text_Diff_Renderer_Table magic methods for PHP 8.2 dynamic
properties
-------------------------------------------------+-------------------------
 Reporter:  antonvlasenko                        |       Owner:
                                                 |  hellofromTonya
     Type:  defect (bug)                         |      Status:  closed
 Priority:  normal                               |   Milestone:  6.4
Component:  Administration                       |     Version:  4.0
 Severity:  normal                               |  Resolution:  fixed
 Keywords:  php82 has-patch has-unit-tests       |     Focuses:  php-
  commit                                         |  compatibility
-------------------------------------------------+-------------------------

Comment (by hellofromTonya):

 In [changeset:"56938" 56938]:
 {{{
 #!CommitTicketReference repository="" revision="56938"
 Code Modernization: Declare dynamic properties on
 WP_Text_Diff_Renderer_Table.

 Core uses 3 known, named, dynamic properties on the
 `WP_Text_Diff_Renderer_Table` class:

 * `_title`
 * `_title_left`
 * `_title_right`

 When reviewing revisions (within the admin UI), `wp_text_diff()` passes
 the arguments (without the leading `_`) to a new instance, which raised
 deprecation notices (see [56354]).

 Note: the parent class adds the leading `_` to each of these properties
 (see [7747]).

 The deprecation notices are resolved by declaring each of these known,
 named, dynamic properties on the class, per the #56034 mitigation
 strategy. These new properties are not initialized to retain their
 previous `null` behavior.

 Follow-up to [56354], [23506], [7747].

 Props wildworks, antonvlasenko, hellofromTonya, ironprogrammer, kafleg,
 mukesh27, nicolefurlan, presskopp, sabernhardt.
 Fixes #59431.
 See #58898, #56034.
 }}}

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


More information about the wp-trac mailing list