[wp-trac] [WordPress Trac] #59431: Revisions: get a deprecation error about WP_Text_Diff_Renderer_Table::__isset()
WordPress Trac
noreply at wordpress.org
Sun Oct 15 14:53:28 UTC 2023
#59431: Revisions: get a deprecation error about
WP_Text_Diff_Renderer_Table::__isset()
-----------------------------------------------+---------------------------
Reporter: wildworks | Owner:
| hellofromTonya
Type: defect (bug) | Status: closed
Priority: normal | Milestone: 6.4
Component: Revisions | Version: trunk
Severity: minor | Resolution: fixed
Keywords: has-patch has-testing-info commit | Focuses:
-----------------------------------------------+---------------------------
Changes (by hellofromTonya):
* status: assigned => closed
* resolution: => fixed
Comment:
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/59431#comment:26>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list