[wp-trac] [WordPress Trac] #58298: Deprecated errors on /wp-admin/revision.php?revision=2895&gutenberg=true

[WordPress Trac]

#58298: Deprecated errors on /wp-admin/revision.php?revision=2895&gutenberg=true
-----------------------------+-----------------------------
 Reporter:  thomask          |       Owner:  SergeyBiryukov
     Type:  defect (bug)     |      Status:  accepted
 Priority:  normal           |   Milestone:  6.3
Component:  Revisions        |     Version:  6.2
 Severity:  normal           |  Resolution:
 Keywords:  php82 has-patch  |     Focuses:
-----------------------------+-----------------------------

Comment (by SergeyBiryukov):

 In [changeset:"55752" 55752]:
 {{{
 #!CommitTicketReference repository="" revision="55752"
 Code Modernization: Explicitly declare all properties in
 `Text_Diff_Engine_native`.

 Dynamic (non-explicitly declared) properties are deprecated as of PHP 8.2
 and are expected to become a fatal error in PHP 9.0.

 There are a number of ways to mitigate this:
 * If it is an accidental typo for a declared property: fix the typo.
 * For known properties: declare them on the class.
 * For unknown properties: add the magic `__get()`, `__set()`, et al.
 methods to the class or let the class extend `stdClass` which has highly
 optimized versions of these magic methods built in.
 * For unknown ''use'' of dynamic properties, the
 `#[AllowDynamicProperties]` attribute can be added to the class. The
 attribute will automatically be inherited by child classes.

 In this case, the properties, as used in the class methods, fall in the
 “known property” category.

 Reference: [https://wiki.php.net/rfc/deprecate_dynamic_properties PHP RFC:
 Deprecate dynamic properties].

 Follow-up to [53942], [53948], [53949], [53952], [53953], [53954],
 [53957], [54037].

 Props jrf, thomask.
 See #58298.
 }}}

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