[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:39:37 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):

 Replying to [comment:6 antonvlasenko]:
 > I believe the `WP_Term::$object_id` property should also be added to the
 list of known properties (i.e., declared on the `WP_Term` class), as it is
 added by the `WP_Term_Query::get_terms()` method.
 >
 > Additionally, the `WP_Term::$auto_add` property is added by the
 `WP_REST_Menus_Controller::get_term()` method.

 Thanks for finding those. I added them in the
 [https://github.com/WordPress/wordpress-develop/pull/7224 working patch].

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


More information about the wp-trac mailing list