[wp-trac] [WordPress Trac] #56034: PHP 8.2: proposal for handling unknown dynamic properties deprecations

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


#56034: PHP 8.2: proposal for handling unknown dynamic properties deprecations
----------------------------+--------------------------------
 Reporter:  jrf             |       Owner:  hellofromTonya
     Type:  task (blessed)  |      Status:  accepted
 Priority:  normal          |   Milestone:  6.5
Component:  General         |     Version:
 Severity:  normal          |  Resolution:
 Keywords:  php82           |     Focuses:  php-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/56034#comment:66>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list