[wp-trac] [WordPress Trac] #61890: Handle WP_Term known, named dynamic properties for PHP 8.2

WordPress Trac noreply at wordpress.org
Wed Aug 21 21:04:31 UTC 2024


#61890: Handle WP_Term known, named dynamic properties for PHP 8.2
-------------------------------------+-------------------------------------
 Reporter:  hellofromTonya           |       Owner:  (none)
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  6.7
Component:  Taxonomy                 |     Version:
 Severity:  minor                    |  Resolution:
 Keywords:  php82 has-patch has-     |     Focuses:  coding-standards, php-
  unit-tests                         |  compatibility
-------------------------------------+-------------------------------------

Comment (by hellofromTonya):

 In [changeset:"58920" 58920]:
 {{{
 #!CommitTicketReference repository="" revision="58920"
 Tests: Remove 'errors' assertion when not WP_Error in
 Tests_Term_WpInsertTerm.

 Removes the assertion for 'errors' being empty when the instance is
 `WP_Term` and not `WP_Error`. This property exists on `WP_Error`.

 This assertion always passed because it was checking a dynamic property on
 `WP_Term` that does not exist and is not added within Core. Thus, this
 assertion is not needed and fails with dynamic property deprecations.

 Follow-up to [51403], [34646], [29830].

 See #61890, #61530.
 }}}

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


More information about the wp-trac mailing list