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

[WordPress Trac]

#56034: PHP 8.2: proposal for handling unknown dynamic properties deprecations
-------------------------------+-----------------------------
 Reporter:  jrf                |       Owner:  hellofromTonya
     Type:  task (blessed)     |      Status:  accepted
 Priority:  normal             |   Milestone:  6.2
Component:  General            |     Version:
 Severity:  normal             |  Resolution:
 Keywords:  php82 needs-patch  |     Focuses:
-------------------------------+-----------------------------

Comment (by peterwilsoncc):

 Replying to [comment:21 jrf]:
 > **Open question:**
 > Should we - TEMPORARILY - add the `#[AllowDynamicProperties]` attribute
 to all (parent) classes in WP Core for the time being to reduce the volume
 of notices being logged in error logs for sites using those logs ?
 >
 > Once we have the ''proper'' patches ready, we can then decide if and if
 so, when to remove the attributes.
 >
 > Please leave opinions on this in this thread.

 Do you have a preference? I'm happy to defer to you, @markjaquith and
 @hellofromTonya on this.

 Core already uses `#[ReturnTypeWillChange]` to reduce notices so it's not
 a new pattern.

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